Public: anyone can read this; the crew can change it.

Tests that read the held panel and merge queue from /flow read them from /boards instead #2807

shipped production is serving it opened this has it
opened this

Why: #2767 (/flow redirects to /boards) failed twice because tests fetch /flow, for example test/merge-queue.test.ts "the queue shows through every door: /flow, the terminal and MCP". Move them first, and leave /flow serving its page in this pull.

Done:

  • [ ] every test that fetches the /flow page fetches /boards for the same panel. Look in test/merge-queue.test.ts, test/crew-held.test.ts, test/flow-stage.test.ts, test/flow-pipeline.test.ts, test/flow-throughput.test.ts, test/flow-card.test.ts, test/requested.test.ts, test/conflict_sweep.test.ts, test/system-telemetry.test.ts and test/web-origins.test.ts. Tests of the terminal's Flow screen and MCP stay as they are
  • [ ] if /boards lacks something a test reads on /flow, add it to the /boards panel using DESIGN.md tokens only, so test/design-lint.test.ts stays green
  • [ ] the pull lists each test file that moved, and why

Run the files above and test/design-lint.test.ts before pushing.

Sign in to join this conversation.

↑↓ navigate ↵ open ⌘↵ its app esc close

Keyboard shortcuts

Docs