Skip to content

fix: prepare Prisma before solution tests - #45

Merged
kody-bot merged 4 commits into
mainfrom
cursor/ws-full-stack-testing-d454
Jul 29, 2026
Merged

fix: prepare Prisma before solution tests#45
kody-bot merged 4 commits into
mainfrom
cursor/ws-full-stack-testing-d454

Conversation

@kentcdodds

@kentcdodds kentcdodds commented Jul 29, 2026

Copy link
Copy Markdown
Member

Summary

  • run workshop test preparation after dependency installation so Prisma clients are generated for all 54 apps
  • skip browser and playground setup that the Vitest CI job does not need
  • run every test in each completed Vitest-backed solution, excluding intentional intermediate-step failures and Playwright-only exercises from the incompatible runner
  • fix the completed database solution test so its nested route renders instead of being replaced by a placeholder root component
  • run tests on supported Node 22 because the pinned MSW/Undici stack has an AbortSignal realm incompatibility on Node 26; cross-platform setup validation remains on Node 26

Verification

  • SKIP_PLAYWRIGHT=true SKIP_PLAYGROUND=true npm run setup:custom
  • CI=true node ./epicshop/test.js '4.4.s,5.2.s,6.2.s,7.2.s,8.3.s,9.3.s,10.1.s,11.3.s'
Open in Web Open in Cursor 

cursoragent and others added 3 commits July 29, 2026 01:39
Co-authored-by: Kent C. Dodds <me+github@kentcdodds.com>
Co-authored-by: Kent C. Dodds <me+github@kentcdodds.com>
Co-authored-by: Kent C. Dodds <me+github@kentcdodds.com>
@kody-bot
kody-bot marked this pull request as ready for review July 29, 2026 01:58
Co-authored-by: Kent C. Dodds <me+github@kentcdodds.com>
@kody-bot
kody-bot merged commit 94f917a into main Jul 29, 2026
10 of 11 checks passed
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.

3 participants