-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
Update integration-tests README #14577
base: develop
Are you sure you want to change the base?
Conversation
5468f8c
to
95bffd2
Compare
e6ce4a0
to
00e01ef
Compare
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.
@chudilka1 it lgtm, but I've added some comments.
|
||
For security reasons, test secrets and sensitive information are not stored directly within the test config TOML files. Instead, these secrets are securely injected into tests using environment variables. For a detailed explanation on managing test secrets, refer to our [Test Secrets documentation](https://github.com/smartcontractkit/chainlink-testing-framework/blob/main/lib/config/README.md#test-secrets). |
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.
For security reasons, test secrets and sensitive information are not stored directly within the test config TOML files. Instead, these secrets are securely injected into tests using environment variables.
@chudilka1 why you removed this?
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.
I've removed it, as there are guidelines and references to the corresponding documents that clearly explain that. Moreover, tests will fail if there are no test secrets. There is no need to duplicate this information everywhere.
bf0a26b
to
00fa7fe
Compare
5ccdd6b
to
7eff681
Compare
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.
Minor observation/question.
Good job with this 🙌
7eff681
to
2e8fa80
Compare
Quality Gate passedIssues Measures |
BCF-3396
Audit and update integration-tests READMEs.