From 9dc1a5f045f89afc305f2fd6f5d68f8f120f89e0 Mon Sep 17 00:00:00 2001 From: Ben Bangert Date: Mon, 25 Jun 2018 10:28:26 -0700 Subject: [PATCH] fix: remove legacy notification saving to address intermittent fails 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 --- autopush/tests/test_integration.py | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/autopush/tests/test_integration.py b/autopush/tests/test_integration.py index e9bb79e3..87628d80 100644 --- a/autopush/tests/test_integration.py +++ b/autopush/tests/test_integration.py @@ -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), " ") @@ -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), " ")