Skip to content

Commit 298e5b2

Browse files
committed
fix: extract only required test databases for integration tests
The default (integration) scope was extracting all tools/test databases for all 10 languages (~60 databases) when only 7 specific tools databases plus javascript/examples are referenced by client integration test fixtures. Replace the blanket 'for lang in all languages' loop with an explicit list of the 7 tools/test directories actually used: - cpp/tools/test/{CallGraphFrom,CallGraphFromTo,CallGraphTo} - javascript/tools/test/CallGraphFromTo - python/tools/test/CallGraphFromTo - rust/tools/test/{CallGraphFrom,PrintAST} This restores the performance fix from main where integration scope was intentionally minimal, and avoids a significant CI time regression from extracting ~53 unnecessary databases.
1 parent 0414247 commit 298e5b2

File tree

4 files changed

+464
-98
lines changed

4 files changed

+464
-98
lines changed

.github/workflows/client-integration-tests.yml

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -99,7 +99,8 @@ jobs:
9999
run: ./server/scripts/install-packs.sh
100100

101101
## Extract test databases used in the integration tests.
102-
## Defaults to integration scope (javascript/examples + all tools databases).
102+
## Defaults to integration scope (javascript/examples + specific tools
103+
## databases referenced by integration test fixtures).
103104
## Query unit tests auto-extract their own databases via `codeql test run`.
104105
- name: MCP Integration Tests - Extract test databases
105106
shell: bash

0 commit comments

Comments
 (0)