Skip to content

Commit cd38655

Browse files
committed
server/test: fix some incorrect tests
1 parent e43b2c7 commit cd38655

File tree

2 files changed

+4
-3
lines changed

2 files changed

+4
-3
lines changed

server/szurubooru/tests/api/test_post_merging.py

+1-1
Original file line numberDiff line numberDiff line change
@@ -32,7 +32,7 @@ def test_merging(user_factory, context_factory, post_factory):
3232
user=auth_user,
3333
)
3434
)
35-
posts.merge_posts.assert_called_once_with(source_post, target_post)
35+
posts.merge_posts.assert_called_once_with(source_post, target_post, False)
3636
snapshots.merge.assert_called_once_with(
3737
source_post, target_post, auth_user
3838
)

server/szurubooru/tests/api/test_user_creating.py

+3-2
Original file line numberDiff line numberDiff line change
@@ -28,6 +28,7 @@ def test_creating_user(user_factory, context_factory, fake_datetime):
2828
):
2929
users.serialize_user.return_value = "serialized user"
3030
users.create_user.return_value = user
31+
auth_user = user_factory(rank=model.User.RANK_REGULAR)
3132
result = api.user_api.create_user(
3233
context_factory(
3334
params={
@@ -38,7 +39,7 @@ def test_creating_user(user_factory, context_factory, fake_datetime):
3839
"avatarStyle": "manual",
3940
},
4041
files={"avatar": b"..."},
41-
user=user_factory(rank=model.User.RANK_REGULAR),
42+
user=auth_user,
4243
)
4344
)
4445
assert result == "serialized user"
@@ -48,7 +49,7 @@ def test_creating_user(user_factory, context_factory, fake_datetime):
4849
assert not users.update_user_name.called
4950
assert not users.update_user_password.called
5051
assert not users.update_user_email.called
51-
users.update_user_rank.assert_called_once_with(user, "moderator")
52+
users.update_user_rank.assert_called_once_with(user, "moderator", auth_user)
5253
users.update_user_avatar.assert_called_once_with(user, "manual", b"...")
5354

5455

0 commit comments

Comments
 (0)