Add general reload

Author: ushiboy

README.md

@@ -44,7 +44,7 @@ except Exception as e:
 | general    | hostname     | supported     |
 | general    | permissions  | not supported |
 | general    | logging      | not supported |
-| general    | reload       | not supported |
+| general    | reload       | supported     |
 | networking |              | supported     |
 | networking | on           | supported     |
 | networking | off          | supported     |
@@ -355,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

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 @@ def get_hostname(self) -> str:
     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 @@ def get_hostname(self) -> str:
 
     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/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 @@ def __init__(self,
         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 @@ def set_hostname(self, hostname: str):
         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

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)