Skip to content

feature/playwright e2e#334

Merged
joescottdave merged 11 commits into
devfrom
feature/playwright-e2e
May 21, 2026
Merged

feature/playwright e2e#334
joescottdave merged 11 commits into
devfrom
feature/playwright-e2e

Conversation

@joescottdave
Copy link
Copy Markdown
Contributor

@joescottdave joescottdave commented May 21, 2026

Summary

Introduce Node/Yarn tooling scoped exclusively to Playwright browser
automation — no impact on the Rails build or asset pipeline. Adds
.nvmrc (Node 24), Yarn 4.15.0 via yarnPath, a minimal package.json
with @playwright/test, playwright.config.ts targeting port 3001, a
reusable setup-node GitHub Action, and a workflow_dispatch e2e.yml
workflow. Updates README to document the E2E setup and clarify that
Node is only present for test automation.
@joescottdave joescottdave marked this pull request as ready for review May 21, 2026 11:28
description: 'https://github.com/epimorphics/ppd-explorer/issues/335',
},
}, async ({ page }) => {
test.fixme()
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is there a ticket to track the two test.fixme() tests

Copy link
Copy Markdown
Contributor

@DanielCouzens DanielCouzens left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 👍 just a comment about the fixme()

@joescottdave
Copy link
Copy Markdown
Contributor Author

One of them has a ticket annotation, the other I've tweaked to not run against the dev environment, so it no longer has the fixme @DanielCouzens

@joescottdave joescottdave merged commit cc8cf67 into dev May 21, 2026
5 checks passed
@joescottdave joescottdave deleted the feature/playwright-e2e branch May 21, 2026 14:15
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.

2 participants