chore: run all e2e tests on CI (#1609)

.aspect/workflows/config.yaml

@@ -18,6 +18,19 @@ workspaces:
                   without: true
             - delivery:
                   without: true
+    e2e/gyp_no_install_script:
+        icon: npm
+        tasks:
+            - test:
+                  queue: aspect-medium
+            - format:
+                  without: true
+            - buildifier:
+                  without: true
+            - configure:
+                  without: true
+            - delivery:
+                  without: true
     # rules_docker not compatible with Bazel 7.
     # See https://github.com/bazelbuild/bazel/issues/20494#issuecomment-1852401451.
     # e2e/js_image_docker:

.github/workflows/ci.yaml

@@ -4,9 +4,9 @@ name: CI
 on:
     # Triggers the workflow on push or pull request events but only for the main branch
     push:
-        branches: [main]
+        branches: [main, 2.x]
     pull_request:
-        branches: [main]
+        branches: [main, 2.x]
 
     # Allows you to run this workflow manually from the Actions tab
     workflow_dispatch:
@@ -55,6 +55,7 @@ jobs:
                   a=(
                     .
                     e2e/bzlmod
+                    e2e/gyp_no_install_script
                     e2e/js_image_docker
                     e2e/js_image_oci
                     e2e/js_run_devserver
@@ -174,31 +175,37 @@ jobs:
                           major: 7
                       bzlmod: 1
                       folder: e2e/update_pnpm_lock_with_import
+                    # gyp_no_install_script is broken in an usual way on 6.5.0
+                    # that is not worth investigating as we're dropping Bazel 6 support soon
+                    - bazel-version:
+                          major: 6
+                      bzlmod: 0
+                      folder: e2e/gyp_no_install_script
                 include:
                     - bazel-version:
                           major: 7
-                          version: 7.0.0
+                          version: 7.1.1
                       bzlmod: 1
                       os: windows
                       config: local
                       folder: e2e/bzlmod
                     - bazel-version:
                           major: 7
-                          version: 7.0.0
+                          version: 7.1.1
                       bzlmod: 1
                       os: macos
                       config: local
                       folder: e2e/bzlmod
                     - bazel-version:
                           major: 6
-                          version: 6.4.0
+                          version: 6.5.0
                       bzlmod: 1
                       os: windows
                       config: local
                       folder: e2e/bzlmod
                     - bazel-version:
                           major: 6
-                          version: 6.4.0
+                          version: 6.5.0
                       bzlmod: 1
                       os: macos
                       config: local

e2e/gyp_no_install_script/MODULE.bazel

@@ -1,5 +1,7 @@
 bazel_dep(name = "aspect_bazel_lib", version = "2.7.0")
 bazel_dep(name = "aspect_rules_js", version = "0.0.0")
+bazel_dep(name = "platforms", version = "0.0.8")
+
 local_path_override(
     module_name = "aspect_rules_js",
     path = "../..",

e2e/patch_from_repo/MODULE.bazel

@@ -24,10 +24,10 @@ npm.npm_translate_lock(
     },
     patches = {
         # TODO: Because `patches` is a string_list_dict that doesn't take labels, we can't
-        # pass in a label like `@local_repo//:....` because only the root repository knows
+        # pass in a label like `@local_repo~//:....` because only the root repository knows
         # how to map this repo to its local override. This causes rules_js to fail when trying
         # to convert the label into a path.
-        "[email protected]": ["@@local_repo~override//:[email protected]"],
+        "[email protected]": ["@@local_repo~//:[email protected]"],
     },
     pnpm_lock = "//:pnpm-lock.yaml",
 )

e2e/vendored_node/WORKSPACE

@@ -7,6 +7,25 @@ load("@aspect_rules_js//js:repositories.bzl", "rules_js_dependencies")
 
 rules_js_dependencies()
 
+load(
+    "@aspect_bazel_lib//lib:repositories.bzl",
+    "register_copy_directory_toolchains",
+    "register_copy_to_directory_toolchains",
+    "register_coreutils_toolchains",
+    "register_tar_toolchains",
+    "register_yq_toolchains",
+)
+
+register_copy_directory_toolchains()
+
+register_copy_to_directory_toolchains()
+
+register_coreutils_toolchains()
+
+register_tar_toolchains()
+
+register_yq_toolchains()
+
 load("@bazel_features//:deps.bzl", "bazel_features_deps")
 
 bazel_features_deps()

e2e/webpack_devserver/MODULE.bazel

@@ -1,3 +1,4 @@
+bazel_dep(name = "aspect_bazel_lib", version = "2.7.0")
 bazel_dep(name = "bazel_skylib", version = "1.5.0")
 bazel_dep(name = "rules_go", version = "0.46.0")
 bazel_dep(name = "aspect_rules_js", version = "0.0.0")

e2e/webpack_devserver_esm/MODULE.bazel

@@ -1,3 +1,4 @@
+bazel_dep(name = "aspect_bazel_lib", version = "2.7.0")
 bazel_dep(name = "bazel_skylib", version = "1.5.0")
 bazel_dep(name = "rules_go", version = "0.46.0")
 bazel_dep(name = "aspect_rules_js", version = "0.0.0")

e2e/worker/WORKSPACE

@@ -15,6 +15,25 @@ nodejs_register_toolchains(
     node_version = "14.20.0",
 )
 
+load(
+    "@aspect_bazel_lib//lib:repositories.bzl",
+    "register_copy_directory_toolchains",
+    "register_copy_to_directory_toolchains",
+    "register_coreutils_toolchains",
+    "register_tar_toolchains",
+    "register_yq_toolchains",
+)
+
+register_copy_directory_toolchains()
+
+register_copy_to_directory_toolchains()
+
+register_coreutils_toolchains()
+
+register_tar_toolchains()
+
+register_yq_toolchains()
+
 load("@bazel_features//:deps.bzl", "bazel_features_deps")
 
 bazel_features_deps()

js/private/test/image/checksum.expected

@@ -1,2 +1,2 @@
-6477f43b8663b593b7eb3ba50f7b617849eca2d2ecccdba37a92b0c9faf1226c  js/private/test/image/cksum_app.tar
+4a332bdfbc2c68da46ff8851a060c04c5e0328c07540c0da992d95e7b0e79ffa  js/private/test/image/cksum_app.tar
 813d54ae63067f19103e13f616726ff1f32cf8d51e27789b8d31415300f0a69e  js/private/test/image/cksum_node_modules.tar

js/private/test/image/custom_owner_app.listing

@@ -10,7 +10,7 @@ drwxr-xr-x 100/0             0 1970-01-01 00:00 app/js/private/test/image/bin.ru
 drwxr-xr-x 100/0             0 1970-01-01 00:00 app/js/private/test/image/bin.runfiles/aspect_rules_js/js/private
 drwxr-xr-x 100/0             0 1970-01-01 00:00 app/js/private/test/image/bin.runfiles/aspect_rules_js/js/private/test
 drwxr-xr-x 100/0             0 1970-01-01 00:00 app/js/private/test/image/bin.runfiles/aspect_rules_js/js/private/test/image
--r-xr-xr-x 100/0         24130 1970-01-01 00:00 app/js/private/test/image/bin.runfiles/aspect_rules_js/js/private/test/image/bin
+-r-xr-xr-x 100/0         23948 1970-01-01 00:00 app/js/private/test/image/bin.runfiles/aspect_rules_js/js/private/test/image/bin
 drwxr-xr-x 100/0             0 1970-01-01 00:00 app/js/private/test/image/bin.runfiles/aspect_rules_js/js/private/test/image/bin_node_bin
 -r-xr-xr-x 100/0           133 1970-01-01 00:00 app/js/private/test/image/bin.runfiles/aspect_rules_js/js/private/test/image/bin_node_bin/node
 -r-xr-xr-x 100/0            20 1970-01-01 00:00 app/js/private/test/image/bin.runfiles/aspect_rules_js/js/private/test/image/main.js

js/private/test/image/default_app.listing

@@ -10,7 +10,7 @@ drwxr-xr-x 0/0               0 1970-01-01 00:00 app/js/private/test/image/bin.ru
 drwxr-xr-x 0/0               0 1970-01-01 00:00 app/js/private/test/image/bin.runfiles/aspect_rules_js/js/private
 drwxr-xr-x 0/0               0 1970-01-01 00:00 app/js/private/test/image/bin.runfiles/aspect_rules_js/js/private/test
 drwxr-xr-x 0/0               0 1970-01-01 00:00 app/js/private/test/image/bin.runfiles/aspect_rules_js/js/private/test/image
--r-xr-xr-x 0/0           24130 1970-01-01 00:00 app/js/private/test/image/bin.runfiles/aspect_rules_js/js/private/test/image/bin
+-r-xr-xr-x 0/0           23948 1970-01-01 00:00 app/js/private/test/image/bin.runfiles/aspect_rules_js/js/private/test/image/bin
 drwxr-xr-x 0/0               0 1970-01-01 00:00 app/js/private/test/image/bin.runfiles/aspect_rules_js/js/private/test/image/bin_node_bin
 -r-xr-xr-x 0/0             133 1970-01-01 00:00 app/js/private/test/image/bin.runfiles/aspect_rules_js/js/private/test/image/bin_node_bin/node
 -r-xr-xr-x 0/0              20 1970-01-01 00:00 app/js/private/test/image/bin.runfiles/aspect_rules_js/js/private/test/image/main.js