Skip to content
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

Policy Group e2e & additional resource and policy tests #159

Open
wants to merge 11 commits into
base: main
Choose a base branch
from

Conversation

pickjasmine
Copy link
Contributor

  • Adds some e2e tests around creating policy group and adding an assignment
    • There is a test for removing assignment but one of the steps is not behaving the way I expect it to and I don't want to waste more time on it. There is a TODO in that step definition. The removal of the assignment shows correctly in the UI; the issue is when you get redirected back to the assignments page and the stubbing in Cypress is not behaving as expected.
  • Adds some additional tests around resource evaluation history, changing resource version, policy assignments, policy version history
  • Also made a couple of updates to some API endpoints that are paginated in Rode but not the UI. I'll make a note in some documentation around those specific calls so we don't forget

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant