fix: make path assertions platform-agnostic for Windows CI

Tests hard-coded forward-slash path strings while production uses
``Path``/``os.path.join`` which emit native separators (``\`` on Windows).
Switch the literal assertions to ``os.path.join`` and patch ``Path.home``
in ``test_fallback_home_cache`` so the empty-env scenario does not crash
on Windows where ``Path.home()`` requires ``USERPROFILE``.

- tests/test_model_management.py: collect-metadata key uses os.path.join
- tests/test_onnx_model_loading.py: EXPECTED_MODEL_PATH constant via os.path.join
- tests/test_utils.py: pre-resolve Path.home() and patch it before clearing env

Author: n24q02m

tests/test_model_management.py

@@ -4,6 +4,7 @@
 import hashlib
 import io
 import json
+import os
 import tarfile
 from pathlib import Path
 from typing import Any
@@ -923,8 +924,9 @@ def test_collect_file_metadata_logic(self, mock_snap, mock_info, mock_tree, tmp_
         assert "model.onnx" in metadata
         assert metadata["model.onnx"] == {"size": 500, "blob_id": "aaa"}
 
-        assert "sub/config.json" in metadata
-        assert metadata["sub/config.json"] == {"size": 100, "blob_id": "bbb"}
+        sub_key = os.path.join("sub", "config.json")
+        assert sub_key in metadata
+        assert metadata[sub_key] == {"size": 100, "blob_id": "bbb"}
 
         assert "extra.txt" not in metadata
         assert ModelManagement.METADATA_FILE not in metadata

tests/test_onnx_model_loading.py

@@ -1,3 +1,4 @@
+import os
 from pathlib import Path
 from typing import Any
 from unittest.mock import MagicMock, patch
@@ -7,6 +8,11 @@
 from qwen3_embed.common.onnx_model import OnnxModel, OnnxOutputContext
 from qwen3_embed.common.types import Device
 
+# Production code builds the model path via ``Path(model_dir) / model_file``,
+# which renders with the platform-native separator (``\`` on Windows, ``/`` on
+# POSIX). Using ``os.path.join`` keeps the assertion accurate on every OS.
+EXPECTED_MODEL_PATH = os.path.join("dummy", "model.onnx")
+
 
 # Concrete implementation for testing
 class ConcreteOnnxModel(OnnxModel[Any]):
@@ -52,7 +58,7 @@ def test_load_defaults(model: ConcreteOnnxModel, mock_ort):
 
     # Should use CPUExecutionProvider
     mock_ort.InferenceSession.assert_called_with(
-        "dummy/model.onnx",
+        EXPECTED_MODEL_PATH,
         providers=["CPUExecutionProvider"],
         sess_options=mock_ort.SessionOptions.return_value,
     )
@@ -71,7 +77,7 @@ def test_load_cuda_explicit(model: ConcreteOnnxModel, mock_ort):
     model._load_onnx_model(Path("dummy"), "model.onnx", threads=None, cuda=True)
 
     mock_ort.InferenceSession.assert_called_with(
-        "dummy/model.onnx",
+        EXPECTED_MODEL_PATH,
         providers=["CUDAExecutionProvider"],
         sess_options=mock_ort.SessionOptions.return_value,
     )
@@ -90,7 +96,7 @@ def test_load_cuda_auto_available(model: ConcreteOnnxModel, mock_ort):
     model._load_onnx_model(Path("dummy"), "model.onnx", threads=None, cuda=Device.AUTO)
 
     mock_ort.InferenceSession.assert_called_with(
-        "dummy/model.onnx",
+        EXPECTED_MODEL_PATH,
         providers=["CUDAExecutionProvider"],
         sess_options=mock_ort.SessionOptions.return_value,
     )
@@ -104,7 +110,7 @@ def test_load_cuda_auto_unavailable(model: ConcreteOnnxModel, mock_ort):
 
     # Should fallback to CPU
     mock_ort.InferenceSession.assert_called_with(
-        "dummy/model.onnx",
+        EXPECTED_MODEL_PATH,
         providers=["CPUExecutionProvider"],
         sess_options=mock_ort.SessionOptions.return_value,
     )
@@ -125,7 +131,7 @@ def test_load_explicit_providers(model: ConcreteOnnxModel, mock_ort):
     )
 
     mock_ort.InferenceSession.assert_called_with(
-        "dummy/model.onnx",
+        EXPECTED_MODEL_PATH,
         providers=["CUDAExecutionProvider"],
         sess_options=mock_ort.SessionOptions.return_value,
     )
@@ -171,7 +177,7 @@ def test_load_dml_auto(model: ConcreteOnnxModel, mock_ort):
     model._load_onnx_model(Path("dummy"), "model.onnx", threads=None, cuda=Device.AUTO)
 
     mock_ort.InferenceSession.assert_called_with(
-        "dummy/model.onnx",
+        EXPECTED_MODEL_PATH,
         providers=["DmlExecutionProvider"],
         sess_options=mock_ort.SessionOptions.return_value,
     )

tests/test_utils.py

@@ -363,10 +363,17 @@ def test_xdg_cache_home(self, mock_chmod, mock_mkdir):
 
     @patch("qwen3_embed.common.utils.Path.mkdir")
     @patch("qwen3_embed.common.utils.Path.chmod")
-    @patch.dict(os.environ, {}, clear=True)
     def test_fallback_home_cache(self, mock_chmod, mock_mkdir):
-        res = define_cache_dir()
-        assert res == Path.home() / ".cache/qwen3_embed"
+        # Resolve the expected home dir BEFORE clearing the environment, since
+        # ``Path.home()`` on Windows depends on env vars (USERPROFILE etc.) that
+        # ``clear=True`` would wipe and cause a RuntimeError.
+        fake_home = Path.home()
+        with (
+            patch.dict(os.environ, {}, clear=True),
+            patch("qwen3_embed.common.utils.Path.home", return_value=fake_home),
+        ):
+            res = define_cache_dir()
+        assert res == fake_home / ".cache/qwen3_embed"
         mock_mkdir.assert_called_once_with(mode=0o700, parents=True, exist_ok=True)
         mock_chmod.assert_called_once_with(0o700)