-
Notifications
You must be signed in to change notification settings - Fork 32
Add unit tests for vmhost #957
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
Conversation
…it, without fixing.
…it, without fixing.
…it, without fixing.
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## rc/after-supernova #957 +/- ##
======================================================
+ Coverage 36.23% 41.19% +4.95%
======================================================
Files 87 87
Lines 20589 20587 -2
======================================================
+ Hits 7460 8480 +1020
+ Misses 12462 11210 -1252
- Partials 667 897 +230 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
| ) | ||
|
|
||
| func TestAddAsyncArgumentsToOutputTransfers(t *testing.T) { | ||
| t.Parallel() |
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.
Does it help for these tests to run in parallel? It looks like they are only testing object construction.
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.
always help. it is faster.
No description provided.