-
Notifications
You must be signed in to change notification settings - Fork 172
Add E2E tests for secrets management #3485
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
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #3485 +/- ##
==========================================
+ Coverage 65.35% 65.36% +0.01%
==========================================
Files 403 403
Lines 39210 39210
==========================================
+ Hits 25625 25631 +6
+ Misses 11599 11591 -8
- Partials 1986 1988 +2 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
a910ab4 to
4da9af4
Compare
eleftherias
left a comment
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.
The 404 tests are failing with a 405. Also added one question inline.
8de7cb0 to
7b47971
Compare
7b47971 to
4253787
Compare
4253787 to
d5f9b1b
Compare
2f65d9d to
92a793a
Compare
f61a69a to
641fa28
Compare
5126c53 to
8d85a4f
Compare
Due to dependency issues (keychain for encrypted, or 1password), these tests only test the environment type. This does not cover all secrets management functionality, but verifies basic correctness of the secrets endpoints.