Skip to content

test: add end-to-end tests for demo workflow - #95

Merged
rprabhat merged 3 commits into
mainfrom
chore/remove-cardscanner-sensible-analytics
Jun 15, 2026
Merged

test: add end-to-end tests for demo workflow#95
rprabhat merged 3 commits into
mainfrom
chore/remove-cardscanner-sensible-analytics

Conversation

@rprabhat

Copy link
Copy Markdown
Collaborator

Summary

Add 30 E2E tests across 7 phases for the demo workflow on crewRoster.

Changes

  • new apps/web/e2e/demo-workflow.spec.ts — 30 tests covering demo page, persona login, roster view, shift management, week navigation, employee roles, and navigation error handling
  • fix apps/web/playwright.config.ts — baseURL from crewcircle.co to roster.crewcircle.co
  • fix playwright.config.ts (root) — baseURL default from crewcircle.co to roster.crewcircle.co

Test Results

  • 30/30 tests passing against live site (https://roster.crewcircle.co)
  • Test run time: ~1.5 minutes

Verification

  • Build passes
  • All 30 E2E tests pass
  • LSP diagnostics clean on changed files
  • No TypeScript errors

Pre-existing Issues (not caused by this PR)

  • ESLint broken project-wide: eslint-plugin-react v7.37.5 incompatible with ESLint 10.4.1 (react/display-name rule crash)
  • .next/ build artifacts tracked in git (missing from root .gitignore)

- Add demo-workflow.spec.ts with 30 tests across 7 phases
- Fix baseURL from crewcircle.co to roster.crewcircle.co
- Tests cover: demo page personas, roster view, shift management, week navigation, employee roles, navigation error handling

Note: eslint pre-existing failure (eslint-plugin-react vs ESLint 10.4.1 incompatibility)
@vercel

vercel Bot commented Jun 15, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
crew-roster Building Building Preview, Comment Jun 15, 2026 6:41am

@rprabhat
rprabhat merged commit 1deb4c7 into main Jun 15, 2026
6 of 7 checks passed
@rprabhat
rprabhat deleted the chore/remove-cardscanner-sensible-analytics branch June 15, 2026 06:42
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