test: move pnpm_lockfiles/cases to isolated test directories (#2208)

Author: jbedard

e2e/pnpm_lockfiles/.bazelignore

@@ -1,5 +1,10 @@
 node_modules/
-cases/node_modules
+cases/versionless-patch-v9/node_modules
+cases/docusaurus-direct-peer-v9/node_modules
+cases/isaacs-cliui-v90/node_modules
+cases/override-with-alias-url-v9/node_modules
+cases/docusaurus-direct-peer-v6/node_modules
+cases/tarball-no-url-v54/node_modules
 projects/a/node_modules
 projects/a-types/node_modules
 projects/b/node_modules

e2e/pnpm_lockfiles/MODULE.bazel

@@ -14,12 +14,12 @@ PNPM_LOCK_VERSIONS = [
 
 # Lockfiles with unique test cases
 PNPM_LOCK_TEST_CASES = [
-    "tarball-no-url-v54.yaml",
-    "override-with-alias-url-v9.yaml",
-    "isaacs-cliui-v90.yaml",
-    "docusaurus-direct-peer-v6.yaml",
-    "docusaurus-direct-peer-v9.yaml",
-    "versionless-patch-v9.yaml",
+    "tarball-no-url-v54",
+    "override-with-alias-url-v9",
+    "isaacs-cliui-v90",
+    "docusaurus-direct-peer-v6",
+    "docusaurus-direct-peer-v9",
+    "versionless-patch-v9",
 ]
 
 bazel_dep(name = "aspect_bazel_lib", version = "2.14.0")
@@ -94,7 +94,7 @@ npm = use_extension(
 [
     npm.npm_translate_lock(
         name = lockfile.replace(".yaml", ""),
-        pnpm_lock = "//cases:%s" % lockfile,
+        pnpm_lock = "//cases/%s:pnpm-lock.yaml" % lockfile,
         verify_node_modules_ignored = "//:.bazelignore",
     )
     for lockfile in PNPM_LOCK_TEST_CASES

e2e/pnpm_lockfiles/WORKSPACE

@@ -13,12 +13,12 @@ PNPM_LOCK_VERSIONS = [
 
 # Lockfiles with unique test cases
 PNPM_LOCK_TEST_CASES = [
-    "tarball-no-url-v54.yaml",
-    "override-with-alias-url-v9.yaml",
-    "isaacs-cliui-v90.yaml",
-    "docusaurus-direct-peer-v6.yaml",
-    "docusaurus-direct-peer-v9.yaml",
-    "versionless-patch-v9.yaml",
+    "tarball-no-url-v54",
+    "override-with-alias-url-v9",
+    "isaacs-cliui-v90",
+    "docusaurus-direct-peer-v6",
+    "docusaurus-direct-peer-v9",
+    "versionless-patch-v9",
 ]
 
 load("@aspect_rules_js//js:repositories.bzl", "rules_js_dependencies")
@@ -60,7 +60,7 @@ npm_repositories_v90()
 [
     npm_translate_lock(
         name = lockfile.replace(".yaml", ""),
-        pnpm_lock = "//cases:%s" % lockfile,
+        pnpm_lock = "//cases/%s:pnpm-lock.yaml" % lockfile,
         verify_node_modules_ignored = "//:.bazelignore",
     )
     for lockfile in PNPM_LOCK_TEST_CASES

e2e/pnpm_lockfiles/cases/BUILD.bazel

@@ -0,0 +1,14 @@
+load("@bazel_skylib//rules:build_test.bzl", "build_test")
+load("@docusaurus-direct-peer-v6//:defs.bzl", docusaurus_direct_with_peers_v6_link_all = "npm_link_all_packages")
+
+exports_files(["pnpm-lock.yaml"])
+
+docusaurus_direct_with_peers_v6_link_all()
+
+build_test(
+    name = "docusaurus_direct_with_peers",
+    targets = [
+        ":node_modules",
+        ":node_modules/@docusaurus/module-type-aliases",
+    ],
+)

e2e/pnpm_lockfiles/cases/docusaurus-direct-peer-v6/BUILD.bazel

@@ -0,0 +1,14 @@
+load("@bazel_skylib//rules:build_test.bzl", "build_test")
+load("@docusaurus-direct-peer-v9//:defs.bzl", docusaurus_direct_with_peers_v9_link_all = "npm_link_all_packages")
+
+exports_files(["pnpm-lock.yaml"])
+
+docusaurus_direct_with_peers_v9_link_all()
+
+build_test(
+    name = "docusaurus_direct_with_peers",
+    targets = [
+        ":node_modules",
+        ":node_modules/@docusaurus/module-type-aliases",
+    ],
+)

e2e/pnpm_lockfiles/cases/package.json renamed to e2e/pnpm_lockfiles/cases/docusaurus-direct-peer-v6/package.json

@@ -0,0 +1,5 @@
+{
+    "pnpm": {
+        "onlyBuiltDependencies": []
+    }
+}