Skip to content

Commit 69caace

Browse files
test: prevent periodic failure of test_send_reminder_all_open_reviews() (ietf-tools#3584)
1 parent f196f50 commit 69caace

1 file changed

Lines changed: 10 additions & 4 deletions

File tree

ietf/review/tests.py

Lines changed: 10 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,8 @@
1111
from .utils import (email_secretary_reminder, review_assignments_needing_secretary_reminder,
1212
email_reviewer_reminder, review_assignments_needing_reviewer_reminder,
1313
send_reminder_unconfirmed_assignments, send_review_reminder_overdue_assignment,
14-
send_reminder_all_open_reviews, send_unavailability_period_ending_reminder)
14+
send_reminder_all_open_reviews, send_unavailability_period_ending_reminder,
15+
ORIGIN_DATE_PERIODIC_REMINDERS)
1516

1617
class HashTest(TestCase):
1718

@@ -28,7 +29,7 @@ def test_hash_list_message_id(self):
2829
(b'xml2rfc', b'3A0F4CD6-451F-44E2-9DA4-28235C638588@rfc-editor.org','g6DN4SxJGDrlSuKsubwb6rRSePU'),
2930
):
3031
self.assertEqual(hash, hash_list_message_id(list, msgid))
31-
32+
3233

3334
class ReviewAssignmentTest(TestCase):
3435
def do_test_update_review_req_status(self, assignment_state, expected_state):
@@ -468,6 +469,7 @@ def test_send_review_reminder_overdue_assignment(self):
468469
self.assertIn('1 overdue review', log[0])
469470

470471
def test_send_reminder_all_open_reviews(self):
472+
today = datetime.date.today()
471473
self.make_secretary(username='reviewsecretary')
472474
ReviewerSettingsFactory(team=self.team, person=self.reviewer, remind_days_open_reviews=1)
473475

@@ -483,10 +485,14 @@ def test_send_reminder_all_open_reviews(self):
483485
assigned_on=second_req.time,
484486
reviewer=self.reviewer.email(),
485487
)
486-
ReviewerSettingsFactory(team=second_team, person=self.reviewer, remind_days_open_reviews=13)
488+
ReviewerSettingsFactory(
489+
team=second_team,
490+
person=self.reviewer,
491+
# set the reminder never to be due to be sent today for this team
492+
remind_days_open_reviews=(today - ORIGIN_DATE_PERIODIC_REMINDERS).days + 1,
493+
)
487494

488495
empty_outbox()
489-
today = datetime.date.today()
490496
log = send_reminder_all_open_reviews(today)
491497

492498
self.assertEqual(len(outbox), 1)

0 commit comments

Comments
 (0)