Skip to content

Commit d4e8102

Browse files
bloveclaude
andcommitted
fix(cockpit-e2e-wiring): accept matrix python entry as uv-sync evidence
The wiring spec previously required a literal `working-directory: cockpit/<cap>/python` step in ci.yml. After the matrix migration in this PR, that path moves into the matrix entry (`python: cockpit/...`) and the working-directory is templated as `${{ matrix.cap.python }}`. Spec now accepts either form, so matrix-templated jobs and any remaining literal-step jobs both pass the cross-check. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
1 parent e39e4bf commit d4e8102

1 file changed

Lines changed: 7 additions & 1 deletion

File tree

apps/cockpit/cockpit-e2e-wiring.spec.ts

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -184,7 +184,13 @@ describe('cockpit e2e wiring', () => {
184184
if (!workflow.includes(wiring.project)) {
185185
errors.push(`${wiring.project}: ${workflowPath} does not run the e2e target`);
186186
}
187-
if (!workflow.includes(`working-directory: ${wiring.langgraphCwd}`)) {
187+
// Matrix-migrated jobs (cockpit-e2e) template the working-directory via
188+
// `${{ matrix.cap.python }}`; the python path appears in the matrix
189+
// entry (e.g. `python: cockpit/chat/foo/python`) instead. Accept either
190+
// form so matrix and non-matrix jobs both pass.
191+
const literalUvSync = workflow.includes(`working-directory: ${wiring.langgraphCwd}`);
192+
const matrixEntry = workflow.includes(`python: ${wiring.langgraphCwd}`);
193+
if (!literalUvSync && !matrixEntry) {
188194
errors.push(`${wiring.project}: ${workflowPath} does not pre-sync ${wiring.langgraphCwd}`);
189195
}
190196
}

0 commit comments

Comments
 (0)