tests: fix e2e network

ozapotichnyi · ozapotichnyi · commit 3e45e3b8f73e · 2025-03-20T16:38:00.000+02:00
diff --git a/.github/workflows/tests.yaml b/.github/workflows/tests.yaml
@@ -22,60 +22,57 @@ jobs:
           cache: 'npm'
       - name: Install dependencies
         run: npm ci
-#      - name: Typecheck
-#        run: npm run typecheck
-#      - name: Unit Tests
-#        run: npm run unit
+      - name: Typecheck
+        run: npm run typecheck
+      - name: Unit Tests
+        run: npm run unit
       - name: E2E Tests
-        run: |
-          for (( i = 0; i < 50; i++ )); do
-            npx jest __tests__/respect/workflow-success-actions/workflow-success-actions.test.ts __tests__/respect/workflow-failure-actions/workflow-failure-actions.test.ts
-          done
+        run: npm run e2e
 
-#  cli-package-test:
-#    runs-on: ubuntu-latest
-#    steps:
-#      - uses: actions/checkout@v3
-#      - uses: actions/setup-node@v3
-#        with:
-#          node-version: 20
-#          cache: 'npm'
-#      - name: Install dependencies
-#        run: npm ci
-#      - name: Prepare CLI and Core packages
-#        run: npm run pack:prepare
-#      - name: Install CLI Package
-#        run: npm install -g redocly-cli.tgz
-#      - name: Redocly version
-#        run: redocly --version
-#      - name: Definition test
-#        run: redocly lint packages/core/src/benchmark/benches/rebilly.yaml --format=stylish
-#
-#  coverage-report:
-#    runs-on: ubuntu-latest
-#    steps:
-#      - uses: actions/checkout@v3
-#      - uses: actions/setup-node@v3
-#        with:
-#          node-version: 20
-#          cache: 'npm'
-#      - name: Coverage Report
-#        uses: artiomtr/jest-coverage-report-action@v2.0.8
-#        continue-on-error: true
-#        with:
-#          skip-step: none
-#          annotations: none
-#          test-script: npm run jest
-#
-#  code-style-check:
-#    runs-on: ubuntu-latest
-#    steps:
-#      - uses: actions/checkout@v3
-#      - uses: actions/setup-node@v3
-#        with:
-#          node-version: 20
-#          cache: 'npm'
-#      - name: Install dependencies
-#        run: npm ci
-#      - run: npm run eslint
-#      - run: npm run prettier:check
+  cli-package-test:
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v3
+      - uses: actions/setup-node@v3
+        with:
+          node-version: 20
+          cache: 'npm'
+      - name: Install dependencies
+        run: npm ci
+      - name: Prepare CLI and Core packages
+        run: npm run pack:prepare
+      - name: Install CLI Package
+        run: npm install -g redocly-cli.tgz
+      - name: Redocly version
+        run: redocly --version
+      - name: Definition test
+        run: redocly lint packages/core/src/benchmark/benches/rebilly.yaml --format=stylish
+
+  coverage-report:
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v3
+      - uses: actions/setup-node@v3
+        with:
+          node-version: 20
+          cache: 'npm'
+      - name: Coverage Report
+        uses: artiomtr/jest-coverage-report-action@v2.0.8
+        continue-on-error: true
+        with:
+          skip-step: none
+          annotations: none
+          test-script: npm run jest
+
+  code-style-check:
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v3
+      - uses: actions/setup-node@v3
+        with:
+          node-version: 20
+          cache: 'npm'
+      - name: Install dependencies
+        run: npm ci
+      - run: npm run eslint
+      - run: npm run prettier:check
diff --git a/__tests__/respect/case-insensitive-headers/case-insensitive-headers.test.ts b/__tests__/respect/case-insensitive-headers/case-insensitive-headers.test.ts
@@ -1,11 +1,12 @@
 import { join } from 'path';
-import { getCommandOutput, getParams } from '../utils';
+import { getParams } from '../utils';
+import { getCommandOutput } from '../../helpers';
 
 test('should send in request and proceed case-insensitive headers in runtime expressions', () => {
   const indexEntryPoint = join(process.cwd(), 'packages/cli/lib/index.js');
   const fixturesPath = join(__dirname, 'case-insensitive-headers.arazzo.yaml');
   const args = getParams(indexEntryPoint, ['respect', fixturesPath]);
 
-  const result = getCommandOutput(args);
+  const result = getCommandOutput(args, process.cwd());
   expect(result).toMatchSnapshot();
 });
diff --git a/__tests__/respect/cat-fact-api/cat-fact-api.test.ts b/__tests__/respect/cat-fact-api/cat-fact-api.test.ts
@@ -1,5 +1,5 @@
 import { join } from 'path';
-import { getParams, getCommandOutput, cleanColors } from '../utils';
+import { getParams, cleanColors, getCommandOutput } from '../utils';
 
 test('cats api test case', () => {
   const indexEntryPoint = join(process.cwd(), 'packages/cli/lib/index.js');
diff --git a/__tests__/respect/consider-severity-in-next-step-execution/consider-severity-in-next-step-execution.test.ts b/__tests__/respect/consider-severity-in-next-step-execution/consider-severity-in-next-step-execution.test.ts
@@ -1,4 +1,5 @@
-import { getParams, getCommandOutput } from '../utils';
+import { getParams } from '../utils';
+import { getCommandOutput } from '../../helpers';
 import { join } from 'path';
 
 test('should not follow the default behavior to break and return if onFailure omitted with warn severity and continue execution of the next step', () => {
@@ -18,6 +19,6 @@ test('should not follow the default behavior to break and return if onFailure om
     'CONTENT_TYPE_CHECK=off',
   ]);
 
-  const result = getCommandOutput(args);
+  const result = getCommandOutput(args, process.cwd());
   expect(result).toMatchSnapshot();
 });
diff --git a/__tests__/respect/free-apis/free-apis.test.ts b/__tests__/respect/free-apis/free-apis.test.ts
@@ -1,11 +1,12 @@
 import { join } from 'path';
-import { getParams, getCommandOutput } from '../utils';
+import { getParams } from '../utils';
+import { getCommandOutput } from '../../helpers';
 
 test('free apis test case', () => {
   const indexEntryPoint = join(process.cwd(), 'packages/cli/lib/index.js');
   const fixturesPath = join(__dirname, 'free.arazzo.yaml');
   const args = getParams(indexEntryPoint, ['respect', fixturesPath]);
 
-  const result = getCommandOutput(args);
+  const result = getCommandOutput(args, process.cwd());
   expect(result).toMatchSnapshot();
 });
diff --git a/__tests__/respect/implicit/implicit.test.ts b/__tests__/respect/implicit/implicit.test.ts
@@ -1,4 +1,5 @@
-import { getParams, getCommandOutput } from '../utils';
+import { getParams } from '../utils';
+import { getCommandOutput } from '../../helpers';
 import { join } from 'path';
 
 test('should implicitly add content type header based on requestBody.content field (the first one) if such does not specified', () => {
@@ -11,6 +12,6 @@ test('should implicitly add content type header based on requestBody.content fie
     '--verbose',
   ]);
 
-  const result = getCommandOutput(args);
+  const result = getCommandOutput(args, process.cwd());
   expect(result).toMatchSnapshot();
 });
diff --git a/__tests__/respect/inputs-with-cli-and-env/inputs-with-cli-and-env.test.ts b/__tests__/respect/inputs-with-cli-and-env/inputs-with-cli-and-env.test.ts
@@ -1,4 +1,5 @@
-import { getParams, getCommandOutput } from '../utils';
+import { getParams } from '../utils';
+import { getCommandOutput } from '../../helpers';
 import { join } from 'path';
 
 test('should use inputs from CLI and env', () => {
@@ -14,7 +15,7 @@ test('should use inputs from CLI and env', () => {
     '{"username":"John","password":"password"}',
   ]);
 
-  const result = getCommandOutput(args);
+  const result = getCommandOutput(args, process.cwd());
   expect(result).toMatchSnapshot();
 
   delete process.env.AUTH_TOKEN;
diff --git a/__tests__/respect/local-json-server/local-json-server.test.ts b/__tests__/respect/local-json-server/local-json-server.test.ts
@@ -1,5 +1,6 @@
 import { spawn } from 'child_process';
-import { getParams, getCommandOutput } from '../utils';
+import { getParams } from '../utils';
+import { getCommandOutput } from '../../helpers';
 import { join } from 'path';
 import * as fs from 'fs';
 
@@ -32,7 +33,7 @@ describe('local-json-server', () => {
     const fixturesPath = join(__dirname, 'local-json-server.arazzo.yaml');
     const args = getParams(indexEntryPoint, ['respect', fixturesPath]);
 
-    const result = getCommandOutput(args);
+    const result = getCommandOutput(args, process.cwd());
     expect(result).toMatchSnapshot();
   });
 });
diff --git a/__tests__/respect/mask-input-secrets/mask-input-secrets.test.ts b/__tests__/respect/mask-input-secrets/mask-input-secrets.test.ts
@@ -1,4 +1,5 @@
-import { getParams, getCommandOutput } from '../utils';
+import { getParams } from '../utils';
+import { getCommandOutput } from '../../helpers';
 import { join } from 'path';
 
 test('should hide sensitive input values', () => {
@@ -14,7 +15,7 @@ test('should hide sensitive input values', () => {
     '{"username":"John","password":"password","secret": {"secretValue":"secretToken"}}',
   ]);
 
-  const result = getCommandOutput(args);
+  const result = getCommandOutput(args, process.cwd());
   expect(result).toMatchSnapshot();
 
   delete process.env.AUTH_TOKEN;
diff --git a/__tests__/respect/outputs-access-syntax-variations/outputs-access-syntax-variations.test.ts b/__tests__/respect/outputs-access-syntax-variations/outputs-access-syntax-variations.test.ts
@@ -1,11 +1,12 @@
-import { getParams, getCommandOutput } from '../utils';
+import { getParams } from '../utils';
+import { getCommandOutput } from '../../helpers';
 import { join } from 'path';
 
 test('should resolve outputs access syntax variations', () => {
   const indexEntryPoint = join(process.cwd(), 'packages/cli/lib/index.js');
   const fixturesPath = join(__dirname, 'outputs-access-syntax-variations.arazzo.yaml');
   const args = getParams(indexEntryPoint, ['respect', fixturesPath, '--verbose']);
 
-  const result = getCommandOutput(args);
+  const result = getCommandOutput(args, process.cwd());
   expect(result).toMatchSnapshot();
 });
diff --git a/__tests__/respect/replacements/replacements.test.ts b/__tests__/respect/replacements/replacements.test.ts
@@ -1,11 +1,12 @@
-import { getParams, getCommandOutput } from '../utils';
+import { getParams } from '../utils';
+import { getCommandOutput } from '../../helpers';
 import { join } from 'path';
 
 test('should replace values in the request body', () => {
   const indexEntryPoint = join(process.cwd(), 'packages/cli/lib/index.js');
   const fixturesPath = join(__dirname, 'replacements.arazzo.yaml');
   const args = getParams(indexEntryPoint, ['respect', fixturesPath, '--verbose']);
 
-  const result = getCommandOutput(args);
+  const result = getCommandOutput(args, process.cwd());
   expect(result).toMatchSnapshot();
 });
diff --git a/__tests__/respect/reusable-components/reusable-components.test.ts b/__tests__/respect/reusable-components/reusable-components.test.ts
@@ -1,4 +1,5 @@
-import { getParams, getCommandOutput } from '../utils';
+import { getParams } from '../utils';
+import { getCommandOutput } from '../../helpers';
 import { join } from 'path';
 
 test('should use inputs from CLI and env to mapp with resolved refs', () => {
@@ -13,7 +14,7 @@ test('should use inputs from CLI and env to mapp with resolved refs', () => {
     '--input',
     'reusable-test="123"',
   ]);
-  const result = getCommandOutput(args);
+  const result = getCommandOutput(args, process.cwd());
 
   expect(result).toMatchSnapshot();
 });
diff --git a/__tests__/respect/server-override-with-console-parameters/server-override-with-console-parameters.test.ts b/__tests__/respect/server-override-with-console-parameters/server-override-with-console-parameters.test.ts
@@ -1,4 +1,5 @@
-import { getParams, getCommandOutput } from '../utils';
+import { getParams } from '../utils';
+import { getCommandOutput } from '../../helpers';
 import { join } from 'path';
 
 // Snapshot is intentionally should show failed request to museum-api-bad-endpoint
@@ -10,7 +11,7 @@ test('should use server override from CLI and env', () => {
   const fixturesPath = join(__dirname, 'server-override-with-console-parameters.arazzo.yaml');
   const args = getParams(indexEntryPoint, ['respect', fixturesPath, '--verbose']);
 
-  const result = getCommandOutput(args);
+  const result = getCommandOutput(args, process.cwd());
   expect(result).toMatchSnapshot();
 
   delete process.env.REDOCLY_CLI_RESPECT_SERVER;
diff --git a/__tests__/respect/severity-error-level/severity-error-level.test.ts b/__tests__/respect/severity-error-level/severity-error-level.test.ts
@@ -1,4 +1,5 @@
-import { getParams, getCommandOutput } from '../utils';
+import { getParams } from '../utils';
+import { getCommandOutput } from '../../helpers';
 import { join } from 'path';
 
 test('should use error severity level', () => {
@@ -18,6 +19,6 @@ test('should use error severity level', () => {
     'CONTENT_TYPE_CHECK=error',
   ]);
 
-  const result = getCommandOutput(args);
+  const result = getCommandOutput(args, process.cwd());
   expect(result).toMatchSnapshot();
 });
diff --git a/__tests__/respect/severity-off-level/severity-off-level.test.ts b/__tests__/respect/severity-off-level/severity-off-level.test.ts
@@ -1,4 +1,5 @@
-import { getParams, getCommandOutput } from '../utils';
+import { getParams } from '../utils';
+import { getCommandOutput } from '../../helpers';
 import { join } from 'path';
 
 test('should use off severity level', () => {
@@ -18,6 +19,6 @@ test('should use off severity level', () => {
     'CONTENT_TYPE_CHECK=off',
   ]);
 
-  const result = getCommandOutput(args);
+  const result = getCommandOutput(args, process.cwd());
   expect(result).toMatchSnapshot();
 });
diff --git a/__tests__/respect/severity-warn-level/severity-warn-level.test.ts b/__tests__/respect/severity-warn-level/severity-warn-level.test.ts
@@ -1,4 +1,5 @@
-import { getParams, getCommandOutput } from '../utils';
+import { getParams } from '../utils';
+import { getCommandOutput } from '../../helpers';
 import { join } from 'path';
 
 test('should use warn severity level', () => {
@@ -18,6 +19,6 @@ test('should use warn severity level', () => {
     'CONTENT_TYPE_CHECK=warn',
   ]);
 
-  const result = getCommandOutput(args);
+  const result = getCommandOutput(args, process.cwd());
   expect(result).toMatchSnapshot();
 });
diff --git a/__tests__/respect/step-on-failure-type-end-action/step-on-failure-type-end-action.test.ts b/__tests__/respect/step-on-failure-type-end-action/step-on-failure-type-end-action.test.ts
@@ -1,11 +1,12 @@
-import { getParams, getCommandOutput } from '../utils';
+import { getParams } from '../utils';
+import { getCommandOutput } from '../../helpers';
 import { join } from 'path';
 // Snapshot should have two workflows, and first workflow should run only first step
 test('should end workflow execution, context returns to the caller with applicable outputs, when step fails and onFailure action is of type `end`', () => {
   const indexEntryPoint = join(process.cwd(), 'packages/cli/lib/index.js');
   const fixturesPath = join(__dirname, 'step-on-failure-type-end-action.arazzo.yaml');
   const args = getParams(indexEntryPoint, ['respect', fixturesPath]);
 
-  const result = getCommandOutput(args);
+  const result = getCommandOutput(args, process.cwd());
   expect(result).toMatchSnapshot();
 });
diff --git a/__tests__/respect/step-on-success-type-end-action/step-on-success-type-end-action.test.ts b/__tests__/respect/step-on-success-type-end-action/step-on-success-type-end-action.test.ts
@@ -1,4 +1,5 @@
-import { getParams, getCommandOutput } from '../utils';
+import { getParams } from '../utils';
+import { getCommandOutput } from '../../helpers';
 import { join } from 'path';
 
 // Snapshot should have two workflows, and first workflow should run only first step
@@ -7,6 +8,6 @@ test('should end workflow execution, context returns to the caller with applicab
   const fixturesPath = join(__dirname, 'step-on-success-type-end-action.arazzo.yaml');
   const args = getParams(indexEntryPoint, ['respect', fixturesPath]);
 
-  const result = getCommandOutput(args);
+  const result = getCommandOutput(args, process.cwd());
   expect(result).toMatchSnapshot();
 });
diff --git a/__tests__/respect/workflow-failure-actions/workflow-failure-actions.arazzo.yaml b/__tests__/respect/workflow-failure-actions/workflow-failure-actions.arazzo.yaml
@@ -14,18 +14,18 @@ sourceDescriptions:
 
 workflows:
   - workflowId: get-museum-hours
-#    failureActions:
-#      - name: repeated-failure-action-1
-#        type: retry
-#        retryLimit: 1
-#        workflowId: $sourceDescriptions.tickets-from-museum-api.workflows.get-museum-tickets
-#        criteria:
-#          - condition: $statusCode == 200
-#      - name: repeated-failure-action-2 # this should not be executed as there is another action with the s
-#        type: goto
-#        workflowId: $sourceDescriptions.tickets-from-museum-api.workflows.get-museum-tickets
-#        criteria:
-#          - condition: $statusCode == 200
+    #    failureActions:
+    #      - name: repeated-failure-action-1
+    #        type: retry
+    #        retryLimit: 1
+    #        workflowId: $sourceDescriptions.tickets-from-museum-api.workflows.get-museum-tickets
+    #        criteria:
+    #          - condition: $statusCode == 200
+    #      - name: repeated-failure-action-2 # this should not be executed as there is another action with the s
+    #        type: goto
+    #        workflowId: $sourceDescriptions.tickets-from-museum-api.workflows.get-museum-tickets
+    #        criteria:
+    #          - condition: $statusCode == 200
     description: >-
       This workflow demonstrates how to get the museum opening hours and buy tickets.
     parameters:
diff --git a/__tests__/respect/workflow-success-actions/workflow-success-actions.arazzo.yaml b/__tests__/respect/workflow-success-actions/workflow-success-actions.arazzo.yaml
diff --git a/__tests__/respect/workflow-success-actions/workflow-success-actions.test.ts b/__tests__/respect/workflow-success-actions/workflow-success-actions.test.ts
