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
chore: resource state tests #9307
chore: resource state tests #9307
Conversation
Signed-off-by: pashavictorovich <pavel@codefresh.io>
Codecov Report
@@ Coverage Diff @@
## master #9307 +/- ##
==========================================
+ Coverage 46.06% 46.12% +0.06%
==========================================
Files 217 217
Lines 25908 25908
==========================================
+ Hits 11934 11951 +17
+ Misses 12317 12299 -18
- Partials 1657 1658 +1
Continue to review full report at Codecov.
|
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.
Just nitpicks. :-)
cmd/argocd/commands/app_test.go
Outdated
} | ||
|
||
key := rst.Key() | ||
assert.Equal(t, key, "group/kind/namespace/name") |
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 think the assertion is backwards here. shouldn't the expected value be first?
assert.False(t, updated) | ||
} | ||
|
||
func TestCheckResourceStatus(t *testing.T) { |
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.
Not required but would be nice: could you add a docstring to the checkResourceStatus function? It's not immediately obvious what it's meant to do.
Signed-off-by: pashavictorovich <pavel@codefresh.io>
Resource state tests inside app.cmd