-
Notifications
You must be signed in to change notification settings - Fork 84
Add cypress test coverage for new request manager page #7164
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add cypress test coverage for new request manager page #7164
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
1 Skipped Deployment
|
Greptile SummaryAdds comprehensive Cypress test coverage for the new request manager page, testing dashboard rendering, filters, pagination, bulk actions, tab navigation, permissions, and request creation. Key Changes:
Minor Issue:
Confidence Score: 4/5
Important Files Changed
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
1 file reviewed, 1 comment
Ticket ENG-1626
Description Of Changes
Adds cypress test coverage for the new privacy request. There will be a followup PR that adds more unit tests as well.
The tests here aim to do a general testing on the page on rendering and the main action, but intentionally doesn't cover everything. eg I added a test for a search filter and a dropdown filter, but not all filters. the unit tests instead will test every single filter.
Here's the list of tests added:
New Privacy Requests
The requests dashboard
Request list items
Bulk actions
Tab navigation
privacy request creation
showing button depending on role
submitting a request
Code Changes
Steps to Confirm
Pre-Merge Checklist
CHANGELOG.mdupdatedmaindowngrade()migration is correct and works