test: migrate legacy CLI tests to e2e directory

Remove an extra level.

Author: alan-agius4

.github/shared-actions/windows-bazel-test/action.yml

@@ -24,7 +24,7 @@ runs:
     - name: Convert symlinks for Windows host
       shell: pwsh
       run: |
-        $runfiles_dir = "./dist/bin/tests/legacy-cli/${{inputs.test_target_name}}_/${{inputs.test_target_name}}.bat.runfiles"
+        $runfiles_dir = "./dist/bin/tests/${{inputs.test_target_name}}_/${{inputs.test_target_name}}.bat.runfiles"
 
         # Needed for resolution because Aspect/Bazel looks for repositories at `<workspace>/external`.
         # TODO(devversion): consult with Aspect on why this is needed.
@@ -38,5 +38,5 @@ runs:
         E2E_TEMP: ${{ inputs.e2e_temp_dir }}
       run: |
         node ./scripts/windows-testing/parallel-executor.mjs \
-          "./dist/bin/tests/legacy-cli/${{ inputs.test_target_name }}_/${{ inputs.test_target_name }}.bat.runfiles" \
+          "./dist/bin/tests/${{ inputs.test_target_name }}_/${{ inputs.test_target_name }}.bat.runfiles" \
           ${{ inputs.test_target_name }}

.github/workflows/ci.yml

@@ -71,7 +71,7 @@ jobs:
       - name: Install node modules
         run: pnpm install --frozen-lockfile
       - name: Run module and package tests
-        run: pnpm bazel test -- //... -//tests/legacy-cli/...
+        run: pnpm bazel test -- //... -//tests/...
 
   e2e:
     needs: test
@@ -121,7 +121,7 @@ jobs:
         with:
           name: win-e2e-build-artifacts
           path: |
-            dist/bin/tests/legacy-cli/**
+            dist/bin/tests/**
             !**/node_modules/**
           retention-days: 1
           if-no-files-found: 'error'
@@ -144,7 +144,7 @@ jobs:
         uses: actions/download-artifact@37930b1c2abaa49bbe596cd826c3c89aef350131 # v7.0.0
         with:
           name: win-e2e-build-artifacts
-          path: dist/bin/tests/legacy-cli/
+          path: dist/bin/tests/
       - name: Run CLI E2E tests
         uses: ./.github/shared-actions/windows-bazel-test
         with:

.github/workflows/pr.yml

@@ -28,7 +28,7 @@ jobs:
         with:
           filters: |
             snapshots:
-              - 'tests/legacy-cli/e2e/ng-snapshot/package.json'
+              - 'tests/e2e/ng-snapshot/package.json'
 
   lint:
     runs-on: ubuntu-latest
@@ -101,7 +101,7 @@ jobs:
       - name: Install node modules
         run: pnpm install --frozen-lockfile
       - name: Run module and package tests
-        run: pnpm bazel test -- //... -//tests/legacy-cli/...
+        run: pnpm bazel test -- //... -//tests/...
 
   e2e:
     needs: build
@@ -146,7 +146,7 @@ jobs:
         with:
           name: win-e2e-build-artifacts
           path: |
-            dist/bin/tests/legacy-cli/**
+            dist/bin/tests/**
             !**/node_modules/**
           retention-days: 1
           if-no-files-found: 'error'
@@ -163,7 +163,7 @@ jobs:
         uses: actions/download-artifact@37930b1c2abaa49bbe596cd826c3c89aef350131 # v7.0.0
         with:
           name: win-e2e-build-artifacts
-          path: dist/bin/tests/legacy-cli/
+          path: dist/bin/tests/
       - name: Run CLI E2E tests
         uses: ./.github/shared-actions/windows-bazel-test
         with:

.prettierignore

@@ -13,6 +13,6 @@
 /CONTRIBUTING.md
 .yarn/
 dist/
-/tests/legacy-cli/e2e/assets/
+/tests/e2e/assets/
 /tools/test/*.json
 pnpm-lock.yaml

renovate.json

@@ -3,11 +3,11 @@
   "baseBranchPatterns": ["main", "21.0.x"],
   "extends": ["github>angular/dev-infra//renovate-presets/default.json5"],
   "ignoreDeps": ["less"],
-  "ignorePaths": ["tests/legacy-cli/e2e/assets/**", "tests/schematics/update/packages/**"],
+  "ignorePaths": ["tests/e2e/assets/**", "tests/schematics/update/packages/**"],
   "packageRules": [
     {
       "enabled": false,
-      "matchFileNames": ["tests/legacy-cli/e2e/ng-snapshot/package.json"],
+      "matchFileNames": ["tests/e2e/ng-snapshot/package.json"],
       "matchBaseBranches": ["!main"]
     },
     {

tests/BUILD.bazel

@@ -1,11 +1,78 @@
+load("@aspect_bazel_lib//lib:directory_path.bzl", "directory_path")
 load("@npm//:defs.bzl", "npm_link_all_packages")
+load("@npm//:rollup/package_json.bzl", rollup = "bin")
+load("//tools:defaults.bzl", "ts_project")
+load(":e2e.bzl", "e2e_suites")
 
-# Copyright Google Inc. All Rights Reserved.
-#
-# Use of this source code is governed by an MIT-style license that can be
-# found in the LICENSE file at https://angular.dev/license
 package(default_visibility = ["//visibility:public"])
 
 licenses(["notice"])
 
 npm_link_all_packages()
+
+ts_project(
+    name = "runner",
+    testonly = True,
+    srcs = [
+        "e2e_runner.ts",
+    ],
+    deps = [
+        "//:node_modules/@types/node",
+        "//:node_modules/fast-glob",
+        "//packages/angular_devkit/core",
+        "//packages/angular_devkit/core/node",
+        "//tests/e2e/utils",
+    ],
+)
+
+rollup.rollup(
+    name = "runner_bundled",
+    testonly = True,
+    srcs = [
+        "rollup.config.mjs",
+        ":runner",
+        "//:node_modules/@rollup/plugin-alias",
+        "//:node_modules/@rollup/plugin-commonjs",
+        "//:node_modules/@rollup/plugin-json",
+        "//:node_modules/@rollup/plugin-node-resolve",
+        "//:node_modules/fast-glob",
+        "//tests/e2e/initialize",
+        "//tests/e2e/ng-snapshot",
+        "//tests/e2e/setup",
+        "//tests/e2e/tests",
+    ],
+    args = [
+        "--format=cjs",
+        "--config=./rollup.config.mjs",
+    ],
+    chdir = package_name(),
+    out_dirs = ["runner_bundled_out"],
+    progress_message = "Bundling e2e test runner",
+)
+
+directory_path(
+    name = "runner_entrypoint",
+    testonly = True,
+    directory = ":runner_bundled",
+    path = "./e2e_runner.js",
+)
+
+e2e_suites(
+    name = "e2e",
+    data = [
+        ":runner_bundled",
+        "verdaccio.yaml",
+        "verdaccio_auth.yaml",
+
+        # Dynamically loaded.
+        "//tests/e2e/assets",
+        "//:node_modules/verdaccio",
+        "//:node_modules/verdaccio-auth-memory",
+
+        # Extra runtime deps due to bundling issues.
+        # TODO: Clean this up.
+        "//:node_modules/express",
+        "//:node_modules/undici",
+    ],
+    runner = ":runner_entrypoint",
+)