test: coverage

Author: 15r10nk

src/inline_snapshot/_external/_storage.py

@@ -72,10 +72,11 @@ def load(self, location: ExternalLocation) -> Generator[Path]:
 
     @property
     def _external_files(self):
-        if not hasattr(self, "_external_files_cache"):
-            from inline_snapshot._global_state import state
+        from inline_snapshot._global_state import state
+
+        if not hasattr(state(), "_external_files_cache"):
 
-            self._external_files_cache = {}
+            state()._external_files_cache = {}
 
             base_folders = {file.parent for file in state().files_with_snapshots}
 
@@ -85,8 +86,8 @@ def _external_files(self):
 
             for folder in base_folders:
                 for file in folder.rglob("????????-????-????-????-????????????.*"):
-                    self._external_files_cache[file.name] = file
-        return self._external_files_cache
+                    state()._external_files_cache[file.name] = file
+        return state()._external_files_cache
 
     def _lookup_path(self, location: ExternalLocation):
         if location.filename and location.qualname:
@@ -248,12 +249,6 @@ def _lookup_path(self, name) -> Path:
     def lookup_all(self, name: str) -> set[str]:
         return {file.name for file in self.directory.glob(name)}
 
-    def lookup_all_path(self, name: str) -> set[Path]:
-        return {file for file in self.directory.glob(name)}
-
-    def remove(self, name):
-        self._lookup_path(name).unlink()
-
 
 def default_storages(storage_dir: Path):
     return {"hash": HashStorage(storage_dir / "external"), "uuid": UuidStorage()}

tests/external/test_external.py

@@ -26,34 +26,6 @@ def test_basic(check_update):
 #     )
 
 
-def test_max_hash():
-    Example(
-        {
-            "test_something.py": """\
-from inline_snapshot import external
-def test_a():
-    assert "a" == external()
-""",
-            "pyproject.toml": """\
-[tool.inline-snapshot]
-hash-length=64
-""",
-        }
-    ).run_inline(
-        ["--inline-snapshot=create"],
-        changed_files=snapshot(
-            {
-                "__inline_snapshot__/test_something/test_a/e3e70682-c209-4cac-a29f-6fbed82c07cd.txt": "a",
-                "test_something.py": """\
-from inline_snapshot import external
-def test_a():
-    assert "a" == external("uuid:e3e70682-c209-4cac-a29f-6fbed82c07cd.txt")
-""",
-            }
-        ),
-    )
-
-
 def test_outsource():
 
     Example(

tests/external/test_generic.py

@@ -13,6 +13,9 @@ def test_generic(storage):
 def test_a():
     assert "testa".upper()==external("{storage}:")
 """
+    ).run_pytest(
+        ["--inline-snapshot=report"],
+        returncode=snapshot(1),
     ).run_inline(
         ["--inline-snapshot=create"],
         changed_files=snapshot(

tests/external/test_storage_hash.py

@@ -0,0 +1,31 @@
+from inline_snapshot._inline_snapshot import snapshot
+from inline_snapshot.testing._example import Example
+
+
+def test_max_hash():
+    Example(
+        {
+            "test_something.py": """\
+from inline_snapshot import external
+def test_a():
+    assert "a" == external()
+""",
+            "pyproject.toml": """\
+[tool.inline-snapshot]
+hash-length=64
+default-storage="hash"
+""",
+        }
+    ).run_inline(
+        ["--inline-snapshot=create"],
+        changed_files=snapshot(
+            {
+                ".inline-snapshot/external/ca978112ca1bbdcafac231b39a23dc4da786eff8147c4e72b9807785afee48bb.txt": "a",
+                "test_something.py": """\
+from inline_snapshot import external
+def test_a():
+    assert "a" == external("hash:ca978112ca1bbdcafac231b39a23dc4da786eff8147c4e72b9807785afee48bb.txt")
+""",
+            }
+        ),
+    )