github-starred/vikunja
2
0
Fork 0
mirror of https://github.com/go-vikunja/vikunja.git synced 2026-08-30 09:07:40 -05:00
Code Issues 690 Packages Projects Releases 13 Wiki Activity

[PR #1976] [MERGED] chore(tests): remove Cypress, use Playwright exclusively #9680

New Issue
Closed
opened 2026-04-23 09:08:52 -05:00 by GiteaMirror · 0 comments
GiteaMirror commented 2026-04-23 09:08:52 -05:00
Owner
Copy Link
Copy Source

📋 Pull Request Information

Original PR: https://github.com/go-vikunja/vikunja/pull/1976
Author: @kolaente
Created: 12/12/2025
Status: ✅ Merged
Merged: 12/12/2025
Merged by: @kolaente

Base: main ← Head: chore-remote-cypress

📝 Commits (2)

  • 77d0c6a chore(tests): remove Cypress, use Playwright exclusively
  • 0d220b8 chore(ci): remove Cypress E2E test job from CI workflow

📊 Changes

52 files changed (+83 additions, -2227 deletions)

View changed files

📝 .github/workflows/test.yml (+0 -63)
📝 frontend/.gitignore (+0 -2)
📝 frontend/.npmrc (+0 -4)
📝 frontend/.stylelintrc.json (+0 -1)
➖ frontend/cypress.config.ts (+0 -28)
➖ frontend/cypress/README.md (+0 -48)
➖ frontend/cypress/docker-compose.yml (+0 -15)
➖ frontend/cypress/e2e/project/prepareProjects.ts (+0 -61)
➖ frontend/cypress/e2e/project/project-view-list.spec.ts (+0 -58)
➖ frontend/cypress/e2e/task/overview.spec.ts (+0 -109)
➖ frontend/cypress/e2e/tsconfig.json (+0 -11)
➖ frontend/cypress/e2e/user/login.spec.ts (+0 -53)
➖ frontend/cypress/e2e/user/settings.spec.ts (+0 -43)
➖ frontend/cypress/factories/bucket.ts (+0 -19)
➖ frontend/cypress/factories/label_task.ts (+0 -16)
➖ frontend/cypress/factories/labels.ts (+0 -21)
➖ frontend/cypress/factories/link_sharing.ts (+0 -21)
➖ frontend/cypress/factories/project.ts (+0 -26)
➖ frontend/cypress/factories/project_view.ts (+0 -19)
➖ frontend/cypress/factories/task.ts (+0 -21)

...and 32 more files

📄 Description

Summary

  • Removes Cypress test framework entirely, using only Playwright for E2E tests
  • All Cypress tests were already covered by Playwright; added 2 missing tests for URL filter/search parameters
  • Removes ~2000 lines of Cypress code and configuration

Changes

  • Deleted cypress/ directory with all tests, factories, and support files
  • Removed Cypress dependencies: @4tw/cypress-drag-drop, @cypress/vue, @cypress/vite-dev-server, cypress, start-server-and-test
  • Removed test:cypress:* npm scripts
  • Cleaned up config files (.gitignore, .npmrc, .stylelintrc.json, eslint.config.js, tsconfig.config.json, env.d.ts)
  • Renamed src/directives/cypress.ts to testid.ts (still used for data-cy attributes which Playwright uses)
  • Added 2 tests to project-view-list.spec.ts:
    • "Should respect filter query parameter from URL"
    • "Should respect search query parameter from URL"

Test plan

  • Run pnpm test:e2e - 173 tests passing
  • Run pnpm lint - passes
  • CI pipeline passes

Summary by CodeRabbit

Release Notes

  • Chores
    • Removed Cypress end-to-end testing infrastructure and configuration
    • Migrated testing framework from Cypress to Playwright
    • Removed test data factories and helper utilities
    • Updated ESLint and Stylelint configurations to reflect testing changes

✏️ Tip: You can customize this high-level summary in your review settings.

🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/go-vikunja/vikunja/pull/1976 **Author:** [@kolaente](https://github.com/kolaente) **Created:** 12/12/2025 **Status:** ✅ Merged **Merged:** 12/12/2025 **Merged by:** [@kolaente](https://github.com/kolaente) **Base:** `main` ← **Head:** `chore-remote-cypress` --- ### 📝 Commits (2) - [`77d0c6a`](https://github.com/go-vikunja/vikunja/commit/77d0c6a929a8b6c198c5f7a8c55e2e7394da02aa) chore(tests): remove Cypress, use Playwright exclusively - [`0d220b8`](https://github.com/go-vikunja/vikunja/commit/0d220b8bb64752601f5b7b5d9fa9d2eedc44d490) chore(ci): remove Cypress E2E test job from CI workflow ### 📊 Changes **52 files changed** (+83 additions, -2227 deletions) <details> <summary>View changed files</summary> 📝 `.github/workflows/test.yml` (+0 -63) 📝 `frontend/.gitignore` (+0 -2) 📝 `frontend/.npmrc` (+0 -4) 📝 `frontend/.stylelintrc.json` (+0 -1) ➖ `frontend/cypress.config.ts` (+0 -28) ➖ `frontend/cypress/README.md` (+0 -48) ➖ `frontend/cypress/docker-compose.yml` (+0 -15) ➖ `frontend/cypress/e2e/project/prepareProjects.ts` (+0 -61) ➖ `frontend/cypress/e2e/project/project-view-list.spec.ts` (+0 -58) ➖ `frontend/cypress/e2e/task/overview.spec.ts` (+0 -109) ➖ `frontend/cypress/e2e/tsconfig.json` (+0 -11) ➖ `frontend/cypress/e2e/user/login.spec.ts` (+0 -53) ➖ `frontend/cypress/e2e/user/settings.spec.ts` (+0 -43) ➖ `frontend/cypress/factories/bucket.ts` (+0 -19) ➖ `frontend/cypress/factories/label_task.ts` (+0 -16) ➖ `frontend/cypress/factories/labels.ts` (+0 -21) ➖ `frontend/cypress/factories/link_sharing.ts` (+0 -21) ➖ `frontend/cypress/factories/project.ts` (+0 -26) ➖ `frontend/cypress/factories/project_view.ts` (+0 -19) ➖ `frontend/cypress/factories/task.ts` (+0 -21) _...and 32 more files_ </details> ### 📄 Description ## Summary - Removes Cypress test framework entirely, using only Playwright for E2E tests - All Cypress tests were already covered by Playwright; added 2 missing tests for URL filter/search parameters - Removes ~2000 lines of Cypress code and configuration ## Changes - Deleted `cypress/` directory with all tests, factories, and support files - Removed Cypress dependencies: `@4tw/cypress-drag-drop`, `@cypress/vue`, `@cypress/vite-dev-server`, `cypress`, `start-server-and-test` - Removed `test:cypress:*` npm scripts - Cleaned up config files (`.gitignore`, `.npmrc`, `.stylelintrc.json`, `eslint.config.js`, `tsconfig.config.json`, `env.d.ts`) - Renamed `src/directives/cypress.ts` to `testid.ts` (still used for `data-cy` attributes which Playwright uses) - Added 2 tests to `project-view-list.spec.ts`: - "Should respect filter query parameter from URL" - "Should respect search query parameter from URL" ## Test plan - [x] Run `pnpm test:e2e` - 173 tests passing - [x] Run `pnpm lint` - passes - [ ] CI pipeline passes <!-- This is an auto-generated comment: release notes by coderabbit.ai --> ## Summary by CodeRabbit # Release Notes * **Chores** * Removed Cypress end-to-end testing infrastructure and configuration * Migrated testing framework from Cypress to Playwright * Removed test data factories and helper utilities * Updated ESLint and Stylelint configurations to reflect testing changes <sub>✏️ Tip: You can customize this high-level summary in your review settings.</sub> <!-- end of auto-generated comment: release notes by coderabbit.ai --> --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
GiteaMirror added the pull-request label 2026-04-23 09:08:52 -05:00
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
area/api
area/attachments
area/auth
area/avatars
area/backup-restore
area/caldav
area/calendar-view
area/comments
area/config
area/database
area/desktop
area/docker
area/email
area/favorites
area/filters
area/frontend
area/gantt
area/i18n
area/import-export
area/internal-code
area/kanban
area/labels
area/list-view
area/mobile
area/notifications
area/permissions
area/projects
area/pwa
area/recurring-tasks
area/reminders
area/search
area/shortcuts
area/subtasks
area/sync
area/table-view
area/task-editor
area/task-metadata
area/task-relations
area/teams
area/theming
area/time-tracking
area/typesense
area/views
area/webhooks
bug
changes requested
concern/accessibility
concern/performance
concern/regression
concern/ux
confirmed
db/mysql
dependencies
enhancement
good first issue
help wanted
integration/inbound
integration/outbound
kind/bug
kind/feature
needs reproduction
pull-request
Mirrored from GitHub Pull Request
 pull-request
Mirrored from GitHub Pull Request
 pull-request
Mirrored from GitHub Pull Request
 pull-request
Mirrored from GitHub Pull Request
 question
security
support
upstream issue
waiting for reply
wontfix
No labels pull-request
Milestone
No items
No Milestone
Projects
Clear projects
No projects
Assignees
Clear assignees
GiteaMirror ninjasurge (Kaehlen Rog)
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github-starred/vikunja#9680