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
test (gradle-plugin/it) : Add test for setting ServiceAccount name in generated controller (#2187) #2188
Conversation
Eclipse JKube CI ReportStarted new GH workflow run for #2188 (2023-05-31T14:37:29Z) ⚙️ JKube E2E Tests (5133924218)
|
enricher { | ||
config { | ||
'jkube-serviceaccount' { | ||
skipCreate = true | ||
} | ||
} | ||
} |
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.
What's the purpose of 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.
ServiceAccountEnricher seems to create additional ServiceAccount resource referenced in Deployment. This flag tells ServiceAccountEnricher to not do so.
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 skipCreate
configuration.
static Stream<Arguments> testInput() { | ||
return Stream.of( | ||
// arguments("no-fragment", new String[] {}), | ||
// arguments("fragment", new String[] {"-Pjkube.resourceDir=./regularFragmentDir"}), |
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.
From the test expectations, the assumption here is that a fragment with an unrelated field has no action over the serviceAccountName field, right?
Could we state this more clearly? A simple "fragment" seems to convey the opposite assumption (I have a fragment with a service account -> what we test in the next case). So maybe an unrelated-fragment
name might be more suitable.
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 renamedfragment
-> unrelated-fragment
Codecov Report
@@ Coverage Diff @@
## master #2188 +/- ##
=========================================
Coverage 57.82% 57.82%
Complexity 4398 4398
=========================================
Files 485 485
Lines 21114 21114
Branches 2802 2802
=========================================
Hits 12209 12209
Misses 7675 7675
Partials 1230 1230 📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
… generated controller (eclipse-jkube#2187) Signed-off-by: Rohan Kumar <rohaan@redhat.com>
d6be6b8
to
c7e7818
Compare
Kudos, SonarCloud Quality Gate passed! 0 Bugs No Coverage information |
Description
Test case covering #2187
Failing cases have been commented. I'll remove comment in #1889
Type of change
test, version modification, documentation, etc.)
Checklist