fix: silently skip duplicate TestFiles.add() instead of raising

Java test discovery adds the same instrumented_behavior_file_path with
different fields. The old Pydantic __eq__ treated them as different;
the set-based dedup correctly identifies them as the same test file.

Since get_test_type_by_instrumented_file_path() returns on first path
match anyway, duplicates by path are dead weight. Silent skip (first
write wins) is both correct and O(1).

Author: KRRT7

codeflash/models/models.py

@@ -436,9 +436,6 @@ def add(self, test_file: TestFile) -> None:
         if key not in self._seen_paths:
             self._seen_paths.add(key)
             self.test_files.append(test_file)
-        else:
-            msg = "Test file already exists in the list"
-            raise ValueError(msg)
 
     def get_by_original_file_path(self, file_path: Path) -> TestFile | None:
         normalized = self._normalize_path_for_comparison(file_path)

tests/test_test_files_add.py

@@ -1,7 +1,5 @@
 from pathlib import Path
 
-import pytest
-
 from codeflash.models.models import TestFile, TestFiles
 from codeflash.models.test_type import TestType
 
@@ -18,16 +16,16 @@ def test_add_unique_test_file(self) -> None:
         assert len(tf.test_files) == 1
         assert tf.test_files[0] is test_file
 
-    def test_add_duplicate_raises(self) -> None:
+    def test_add_duplicate_is_noop(self) -> None:
         tf = TestFiles(test_files=[])
         test_file = TestFile(
             instrumented_behavior_file_path=Path("/tmp/test_behavior.py"),
             benchmarking_file_path=Path("/tmp/test_perf.py"),
             test_type=TestType.GENERATED_REGRESSION,
         )
         tf.add(test_file)
-        with pytest.raises(ValueError, match="Test file already exists"):
-            tf.add(test_file)
+        tf.add(test_file)  # silent skip — first write wins
+        assert len(tf.test_files) == 1
 
     def test_add_many_files_performance(self) -> None:
         tf = TestFiles(test_files=[])