darkhelm/notifications-api
1
0
Fork 0
mirror of https://github.com/GSA/notifications-api.git synced 2026-02-04 02:11:11 -05:00
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #463 from alphagov/test-refactor

Browse Source 
Remove db.drop_all from notify_db session fixture to speed up tests
This commit is contained in:
Leo Hemsted
2016-06-22 16:53:06 +01:00
committed by GitHub
parent 5f29815870 93618d2a7e
commit 4ce26dc1a4
2 changed files with 8 additions and 9 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
tests/app/dao/test_notification_dao.py
View File

@@ -647,7 +647,7 @@ def test_update_notification(sample_notification, sample_template):
def test_should_delete_notifications_after_seven_days(notify_db, notify_db_session):
assert len(Notification.query.all()) == 0
# create one notification a day between 1st and 9th from 11:00 to 19:00
# create one notification a day between 1st and 10th from 11:00 to 19:00
for i in range(1, 11):
past_date = '2016-01-{0:02d} {0:02d}:00:00.000000'.format(i, i)
with freeze_time(past_date):
@@ -660,7 +660,8 @@ def test_should_delete_notifications_after_seven_days(notify_db, notify_db_sessi
delete_notifications_created_more_than_a_week_ago('failed')
remaining_notifications = Notification.query.all()
assert len(remaining_notifications) == 8
assert remaining_notifications[0].created_at.date() == date(2016, 1, 3)
for notification in remaining_notifications:
assert notification.created_at.date() >= date(2016, 1, 3)
def test_should_not_delete_failed_notifications_before_seven_days(notify_db, notify_db_session):