test: advanced test use cases (#3737)

Author: dreamorosi

Diff for: .github/scripts/get_pr_info.js

@@ -17,20 +17,21 @@ jobs:
     env:
       NODE_ENV: dev
       PR_NUMBER: ${{ inputs.prNumber }}
+      GH_TOKEN: ${{ github.token }}
     permissions:
       id-token: write # needed to interact with GitHub's OIDC Token endpoint.
       contents: read
     strategy:
-      max-parallel: 30
+      max-parallel: 25
       matrix:
         package:
           [
-            layers,
+            packages/idempotency,
             packages/logger,
             packages/metrics,
-            packages/tracer,
             packages/parameters,
-            packages/idempotency,
+            packages/tracer,
+            layers,
           ]
         version: [18, 20, 22]
         arch: [x86_64, arm64]
@@ -42,11 +43,14 @@ jobs:
       - name: Extract PR details
         id: extract_PR_details
         if: ${{ inputs.prNumber != '' }}
-        uses: actions/github-script@60a0d83039c74a4aee543508d2ffcb1c3799cdea # v7.0.1
-        with:
-          script: |
-            const script = require('.github/scripts/get_pr_info.js');
-            await script({github, context, core});
+        run: |
+          # Get the PR number from the input
+          pr_number=${{ inputs.prNumber }}
+          # Get the headSHA of the PR
+          head_sha=$(gh pr view $pr_number --json headRefOid -q '.headRefOid')
+          # Set the headSHA as an output variable
+          echo "headSHA=$head_sha" >> $GITHUB_OUTPUT
+          echo "headSHA=$head_sha"
       # Only if a PR Number was passed and the headSHA of the PR extracted,
       # we checkout the PR at that point in time
       - name: Checkout PR code
@@ -75,4 +79,4 @@ jobs:
           ARCH: ${{ matrix.arch }}
           JSII_SILENCE_WARNING_DEPRECATED_NODE_VERSION: true
           RUNNER_DEBUG: ${{ env.RUNNER_DEBUG }}
-        run: npm run test:e2e -w ${{ matrix.package }}
+        run: npm run test:e2e -w ${{ matrix.package }}

Diff for: .github/workflows/run-e2e-tests.yml

@@ -1,5 +1 @@
 export const RESOURCE_NAME_PREFIX = 'Layers-E2E';
-export const ONE_MINUTE = 60 * 1000;
-export const TEST_CASE_TIMEOUT = 3 * ONE_MINUTE;
-export const SETUP_TIMEOUT = 7 * ONE_MINUTE;
-export const TEARDOWN_TIMEOUT = 5 * ONE_MINUTE;

Diff for: layers/tests/e2e/constants.ts

@@ -11,11 +11,7 @@ import { LayerVersion } from 'aws-cdk-lib/aws-lambda';
 import { afterAll, beforeAll, describe, expect, it, vi } from 'vitest';
 import packageJson from '../../package.json';
 import { LayerPublisherStack } from '../../src/layer-publisher-stack.js';
-import {
-  RESOURCE_NAME_PREFIX,
-  SETUP_TIMEOUT,
-  TEARDOWN_TIMEOUT,
-} from './constants.js';
+import { RESOURCE_NAME_PREFIX } from './constants.js';
 
 /**
  * This test has two stacks:
@@ -121,7 +117,7 @@ describe('Layers E2E tests', () => {
         })
       );
     }
-  }, SETUP_TIMEOUT);
+  });
 
   it.each(cases)(
     'imports and instantiates all utilities (%s)',
@@ -198,5 +194,5 @@ describe('Layers E2E tests', () => {
       await testLayerStack.destroy();
       await testStack.destroy();
     }
-  }, TEARDOWN_TIMEOUT);
+  });
 });

Diff for: layers/tests/e2e/layerPublisher.test.ts

@@ -3,5 +3,7 @@ import { defineProject } from 'vitest/config';
 export default defineProject({
   test: {
     environment: 'node',
+    hookTimeout: 1_000 * 60 * 10, // 10 minutes
+    testTimeout: 1_000 * 60 * 3, // 3 minutes
   },
 });

Diff for: layers/vitest.config.ts

@@ -65,7 +65,7 @@
     "typedoc-plugin-missing-exports": "^3.1.0",
     "typedoc-plugin-zod": "^1.4.0",
     "typescript": "^5.8.2",
-    "vitest": "^3.0.5"
+    "vitest": "^3.0.9"
   },
   "lint-staged": {
     "*.{js,ts}": "biome check --write",

Diff for: package-lock.json

@@ -1,6 +1 @@
 export const RESOURCE_NAME_PREFIX = 'Idempotency';
-
-export const ONE_MINUTE = 60 * 1_000;
-export const TEARDOWN_TIMEOUT = 5 * ONE_MINUTE;
-export const SETUP_TIMEOUT = 7 * ONE_MINUTE;
-export const TEST_CASE_TIMEOUT = 5 * ONE_MINUTE;