fix: flaky assignment test#103
Conversation
There was a problem hiding this comment.
Pull request overview
Stabilizes a flaky content assignment task test by making the “start_date” trigger property deterministic via frozen time, aligning the expected Braze payload with the time-dependent logic in get_self_paced_normalized_start_date().
Changes:
- Freeze time for
test_send_email_for_new_assignmentto eliminate dependence on the wall clock. - Replace the expected
start_datefromdatetime.now()-derived to a fixed timestamp matching the frozen time.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #103 +/- ##
=======================================
Coverage 84.01% 84.01%
=======================================
Files 144 144
Lines 12226 12226
Branches 1170 1170
=======================================
Hits 10272 10272
Misses 1625 1625
Partials 329 329 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
957aeaa to
e40102c
Compare
e40102c to
875517d
Compare
There was a problem hiding this comment.
Pull request overview
This pull request updates a Braze email task test to be deterministic by freezing the system time and aligning the expected start_date trigger property with the frozen timestamp.
Changes:
- Freeze time for
test_send_email_for_new_assignmentusingfreezegun.freeze_time. - Replace the previously dynamic
start_dateexpectation with a fixed value matching the frozen time.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
No description provided.