search

permitio / opal

Policy and data administration, distribution, and real-time updates on top of Policy Agents (OPA, Cedar, ...)
https://opal.ac
Apache License 2.0
5.14k stars 180 forks source link

feat: E2E tests framework using PyTest #681

Open varshith257 opened 1 month ago

varshith257 commented 1 month ago

Fixes Issue

Closes #677 /claim #677

Changes proposed

Check List (Check all the applicable boxes)

Screenshots

Note to reviewers

netlify[bot] commented 1 month ago

Deploy Preview for opal-docs canceled.

Name Link
Latest commit 1c6b3ea594ada73787ac707d4f56c9ebc7f31737
Latest deploy log https://app.netlify.com/sites/opal-docs/deploys/672340baa950690008fbb4c8
varshith257 commented 1 month ago

@gemanor / @danyi1212 Have a check on the initial configuration done. Once it is on the way expected I proceed for next steps as outlined in the issue

varshith257 commented 1 month ago

@gemanor Any review on this?

gemanor commented 1 month ago

Hey, @varshith257, this is a very basic PR, and we are looking for something much more extensive from that:

  1. Explain your design and why you choose to run the docker as a sub-process
  2. Why is there a new docker requirement?
  3. Write a guidelines for new tests
  4. Run the assertions and tests we asked
  5. Create more stable tests of the log checks etc.

You can continue the discussion and show the plan here before continue with the code if you prefer

varshith257 commented 1 month ago

@gemanor Yeah I have just laid out foundation and pushed here to know whether it aligns with the same.

Yeah! I will document all the above :)