Skip to content

Commit e3ab546

Browse files
committed
feat: TT-122 update tests
1 parent 496b801 commit e3ab546

File tree

2 files changed

+18
-18
lines changed

2 files changed

+18
-18
lines changed

tests/time_tracker_api/users/users_namespace_test.py

Lines changed: 1 addition & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@
22
from flask import json
33
from flask.testing import FlaskClient
44
from flask_restplus._http import HTTPStatus
5-
from utils.azure_users import AzureConnection, ROLE_FIELD_VALUES, AzureUser_v2
5+
from utils.azure_users import AzureConnection
66
from pytest import mark
77

88

@@ -186,19 +186,3 @@ def test_update_role_is_called_properly_on_each_action(
186186
update_role_mock.assert_called_once_with(
187187
user_id, role_id, is_grant=is_grant
188188
)
189-
190-
191-
@patch('msal.ConfidentialClientApplication', Mock())
192-
@patch('utils.azure_users.AzureConnection.get_token', Mock())
193-
@patch('utils.azure_users.AzureConnection.get_test_user_ids')
194-
@patch('utils.azure_users.AzureConnection.users_v2')
195-
def test_azure_connection_get_non_test_users(
196-
users_v2_mock, get_test_user_ids_mock
197-
):
198-
test_user = AzureUser_v2('ID1', None, None, [])
199-
non_test_user = AzureUser_v2('ID2', None, None, [])
200-
users_v2_mock.return_value = [test_user, non_test_user]
201-
get_test_user_ids_mock.return_value = ['ID1']
202-
non_test_users = [non_test_user]
203-
az_conn = AzureConnection()
204-
assert az_conn.get_non_test_users() == non_test_users

tests/utils/azure_users_test.py

Lines changed: 17 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
from unittest.mock import Mock, patch
2-
from utils.azure_users import AzureConnection, ROLE_FIELD_VALUES
2+
from utils.azure_users import AzureConnection, ROLE_FIELD_VALUES, AzureUser_v2
33
from pytest import mark
44

55

@@ -40,3 +40,19 @@ def test_azure_connection_get_test_user_ids(get_mock):
4040
ids = ['ID1', 'ID2']
4141
az_conn = AzureConnection()
4242
assert az_conn.get_test_user_ids() == ids
43+
44+
45+
@patch('msal.ConfidentialClientApplication', Mock())
46+
@patch('utils.azure_users.AzureConnection.get_token', Mock())
47+
@patch('utils.azure_users.AzureConnection.get_test_user_ids')
48+
@patch('utils.azure_users.AzureConnection.users_v2')
49+
def test_azure_connection_get_non_test_users(
50+
users_v2_mock, get_test_user_ids_mock
51+
):
52+
test_user = AzureUser_v2('ID1', None, None, [])
53+
non_test_user = AzureUser_v2('ID2', None, None, [])
54+
users_v2_mock.return_value = [test_user, non_test_user]
55+
get_test_user_ids_mock.return_value = ['ID1']
56+
non_test_users = [non_test_user]
57+
az_conn = AzureConnection()
58+
assert az_conn.get_non_test_users() == non_test_users

0 commit comments

Comments
 (0)