-
Notifications
You must be signed in to change notification settings - Fork 15
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
feat(secret): add source secrets #82
base: main
Are you sure you want to change the base?
feat(secret): add source secrets #82
Conversation
fdf1db5
to
6d49c91
Compare
Codecov ReportAttention: Patch coverage is
❌ Your patch check has failed because the patch coverage (55.88%) is below the target coverage (80.00%). You can increase the patch coverage or adjust the target coverage. Additional details and impacted files@@ Coverage Diff @@
## main #82 +/- ##
==========================================
+ Coverage 80.06% 80.07% +0.01%
==========================================
Files 544 544
Lines 21155 21189 +34
Branches 5326 5072 -254
==========================================
+ Hits 16937 16967 +30
- Misses 4194 4198 +4
Partials 24 24
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report in Codecov by Sentry.
|
LGTM All the modifications are working properly.
Probably there are other things to fix since there were changes on how the build time secret was model Is it possible to cover the missing lines in Codecov/patch? |
@JoaoPedroPP thanks for the references JP, I'll look into these |
8a9c729
to
85f9b23
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.
LGTM
402d890
to
da46146
Compare
/test |
{currentType === SecretTypeDropdownLabel.source && <SourceSecretForm />} | ||
{currentType !== SecretTypeDropdownLabel.source && ( | ||
<KeyValueFileInputField | ||
required | ||
name={ | ||
currentType === SecretTypeDropdownLabel.opaque ? 'opaque.keyValues' : 'image.keyValues' | ||
} | ||
entries={defaultKeyValues} | ||
disableRemoveAction={values.opaque.keyValues.length === 1} | ||
/> | ||
)} |
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.
you need to test this conditional rendering to make the codecov happy.
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.
added by mocking the component
24ac319
to
25eb65f
Compare
LGTM |
Fixes
https://issues.redhat.com/browse/HAC-5834
Description
PR ready for review
Type of change
Screen shots / Gifs for design review
How to test or reproduce?
Browser conformance: