Improve temp_file fixture cleanup on Windows

Enhances the temp_file pytest fixture to handle delayed file handle release on Windows by retrying file deletion with a short delay. This prevents test failures due to locked files and provides more robust cleanup across platforms.

Author: SmartManoj

tests/tools/file_editor/conftest.py

@@ -1,4 +1,7 @@
+import os
+import sys
 import tempfile
+import time
 from pathlib import Path
 
 import pytest
@@ -14,11 +17,28 @@
 def temp_file():
     """Create a temporary file for testing."""
     with tempfile.NamedTemporaryFile(delete=False) as f:
-        yield Path(f.name)
-        try:
-            Path(f.name).unlink()
-        except FileNotFoundError:
-            pass
+        file_path = Path(f.name)
+        yield file_path
+        # On Windows, files may be locked briefly after handles are closed.
+        # This is due to Windows file system behavior where file handles may not be
+        # immediately released, opportunistic locks (oplocks), or background processes
+        # like File Explorer holding handles. See:
+        # - https://learn.microsoft.com/en-us/windows-hardware/drivers/ifs/fsctl-opbatch-ack-close-pending
+        # - https://learn.microsoft.com/en-us/answers/questions/5559106/file-explorer-is-opening-files-and-folders-prevent
+        # Retry deletion with a small delay if it fails
+        max_retries = 5 if sys.platform == "win32" else 1
+        for attempt in range(max_retries):
+            try:
+                if file_path.exists():
+                    file_path.unlink()
+                break
+            except (FileNotFoundError, PermissionError):
+                if attempt < max_retries - 1:
+                    time.sleep(0.1)  # Small delay before retry
+                elif attempt == max_retries - 1:
+                    # Last attempt failed, but don't fail the test
+                    # The file will be cleaned up by the OS eventually
+                    pass
 
 
 @pytest.fixture