-
Notifications
You must be signed in to change notification settings - Fork 12
feat(e2e-tests): infisical proxy coverage #108
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
Greptile SummaryAdded comprehensive end-to-end tests for Infisical Proxy functionality. The tests validate cache hit/miss behavior, mutation-based cache purging (for updates and deletes), token invalidation, high availability when backend is down, background refresh of cached secrets, multiple secret handling, and single secret endpoint caching. Key changes:
The test suite follows existing patterns and provides good coverage of the proxy's caching behavior and resilience features. Confidence Score: 5/5
Important Files Changed
|
akhilmhdh
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.
Overal looking good. I would recommend to remove all time.sleep and then check for stdErr.
Even though it's not affecting us now - others or llm will keep picking that pattern and ends up using it more often. We should try to use the testing library one waitForStdErr as it contains timeout, retry mechanism in place - so every it's used as it is.
|
Application testing pending |
Description 📣
End to end tests for Infisical Proxy.
We are covering:
Type ✨
Tests 🛠️