windows fixes

Author: jensens

src/mxdev/including.py

@@ -26,7 +26,10 @@ def resolve_dependencies(
         if http_parent:
             file_or_url = parse.urljoin(http_parent, file_or_url)
         parsed = parse.urlparse(str(file_or_url))
-        if parsed.scheme:
+        # Check if it's a real URL scheme (not a Windows drive letter)
+        # Windows drive letters are single characters, URL schemes are longer
+        is_url = parsed.scheme and len(parsed.scheme) > 1
+        if is_url:
             with request.urlopen(str(file_or_url)) as fio:
                 tf = tempfile.NamedTemporaryFile(
                     suffix=".ini",
@@ -54,7 +57,10 @@ def resolve_dependencies(
         include = include.strip()
         if not include:
             continue
-        if http_parent or parse.urlparse(include).scheme:
+        # Check if it's a real URL scheme (not a Windows drive letter)
+        parsed_include = parse.urlparse(include)
+        is_include_url = parsed_include.scheme and len(parsed_include.scheme) > 1
+        if http_parent or is_include_url:
             file_list += resolve_dependencies(include, tmpdir, http_parent)
         else:
             file_list += resolve_dependencies(file.parent / include, tmpdir)

tests/test_including.py

@@ -54,3 +54,21 @@ def test_read_with_included():
     assert cfg["settings"]["unique_2"] == "2"
     assert cfg["settings"]["unique_3"] == "3"
     assert cfg["settings"]["unique_4"] == "4"
+
+
+def test_resolve_dependencies_windows_path(tmp_path):
+    """Test that Windows absolute paths with drive letters are handled correctly.
+
+    On Windows, paths like 'D:\\path\\to\\file.ini' should be treated as
+    file paths, not URLs (even though urlparse() interprets 'D:' as a scheme).
+    """
+    from mxdev.including import resolve_dependencies
+
+    # Create a test file with no includes
+    test_file = tmp_path / "test_config.ini"
+    test_file.write_text("[settings]\ntest = value\n")
+
+    # Test with the actual path (on Windows this would be like D:\...\test_config.ini)
+    file_list = resolve_dependencies(str(test_file), str(tmp_path))
+    assert len(file_list) == 1
+    assert file_list[0] == test_file