Skip to content

[DRAFT/WIP] test(web): add Stovepipe coverage and align E2E suites - #838

Draft
mnoah1 wants to merge 1 commit into
mnoah1/stovepipe-webfrom
mnoah1/stovepipe-web-e2e
Draft

mnoah1 wants to merge 1 commit into
mnoah1/stovepipe-webfrom
mnoah1/stovepipe-web-e2e

Conversation

@mnoah1

@mnoah1 mnoah1 commented Oct 12, 2026 •

Copy link
Copy Markdown
Contributor

Why?

The new Stovepipe UI needs browser coverage against a real backend, in addition to its unit and component tests. Align both services' browser suites under web/test/e2e/<domain> so they follow the repository layout and reuse stack setup helpers.

What?

  • Move the existing SubmitQueue browser suite into web/test/e2e/submitqueue, preserving its browser spec and Compose overlay.
  • Add web/test/e2e/stovepipe with real ingestion and seeded pagination data. Cover queue navigation and cursors, encoded change URIs, project pagination (including pages with no recorded results), raw project JSON containing only returned entries across all backend pages, history, refresh, empty queues, and accessibility.
  • Share image loading, schema application, and port lookup helpers. Both browser runners use the Bazel-pinned Compose executable from chore(docker): supply docker compose via Bazel #830.
  • Update Bazel targets and visibility, workspace entries, runner commands, and test documentation for the new layout. Tests emit queue and change-page screenshots as Bazel test outputs.

Test Plan

Latest validation passed all 31 targets in //web:check plus //web/test/e2e/stovepipe:web_test, including coverage that absent projects are omitted from the table and JSON export.

All 33 targets passed on the rebased stack, including both real-stack browser suites and the pinned Compose checks:

./tool/bazel test //web:check //web/test/e2e/submitqueue:web_test //web/test/e2e/stovepipe:web_test //tool/compose:compose_test --test_output=errors

Repository formatting, lint, module tidy, and Gazelle checks also passed. Browser tests require a running Docker daemon; Bazel supplies Compose and the browser.

Revert Plan

Revert this PR to restore the previous SubmitQueue test location and remove the Stovepipe browser suite. The frontend and shared-package PRs can remain.

Issue Links

None — no issue was supplied for this work.

Stack

  1. refactor(web): extract shared UI and host packages #842 — shared-package refactoring (based on main).
  2. [DRAFT/WIP] feat(web): add Stovepipe change status UI #837 — Stovepipe frontend (based on refactor(web): extract shared UI and host packages #842).
  3. [DRAFT/WIP] test(web): add Stovepipe coverage and align E2E suites #838 — browser coverage and E2E organization (this PR; based on [DRAFT/WIP] feat(web): add Stovepipe change status UI #837).

Merge in this order.

@mnoah1
mnoah1 added this pull request to stack #839 October 12, 2026 14:22
@mnoah1
mnoah1 force-pushed the mnoah1/stovepipe-web-e2e branch from 95f5825 to 99bb189 Compare October 12, 2026 14:51
@mnoah1
mnoah1 force-pushed the mnoah1/stovepipe-web-e2e branch from 99bb189 to 2df95bd Compare October 12, 2026 15:00
@mnoah1
mnoah1 force-pushed the mnoah1/stovepipe-web-e2e branch from 2df95bd to 9e37974 Compare October 12, 2026 15:13
@mnoah1
mnoah1 removed this pull request from stack #839 October 12, 2026 15:29
@mnoah1
mnoah1 added this pull request to stack #843 October 12, 2026 15:29
@mnoah1
mnoah1 force-pushed the mnoah1/stovepipe-web-e2e branch from 9e37974 to 96605a6 Compare October 12, 2026 15:44
@mnoah1
mnoah1 marked this pull request as ready for review October 12, 2026 15:50
@mnoah1
mnoah1 requested review from a team, behinddwalls and sbalabanov as code owners October 12, 2026 15:50
mnoah1 added a commit that referenced this pull request Oct 12, 2026
Send web logs through per-request callbacks so hosts can use their own logging context. Keep Pino in the reference hosts, preserve original errors and structured fields, and isolate callback failures from page results.

Builds on the web E2E layer in #838.
@mnoah1
mnoah1 removed this pull request from stack #843 October 12, 2026 20:08
@mnoah1
mnoah1 added this pull request to stack #847 October 12, 2026 20:08
@mnoah1 mnoah1 changed the title test(web): add Stovepipe coverage and align E2E suites [DRAFT/WIP] test(web): add Stovepipe coverage and align E2E suites Oct 12, 2026
@mnoah1
mnoah1 marked this pull request as draft October 12, 2026 22:05

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant