build: slim down npm tasks and make them more consistent

Notable changes here:

- Make testing targets more consistent. E.g. no more `npm run e2e`,
- Slim down how many tasks we have, including by getting rid of
  `test:ci` tasks. We already pass additional flags to Vitest in several
  CI workflows, so I don't see the benefit in maintaining these tasks.
- Don't automatically run `test:init`. We already manually run it in the
  integration CI workflow, so I'm following that workflow's lead.
  (Hopefully) soon we'll incorporate these setup steps into the tests
  themselves so you don't have to run any pre-test setup steps at all.
- Remove the explicit Vitest reporter. (This will make Vitest
  automatically use the GitHub Actions reporter in CI.)

Author: ndhoule

.github/workflows/e2e-tests.yml

@@ -49,7 +49,7 @@ jobs:
         run: npm run build
 
       - name: Run E2E tests
-        run: npm run e2e
+        run: npm run test:e2e
 
       - name: Notify Slack
         uses: 8398a7/action-slack@v3
@@ -103,7 +103,7 @@ jobs:
         run: npm run build
 
       - name: Run E2E tests
-        run: npm run e2e
+        run: npm run test:e2e
 
       - name: Notify Slack
         uses: 8398a7/action-slack@v3

.github/workflows/integration-tests.yml

@@ -76,7 +76,7 @@ jobs:
           timeout_minutes: 15
           max_attempts: 3
           retry_on: error
-          command: npm run test:ci:vitest:integration -- --shard=${{ matrix.shard }}
+          command: npm run test:integration -- --coverage --shard=${{ matrix.shard }}
         env:
           # GitHub secrets are not available when running on PR from forks
           # We set a flag so we can skip tests that access Netlify API
@@ -169,7 +169,7 @@ jobs:
           timeout_minutes: 15
           max_attempts: 3
           retry_on: error
-          command: npm exec vitest -- run tests/integration/commands/dev/dev.test.ts
+          command: npm exec vitest -- run tests/integration/commands/dev/dev.test.ts --coverage
         env:
           # GitHub secrets are not available when running on PR from forks
           # We set a flag so we can skip tests that access Netlify API

.github/workflows/release-please.yml

@@ -52,7 +52,7 @@ jobs:
         run: npm run build
 
       - name: Run E2E tests
-        run: npm run e2e
+        run: npm run test:e2e
 
       - name: Publish package
         run: npm publish --provenance

.github/workflows/unit-tests.yml

@@ -45,4 +45,4 @@ jobs:
           timeout_minutes: 15
           max_attempts: 3
           retry_on: error
-          command: npm run test:ci:vitest:unit
+          command: npm run test:unit -- --coverage

package-lock.json

@@ -36,30 +36,26 @@
     "clean": "rm -rf dist/",
     "dev": "tsc --project tsconfig.build.json --watch",
     "docs": "npm run --prefix=site build",
-    "e2e": "vitest run --config vitest.e2e.config.ts",
-    "e2e:debug": "DEBUG='netlify-cli:test' npm run e2e",
     "format": "npm run _format -- --write",
     "format:check": "npm run _format -- --check",
-    "start": "node ./bin/run.js",
-    "test": "npm run test:dev",
     "lint": "eslint --cache \"{src,scripts,tests,.github}/**/*.{mjs,cjs,js,md,html}\" \"*.{mjs,cjs,js,md,html}\"",
     "lint:fix": "npm run lint --fix",
-    "test:dev": "run-s test:init:* test:dev:*",
-    "test:dev:vitest": "vitest run tests/unit/ && vitest run tests/integration",
+    "start": "node ./bin/run.js",
+    "test": "run-s test:unit test:integration test:e2e",
+    "test:e2e": "vitest run --config vitest.e2e.config.ts",
     "test:init": "run-s test:init:*",
     "test:init:cli-help": "npm run start -- --help",
     "test:init:cli-version": "npm run start -- --version",
     "test:init:eleventy-deps": "cd tests/integration/__fixtures__/eleventy-site && pnpm install --frozen-lockfile",
     "test:init:hugo-deps": "npm ci --prefix tests/integration/__fixtures__/hugo-site --no-audit",
     "test:init:monorepo-deps": "cd tests/integration/__fixtures__/monorepo && pnpm install --frozen-lockfile",
     "test:init:next-deps": "npm ci --prefix tests/integration/__fixtures__/next-app-without-config --no-audit && npm ci --prefix tests/integration/__fixtures__/next-app --no-audit",
-    "test:ci:vitest:integration": "vitest run --coverage tests/integration/",
-    "test:ci:vitest:unit": "vitest run --coverage tests/unit/",
+    "test:integration": "vitest run tests/integration/",
+    "test:unit": "vitest run tests/unit/",
     "postinstall": "node ./scripts/postinstall.js",
     "prepublishOnly": "npm shrinkwrap",
     "typecheck": "tsc",
-    "typecheck:watch": "tsc --watch",
-    "watch": "c8 --reporter=lcov vitest --watch"
+    "typecheck:watch": "tsc --watch"
   },
   "dependencies": {
     "@fastify/static": "7.0.4",

package.json

@@ -1,5 +1,3 @@
-import process from 'process'
-
 import { defineConfig } from 'vitest/config'
 
 export default defineConfig({
@@ -19,7 +17,6 @@ export default defineConfig({
         singleThread: true,
       },
     },
-    reporters: [process.env.DEBUG_TESTS ? 'tap' : 'default'],
     coverage: {
       provider: 'v8',
       reporter: ['text', 'lcov'],