chore: Disable n8n node API-coverage contract test until API team resync (#34333)
Some checks are pending
Build: Benchmark Image / build (push) Waiting to run
CI: Master (Build, Test, Lint) / Build for Github Cache (push) Waiting to run
CI: Master (Build, Test, Lint) / Unit tests (22.22.3) (push) Waiting to run
CI: Master (Build, Test, Lint) / Unit tests (24.16.0) (push) Waiting to run
CI: Master (Build, Test, Lint) / Lint (push) Waiting to run
CI: Master (Build, Test, Lint) / Performance (push) Waiting to run
CI: Master (Build, Test, Lint) / Notify Slack on failure (push) Blocked by required conditions
Util: Sync API Docs / sync-public-api (push) Waiting to run

Co-authored-by: n8n-cat-bot[bot] <n8n-cat-bot[bot]@users.noreply.github.com>
Co-authored-by: Claude Opus 4.8 <noreply@anthropic.com>
This commit is contained in:
n8n-cat-bot[bot] 2026-07-16 11:23:44 +00:00 committed by GitHub
parent cc38d09efd
commit 1e52ffeca0
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
2 changed files with 7 additions and 13 deletions

View File

@ -27,8 +27,6 @@ const MANIFEST_RELATIVE = 'packages/nodes-base/nodes/N8n/n8n-api-coverage.json';
const HTTP_METHODS = new Set(['get', 'post', 'put', 'patch', 'delete', 'head', 'options']);
// This test reads OpenAPI files from packages/cli. Keep Turbo inputs for
// n8n-nodes-base#test in sync (see turbo.json) so CLI spec changes invalidate cache.
// Matches path entries in openapi.yml: " /some/path:\n $ref: './relative/file.yml'"
const PATH_REF_PATTERN = /^ {2}(\/\S+):\s*\n\s+\$ref:\s*'([^']+)'/gm;
@ -83,7 +81,13 @@ function loadManifest(): Manifest {
}
}
describe('n8n node API coverage', () => {
// Skipped pending an API-team resync of node ↔ public-API coverage. This is a
// coverage-backlog ledger with a drift guard, not a behavioural test, and its
// real input (the CLI public API spec) lives in another package that Test
// Impact Analysis can't link to this file. When re-enabled it should return as
// an always-run invariant/check rather than a per-file unit test. See DEVP-656.
// eslint-disable-next-line n8n-local-rules/no-skipped-tests
describe.skip('n8n node API coverage', () => {
let specEndpoints: string[];
let manifest: Manifest;

View File

@ -46,16 +46,6 @@
"outputs": ["coverage/**", "*.xml"],
"env": ["COVERAGE_ENABLED"]
},
"n8n-nodes-base#test": {
"inputs": ["$TURBO_DEFAULT$", "../cli/src/public-api/v1/**/*.yml"]
},
"n8n-nodes-base#test:changed": {
"dependsOn": ["^build", "build"],
"inputs": ["$TURBO_DEFAULT$", "../cli/src/public-api/v1/**/*.yml"],
"outputs": ["coverage/**", "*.xml"],
"env": ["COVERAGE_ENABLED", "CHANGED_FILES"],
"cache": false
},
"test:changed": {
"dependsOn": ["^build", "build"],
"outputs": ["coverage/**", "*.xml"],