Improve test coverage

codingjoe · Jan 25, 2024 · 251cbd2 · 251cbd2
1 parent 9fbe375
commit 251cbd2
Show file tree

Hide file tree

Showing 17 changed files with 70 additions and 3 deletions.
diff --git a/django_esm/utils.py b/django_esm/utils.py
@@ -102,7 +102,7 @@ def parse_package_json(path: Path = None):
     try:
         module = exports["default"]
         yield from get_static_from_abs_path(
-            str(Path(name) / module),
+            name,
             path / module,
             settings.BASE_DIR / "node_modules",
         )

diff --git a/tests/node_modules/.package-lock.json b/tests/node_modules/.package-lock.json
diff --git a/tests/package.json b/tests/package.json
@@ -9,7 +9,12 @@
     "#htmx": "https://unpkg.com/htmx.org@1.9.10"
   },
   "dependencies": {
+    "deep": "file:packages/deep",
+    "flat": "file:packages/flat",
+    "flat2": "file:packages/flat2",
     "htmx.org": "^1.9.9",
-    "lit": "^3.1.0"
+    "list": "file:packages/list",
+    "lit": "^3.1.0",
+    "string": "file:packages/string"
   }
 }
diff --git a/tests/packages/deep/features/a.js b/tests/packages/deep/features/a.js
diff --git a/tests/packages/deep/features/b.js b/tests/packages/deep/features/b.js
diff --git a/tests/packages/deep/index.js b/tests/packages/deep/index.js
diff --git a/tests/packages/deep/package.json b/tests/packages/deep/package.json
@@ -0,0 +1,10 @@
+{
+  "name": "deep",
+  "exports": {
+    ".": {
+      "import": "./index.js"
+    },
+    "./*": "./*",
+    "./utils": ["./utils/a.js"]
+  }
+}
diff --git a/tests/packages/deep/utils/a.js b/tests/packages/deep/utils/a.js
diff --git a/tests/packages/flat/index.js b/tests/packages/flat/index.js
diff --git a/tests/packages/flat/package.json b/tests/packages/flat/package.json
@@ -0,0 +1,6 @@
+{
+  "name": "flat",
+  "exports": {
+    "default": "./index.js"
+  }
+}
diff --git a/tests/packages/flat2/index.js b/tests/packages/flat2/index.js
diff --git a/tests/packages/flat2/package.json b/tests/packages/flat2/package.json
@@ -0,0 +1,7 @@
+{
+  "name": "flat2",
+  "exports": {
+    "node": "./index.js",
+    "import": "./index.js"
+  }
+}
diff --git a/tests/packages/list/index.js b/tests/packages/list/index.js
diff --git a/tests/packages/list/package.json b/tests/packages/list/package.json
@@ -0,0 +1,4 @@
+{
+  "name": "list",
+  "exports": ["./index.js"]
+}
diff --git a/tests/packages/string/package.json b/tests/packages/string/package.json
@@ -0,0 +1,4 @@
+{
+  "name": "string",
+  "exports": "./index.js"
+}
diff --git a/tests/test_utils.py b/tests/test_utils.py
@@ -24,6 +24,12 @@ def test_parse_dependencies(package_json):
         import_map["@lit/reactive-element"]
         == "@lit/reactive-element/reactive-element.js"
     )
+    assert import_map["string"] == "string/index.js"
+    assert import_map["list/index.js"] == "list/index.js"
+    assert import_map["flat"] == "flat/index.js"
+    assert import_map["flat2"] == "flat2/index.js"
+    assert import_map["deep"] == "deep/index.js"
+    assert import_map["deep/features/a.js"] == "deep/features/a.js"
 
 
 def test_parse_root_package__bad_imports(package_json):

diff --git a/tests/testapp/static/js/index.js.map b/tests/testapp/static/js/index.js.map