From 28847cdf6f2723f1c5669abfa24fe8ecb92a7500 Mon Sep 17 00:00:00 2001 From: mongkok Date: Sun, 2 Aug 2020 08:54:05 +0700 Subject: [PATCH] Improved assertions --- tests/test_middleware.py | 20 ++++++++++---------- 1 file changed, 10 insertions(+), 10 deletions(-) diff --git a/tests/test_middleware.py b/tests/test_middleware.py index 74a94363..6ede12a4 100644 --- a/tests/test_middleware.py +++ b/tests/test_middleware.py @@ -28,7 +28,7 @@ def test_authenticate(self): self.middleware.resolve(next_mock, None, info_mock) - next_mock.assert_called_with(None, info_mock) + next_mock.assert_called_once_with(None, info_mock) self.assertEqual(info_mock.context.user, self.user) @override_jwt_settings(JWT_ALLOW_ANY_HANDLER=lambda *args: False) @@ -44,8 +44,8 @@ def test_not_authenticate(self, authenticate_mock): self.middleware.resolve(next_mock, None, info_mock) - next_mock.assert_called_with(None, info_mock) - authenticate_mock.assert_called_with(request=info_mock.context) + next_mock.assert_called_once_with(None, info_mock) + authenticate_mock.assert_called_once_with(request=info_mock.context) self.assertIsInstance(info_mock.context.user, AnonymousUser) @override_jwt_settings(JWT_ALLOW_ANY_HANDLER=lambda *args: False) @@ -75,7 +75,7 @@ def test_already_authenticated(self, authenticate_mock): self.middleware.resolve(next_mock, None, info_mock) - next_mock.assert_called_with(None, info_mock) + next_mock.assert_called_once_with(None, info_mock) authenticate_mock.assert_not_called() @override_jwt_settings(JWT_ALLOW_ANY_HANDLER=lambda *args: True) @@ -90,7 +90,7 @@ def test_allow_any(self): self.middleware.resolve(next_mock, None, info_mock) - next_mock.assert_called_with(None, info_mock) + next_mock.assert_called_once_with(None, info_mock) self.assertIsInstance(info_mock.context.user, AnonymousUser) def test_authenticate_context(self): @@ -122,7 +122,7 @@ def test_authenticate(self): self.middleware.resolve(next_mock, None, info_mock, **kwargs) - next_mock.assert_called_with(None, info_mock, **kwargs) + next_mock.assert_called_once_with(None, info_mock, **kwargs) self.assertEqual(info_mock.context.user, self.user) user = self.middleware.cached_authentication[tuple(info_mock.path)] @@ -137,7 +137,7 @@ def test_authenticate_parent(self): self.middleware.cached_authentication.insert(['0'], self.user) self.middleware.resolve(next_mock, None, info_mock) - next_mock.assert_called_with(None, info_mock) + next_mock.assert_called_once_with(None, info_mock) self.assertEqual(info_mock.context.user, self.user) @override_jwt_settings(JWT_ALLOW_ARGUMENT=True) @@ -147,7 +147,7 @@ def test_clear_authentication(self): self.middleware.resolve(next_mock, None, info_mock) - next_mock.assert_called_with(None, info_mock) + next_mock.assert_called_once_with(None, info_mock) self.assertIsInstance(info_mock.context.user, AnonymousUser) @override_jwt_settings(JWT_ALLOW_ARGUMENT=True) @@ -158,7 +158,7 @@ def test_clear_session_authentication(self): self.middleware.resolve(next_mock, None, info_mock) - next_mock.assert_called_with(None, info_mock) + next_mock.assert_called_once_with(None, info_mock) self.assertIsInstance(info_mock.context.user, AnonymousUser) @override_jwt_settings(JWT_ALLOW_ARGUMENT=True) @@ -168,7 +168,7 @@ def test_context_has_not_attr_user(self): self.middleware.resolve(next_mock, None, info_mock) - next_mock.assert_called_with(None, info_mock) + next_mock.assert_called_once_with(None, info_mock) self.assertFalse(hasattr(info_mock.context, 'user'))