chore: fix test links and add coverage report (#2393)

Author: tatomyr

.github/workflows/tests.yaml

@@ -26,6 +26,12 @@ jobs:
         run: npm run typecheck
       - name: Unit Tests
         run: npm run unit
+
+      - name: Coverage Report
+        if: always() # Also generate the report if tests are failing
+        continue-on-error: true # Do not fail if there is an error during reporting
+        uses: davelosert/vitest-coverage-report-action@v2
+
       - name: E2E Tests
         run: npm run e2e
 
@@ -48,16 +54,6 @@ jobs:
       - name: Definition test
         run: redocly lint resources/rebilly.yaml --format=stylish
 
-  coverage-report:
-    runs-on: ubuntu-latest
-    steps:
-      - uses: actions/checkout@v4
-      - uses: actions/setup-node@v3
-        with:
-          node-version: 24
-          cache: npm
-      # FIXME: add vitest coverage report (2.0)
-
   code-style-check:
     runs-on: ubuntu-latest
     steps:

tests/e2e/commands.test.ts

@@ -9,8 +9,7 @@ export function getParams(indexEntryPoint: string, args: string[] = []): string[
 
 export function getCommandOutput(
   args: string[],
-  env?: Record<string, string>,
-  options?: { testPath?: string }
+  options?: { env?: Record<string, string>; testPath?: string }
 ) {
   const result = spawnSync('node', args, {
     encoding: 'utf-8',
@@ -20,7 +19,7 @@ export function getCommandOutput(
       NODE_ENV: 'test',
       NO_COLOR: 'TRUE',
       FORCE_COLOR: '0',
-      ...env,
+      ...options?.env,
     },
     cwd: options?.testPath,
   });

tests/e2e/helpers.ts

@@ -7,7 +7,7 @@ apis:
     rules:
       info-license: error
   test@external-url:
-    root: https://raw.githubusercontent.com/Redocly/redocly-cli/refs/heads/main/__tests__/miscellaneous/apply-per-api-decorators/nested/openapi/main.yaml
+    root: https://raw.githubusercontent.com/Redocly/redocly-cli/refs/heads/main/tests/e2e/miscellaneous/apply-per-api-decorators/nested/openapi/main.yaml
     rules:
       info-contact: error
 plugins:

tests/e2e/miscellaneous/apply-per-api-decorators/nested/redocly.yaml

@@ -15,5 +15,5 @@ components:
 bundling nested/openapi/main.yaml using configuration for api 'test@fs'...
 📦 Created a bundle for nested/openapi/main.yaml at stdout <test>ms.
 🧹 Removed 1 unused components.
-bundling https://raw.githubusercontent.com/Redocly/redocly-cli/refs/heads/main/__tests__/miscellaneous/apply-per-api-decorators/nested/openapi/main.yaml using configuration for api 'test@external-url'...
-📦 Created a bundle for https://raw.githubusercontent.com/Redocly/redocly-cli/refs/heads/main/__tests__/miscellaneous/apply-per-api-decorators/nested/openapi/main.yaml at stdout <test>ms.
+bundling https://raw.githubusercontent.com/Redocly/redocly-cli/refs/heads/main/tests/e2e/miscellaneous/apply-per-api-decorators/nested/openapi/main.yaml using configuration for api 'test@external-url'...
+📦 Created a bundle for https://raw.githubusercontent.com/Redocly/redocly-cli/refs/heads/main/tests/e2e/miscellaneous/apply-per-api-decorators/nested/openapi/main.yaml at stdout <test>ms.

tests/e2e/miscellaneous/apply-per-api-decorators/snapshot.txt

@@ -1,4 +1,4 @@
-[1] https://raw.githubusercontent.com/Redocly/redocly-cli/refs/heads/main/__tests__/miscellaneous/apply-per-api-decorators/nested/openapi/main.yaml:2:1 at #/info/contact
+[1] https://raw.githubusercontent.com/Redocly/redocly-cli/refs/heads/main/tests/e2e/miscellaneous/apply-per-api-decorators/nested/openapi/main.yaml:2:1 at #/info/contact
 
 Info object should contain `contact` field.
 
@@ -12,8 +12,8 @@ Error was generated by the info-contact rule.
 
 
 
-validating https://raw.githubusercontent.com/Redocly/redocly-cli/refs/heads/main/__tests__/miscellaneous/apply-per-api-decorators/nested/openapi/main.yaml using lint rules for api 'test@external-url'...
-https://raw.githubusercontent.com/Redocly/redocly-cli/refs/heads/main/__tests__/miscellaneous/apply-per-api-decorators/nested/openapi/main.yaml: validated in <test>ms
+validating https://raw.githubusercontent.com/Redocly/redocly-cli/refs/heads/main/tests/e2e/miscellaneous/apply-per-api-decorators/nested/openapi/main.yaml using lint rules for api 'test@external-url'...
+https://raw.githubusercontent.com/Redocly/redocly-cli/refs/heads/main/tests/e2e/miscellaneous/apply-per-api-decorators/nested/openapi/main.yaml: validated in <test>ms
 
 ❌ Validation failed with 1 error.
 run `redocly lint --generate-ignore-file` to add all problems to the ignore file.

tests/e2e/miscellaneous/apply-per-api-decorators/snapshot_3.txt

@@ -16,7 +16,9 @@ test('should quit an infinite loop on REDOCLY_CLI_RESPECT_MAX_STEPS', () => {
   ]);
 
   const result = getCommandOutput(args, {
-    REDOCLY_CLI_RESPECT_MAX_STEPS: '10',
+    env: {
+      REDOCLY_CLI_RESPECT_MAX_STEPS: '10',
+    },
   });
 
   expect(result).toMatchSnapshot();

tests/e2e/respect/max-steps/max-steps.test.ts

@@ -4,7 +4,7 @@
   "description": "Test benchmark for Redocly CLI",
   "private": true,
   "scripts": {
-    "chart": "node chart.mjs > benchmark_chart.md",
+    "chart": "node chart.js > benchmark_chart.md",
     "make-test": "bash make-test-command.sh"
   },
   "dependencies": {

tests/performance/chart.mjs tests/performance/chart.jstests/performance/chart.mjs renamed to tests/performance/chart.js

@@ -8,6 +8,8 @@ const configExtension: { [key: string]: ViteUserConfig } = {
       include: ['packages/*/src/**/*.test.ts'],
       coverage: {
         enabled: true,
+        reporter: ['text', 'json-summary', 'json'],
+        reportOnFailure: true,
         include: [
           'packages/cli/src/**/*.ts',
           'packages/core/src/**/*.ts',