fix(tests): json files cache

Author: marioevz

tests/json_infra/conftest.py

@@ -309,8 +309,9 @@ def pytest_runtest_teardown(item: Item, nextitem: Item) -> None:
     same type or does not belong to the same fixtures file.
     """
     if isinstance(item, FixtureTestItem):
-        if not isinstance(nextitem, FixtureTestItem):
+        if (
+            nextitem is None
+            or not isinstance(nextitem, FixtureTestItem)
+            or item.fixtures_file != nextitem.fixtures_file
+        ):
             item.fixtures_file.clear_data_cache()
-        else:
-            if item.fixtures_file != nextitem.fixtures_file:
-                item.fixtures_file.clear_data_cache()

tests/json_infra/helpers/fixtures.py

@@ -75,7 +75,7 @@ def data(self) -> Dict[str, Any]:
 
     def clear_data_cache(self) -> None:
         """Drop the data cache."""
-        del self.data
+        self.__dict__.pop("data", None)
 
     def collect(
         self: Self,
@@ -85,22 +85,21 @@ def collect(
             loaded_file = self.data
         except Exception:
             return  # Skip *.json files that are unreadable.
-        if not isinstance(loaded_file, dict):
-            return
-        for key, test_dict in loaded_file.items():
-            if not isinstance(test_dict, dict):
-                continue
-            for fixture_type in ALL_FIXTURE_TYPES:
-                if not fixture_type.is_format(test_dict):
+        if isinstance(loaded_file, dict):
+            for key, test_dict in loaded_file.items():
+                if not isinstance(test_dict, dict):
                     continue
-                name = key
-                if "::" in name:
-                    name = name.split("::")[1]
-                yield fixture_type.from_parent(  # type: ignore
-                    parent=self,
-                    name=name,
-                    test_file=str(self.path),
-                    test_key=key,
-                )
+                for fixture_type in ALL_FIXTURE_TYPES:
+                    if not fixture_type.is_format(test_dict):
+                        continue
+                    name = key
+                    if "::" in name:
+                        name = name.split("::")[1]
+                    yield fixture_type.from_parent(  # type: ignore
+                        parent=self,
+                        name=name,
+                        test_file=str(self.path),
+                        test_key=key,
+                    )
         # Make sure we don't keep anything from collection in memory.
         self.clear_data_cache()