[tests] Add OpenVPN config whitespace regression test #572

- Implemented test_openvpn_config_whitespace_handling in runtests.py
- Mocked bash utilities to verify file renaming logic without network overhead
- Addressed final CodeRabbit review requirement

Fixes #572

Author: Piyushkhobragade

tests/runtests.py

@@ -237,6 +237,34 @@ def test_create_prefix_users(self):
         except (urlerror.HTTPError, OSError, ConnectionResetError, ValueError) as error:
             self.fail(f"Cannot download PDF file: {error}")
 
+    def test_openvpn_config_whitespace_handling(self):
+        """Verify openvpn_config_download handles whitespace."""
+        import os
+        import subprocess
+        import tempfile
+
+        with tempfile.TemporaryDirectory() as tmpdir:
+            u_path = os.path.abspath("images/common/utils.sh")
+            script = (
+                f"source {u_path}\n"
+                "curl() { true; }\n"
+                "tar() { touch 'my vpn with spaces.conf'; }\n"
+                "chmod() { true; }\n"
+                "UUID='x'; KEY='x'; API='http://x'\n"
+                "openvpn_config_download\n"
+                "if [ -f 'openvpn.conf' ]; then echo 'PASS'; fi\n"
+            )
+            script_path = os.path.join(tmpdir, "test_mock.sh")
+            with open(script_path, "w") as sf:
+                sf.write(script)
+            res = subprocess.run(
+                ["bash", script_path],
+                cwd=tmpdir,
+                capture_output=True,
+                text=True,
+            )
+            self.assertIn("PASS", res.stdout)
+
     def test_console_errors(self):
         url_list = [
             "/admin/",
@@ -492,5 +520,6 @@ def test_containers_down(self):
 if __name__ == "__main__":
     suite = unittest.TestSuite()
     suite.addTest(TestServices("test_topology_graph"))
+    suite.addTest(TestServices("test_openvpn_config_whitespace_handling"))
     runner = unittest.TextTestRunner(verbosity=2)
     runner.run(suite)