groombook/app
Archived
13
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix(e2e): block service workers to prevent route mock bypass #66

Closed
ghost wants to merge 5 commits from fix/e2e-block-service-worker into main
pull from: fix/e2e-block-service-worker
merge into: groombook:main
Conversation 3 Commits 5 Files Changed 12
+50 -1077

5 Commits

Author SHA1 Message Date
Groom Book CTO 4289abaa7f fix(e2e): block service workers to prevent route mock bypass 
The PWA service worker (VitePWA workbox runtimeCaching) intercepts
/api/* requests, which prevents Playwright's page.route() mocks from
working. This caused the booking flow E2E test to fail because the
availability request was handled by the service worker instead of the
test mock, resulting in real (empty) API responses.

Fixes #65

Co-Authored-By: Paperclip <noreply@paperclip.ing>
 2026-03-19 12:56:30 +00:00
Groom Book CTO dc37a5f59e Fix E2E tests for split-element brand name 
The brand is now <span>Groom</span>Book (no space), so Playwright's
getByText needs "GroomBook" instead of "Groom Book".

Co-Authored-By: Paperclip <noreply@paperclip.ing>
 2026-03-19 03:27:54 +00:00
Groom Book CTO ad4cc56546 Fix brand text test to handle split-element rendering 
The nav brand was changed to <span>Groom</span>Book for color styling,
but getByText with a regex can't match text split across child elements.
Use a custom text matcher that checks the STRONG element's textContent.

Co-Authored-By: Paperclip <noreply@paperclip.ing>
 2026-03-19 03:20:09 +00:00
Groom Book CTO f21ac59040 Fix test selectors for branded nav text 
- Use regex /Groom\s*Book/ to match split-element brand text
- Use getByRole("link") for Book CTA to avoid matching brand <strong>

Co-Authored-By: Paperclip <noreply@paperclip.ing>
 2026-03-19 03:16:46 +00:00
Groom Book CTO 123cae65c2 Improve admin UI visual design — polish look and feel 
- Sticky nav bar with subtle shadow, branded GroomBook wordmark, green gradient Book button
- Consistent brand green (#4f8a6f) for primary buttons across all admin pages
- Tables wrapped in white cards with rounded corners and soft shadows
- Uppercase table headers with better spacing and hierarchy
- Input/button border-radius increased to 6px for softer feel
- Global CSS: button transitions, input focus states with brand green ring, subtle card shadows
- Background changed from plain white to light gray (#f0f2f5) for depth
- Reports: polished stat cards with shadows, refined section headers, card-wrapped tables
- Custom scrollbar styling for a cleaner look

Closes groombook/groombook#58

Co-Authored-By: Paperclip <noreply@paperclip.ing>
 2026-03-19 03:00:23 +00:00