Run end-to-end tests in CI by waisingyiu · Pull Request #2102 · guardian/facia-tool · GitHub
Skip to content

Run end-to-end tests in CI - #2102

Open
waisingyiu wants to merge 3 commits into
dk-ws/e2e-phase-5-docsfrom
dk-ws/e2e-phase-6-ci
Open

waisingyiu wants to merge 3 commits into
dk-ws/e2e-phase-5-docsfrom
dk-ws/e2e-phase-6-ci

Conversation

@waisingyiu

@waisingyiu waisingyiu commented Sep 29, 2026 •

Copy link
Copy Markdown
Contributor

Summary

  • add a dedicated GitHub Actions workflow for the Playwright/Testcontainers suite
  • pin all third-party actions to immutable Node 24-based v5 commit SHAs
  • cache Yarn dependencies and install only the Chromium headless shell
  • upload Playwright traces, videos, and screenshots for failed runs
  • document the implemented CI behavior in the e2e README

Dependency and merge order

This is a stacked Phase 6 PR based on Phase 5 PR #2101. Merge #2101 first, then this PR.

How to validate

  1. Let the End-to-end tests pull request workflow run on ubuntu-22.04.
  2. Find and watch it with:
gh run list --workflow e2e-tests.yml --branch dk-ws/e2e-phase-6-ci --event pull_request --limit 1
gh run watch <run-id> --exit-status
  1. Inspect the complete job log with gh run view <run-id> --log. Confirm the suite passes, container health checks are clean, no request reaches an amazonaws.com endpoint, and no credentials are printed.

Validation result

Passed on ubuntu-22.04 in workflow run 36584617728. Both scenarios passed in 3.3 minutes. The completed log contained no exceptions, failed health checks, amazonaws.com requests, credential output, or deprecated-action warning; the only annotation was the Playwright 2 passed notice.

The failure-only artifact step was skipped on the successful run. Artifact upload was not deliberately triggered because the controlled-failure validation option was not selected.

@waisingyiu
waisingyiu requested a review from a team as a code owner September 29, 2026 14:26
@github-actions

github-actions Bot commented Sep 29, 2026 •

Copy link
Copy Markdown

@waisingyiu
waisingyiu force-pushed the dk-ws/e2e-phase-5-docs branch from 4382b93 to 92693af Compare October 1, 2026 14:57
@waisingyiu
waisingyiu force-pushed the dk-ws/e2e-phase-6-ci branch from 0dd4a12 to f704b38 Compare October 1, 2026 14:57
@waisingyiu
waisingyiu force-pushed the dk-ws/e2e-phase-5-docs branch from 92693af to dd5a02c Compare October 1, 2026 15:34
@waisingyiu
waisingyiu force-pushed the dk-ws/e2e-phase-6-ci branch from f704b38 to 188a92f Compare October 1, 2026 15:34
@waisingyiu
waisingyiu force-pushed the dk-ws/e2e-phase-5-docs branch from dd5a02c to 23a8d40 Compare October 1, 2026 15:39
@waisingyiu
waisingyiu force-pushed the dk-ws/e2e-phase-6-ci branch from 188a92f to 39e9e36 Compare October 1, 2026 15:39
@waisingyiu
waisingyiu force-pushed the dk-ws/e2e-phase-6-ci branch from 39e9e36 to 0c8b656 Compare October 9, 2026 11:59
@waisingyiu
waisingyiu force-pushed the dk-ws/e2e-phase-5-docs branch from 23a8d40 to 10c7c72 Compare October 9, 2026 11:59
@waisingyiu
waisingyiu force-pushed the dk-ws/e2e-phase-5-docs branch from 10c7c72 to fadebc0 Compare October 9, 2026 13:43
@waisingyiu
waisingyiu force-pushed the dk-ws/e2e-phase-6-ci branch from 0c8b656 to e90eaac Compare October 9, 2026 13:43
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