Add Playwright E2E smoke tests#335
Conversation
|
@saurabhhhcodes is attempting to deploy a commit to the PRIYANSHU DOSHI's projects Team on Vercel. A member of the Team first needs to authorize it. |
|
Hi @Priyanshu-byte-coder, this PR is for assigned GSSoC issue #236. Could you please mirror the issue scoring labels when reviewing: I kept the E2E setup CI-friendly by avoiding lockfile churn and running Playwright through pinned |
Priyanshu-byte-coder
left a comment
There was a problem hiding this comment.
@playwright/test is not in package.json — the CI workflow tries to run the config which imports it, causing the ERR_MODULE_NOT_FOUND crash before any tests run.
Fix:
npm install --save-dev @playwright/testThen commit the updated package.json and lockfile. Also consider adding:
playwright install --with-depsstep in the workflow (or confirm browsers are pre-installed in the CI image)- A
.gitignoreentry forplaywright-report/andtest-results/
|
Synced this branch with the latest Validation after sync:
I also tried |
60da70e
into
Priyanshu-byte-coder:main
|
Merged! Solid E2E test suite — cookie injection for auth bypass, |
Summary
/u/[username]renders without requiring authCloses #236
Type of Change
Changes Made
playwright.config.mjse2e/.github/workflows/e2e.ymlHow to Test
node --check playwright.config.mjs e2e/*.jsnode node_modules/typescript/bin/tsc --noEmitnode node_modules/next/dist/bin/next lintE2E / Playwright smoke testsworkflow runsnpx -y @playwright/test@1.49.1 testNotes
package.json/package-lock.jsonbecause this workspace does not have npm available locally to safely regenerate the lockfile. The workflow invokes Playwright throughnpx -y @playwright/test@1.49.1, so existingnpm ciremains unchanged.next buildis blocked on this macOS runtime by the existing Next SWC native binary code-signature issue (failed-loading-swc). Type-check and lint complete locally; lint reports only pre-existing warnings inBadgeSection.tsxandCommitTimeChart.tsx.Checklist
npm run lintequivalent passes locally vianode node_modules/next/dist/bin/next lint(warnings only, matching current repo behavior)node node_modules/typescript/bin/tsc --noEmit)