-
Notifications
You must be signed in to change notification settings - Fork 348
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
test: Use DEADLINE_TZINFO to validate reminder due date calculation (#…
- Loading branch information
1 parent
43c7e92
commit c35afca
Showing
1 changed file
with
11 additions
and
5 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -34,7 +34,7 @@ | |
from ietf.utils.test_utils import login_testing_unauthorized | ||
from ietf.utils.mail import outbox, empty_outbox, get_payload_text | ||
from ietf.utils.test_utils import TestCase | ||
from ietf.utils.timezone import date_today, datetime_from_date | ||
from ietf.utils.timezone import date_today, datetime_from_date, DEADLINE_TZINFO | ||
|
||
|
||
class ChangeStateTests(TestCase): | ||
|
@@ -1749,8 +1749,11 @@ def test_set_initial_state(self): | |
self.assertEqual(draft.docevent_set.count() - events_before, 2) | ||
reminder = DocReminder.objects.filter(event__doc=draft, type="stream-s") | ||
self.assertEqual(len(reminder), 1) | ||
due = timezone.now() + datetime.timedelta(weeks=10) | ||
self.assertTrue(due - datetime.timedelta(days=1) <= reminder[0].due <= due + datetime.timedelta(days=1)) | ||
due = timezone.now().astimezone(DEADLINE_TZINFO) + datetime.timedelta(weeks=10) | ||
self.assertTrue( | ||
due - datetime.timedelta(days=1) <= reminder[0].due <= due + datetime.timedelta(days=1), | ||
f'Due date {reminder[0].due} should be {due} +/- 1 day' | ||
) | ||
self.assertEqual(len(outbox), 1) | ||
self.assertTrue("state changed" in outbox[0]["Subject"].lower()) | ||
self.assertTrue("[email protected]" in outbox[0].as_string()) | ||
|
@@ -1794,8 +1797,11 @@ def test_set_state(self): | |
self.assertEqual(draft.docevent_set.count() - events_before, 2) | ||
reminder = DocReminder.objects.filter(event__doc=draft, type="stream-s") | ||
self.assertEqual(len(reminder), 1) | ||
due = timezone.now() + datetime.timedelta(weeks=10) | ||
self.assertTrue(due - datetime.timedelta(days=1) <= reminder[0].due <= due + datetime.timedelta(days=1)) | ||
due = timezone.now().astimezone(DEADLINE_TZINFO) + datetime.timedelta(weeks=10) | ||
self.assertTrue( | ||
due - datetime.timedelta(days=1) <= reminder[0].due <= due + datetime.timedelta(days=1), | ||
f'Due date {reminder[0].due} should be {due} +/- 1 day' | ||
) | ||
self.assertEqual(len(outbox), 1) | ||
self.assertTrue("state changed" in outbox[0]["Subject"].lower()) | ||
self.assertTrue("[email protected]" in outbox[0].as_string()) | ||
|