3.13 test

Author: OliverBryant

.github/workflows/python.yaml

@@ -88,14 +88,9 @@ jobs:
           - { os: windows-latest, python-version: 3.10 }
           - { os: windows-latest, python-version: 3.11 }
           - { os: windows-latest, python-version: 3.12 }
-          - { os: windows-latest, python-version: 3.9 }
-          - { os: ubuntu-latest, python-version: 3.9 }
-          - { os: ubuntu-latest, python-version: 3.10 }
-          - { os: ubuntu-latest, python-version: 3.11 }
-          - { os: ubuntu-latest, python-version: 3.12 }
-#        include:
-#          - { os: self-hosted, module: gpu, python-version: 3.9}
-#          - { os: macos-latest, module: metal, python-version: "3.10" }
+        include:
+          - { os: self-hosted, module: gpu, python-version: 3.9}
+          - { os: macos-latest, module: metal, python-version: "3.10" }
 
     steps:
       - name: Check out code

xinference/model/embedding/tests/test_embedding_models.py

@@ -264,7 +264,7 @@ def test_register_fault_embedding():
     with open(file_path, "w") as f:
         json.dump(data, f, indent=4)
 
-        # Debug: Verify file was created
+    # Debug: Verify file was created
     print(f"DEBUG: File exists after writing: {os.path.exists(file_path)}")
     if os.path.exists(file_path):
         with open(file_path, "r") as f:
@@ -302,8 +302,10 @@ def custom_warning_handler(
         # Restore original warning handler
         warnings.showwarning = old_showwarning
 
-        # Now run with pytest.warns to capture the official warningswith pytest.warns(UserWarning) as record:
+        # Now run with pytest.warns to capture the official warnings
+        with pytest.warns(UserWarning) as record:
             _install()
+
         print(f"DEBUG: pytest.warns captured {len(record)} warnings")
         for i, warning in enumerate(record):
             print(f"DEBUG: pytest warning {i+1}: {warning.message}")
@@ -317,8 +319,8 @@ def custom_warning_handler(
             print(f"DEBUG: Checking warning message: {message}")
             if (
                 "has error" in message
-                and(
-            "Invalid model URI " in message
+                and (
+                    "Invalid model URI" in message
                     or "Model URI cannot be a relative path" in message
                 )
                 and "/new_data/cache/gte-Qwen2" in message
@@ -329,9 +331,9 @@ def custom_warning_handler(
 
         assert (
             found_warning
-        ), f"Expected warning about invalid model URI not found. Warnings: {[str(w.message)
-            for w in record]}"
-        finally:
+        ), f"Expected warning about invalid model URI not found. Warnings: {[str(w.message) for w in record]}"
+
+    finally:
         # Restore original warning handler
         warnings.showwarning = old_showwarning
 
@@ -342,10 +344,6 @@ def custom_warning_handler(
     if os.path.exists(file_path):
         os.remove(file_path)
         print("DEBUG: Test file cleaned up")
-    finally:
-        # Clean up the test file to avoid interference with subsequent test runs
-        if os.path.exists(file_path):
-            os.remove(file_path)
 
 
 def test_convert_ids_to_tokens():