fix windows tests

Author: misrasaurabh1

codeflash/code_utils/config_js.py

@@ -81,21 +81,21 @@ def detect_module_root(project_root: Path, package_data: dict[str, Any]) -> str:
         if entry_path and isinstance(entry_path, str):
             parent = Path(entry_path).parent
             if parent != Path() and (project_root / parent).is_dir():
-                return str(parent)
+                return parent.as_posix()
 
     # Check module field (ESM)
     module_field = package_data.get("module")
     if module_field and isinstance(module_field, str):
         parent = Path(module_field).parent
         if parent != Path() and (project_root / parent).is_dir():
-            return str(parent)
+            return parent.as_posix()
 
     # Check main field (CJS)
     main_field = package_data.get("main")
     if main_field and isinstance(main_field, str):
         parent = Path(main_field).parent
         if parent != Path() and (project_root / parent).is_dir():
-            return str(parent)
+            return parent.as_posix()
 
     # Check for src/ directory convention
     if (project_root / "src").is_dir():

codeflash/languages/base.py

@@ -177,9 +177,10 @@ class TestInfo:
     @property
     def full_test_path(self) -> str:
         """Get full test path in pytest format (file::class::function)."""
+        file_path = self.test_file.as_posix()
         if self.test_class:
-            return f"{self.test_file}::{self.test_class}::{self.test_name}"
-        return f"{self.test_file}::{self.test_name}"
+            return f"{file_path}::{self.test_class}::{self.test_name}"
+        return f"{file_path}::{self.test_name}"
 
 
 @dataclass

tests/code_utils/test_config_js.py

@@ -3,6 +3,7 @@
 from __future__ import annotations
 
 import json
+import sys
 from pathlib import Path
 
 import pytest
@@ -72,6 +73,7 @@ def test_returns_none_for_nonexistent_file(self, tmp_path: Path) -> None:
 
         assert result is None
 
+    @pytest.mark.skipif(sys.platform == "win32", reason="chmod doesn't restrict read access on Windows")
     def test_returns_none_for_unreadable_file(self, tmp_path: Path) -> None:
         """Should return None if file cannot be read."""
         package_json = tmp_path / "package.json"

tests/test_languages/test_javascript_instrumentation.py

@@ -35,7 +35,7 @@ def test_line_profiler_generates_init_code(self):
         assert profiler.profiler_var in init_code
         assert "hit" in init_code  # Changed from recordLine to hit
         assert "save" in init_code
-        assert str(output_file) in init_code
+        assert output_file.as_posix() in init_code
 
     def test_line_profiler_instruments_simple_function(self):
         """Test line profiler can instrument a simple function."""
@@ -101,7 +101,7 @@ def test_tracer_generates_init_code(self):
         assert tracer.tracer_var in init_code
         assert "serialize" in init_code
         assert "wrap" in init_code
-        assert str(output_db) in init_code
+        assert output_db.as_posix() in init_code
 
     def test_tracer_instruments_simple_function(self):
         """Test tracer can instrument a simple function."""

tests/test_languages/test_javascript_test_discovery.py

@@ -1003,7 +1003,7 @@ def test_circular_imports(self, js_support):
 
     def test_unicode_in_test_names(self, js_support):
         """Test handling of unicode characters in test names."""
-        with tempfile.NamedTemporaryFile(suffix=".test.js", mode="w", delete=False) as f:
+        with tempfile.NamedTemporaryFile(suffix=".test.js", mode="w", delete=False, encoding="utf-8") as f:
             f.write("""
 test('handles emoji 🎉', () => {});
 describe('日本語テスト', () => {
@@ -1013,7 +1013,7 @@ def test_unicode_in_test_names(self, js_support):
             f.flush()
             file_path = Path(f.name)
 
-            source = file_path.read_text()
+            source = file_path.read_text(encoding="utf-8")
             from codeflash.languages.treesitter_utils import get_analyzer_for_file
             analyzer = get_analyzer_for_file(file_path)
             test_names = js_support._find_jest_tests(source, analyzer)

tests/test_languages/test_language_parity.py

@@ -288,7 +288,7 @@ def js_support():
 
 def write_temp_file(content: str, suffix: str) -> Path:
     """Write content to a temporary file and return the path."""
-    with tempfile.NamedTemporaryFile(suffix=suffix, mode="w", delete=False) as f:
+    with tempfile.NamedTemporaryFile(suffix=suffix, mode="w", delete=False, encoding="utf-8") as f:
         f.write(content)
         f.flush()
         return Path(f.name)