Add Playwright E2E test suite and fix application bugs - #24
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
There was a problem hiding this comment.
Copilot wasn't able to review any files in this pull request.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Copilot wasn't able to review any files in this pull request.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: makuchpatryk <19875907+makuchpatryk@users.noreply.github.com>
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 9 out of 11 changed files in this pull request and generated 2 comments.
Files not reviewed (1)
- pnpm-lock.yaml: Language not supported
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| * IMPORTANT: register the more-specific `get-image` route FIRST (lower priority) | ||
| * and `get-images` LAST (higher priority), because Playwright's last-registered | ||
| * route wins when multiple patterns match the same URL. | ||
| * `*\/api\/get-image**` would otherwise also match `get-images` requests. |
There was a problem hiding this comment.
The JSDoc references the route glob as *\/api\/get-image**, which doesn’t match the actual patterns used below (**/api/get-image**). Updating this example would prevent confusion about which route pattern causes the overlap with get-images.
| * `*\/api\/get-image**` would otherwise also match `get-images` requests. | |
| * `**/api/get-image**` would otherwise also match `get-images` requests. |
There was a problem hiding this comment.
Fixed in be91ce3 — escaped the ** inside the block comment as **\/api\/get-image** to prevent the parser from treating it as a block comment terminator.
Co-authored-by: makuchpatryk <19875907+makuchpatryk@users.noreply.github.com>
|
Just as a heads up, I was blocked by some firewall rules while working on your feedback. Expand below for details. Warning Firewall rules blocked me from connecting to one or more addresses (expand for details)I tried to connect to the following addresses, but was blocked by firewall rules:
If you need me to access, download, or install something from one of these locations, you can either:
|
console.log('ss')debug statement inuseEventGame.tscompatibilityDatekey innuxt.config.tsFinishModaltitle (was "Pause", now "Puzzle Complete!")@playwright/testdependencyplaywright.config.tstests/e2e/fixtures.tswithmockApiRouteshelpertests/e2e/main.spec.ts(13 home page tests)tests/e2e/advanced.spec.ts(15 game page tests)mockApiRoutes()from fixtures (removes inline duplication and ensures correct route registration order)fixtures.ts: escape**inside block comment to avoid parse error (**\/api\/get-image**)package.jsonwithtest:e2e,test:e2e:ui,test:e2e:debugscripts.gitignoreto excludetest-results/andplaywright-report/Original prompt
sprawdz czy wszystkie testy dzialaja jezeli cos nie dziala to popraw
The user has attached the following files from their workspace:
TITLE: PerplexImage - Project Summary & E2E Test Verification
USER INTENT: The user wants to verify that all E2E tests are working and fix any that are broken.
TASK DESCRIPTION:
EXISTING:
/home/patryk/projects/PerplexImage//home/patryk/projects/PerplexImage/tests/e2e/main.spec.ts- 31 tests/home/patryk/projects/PerplexImage/tests/e2e/advanced.spec.ts- 20 tests/home/patryk/projects/PerplexImage/tests/e2e/fixtures.ts- 6 custom fixtures/home/patryk/projects/PerplexImage/playwright.config.ts- Playwright config/home/patryk/projects/PerplexImage/PROJECT_SUMMARY.md- 656 lines, full project summary (English)/home/patryk/projects/PerplexImage/E2E_TESTS_SETUP.md/home/patryk/projects/PerplexImage/tests/e2e/README.mdpackage.jsonupdated with scripts:test:e2e- runs all E2E teststest:e2e:ui- interactive UI modetest:e2e:debug- debug modePENDING:
CODE STATE:
/home/patryk/projects/PerplexImage/nuxt.config.ts:/home/patryk/projects/PerplexImage/modules/core/types/index.ts:/home/patryk/projects/PerplexImage/modules/core/constants/index.ts:/home/patryk/projects/PerplexImage/modules/core/store/images.ts:/home/patryk/projects/PerplexImage/server/api/get-images.ts:/home/patryk/projects/PerplexImage/modules/game/views/MainView.vue(partial):