ddns 4.1.0b3__tar.gz → 4.1.0b4__tar.gz

{ddns-4.1.0b3 → ddns-4.1.0b4}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: ddns
-Version: 4.1.0b3
+Version: 4.1.0b4
 Summary: Dynamic DNS client for multiple providers, supporting IPv4 and IPv6.
 Author-email: NewFuture <python@newfuture.cc>
 License-Expression: MIT
@@ -138,7 +138,7 @@ Dynamic: license-file
   也可使用一键安装脚本自动下载并安装对应平台的二进制：
 
   ```bash
-  curl -fSL https://ddns.newfuture.cc/install.sh | sh
+  curl -#fSL https://ddns.newfuture.cc/install.sh | sh
   ```
   提示：安装到系统目录（如 /usr/local/bin）可能需要 root 或 sudo 权限；若权限不足，可改为 `sudo sh` 运行。
 

{ddns-4.1.0b3 → ddns-4.1.0b4}/README.md

@@ -94,7 +94,7 @@
   也可使用一键安装脚本自动下载并安装对应平台的二进制：
 
   ```bash
-  curl -fSL https://ddns.newfuture.cc/install.sh | sh
+  curl -#fSL https://ddns.newfuture.cc/install.sh | sh
   ```
   提示：安装到系统目录（如 /usr/local/bin）可能需要 root 或 sudo 权限；若权限不足，可改为 `sudo sh` 运行。
 

{ddns-4.1.0b3 → ddns-4.1.0b4}/ddns/__init__.py

@@ -6,7 +6,7 @@ ddns Package
 __description__ = "automatically update DNS records to my IP [域名自动指向本机IP]"
 
 # 编译时，版本会被替换
-__version__ = "4.1.0b3"
+__version__ = "4.1.0b4"
 
 # 时间也会被替换掉
-build_date = "2025-08-11T15:00:49Z"
+build_date = "2025-08-16T02:41:26Z"

{ddns-4.1.0b3 → ddns-4.1.0b4}/ddns/__main__.py

@@ -4,16 +4,16 @@ DDNS
 @author: NewFuture, rufengsuixing
 """
 
+import sys
 from io import TextIOWrapper
-from subprocess import check_output
 from logging import getLogger
-import sys
+from subprocess import check_output
 
-from .__init__ import __version__, __description__, build_date
-from .config import load_configs, Config  # noqa: F401
-from .provider import get_provider_class, SimpleProvider  # noqa: F401
 from . import ip
+from .__init__ import __description__, __version__, build_date
 from .cache import Cache
+from .config import Config, load_configs  # noqa: F401
+from .provider import SimpleProvider, get_provider_class  # noqa: F401
 
 logger = getLogger()
 
@@ -102,11 +102,16 @@ def run(config):
 
 
 def main():
-    encode = sys.stdout.encoding
-    if encode is not None and encode.lower() != "utf-8" and hasattr(sys.stdout, "buffer"):
+    stdout = sys.stdout  # pythonw 模式无 stdout
+    if stdout and stdout.encoding and stdout.encoding.lower() != "utf-8" and hasattr(stdout, "buffer"):
         # 兼容windows 和部分ASCII编码的老旧系统
         sys.stdout = TextIOWrapper(sys.stdout.buffer, encoding="utf-8")
         sys.stderr = TextIOWrapper(sys.stderr.buffer, encoding="utf-8")
+
+    # Windows 下输出一个空行
+    if stdout and sys.platform.startswith("win"):
+        stdout.write("\r\n")
+
     logger.name = "ddns"
 
     # 使用多配置加载器，它会自动处理单个和多个配置

{ddns-4.1.0b3 → ddns-4.1.0b4}/ddns/ip.py

@@ -16,6 +16,24 @@ IPV4_REG = r"((25[0-5]|(2[0-4]|1{0,1}[0-9]){0,1}[0-9])\.){3,3}(25[0-5]|(2[0-4]|1
 # https://community.helpsystems.com/forums/intermapper/miscellaneous-topics/5acc4fcf-fa83-e511-80cf-0050568460e4
 IPV6_REG = r"((([0-9A-Fa-f]{1,4}:){7}([0-9A-Fa-f]{1,4}|:))|(([0-9A-Fa-f]{1,4}:){6}(:[0-9A-Fa-f]{1,4}|((25[0-5]|2[0-4]\d|1\d\d|[1-9]?\d)(\.(25[0-5]|2[0-4]\d|1\d\d|[1-9]?\d)){3})|:))|(([0-9A-Fa-f]{1,4}:){5}(((:[0-9A-Fa-f]{1,4}){1,2})|:((25[0-5]|2[0-4]\d|1\d\d|[1-9]?\d)(\.(25[0-5]|2[0-4]\d|1\d\d|[1-9]?\d)){3})|:))|(([0-9A-Fa-f]{1,4}:){4}(((:[0-9A-Fa-f]{1,4}){1,3})|((:[0-9A-Fa-f]{1,4})?:((25[0-5]|2[0-4]\d|1\d\d|[1-9]?\d)(\.(25[0-5]|2[0-4]\d|1\d\d|[1-9]?\d)){3}))|:))|(([0-9A-Fa-f]{1,4}:){3}(((:[0-9A-Fa-f]{1,4}){1,4})|((:[0-9A-Fa-f]{1,4}){0,2}:((25[0-5]|2[0-4]\d|1\d\d|[1-9]?\d)(\.(25[0-5]|2[0-4]\d|1\d\d|[1-9]?\d)){3}))|:))|(([0-9A-Fa-f]{1,4}:){2}(((:[0-9A-Fa-f]{1,4}){1,5})|((:[0-9A-Fa-f]{1,4}){0,3}:((25[0-5]|2[0-4]\d|1\d\d|[1-9]?\d)(\.(25[0-5]|2[0-4]\d|1\d\d|[1-9]?\d)){3}))|:))|(([0-9A-Fa-f]{1,4}:){1}(((:[0-9A-Fa-f]{1,4}){1,6})|((:[0-9A-Fa-f]{1,4}){0,4}:((25[0-5]|2[0-4]\d|1\d\d|[1-9]?\d)(\.(25[0-5]|2[0-4]\d|1\d\d|[1-9]?\d)){3}))|:))|(:(((:[0-9A-Fa-f]{1,4}){1,7})|((:[0-9A-Fa-f]{1,4}){0,5}:((25[0-5]|2[0-4]\d|1\d\d|[1-9]?\d)(\.(25[0-5]|2[0-4]\d|1\d\d|[1-9]?\d)){3}))|:)))"  # noqa: E501
 
+# 公网IPv4 API列表，按优先级排序
+PUBLIC_IPV4_APIS = [
+    "https://api.ipify.cn",
+    "https://api.ipify.org",
+    "https://4.ipw.cn/",
+    "https://ipinfo.io/ip",
+    "https://api-ipv4.ip.sb/ip",
+    "http://checkip.amazonaws.com",
+]
+
+# 公网IPv6 API列表，按优先级排序
+PUBLIC_IPV6_APIS = [
+    "https://api6.ipify.org/",
+    "https://6.ipw.cn/",
+    "https://api-ipv6.ip.sb/ip",
+    "http://ipv6.icanhazip.com",
+]
+
 
 def default_v4():  # 默认连接外网的ipv4
     s = socket(AF_INET, SOCK_DGRAM)
@@ -60,12 +78,40 @@ def _open(url, reg):
         error(e)
 
 
-def public_v4(url="https://api-ipv4.ip.sb/ip", reg=IPV4_REG):  # 公网IPV4地址
-    return _open(url, reg)
+def _try_multiple_apis(api_list, reg, ip_type):
+    """
+    Try multiple API endpoints until one succeeds
+    """
+    for url in api_list:
+        try:
+            debug("Trying %s API: %s", ip_type, url)
+            result = _open(url, reg)
+            if result:
+                debug("Successfully got %s from %s: %s", ip_type, url, result)
+                return result
+            else:
+                debug("No valid IP found from %s", url)
+        except Exception as e:
+            debug("Failed to get %s from %s: %s", ip_type, url, e)
+    return None
+
+
+def public_v4(url=None, reg=IPV4_REG):  # 公网IPV4地址
+    if url:
+        # 使用指定URL
+        return _open(url, reg)
+    else:
+        # 使用多个API自动重试
+        return _try_multiple_apis(PUBLIC_IPV4_APIS, reg, "IPv4")
 
 
-def public_v6(url="https://api-ipv6.ip.sb/ip", reg=IPV6_REG):  # 公网IPV6地址
-    return _open(url, reg)
+def public_v6(url=None, reg=IPV6_REG):  # 公网IPV6地址
+    if url:
+        # 使用指定URL
+        return _open(url, reg)
+    else:
+        # 使用多个API自动重试
+        return _try_multiple_apis(PUBLIC_IPV6_APIS, reg, "IPv6")
 
 
 def _ip_regex_match(parrent_regex, match_regex):

{ddns-4.1.0b3 → ddns-4.1.0b4}/ddns/util/fileio.py

@@ -23,7 +23,7 @@ def _ensure_directory_exists(file_path):  # type: (str) -> None
         os.makedirs(directory)
 
 
-def read_file_safely(file_path, encoding="utf-8", default=None):  # type: (str, str, str|None) -> str | None
+def read_file_safely(file_path, encoding="utf-8", default=None):  # type: (str, str, str|None) -> str
     """
     Safely read file content with UTF-8 encoding, return None if file doesn't exist or can't be read
 
@@ -37,7 +37,7 @@ def read_file_safely(file_path, encoding="utf-8", default=None):  # type: (str,
     try:
         return read_file(file_path, encoding)
     except Exception:
-        return default
+        return default  # type: ignore
 
 
 def write_file_safely(file_path, content, encoding="utf-8"):  # type: (str, str, str) -> bool

ddns-4.1.0b4/ddns/util/try_run.py

@@ -0,0 +1,37 @@
+# -*- coding:utf-8 -*-
+"""
+Utility: Safe command execution wrapper used across the project.
+Provides a single try_run function with consistent behavior.
+"""
+
+import subprocess
+import sys
+
+
+def try_run(command, logger=None, **kwargs):
+    # type: (list, object, **object) -> str | None
+    """Safely run a subprocess command and return decoded output or None on failure.
+
+    Args:
+        command (list): Command array to execute
+        logger (object, optional): Logger instance for debug output
+        **kwargs: Additional arguments passed to subprocess.check_output
+
+    Returns:
+        str or None: Command output as string, or None if command failed
+
+    - Adds a default timeout=60s on Python 3 to avoid hangs
+    - Decodes output as text via universal_newlines=True
+    - Logs at debug level when logger is provided
+    """
+    try:
+        if sys.version_info[0] >= 3 and "timeout" not in kwargs:
+            kwargs["timeout"] = 60
+        return subprocess.check_output(command, universal_newlines=True, **kwargs)  # type: ignore
+    except Exception as e:  # noqa: BLE001 - broad for subprocess safety
+        if logger is not None:
+            try:
+                logger.debug("Command failed: %s", e)  # type: ignore
+            except Exception:
+                pass
+        return None

{ddns-4.1.0b3 → ddns-4.1.0b4}/ddns.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: ddns
-Version: 4.1.0b3
+Version: 4.1.0b4
 Summary: Dynamic DNS client for multiple providers, supporting IPv4 and IPv6.
 Author-email: NewFuture <python@newfuture.cc>
 License-Expression: MIT
@@ -138,7 +138,7 @@ Dynamic: license-file
   也可使用一键安装脚本自动下载并安装对应平台的二进制：
 
   ```bash
-  curl -fSL https://ddns.newfuture.cc/install.sh | sh
+  curl -#fSL https://ddns.newfuture.cc/install.sh | sh
   ```
   提示：安装到系统目录（如 /usr/local/bin）可能需要 root 或 sudo 权限；若权限不足，可改为 `sudo sh` 运行。
 

{ddns-4.1.0b3 → ddns-4.1.0b4}/ddns.egg-info/SOURCES.txt

@@ -34,6 +34,7 @@ ddns/util/__init__.py
 ddns/util/comment.py
 ddns/util/fileio.py
 ddns/util/http.py
+ddns/util/try_run.py
 tests/test_cache.py
 tests/test_config_cli.py
 tests/test_config_cli_task.py
@@ -44,6 +45,7 @@ tests/test_config_file_remote.py
 tests/test_config_init.py
 tests/test_config_init_multi.py
 tests/test_config_schema_v4_1.py
+tests/test_ip.py
 tests/test_provider__signature.py
 tests/test_provider_alidns.py
 tests/test_provider_aliesa.py

ddns-4.1.0b4/tests/test_ip.py

@@ -0,0 +1,239 @@
+# -*- coding:utf-8 -*-
+"""
+Tests for ddns.ip module including integration tests
+"""
+from __init__ import unittest, patch, MagicMock
+from ddns import ip
+from ddns.__main__ import get_ip
+from ddns.util.http import HttpResponse
+
+
+class TestIpModule(unittest.TestCase):
+    """测试IP获取模块"""
+
+    def setUp(self):
+        """设置测试环境"""
+        self.original_ssl_verify = ip.ssl_verify
+
+    def tearDown(self):
+        """清理测试环境"""
+        ip.ssl_verify = self.original_ssl_verify
+
+    @patch('ddns.ip.request')
+    def test_url_v4_success(self, mock_request):
+        """测试自定义URL获取IPv4 - 成功"""
+        # 模拟成功响应
+        mock_response = MagicMock()
+        mock_response.body = "1.2.3.4"
+        mock_request.return_value = mock_response
+
+        result = ip.public_v4("https://test.example.com/ip")
+
+        self.assertEqual(result, "1.2.3.4")
+        mock_request.assert_called_once_with("GET", "https://test.example.com/ip", verify=ip.ssl_verify, retries=2)
+
+    @patch('ddns.ip.request')
+    def test_url_v6_success(self, mock_request):
+        """测试自定义URL获取IPv6 - 成功"""
+        # 模拟成功响应
+        mock_response = MagicMock()
+        mock_response.body = "2001:db8::1"
+        mock_request.return_value = mock_response
+
+        result = ip.public_v6("https://test.example.com/ipv6")
+
+        self.assertEqual(result, "2001:db8::1")
+        mock_request.assert_called_once_with("GET", "https://test.example.com/ipv6", verify=ip.ssl_verify, retries=2)
+
+    @patch('ddns.ip.request')
+    def test_url_v4_request_failure(self, mock_request):
+        """测试自定义URL获取IPv4 - 请求失败"""
+        # 模拟请求异常
+        mock_request.side_effect = Exception("Network error")
+
+        result = ip.public_v4("https://test.example.com/ip")
+
+        self.assertIsNone(result)
+        mock_request.assert_called_once_with("GET", "https://test.example.com/ip", verify=ip.ssl_verify, retries=2)
+
+    @patch('ddns.ip.request')
+    def test_url_v4_invalid_response(self, mock_request):
+        """测试自定义URL获取IPv4 - 无效响应"""
+        # 模拟无效响应
+        mock_response = MagicMock()
+        mock_response.body = "invalid response"
+        mock_request.return_value = mock_response
+
+        result = ip.public_v4("https://test.example.com/ip")
+
+        self.assertIsNone(result)
+        mock_request.assert_called_once_with("GET", "https://test.example.com/ip", verify=ip.ssl_verify, retries=2)
+
+    @patch('ddns.ip.request')
+    def test_public_v4_multiple_apis_first_success(self, mock_request):
+        """测试公网IPv4获取 - 多个API第一个成功"""
+        # 模拟第一个API成功
+        mock_response = MagicMock()
+        mock_response.body = "1.2.3.4"
+        mock_request.return_value = mock_response
+
+        result = ip.public_v4()
+
+        self.assertEqual(result, "1.2.3.4")
+        # 应该只调用第一个API
+        mock_request.assert_called_once_with("GET", ip.PUBLIC_IPV4_APIS[0], verify=ip.ssl_verify, retries=2)
+
+    @patch('ddns.ip.request')
+    def test_public_v4_multiple_apis_fallback_success(self, mock_request):
+        """测试公网IPv4获取 - 多个API第一个失败第二个成功"""
+        def mock_request_side_effect(method, url, **kwargs):
+            if url == ip.PUBLIC_IPV4_APIS[0]:
+                raise Exception("First API failed")
+            else:
+                mock_response = MagicMock()
+                mock_response.body = "1.2.3.4"
+                return mock_response
+
+        mock_request.side_effect = mock_request_side_effect
+
+        result = ip.public_v4()
+
+        self.assertEqual(result, "1.2.3.4")
+        # 应该调用前两个API
+        self.assertEqual(mock_request.call_count, 2)
+        mock_request.assert_any_call("GET", ip.PUBLIC_IPV4_APIS[0], verify=ip.ssl_verify, retries=2)
+        mock_request.assert_any_call("GET", ip.PUBLIC_IPV4_APIS[1], verify=ip.ssl_verify, retries=2)
+
+    @patch('ddns.ip.request')
+    def test_public_v4_multiple_apis_all_fail(self, mock_request):
+        """测试公网IPv4获取 - 多个API全部失败"""
+        # 模拟所有API都失败
+        mock_request.side_effect = Exception("All APIs failed")
+
+        result = ip.public_v4()
+
+        self.assertIsNone(result)
+        # 应该调用所有API
+        self.assertEqual(mock_request.call_count, len(ip.PUBLIC_IPV4_APIS))
+
+    @patch('ddns.ip.request')
+    def test_public_v6_multiple_apis_first_success(self, mock_request):
+        """测试公网IPv6获取 - 多个API第一个成功"""
+        # 模拟第一个API成功
+        mock_response = MagicMock()
+        mock_response.body = "2001:db8::1"
+        mock_request.return_value = mock_response
+
+        result = ip.public_v6()
+
+        self.assertEqual(result, "2001:db8::1")
+        # 应该只调用第一个API
+        mock_request.assert_called_once_with("GET", ip.PUBLIC_IPV6_APIS[0], verify=ip.ssl_verify, retries=2)
+
+    @patch('ddns.ip.request')
+    def test_public_v6_multiple_apis_fallback_success(self, mock_request):
+        """测试公网IPv6获取 - 多个API第一个失败第二个成功"""
+        def mock_request_side_effect(method, url, **kwargs):
+            if url == ip.PUBLIC_IPV6_APIS[0]:
+                raise Exception("First API failed")
+            else:
+                mock_response = MagicMock()
+                mock_response.body = "2001:db8::1"
+                return mock_response
+
+        mock_request.side_effect = mock_request_side_effect
+
+        result = ip.public_v6()
+
+        self.assertEqual(result, "2001:db8::1")
+        # 应该调用前两个API
+        self.assertEqual(mock_request.call_count, 2)
+        mock_request.assert_any_call("GET", ip.PUBLIC_IPV6_APIS[0], verify=ip.ssl_verify, retries=2)
+        mock_request.assert_any_call("GET", ip.PUBLIC_IPV6_APIS[1], verify=ip.ssl_verify, retries=2)
+
+    def test_public_ipv4_apis_list_exists(self):
+        """测试IPv4 API列表存在并包含所需的API"""
+        expected_apis = [
+            "https://api.ipify.cn",
+            "https://api.ipify.org",
+            "https://4.ipw.cn/",
+            "https://ipinfo.io/ip",
+            "https://api-ipv4.ip.sb/ip",
+            "http://checkip.amazonaws.com",
+        ]
+        self.assertEqual(ip.PUBLIC_IPV4_APIS, expected_apis)
+
+    def test_public_ipv6_apis_list_exists(self):
+        """测试IPv6 API列表存在并包含所需的API"""
+        expected_apis = [
+            "https://api6.ipify.org/",
+            "https://6.ipw.cn/",
+            "https://api-ipv6.ip.sb/ip",
+            "http://ipv6.icanhazip.com",
+        ]
+        self.assertEqual(ip.PUBLIC_IPV6_APIS, expected_apis)
+
+    @patch('ddns.ip.request')
+    def test_get_ip_public_mode_fallback(self, mock_request):
+        """测试通过get_ip使用public模式的自动fallback功能"""
+        # 模拟第一个API失败，第二个成功
+        def mock_request_side_effect(method, url, **kwargs):
+            if "api.ipify.cn" in url:
+                raise Exception("First API failed")
+            elif "api.ipify.org" in url:
+                mock_response = MagicMock()
+                mock_response.body = "1.2.3.4"
+                return mock_response
+            else:
+                raise Exception("Unexpected URL")
+
+        mock_request.side_effect = mock_request_side_effect
+
+        # 使用"public"规则获取IPv4地址
+        result = get_ip("4", ["public"])
+
+        self.assertEqual(result, "1.2.3.4")
+        # 应该调用了前两个API
+        self.assertEqual(mock_request.call_count, 2)
+
+    @patch('ddns.ip.request')
+    def test_get_ip_url_mode_backward_compatibility(self, mock_request):
+        """测试通过get_ip使用url:模式的向后兼容性"""
+        # 模拟成功响应
+        mock_response = MagicMock()
+        mock_response.body = "1.2.3.4"
+        mock_request.return_value = mock_response
+
+        # 使用"url:"规则获取IPv4地址
+        result = get_ip("4", ["url:https://custom.api.com/ip"])
+
+        self.assertEqual(result, "1.2.3.4")
+        # 应该只调用指定的API
+        mock_request.assert_called_once()
+        args, kwargs = mock_request.call_args
+        self.assertEqual(args[1], "https://custom.api.com/ip")
+
+    @patch('ddns.ip.request')
+    def test_get_ip_multiple_rules_limitation(self, mock_request):
+        """测试get_ip的多规则限制 - 当前实现的限制"""
+        # 注意：当前get_ip实现有限制 - 如果一个规则返回None，它不会尝试下一个规则
+        # 只有当规则抛出异常时才会尝试下一个规则
+        # 这是一个已知限制，不在本次功能实现范围内
+
+        # 模拟所有public API都失败（返回无效响应）
+        mock_response = MagicMock()
+        mock_response.body = "invalid response"
+        mock_request.return_value = mock_response
+
+        # 使用多个规则：先尝试public，失败后应该尝试url:指定的API
+        # 但由于当前实现限制，public返回None后不会尝试下一个规则
+        result = get_ip("4", ["public", "url:https://backup.api.com/ip"])
+
+        # 由于当前实现限制，返回None而不是继续尝试backup API
+        self.assertIsNone(result)
+        # 只调用了public APIs，没有调用backup API
+        self.assertEqual(mock_request.call_count, len(ip.PUBLIC_IPV4_APIS))
+
+
+if __name__ == '__main__':
+    unittest.main()

{ddns-4.1.0b3 → ddns-4.1.0b4}/tests/test_scheduler_base.py

@@ -4,7 +4,8 @@ Unit tests for ddns.scheduler._base module
 @author: NewFuture
 """
 
-from __init__ import unittest, patch
+from __init__ import patch, unittest
+
 from ddns.scheduler._base import BaseScheduler
 
 
@@ -58,8 +59,10 @@ class TestBaseScheduler(unittest.TestCase):
 
         command = self.scheduler._build_ddns_command(ddns_args)
 
-        self.assertIsInstance(command, str)
-        self.assertIn("python", command.lower())
+        self.assertIsInstance(command, list)
+        command_str = " ".join(command)
+        self.assertIn("python", command_str.lower())
+        self.assertIn("-m", command)
         self.assertIn("ddns", command)
         self.assertIn("--dns", command)
         self.assertIn("debug", command)
@@ -98,8 +101,9 @@ class TestBaseScheduler(unittest.TestCase):
 
         command = self.scheduler._build_ddns_command(ddns_args)
 
-        self.assertIn("--debug true", command)
-        self.assertIn("--cache true", command)
+        self.assertIn("--debug", command)
+        self.assertIn("true", command)
+        self.assertIn("--cache", command)
 
     def test_build_ddns_command_filters_debug_false(self):
         """Test _build_ddns_command filters out debug=False"""
@@ -107,8 +111,10 @@ class TestBaseScheduler(unittest.TestCase):
 
         command = self.scheduler._build_ddns_command(ddns_args)
 
-        self.assertNotIn("--debug false", command)
-        self.assertIn("--cache true", command)
+        command_str = " ".join(command)
+        self.assertNotIn("--debug false", command_str)
+        self.assertIn("--cache", command)
+        self.assertIn("true", command)
 
     def test_build_ddns_command_with_single_values(self):
         """Test _build_ddns_command with single value arguments"""
@@ -153,7 +159,8 @@ class TestBaseScheduler(unittest.TestCase):
         command = self.scheduler._build_ddns_command(ddns_args)
 
         # Should use sys.executable for Python path
-        self.assertIn("python", command.lower())
+        command_str = " ".join(command)
+        self.assertIn("python", command_str.lower())
 
     def test_build_ddns_command_with_special_characters(self):
         """Test _build_ddns_command handles special characters"""
@@ -161,7 +168,7 @@ class TestBaseScheduler(unittest.TestCase):
 
         command = self.scheduler._build_ddns_command(ddns_args)
 
-        self.assertIsInstance(command, str)
+        self.assertIsInstance(command, list)
         self.assertIn("test-domain.example.com", command)
         self.assertIn("test_token_with_special_chars!@#", command)
 
@@ -192,6 +199,27 @@ class TestBaseScheduler(unittest.TestCase):
         result = self.scheduler.disable()
         self.assertIsInstance(result, bool)
 
+    def test_quote_command_array(self):
+        """Test _quote_command_array method"""
+        # Test basic functionality
+        cmd_array = ["python", "script.py"]
+        result = self.scheduler._quote_command_array(cmd_array)
+        self.assertEqual(result, "python script.py")
+
+        # Test with spaces
+        cmd_array = ["python", "script with spaces.py", "normal_arg"]
+        result = self.scheduler._quote_command_array(cmd_array)
+        self.assertEqual(result, 'python "script with spaces.py" normal_arg')
+
+        # Test with multiple spaced arguments
+        cmd_array = ["python", "-m", "ddns", "--config", "config file.json"]
+        result = self.scheduler._quote_command_array(cmd_array)
+        self.assertEqual(result, 'python -m ddns --config "config file.json"')
+
+        # Test empty array
+        result = self.scheduler._quote_command_array([])
+        self.assertEqual(result, "")
+
 
 if __name__ == "__main__":
     unittest.main()