diff --git a/atst/domain/users.py b/atst/domain/users.py index b276a5bd..baf5acb0 100644 --- a/atst/domain/users.py +++ b/atst/domain/users.py @@ -85,7 +85,7 @@ class Users(object): @classmethod def update_last_login(cls, user): - setattr(user, "last_login", datetime.now()) + user.last_login = datetime.now() db.session.add(user) db.session.commit() diff --git a/templates/footer.html b/templates/footer.html index 340b10cf..39e1bf96 100644 --- a/templates/footer.html +++ b/templates/footer.html @@ -9,7 +9,7 @@ {% if g.last_login %}
- Last Login: + Last Login:
{% endif %} diff --git a/tests/domain/test_users.py b/tests/domain/test_users.py index 1c0cdcbc..116e5294 100644 --- a/tests/domain/test_users.py +++ b/tests/domain/test_users.py @@ -69,12 +69,8 @@ def test_update_user_with_dod_id(): def test_update_user_with_last_login(): - new_user = UserFactory.create(last_login=datetime.now()) + new_user = UserFactory.create() Users.update_last_login(new_user) last_login = new_user.last_login - - with pytest.raises(UnauthorizedError): - Users.update(new_user, {"last_login": datetime.now()}) - Users.update_last_login(new_user) assert new_user.last_login > last_login