From b97ffe94df4a2b77b9fa52357592b3abcfffe283 Mon Sep 17 00:00:00 2001 From: Charlie Kolb Date: Tue, 1 Sep 2026 14:12:52 +0200 Subject: [PATCH] ci: Skip the scoped cli integration step when the DB matrix runs The DB matrix's SQLite Pooled leg runs the full cli integration suite with the same command and environment as the scoped step in the unit workflow. Skip the scoped step on those runs. Master runs keep the default and do not change. Co-Authored-By: Claude Fable 5 --- .github/workflows/ci-pull-requests.yml | 3 +++ .github/workflows/test-unit-reusable.yml | 11 ++++++++++- 2 files changed, 13 insertions(+), 1 deletion(-) diff --git a/.github/workflows/ci-pull-requests.yml b/.github/workflows/ci-pull-requests.yml index 13166e5d235..afd6aba0386 100644 --- a/.github/workflows/ci-pull-requests.yml +++ b/.github/workflows/ci-pull-requests.yml @@ -211,6 +211,9 @@ jobs: collectCoverage: ${{ github.event_name != 'merge_group' }} affectedPackages: ${{ needs.install-and-build.outputs.affected_packages }} changedFiles: ${{ needs.install-and-build.outputs.changed_files }} + # The DB matrix's "SQLite Pooled" leg runs the full cli integration + # suite with the same command and environment as the scoped step. + skipCliIntegration: ${{ needs.install-and-build.outputs.db == 'true' }} secrets: inherit typecheck: diff --git a/.github/workflows/test-unit-reusable.yml b/.github/workflows/test-unit-reusable.yml index 4c05b2be742..823f2305dc3 100644 --- a/.github/workflows/test-unit-reusable.yml +++ b/.github/workflows/test-unit-reusable.yml @@ -33,6 +33,15 @@ on: required: false default: '' type: string + skipCliIntegration: + description: | + Skip the scoped cli integration step. The PR workflow sets this when + the DB matrix runs: its "SQLite Pooled" leg runs the full cli + integration suite with the same command and environment, so the + scoped run here adds no coverage. + required: false + default: false + type: boolean env: NODE_OPTIONS: --max-old-space-size=7168 @@ -136,7 +145,7 @@ jobs: run: pnpm turbo test:integration --continue --concurrency=1 --filter='!./packages/frontend/**' --filter='!./packages/modules/**' --filter='!n8n' --summarize - name: Test Integration (cli, scoped) - if: ${{ !cancelled() }} + if: ${{ !cancelled() && !inputs.skipCliIntegration }} run: pnpm turbo test:integration:changed --filter=n8n --summarize - name: Send Test Stats