[openwrt] Dropped obsolete code in OpenVpn backend #58

Closes #58
openwisp · May 25, 2017 · a402079 · a402079
1 parent b305b1a
commit a402079
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 17 deletions.
diff --git a/netjsonconfig/backends/openwrt/converters.py b/netjsonconfig/backends/openwrt/converters.py
@@ -477,9 +477,6 @@ def __get_vpn(self, vpn):
         if 'disabled' in config:
             config['enabled'] = not config['disabled']
             del config['disabled']
-        # TODO: keep 'enabled' check until 0.6 and then drop it
-        elif 'disabled' not in config and 'enabled' not in config:
-            config['enabled'] = True
         config.update({
             '.name': logical_name(config.pop('name')),
             '.type': 'openvpn'

diff --git a/tests/openwrt/test_openvpn.py b/tests/openwrt/test_openvpn.py
@@ -25,8 +25,7 @@ def test_server_mode(self):
                 "down": "",
                 "duplicate_cn": True,
                 "engine": "rsax",
-                # TODO: in 0.6.0 change this to "disabled": False
-                "enabled": True,
+                "disabled": True,
                 "fast_io": True,
                 "fragment": 0,
                 "group": "nogroup",
@@ -109,8 +108,7 @@ def test_client_mode(self):
                     "dev": "tun0",
                     "dev_type": "tun",
                     "down": "/home/user/down-command.sh",
-                    # TODO: in 0.6.0 change this to "disabled": False
-                    "enabled": True,
+                    "disabled": True,
                     "engine": "",
                     "fast_io": False,
                     "fragment": 0,
@@ -205,8 +203,7 @@ def test_no_status_file(self):
                 "dev": "tap0",
                 "dev_type": "tap",
                 "dh": "dh.pem",
-                # TODO: in 0.6.0 change this to "disabled": False
-                "enabled": True,
+                "disabled": True,
                 "key": "key.pem",
                 "mode": "server",
                 "name": "test-no-status",
@@ -240,8 +237,7 @@ def test_additional_properties(self):
                 "dev": "tap0",
                 "dev_type": "tap",
                 "dh": "dh.pem",
-                # TODO: in 0.6.0 change this to "disabled": False
-                "enabled": True,
+                "disabled": True,
                 "key": "key.pem",
                 "mode": "server",
                 "name": "test-properties",
@@ -314,8 +310,7 @@ def test_server_bridge(self):
                 "dev": "tap0",
                 "dev_type": "tap",
                 "dh": "dh.pem",
-                # TODO: in 0.6.0 change this to "disabled": False
-                "enabled": True,
+                "disabled": True,
                 "key": "key.pem",
                 "mode": "server",
                 "name": "bridged",
@@ -349,8 +344,7 @@ def test_server_bridge_proxy(self):
                 "dev": "tap0",
                 "dev_type": "tap",
                 "dh": "dh.pem",
-                # TODO: in 0.6.0 change this to "disabled": False
-                "enabled": True,
+                "disabled": True,
                 "key": "key.pem",
                 "mode": "server",
                 "name": "bridged-proxy",
@@ -384,8 +378,7 @@ def test_server_bridge_routed(self):
                 "dev": "tap0",
                 "dev_type": "tap",
                 "dh": "dh.pem",
-                # TODO: in 0.6.0 change this to "disabled": False
-                "enabled": True,
+                "disabled": True,
                 "key": "key.pem",
                 "mode": "server",
                 "name": "routed",