Skip to content

feat: add Cypress e2e tests for dashboard push actions #1403

Open
fabiovincenzi wants to merge 19 commits intofinos:mainfrom
fabiovincenzi:feat/e2e-dashboard-tests
Open

feat: add Cypress e2e tests for dashboard push actions #1403
fabiovincenzi wants to merge 19 commits intofinos:mainfrom
fabiovincenzi:feat/e2e-dashboard-tests

Conversation

@fabiovincenzi
Copy link
Contributor

@fabiovincenzi fabiovincenzi commented Feb 12, 2026

Fixes #1390

  • Add Cypress e2e tests for the admin UI dashboard's push management flows (approve, reject, cancel)
  • Fix disabled prop not being forwarded in CustomButton component, which prevented the attestation confirm button from being properly disabled
  • Add data-testid attributes to push detail action buttons and attestation dialog for reliable test selectors
  • Organise Cypress tests into cypress/e2e/ (local) and cypress/e2e/docker/ (require Docker); local tests continue to run in the Node/MongoDB matrix in ci.yml, Docker tests run in e2e.yml
  • Add cypress:run:docker script targeting Docker-only tests
  • Add Cypress Docker step to e2e.yml alongside existing vitest e2e tests, with log dump and screenshot upload on failure
  • Make existing Cypress tests (repo.cy.js) compatible with Docker environment by replacing hardcoded URLs with Cypress.env('API_BASE_URL')
  • Pass git credentials via GIT_CONFIG_* env vars in createPush to avoid exposing them in CI logs

@netlify
Copy link

netlify bot commented Feb 12, 2026

Deploy Preview for endearing-brigadeiros-63f9d0 canceled.

Name Link
🔨 Latest commit e36872a
🔍 Latest deploy log https://app.netlify.com/projects/endearing-brigadeiros-63f9d0/deploys/6996d7d5645df30008824b8f

@fabiovincenzi fabiovincenzi changed the title Feat/e2e dashboard tests feat: add Cypress e2e tests for dashboard push actions Feb 12, 2026
@codecov
Copy link

codecov bot commented Feb 12, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 81.37%. Comparing base (468bd12) to head (e36872a).

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1403   +/-   ##
=======================================
  Coverage   81.37%   81.37%           
=======================================
  Files          67       67           
  Lines        4735     4735           
  Branches      819      819           
=======================================
  Hits         3853     3853           
  Misses        867      867           
  Partials       15       15           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@fabiovincenzi fabiovincenzi requested a review from a team as a code owner February 17, 2026 11:36
@fabiovincenzi fabiovincenzi marked this pull request as draft February 17, 2026 18:07
@fabiovincenzi fabiovincenzi marked this pull request as ready for review February 18, 2026 10:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Implement e2e tests for admin UI's dashboard

1 participant

Comments