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

fix failing tests

Browse Source 
make sure we're using notify_db_session to ensure test independence
This commit is contained in:
Leo Hemsted
2016-12-28 13:35:41 +00:00
parent d550893377
commit 996bd2579a
1 changed files with 4 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
tests/app/dao/test_users_dao.py
View File

@@ -46,16 +46,17 @@ def test_get_all_users(notify_db_session):
def test_get_user(notify_db_session):
user = create_user(email='1@test.com')
email = '1@test.com'
user = create_user(email=email)
assert get_user_by_id(user_id=user.id).email_address == email
def test_get_user_not_exists(notify_db, fake_uuid):
def test_get_user_not_exists(notify_db_session, fake_uuid):
with pytest.raises(NoResultFound):
get_user_by_id(user_id=fake_uuid)
def test_get_user_invalid_id(notify_db):
def test_get_user_invalid_id(notify_db_session):
with pytest.raises(DataError):
get_user_by_id(user_id="blah")