Skip to content
This repository has been archived by the owner on Jul 13, 2023. It is now read-only.

Commit

Permalink
fix: remove legacy notification saving to address intermittent fails
Browse files Browse the repository at this point in the history
We don't actually need to test legacy notification saving here, since
this is supposed to test monthly table rotation. The test would fail
randomly based on whether the channel UUID started with the first two
characters being less than '01' as fetch_timestamp_messages requires
that to locate them.

Closes #878
  • Loading branch information
bbangert committed Jun 25, 2018
1 parent 234b64b commit 9dc1a5f
Showing 1 changed file with 2 additions and 4 deletions.
6 changes: 2 additions & 4 deletions autopush/tests/test_integration.py
Original file line number Diff line number Diff line change
Expand Up @@ -1263,8 +1263,7 @@ def test_webpush_monthly_rotation(self):
# Send in a notification, verify it landed in last months notification
# table
data = uuid.uuid4().hex
with self.legacy_endpoint():
yield client.send_notification(data=data)
yield client.send_notification(data=data)
ts, notifs = yield deferToThread(lm_message.fetch_timestamp_messages,
uuid.UUID(client.uaid),
" ")
Expand Down Expand Up @@ -1366,8 +1365,7 @@ def test_webpush_monthly_rotation_prior_record_exists(self):
# Send in a notification, verify it landed in last months notification
# table
data = uuid.uuid4().hex
with self.legacy_endpoint():
yield client.send_notification(data=data)
yield client.send_notification(data=data)
_, notifs = yield deferToThread(lm_message.fetch_timestamp_messages,
uuid.UUID(client.uaid),
" ")
Expand Down

0 comments on commit 9dc1a5f

Please sign in to comment.