Enable CyberArk E2E and integration tests in CI #753
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Enable "ARK" tests in CI; those are tests which connect to real CyberArk Disco
APIs. This will reduce the risk of us releasing a broken the disco-agent chart.
Additionally, use a real UID in integration tests to satisfy real API validation
And add a cluster name, a shorter upload period, and a sample secret so that
it's easier to find the uploaded Secret in the UI.
Testing
The integration tests ran and succeeded, but with one failure due to a slow API response, so perhaps running these new tests for every PR commit, is going to cause frustration:
The E2E CI job ran and succeeded:
Here's a screenshot of the uploaded secrets