Skip to content

Commit 889a73c

Browse files
committed
improve tests
1 parent 39129f2 commit 889a73c

2 files changed

Lines changed: 8 additions & 3 deletions

File tree

server/mergin/tests/test_public_api_v2.py

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@
77
from . import DEFAULT_USER
88
from .utils import (
99
add_user,
10-
login,
10+
logout,
1111
login_as_admin,
1212
create_workspace,
1313
create_project,
@@ -142,11 +142,10 @@ def test_project_members(client):
142142

143143
def test_get_project(client):
144144
"""Test get project info endpoint"""
145-
add_user("test_user", "ilovemergin")
146-
login(client, "test_user", "ilovemergin")
147145
admin = User.query.filter_by(username=DEFAULT_USER[0]).first()
148146
test_workspace = create_workspace()
149147
project = create_project("new_project", test_workspace, admin)
148+
logout(client)
150149
# lack of permissions
151150
response = client.get(f"v2/projects/{project.id}")
152151
assert response.status_code == 403

server/mergin/tests/utils.py

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -387,3 +387,9 @@ def modify_file_times(path, time: datetime, accessed=True, modified=True):
387387
mtime = epoch_time if modified else file_stat.st_mtime
388388

389389
os.utime(path, (atime, mtime))
390+
391+
392+
def logout(client):
393+
"""Test helper to log out the client"""
394+
resp = client.get(url_for("/.mergin_auth_controller_logout"))
395+
assert resp.status_code == 200

0 commit comments

Comments
 (0)