-
Notifications
You must be signed in to change notification settings - Fork 354
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: add intg tests for tasks module #8750
Conversation
✅ Deploy Preview for determined-ui ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #8750 +/- ##
==========================================
- Coverage 47.58% 47.27% -0.31%
==========================================
Files 1049 1049
Lines 167354 167345 -9
Branches 2241 2241
==========================================
- Hits 79633 79118 -515
- Misses 87563 88069 +506
Partials 158 158
Flags with carried forward coverage won't be shown. Click here to find out more.
|
32092d7
to
d1bb042
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.
A few comments but the rest looks great!
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.
Looks good to me, nice work
I would wait for @salonig23 's approval too
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, great work!
8b72e5d
to
da2f941
Compare
Description
Add integration tests for all exported functions in
db/postgres_tasks.go
, and initialize the db in TestMain.Add bun descriptions to the AllocationSession & TaskLog models, so that the ID key is auto-incremented with each addition.
Test Plan
See attached test.
Commentary (optional)
Truncated the start time in
RequireMockAllocation
to match what the db stores.Remove
CheckTaskExists
frompostgres_tasks.go
, as it's used nowhere.Writing the tests first, then bun-ifying db/postgres_tasks.go in #8764
Checklist
docs/release-notes/
.See Release Note for details.
Ticket
DET-10124