nmcli 1.6.0__tar.gz → 1.7.0__tar.gz

{nmcli-1.6.0 → nmcli-1.7.0}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: nmcli
-Version: 1.6.0
+Version: 1.7.0
 Summary: A python wrapper library for the network-manager cli client
 Author: ushiboy
 License-Expression: MIT
@@ -58,6 +58,7 @@ except Exception as e:
 | general    | hostname     | supported     |
 | general    | permissions  | not supported |
 | general    | logging      | not supported |
+| general    | reload       | supported     |
 | networking |              | supported     |
 | networking | on           | supported     |
 | networking | off          | supported     |
@@ -75,6 +76,7 @@ except Exception as e:
 | connection | clone        | not supported |
 | connection | edit         | not supported |
 | connection | delete       | supported     |
+| connection | monitor      | not supported |
 | connection | reload       | supported     |
 | connection | load         | not supported |
 | connection | import       | not supported |
@@ -83,17 +85,20 @@ except Exception as e:
 | device     | status       | supported     |
 | device     | show         | supported     |
 | device     | set          | not supported |
+| device     | up           | supported     |
 | device     | connect      | supported     |
 | device     | reapply      | supported     |
 | device     | modify       | not supported |
+| device     | down         | supported     |
 | device     | disconnect   | supported     |
 | device     | delete       | supported     |
 | device     | monitor      | not supported |
-| device     | wifi         | supported     |
-| device     | wifi connect | supported     |
-| device     | wifi rescan  | supported     |
-| device     | wifi hotspot | supported     |
-| device     | lldp         | not supported |
+| device     | wifi                | supported     |
+| device     | wifi connect        | supported     |
+| device     | wifi rescan         | supported     |
+| device     | wifi hotspot        | supported     |
+| device     | wifi show-password  | not supported |
+| device     | lldp                | not supported |
 | agent      |              | not supported |
 | agent      | secret       | not supported |
 | agent      | polkit       | not supported |
@@ -174,6 +179,16 @@ Use `active` argument to show only the active profile.
 nmcli.connection.show(name: str, show_secrets: bool = False, active: bool = False) -> ConnectionDetails
 ```
 
+#### nmcli.connection.show_all
+
+Show all connections.
+
+Use `active` argument to show only active connections.
+
+```
+nmcli.connection.show_all(active: bool = False) -> List[Connection]
+```
+
 #### nmcli.connection.reload
 
 Reload all connection files from disk.
@@ -220,6 +235,16 @@ The `fields` argument applies the same effect to the command as the `-f | --fiel
 nmcli.device.show_all(fields: str = None) -> List[DeviceDetails]
 ```
 
+#### nmcli.device.up
+
+Connect the device.
+
+The `wait` argument applies the same effect to the command as the `--wait` option. If it is omitted, the default behavior is followed.
+
+```
+nmcli.device.up(ifname: str, wait: int = None) -> None
+```
+
 #### nmcli.device.connect
 
 Connect the device.
@@ -230,6 +255,16 @@ The `wait` argument applies the same effect to the command as the `--wait` optio
 nmcli.device.connect(ifname: str, wait: int = None) -> None
 ```
 
+#### nmcli.device.down
+
+Disconnect a device and prevent the device from automatically activating further connections without user/manual intervention.
+
+The `wait` argument applies the same effect to the command as the `--wait` option. If it is omitted, the default behavior is followed.
+
+```
+nmcli.device.down(ifname: str, wait: int = None) -> None
+```
+
 #### nmcli.device.disconnect
 
 Disconnect devices.
@@ -334,6 +369,21 @@ Change persistent system hostname.
 nmcli.general.set_hostname(hostname: str) -> None
 ```
 
+#### nmcli.general.reload
+
+Reload NetworkManager's configuration and perform certain updates.
+
+The `flags` argument specifies which configurations to reload. Valid flags are:
+- `conf`: Reload NetworkManager.conf configuration from disk
+- `dns-rc`: Update DNS configuration (equivalent to SIGUSR1)
+- `dns-full`: Restart the DNS plugin
+
+If no flags are provided, everything that is supported is reloaded.
+
+```
+nmcli.general.reload(flags: Optional[List[str]] = None) -> None
+```
+
 ### networking
 
 #### nmcli.networking
@@ -473,6 +523,12 @@ nmcli.set_lang(lang: str) -> None
 
 ## Change Log
 
+### 1.7.0
+
+- Added `nmcli.connection.show_all` method with active filtering support
+- Added `nmcli.device.up` and `nmcli.device.down` methods
+- Added `nmcli.general.reload` method with configuration flags support
+
 ### 1.6.0
 
 - Added active option to `nmcli.connection.show`

{nmcli-1.6.0 → nmcli-1.7.0}/README.md

@@ -44,6 +44,7 @@ except Exception as e:
 | general    | hostname     | supported     |
 | general    | permissions  | not supported |
 | general    | logging      | not supported |
+| general    | reload       | supported     |
 | networking |              | supported     |
 | networking | on           | supported     |
 | networking | off          | supported     |
@@ -61,6 +62,7 @@ except Exception as e:
 | connection | clone        | not supported |
 | connection | edit         | not supported |
 | connection | delete       | supported     |
+| connection | monitor      | not supported |
 | connection | reload       | supported     |
 | connection | load         | not supported |
 | connection | import       | not supported |
@@ -69,17 +71,20 @@ except Exception as e:
 | device     | status       | supported     |
 | device     | show         | supported     |
 | device     | set          | not supported |
+| device     | up           | supported     |
 | device     | connect      | supported     |
 | device     | reapply      | supported     |
 | device     | modify       | not supported |
+| device     | down         | supported     |
 | device     | disconnect   | supported     |
 | device     | delete       | supported     |
 | device     | monitor      | not supported |
-| device     | wifi         | supported     |
-| device     | wifi connect | supported     |
-| device     | wifi rescan  | supported     |
-| device     | wifi hotspot | supported     |
-| device     | lldp         | not supported |
+| device     | wifi                | supported     |
+| device     | wifi connect        | supported     |
+| device     | wifi rescan         | supported     |
+| device     | wifi hotspot        | supported     |
+| device     | wifi show-password  | not supported |
+| device     | lldp                | not supported |
 | agent      |              | not supported |
 | agent      | secret       | not supported |
 | agent      | polkit       | not supported |
@@ -160,6 +165,16 @@ Use `active` argument to show only the active profile.
 nmcli.connection.show(name: str, show_secrets: bool = False, active: bool = False) -> ConnectionDetails
 ```
 
+#### nmcli.connection.show_all
+
+Show all connections.
+
+Use `active` argument to show only active connections.
+
+```
+nmcli.connection.show_all(active: bool = False) -> List[Connection]
+```
+
 #### nmcli.connection.reload
 
 Reload all connection files from disk.
@@ -206,6 +221,16 @@ The `fields` argument applies the same effect to the command as the `-f | --fiel
 nmcli.device.show_all(fields: str = None) -> List[DeviceDetails]
 ```
 
+#### nmcli.device.up
+
+Connect the device.
+
+The `wait` argument applies the same effect to the command as the `--wait` option. If it is omitted, the default behavior is followed.
+
+```
+nmcli.device.up(ifname: str, wait: int = None) -> None
+```
+
 #### nmcli.device.connect
 
 Connect the device.
@@ -216,6 +241,16 @@ The `wait` argument applies the same effect to the command as the `--wait` optio
 nmcli.device.connect(ifname: str, wait: int = None) -> None
 ```
 
+#### nmcli.device.down
+
+Disconnect a device and prevent the device from automatically activating further connections without user/manual intervention.
+
+The `wait` argument applies the same effect to the command as the `--wait` option. If it is omitted, the default behavior is followed.
+
+```
+nmcli.device.down(ifname: str, wait: int = None) -> None
+```
+
 #### nmcli.device.disconnect
 
 Disconnect devices.
@@ -320,6 +355,21 @@ Change persistent system hostname.
 nmcli.general.set_hostname(hostname: str) -> None
 ```
 
+#### nmcli.general.reload
+
+Reload NetworkManager's configuration and perform certain updates.
+
+The `flags` argument specifies which configurations to reload. Valid flags are:
+- `conf`: Reload NetworkManager.conf configuration from disk
+- `dns-rc`: Update DNS configuration (equivalent to SIGUSR1)
+- `dns-full`: Restart the DNS plugin
+
+If no flags are provided, everything that is supported is reloaded.
+
+```
+nmcli.general.reload(flags: Optional[List[str]] = None) -> None
+```
+
 ### networking
 
 #### nmcli.networking
@@ -459,6 +509,12 @@ nmcli.set_lang(lang: str) -> None
 
 ## Change Log
 
+### 1.7.0
+
+- Added `nmcli.connection.show_all` method with active filtering support
+- Added `nmcli.device.up` and `nmcli.device.down` methods
+- Added `nmcli.general.reload` method with configuration flags support
+
 ### 1.6.0
 
 - Added active option to `nmcli.connection.show`

{nmcli-1.6.0 → nmcli-1.7.0}/nmcli/_connection.py

@@ -34,6 +34,9 @@ class ConnectionControlInterface:
     def show(self, name: str, show_secrets: bool = False, active: bool = False) -> ConnectionDetails:
         raise NotImplementedError
 
+    def show_all(self, active: bool = False) -> List[Connection]:
+        raise NotImplementedError
+
     def reload(self) -> None:
         raise NotImplementedError
 
@@ -104,5 +107,17 @@ class ConnectionControl(ConnectionControlInterface):
                 results[key] = None if value in ('--', '""') else value
         return results
 
+    def show_all(self, active: bool = False) -> List[Connection]:
+        cmd = ['connection', 'show']
+        if active:
+            cmd += ['--active']
+        r = self._syscmd.nmcli(cmd)
+        results = []
+        for row in r.split('\n')[1:]:
+            if len(row) == 0:
+                continue
+            results.append(Connection.parse(row))
+        return results
+
     def reload(self) -> None:
         self._syscmd.nmcli(['connection', 'reload'])

{nmcli-1.6.0 → nmcli-1.7.0}/nmcli/_device.py

@@ -52,9 +52,15 @@ class DeviceControlInterface:
     def show_all(self, fields: str = None) -> List[DeviceDetails]:
         raise NotImplementedError
 
+    def up(self, ifname: str, wait: int = None) -> None:
+        raise NotImplementedError
+
     def connect(self, ifname: str, wait: int = None) -> None:
         raise NotImplementedError
 
+    def down(self, ifname: str, wait: int = None) -> None:
+        raise NotImplementedError
+
     def disconnect(self, ifname: str, wait: int = None) -> None:
         raise NotImplementedError
 
@@ -131,11 +137,21 @@ class DeviceControl(DeviceControlInterface):
                 details[key] = None if value in ('--', '""') else value
         return results
 
+    def up(self, ifname: str, wait: int = None) -> None:
+        cmd = add_wait_option_if_needed(
+            wait) + ['device', 'up', ifname]
+        self._syscmd.nmcli(cmd)
+
     def connect(self, ifname: str, wait: int = None) -> None:
         cmd = add_wait_option_if_needed(
             wait) + ['device', 'connect', ifname]
         self._syscmd.nmcli(cmd)
 
+    def down(self, ifname: str, wait: int = None) -> None:
+        cmd = add_wait_option_if_needed(
+            wait) + ['device', 'down', ifname]
+        self._syscmd.nmcli(cmd)
+
     def disconnect(self, ifname: str, wait: int = None) -> None:
         cmd = add_wait_option_if_needed(
             wait) + ['device', 'disconnect', ifname]

{nmcli-1.6.0 → nmcli-1.7.0}/nmcli/_general.py

@@ -1,3 +1,5 @@
+from typing import List, Optional
+
 from ._system import SystemCommand, SystemCommandInterface
 from .data import General
 
@@ -16,9 +18,14 @@ class GeneralControlInterface:
     def set_hostname(self, hostname: str):
         raise NotImplementedError
 
+    def reload(self, flags: Optional[List[str]] = None) -> None:
+        raise NotImplementedError
+
 
 class GeneralControl(GeneralControlInterface):
 
+    VALID_RELOAD_FLAGS = ['conf', 'dns-rc', 'dns-full']
+
     def __init__(self, syscmd: SystemCommandInterface = None):
         self._syscmd = syscmd or SystemCommand()
 
@@ -35,3 +42,16 @@ class GeneralControl(GeneralControlInterface):
 
     def set_hostname(self, hostname: str):
         self._syscmd.nmcli(['general', 'hostname', hostname])
+
+    def reload(self, flags: Optional[List[str]] = None) -> None:
+        if flags is not None:
+            for flag in flags:
+                if flag not in self.VALID_RELOAD_FLAGS:
+                    raise ValueError(
+                        f"Invalid reload flag '{flag}'. "
+                        f"Valid flags are: {', '.join(self.VALID_RELOAD_FLAGS)}"
+                    )
+        cmd = ['general', 'reload']
+        if flags:
+            cmd += flags
+        self._syscmd.nmcli(cmd)

{nmcli-1.6.0 → nmcli-1.7.0}/nmcli/dummy/_connection.py

@@ -31,6 +31,10 @@ class DummyConnectionControl(ConnectionControlInterface):
     def show_args(self):
         return self._show_args
 
+    @property
+    def show_all_args(self):
+        return self._show_all_args
+
     @property
     def called_reload(self) -> int:
         return self._called_reload
@@ -38,16 +42,19 @@ class DummyConnectionControl(ConnectionControlInterface):
     def __init__(self,
                  result_call: List[Connection] = None,
                  result_show: ConnectionDetails = None,
+                 result_show_all: List[Connection] = None,
                  raise_error: Exception = None):
         self._raise_error = raise_error
         self._result_call = result_call or []
         self._result_show = result_show
+        self._result_show_all = result_show_all or []
         self._add_args: List[Tuple] = []
         self._modify_args: List[Tuple] = []
         self._delete_args: List[Tuple] = []
         self._up_args: List[Tuple] = []
         self._down_args: List[Tuple] = []
         self._show_args: List[Tuple] = []
+        self._show_all_args: List[Tuple] = []
         self._called_reload = 0
 
     def __call__(self) -> List[Connection]:
@@ -86,6 +93,11 @@ class DummyConnectionControl(ConnectionControlInterface):
             return self._result_show
         raise ValueError("'result_show' is not properly initialized")
 
+    def show_all(self, active: bool = False) -> List[Connection]:
+        self._raise_error_if_needed()
+        self._show_all_args.append((active,))
+        return self._result_show_all
+
     def reload(self) -> None:
         self._raise_error_if_needed()
         self._called_reload += 1

{nmcli-1.6.0 → nmcli-1.7.0}/nmcli/dummy/_device.py

@@ -5,16 +5,24 @@ from ..data.device import Device, DeviceWifi
 from ..data.hotspot import Hotspot
 
 
-class DummyDeviceControl(DeviceControlInterface):
+class DummyDeviceControl(DeviceControlInterface):  # pylint: disable=too-many-public-methods
 
     @property
     def show_args(self):
         return self._show_args
 
+    @property
+    def up_args(self):
+        return self._up_args
+
     @property
     def connect_args(self):
         return self._connect_args
 
+    @property
+    def down_args(self):
+        return self._down_args
+
     @property
     def disconnect_args(self):
         return self._disconnect_args
@@ -57,7 +65,9 @@ class DummyDeviceControl(DeviceControlInterface):
         self._result_show_all = result_show_all or []
         self._result_wifi_hotspot = result_wifi_hotspot
         self._show_args: List[Tuple] = []
+        self._up_args: List[Tuple] = []
         self._connect_args: List[Tuple] = []
+        self._down_args: List[Tuple] = []
         self._disconnect_args: List[Tuple] = []
         self._reapply_args: List[str] = []
         self._delete_args: List[Tuple] = []
@@ -85,10 +95,18 @@ class DummyDeviceControl(DeviceControlInterface):
         self._raise_error_if_needed()
         return self._result_show_all
 
+    def up(self, ifname: str, wait: int = None) -> None:
+        self._raise_error_if_needed()
+        self._up_args.append((ifname, wait))
+
     def connect(self, ifname: str, wait: int = None) -> None:
         self._raise_error_if_needed()
         self._connect_args.append((ifname, wait))
 
+    def down(self, ifname: str, wait: int = None) -> None:
+        self._raise_error_if_needed()
+        self._down_args.append((ifname, wait))
+
     def disconnect(self, ifname: str, wait: int = None) -> None:
         self._raise_error_if_needed()
         self._disconnect_args.append((ifname, wait))

{nmcli-1.6.0 → nmcli-1.7.0}/nmcli/dummy/_general.py

@@ -1,4 +1,4 @@
-from typing import List
+from typing import List, Optional
 
 from .._general import GeneralControlInterface
 from ..data.general import General
@@ -10,6 +10,10 @@ class DummyGeneralControl(GeneralControlInterface):
     def set_hostname_args(self):
         return self._set_hostname_args
 
+    @property
+    def reload_args(self):
+        return self._reload_args
+
     def __init__(self,
                  result_call: General = None,
                  result_status: General = None,
@@ -20,6 +24,7 @@ class DummyGeneralControl(GeneralControlInterface):
         self._result_status = result_status
         self._result_hostname = result_hostname
         self._set_hostname_args: List[str] = []
+        self._reload_args: List[Optional[List[str]]] = []
 
     def __call__(self) -> General:
         self._raise_error_if_needed()
@@ -41,6 +46,10 @@ class DummyGeneralControl(GeneralControlInterface):
         self._raise_error_if_needed()
         self._set_hostname_args.append(hostname)
 
+    def reload(self, flags: Optional[List[str]] = None) -> None:
+        self._raise_error_if_needed()
+        self._reload_args.append(flags)
+
     def _raise_error_if_needed(self):
         if not self._raise_error is None:
             raise self._raise_error

{nmcli-1.6.0 → nmcli-1.7.0}/nmcli.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: nmcli
-Version: 1.6.0
+Version: 1.7.0
 Summary: A python wrapper library for the network-manager cli client
 Author: ushiboy
 License-Expression: MIT
@@ -58,6 +58,7 @@ except Exception as e:
 | general    | hostname     | supported     |
 | general    | permissions  | not supported |
 | general    | logging      | not supported |
+| general    | reload       | supported     |
 | networking |              | supported     |
 | networking | on           | supported     |
 | networking | off          | supported     |
@@ -75,6 +76,7 @@ except Exception as e:
 | connection | clone        | not supported |
 | connection | edit         | not supported |
 | connection | delete       | supported     |
+| connection | monitor      | not supported |
 | connection | reload       | supported     |
 | connection | load         | not supported |
 | connection | import       | not supported |
@@ -83,17 +85,20 @@ except Exception as e:
 | device     | status       | supported     |
 | device     | show         | supported     |
 | device     | set          | not supported |
+| device     | up           | supported     |
 | device     | connect      | supported     |
 | device     | reapply      | supported     |
 | device     | modify       | not supported |
+| device     | down         | supported     |
 | device     | disconnect   | supported     |
 | device     | delete       | supported     |
 | device     | monitor      | not supported |
-| device     | wifi         | supported     |
-| device     | wifi connect | supported     |
-| device     | wifi rescan  | supported     |
-| device     | wifi hotspot | supported     |
-| device     | lldp         | not supported |
+| device     | wifi                | supported     |
+| device     | wifi connect        | supported     |
+| device     | wifi rescan         | supported     |
+| device     | wifi hotspot        | supported     |
+| device     | wifi show-password  | not supported |
+| device     | lldp                | not supported |
 | agent      |              | not supported |
 | agent      | secret       | not supported |
 | agent      | polkit       | not supported |
@@ -174,6 +179,16 @@ Use `active` argument to show only the active profile.
 nmcli.connection.show(name: str, show_secrets: bool = False, active: bool = False) -> ConnectionDetails
 ```
 
+#### nmcli.connection.show_all
+
+Show all connections.
+
+Use `active` argument to show only active connections.
+
+```
+nmcli.connection.show_all(active: bool = False) -> List[Connection]
+```
+
 #### nmcli.connection.reload
 
 Reload all connection files from disk.
@@ -220,6 +235,16 @@ The `fields` argument applies the same effect to the command as the `-f | --fiel
 nmcli.device.show_all(fields: str = None) -> List[DeviceDetails]
 ```
 
+#### nmcli.device.up
+
+Connect the device.
+
+The `wait` argument applies the same effect to the command as the `--wait` option. If it is omitted, the default behavior is followed.
+
+```
+nmcli.device.up(ifname: str, wait: int = None) -> None
+```
+
 #### nmcli.device.connect
 
 Connect the device.
@@ -230,6 +255,16 @@ The `wait` argument applies the same effect to the command as the `--wait` optio
 nmcli.device.connect(ifname: str, wait: int = None) -> None
 ```
 
+#### nmcli.device.down
+
+Disconnect a device and prevent the device from automatically activating further connections without user/manual intervention.
+
+The `wait` argument applies the same effect to the command as the `--wait` option. If it is omitted, the default behavior is followed.
+
+```
+nmcli.device.down(ifname: str, wait: int = None) -> None
+```
+
 #### nmcli.device.disconnect
 
 Disconnect devices.
@@ -334,6 +369,21 @@ Change persistent system hostname.
 nmcli.general.set_hostname(hostname: str) -> None
 ```
 
+#### nmcli.general.reload
+
+Reload NetworkManager's configuration and perform certain updates.
+
+The `flags` argument specifies which configurations to reload. Valid flags are:
+- `conf`: Reload NetworkManager.conf configuration from disk
+- `dns-rc`: Update DNS configuration (equivalent to SIGUSR1)
+- `dns-full`: Restart the DNS plugin
+
+If no flags are provided, everything that is supported is reloaded.
+
+```
+nmcli.general.reload(flags: Optional[List[str]] = None) -> None
+```
+
 ### networking
 
 #### nmcli.networking
@@ -473,6 +523,12 @@ nmcli.set_lang(lang: str) -> None
 
 ## Change Log
 
+### 1.7.0
+
+- Added `nmcli.connection.show_all` method with active filtering support
+- Added `nmcli.device.up` and `nmcli.device.down` methods
+- Added `nmcli.general.reload` method with configuration flags support
+
 ### 1.6.0
 
 - Added active option to `nmcli.connection.show`

{nmcli-1.6.0 → nmcli-1.7.0}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "nmcli"
-version = "1.6.0"
+version = "1.7.0"
 description = "A python wrapper library for the network-manager cli client"
 readme = "README.md"
 authors = [{name = "ushiboy"}]

{nmcli-1.6.0 → nmcli-1.7.0}/tests/test_connection.py

@@ -132,6 +132,28 @@ def test_show():
         'connection', 'show', "--active", name]
 
 
+def test_show_all():
+    s = DummySystemCommand('''NAME            UUID                                  TYPE      DEVICE
+AP1  3eac760c-de77-4823-9ab8-773c276daca3  wifi      wlan0
+Home            700f5b18-cbb3-4d38-9c61-e3bc3a3852b9  ethernet  eth0
+Wired connection 1  700f5b18-cbb3-4d38-9c61-999999999999  ethernet  eth1''')
+    connection = ConnectionControl(s)
+
+    r = connection.show_all()
+    assert s.passed_parameters == ['connection', 'show']
+    assert r == [
+        Connection('AP1', '3eac760c-de77-4823-9ab8-773c276daca3',
+                   'wifi', 'wlan0'),
+        Connection('Home', '700f5b18-cbb3-4d38-9c61-e3bc3a3852b9',
+                   'ethernet', 'eth0'),
+        Connection('Wired connection 1',
+                   '700f5b18-cbb3-4d38-9c61-999999999999', 'ethernet', 'eth1')
+    ]
+
+    connection.show_all(active=True)
+    assert s.passed_parameters == ['connection', 'show', '--active']
+
+
 def test_reload():
     s = DummySystemCommand()
     connection = ConnectionControl(s)

{nmcli-1.6.0 → nmcli-1.7.0}/tests/test_device.py

@@ -171,6 +171,17 @@ IP6.ROUTE[1]:                           dst = ::1/128, nh = ::, mt = 256'''
     assert s2.passed_parameters == ['-f', 'all', 'device', 'show']
 
 
+def test_up():
+    s = DummySystemCommand()
+    device = DeviceControl(s)
+    ifname = 'eth0'
+    device.up(ifname)
+    assert s.passed_parameters == ['device', 'up', ifname]
+
+    device.up(ifname, wait=10)
+    assert s.passed_parameters == ['--wait', '10', 'device', 'up', ifname]
+
+
 def test_connect():
     s = DummySystemCommand()
     device = DeviceControl(s)
@@ -182,6 +193,18 @@ def test_connect():
     assert s.passed_parameters == ['--wait', '10', 'device', 'connect', ifname]
 
 
+def test_down():
+    s = DummySystemCommand()
+    device = DeviceControl(s)
+    ifname = 'eth0'
+    device.down(ifname)
+    assert s.passed_parameters == ['device', 'down', ifname]
+
+    device.down(ifname, wait=10)
+    assert s.passed_parameters == [
+        '--wait', '10', 'device', 'down', ifname]
+
+
 def test_disconnect():
     s = DummySystemCommand()
     device = DeviceControl(s)

{nmcli-1.6.0 → nmcli-1.7.0}/tests/test_general.py

@@ -1,3 +1,5 @@
+import pytest
+
 from nmcli._const import NetworkConnectivity, NetworkManagerState
 from nmcli._general import GeneralControl
 from nmcli.data import General
@@ -37,3 +39,41 @@ def test_set_hostname():
     general = GeneralControl(s)
     general.set_hostname('test')
     assert s.passed_parameters == ['general', 'hostname', 'test']
+
+
+def test_reload_without_flags():
+    s = DummySystemCommand()
+    general = GeneralControl(s)
+    general.reload()
+    assert s.passed_parameters == ['general', 'reload']
+
+
+def test_reload_with_single_flag():
+    s = DummySystemCommand()
+    general = GeneralControl(s)
+    general.reload(['conf'])
+    assert s.passed_parameters == ['general', 'reload', 'conf']
+
+
+def test_reload_with_all_valid_flags():
+    s = DummySystemCommand()
+    general = GeneralControl(s)
+    general.reload(['conf', 'dns-rc', 'dns-full'])
+    assert s.passed_parameters == ['general', 'reload', 'conf', 'dns-rc', 'dns-full']
+
+
+def test_reload_with_invalid_flag():
+    s = DummySystemCommand()
+    general = GeneralControl(s)
+    with pytest.raises(ValueError) as exc_info:
+        general.reload(['invalid-flag'])
+    assert "Invalid reload flag 'invalid-flag'" in str(exc_info.value)
+    assert "Valid flags are: conf, dns-rc, dns-full" in str(exc_info.value)
+
+
+def test_reload_with_mixed_valid_and_invalid_flags():
+    s = DummySystemCommand()
+    general = GeneralControl(s)
+    with pytest.raises(ValueError) as exc_info:
+        general.reload(['conf', 'invalid'])
+    assert "Invalid reload flag 'invalid'" in str(exc_info.value)