Skip to content
This repository was archived by the owner on Apr 26, 2024. It is now read-only.

Commit 23f5106

Browse files
committed
Fix tests on develop
This was broken due to #8617 and #8761.
1 parent b690542 commit 23f5106

File tree

1 file changed

+0
-16
lines changed

1 file changed

+0
-16
lines changed

tests/rest/admin/test_user.py

Lines changed: 0 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -1574,15 +1574,13 @@ def _get_token(self) -> str:
15741574
request, channel = self.make_request(
15751575
"POST", self.url, b"{}", access_token=self.admin_user_tok
15761576
)
1577-
self.render(request)
15781577
self.assertEqual(200, int(channel.result["code"]), msg=channel.result["body"])
15791578
return channel.json_body["access_token"]
15801579

15811580
def test_no_auth(self):
15821581
"""Try to login as a user without authentication.
15831582
"""
15841583
request, channel = self.make_request("POST", self.url, b"{}")
1585-
self.render(request)
15861584

15871585
self.assertEqual(401, int(channel.result["code"]), msg=channel.result["body"])
15881586
self.assertEqual(Codes.MISSING_TOKEN, channel.json_body["errcode"])
@@ -1593,7 +1591,6 @@ def test_not_admin(self):
15931591
request, channel = self.make_request(
15941592
"POST", self.url, b"{}", access_token=self.other_user_tok
15951593
)
1596-
self.render(request)
15971594

15981595
self.assertEqual(403, int(channel.result["code"]), msg=channel.result["body"])
15991596

@@ -1622,7 +1619,6 @@ def test_devices(self):
16221619
request, channel = self.make_request(
16231620
"GET", "devices", b"{}", access_token=self.other_user_tok
16241621
)
1625-
self.render(request)
16261622
self.assertEqual(200, int(channel.result["code"]), msg=channel.result["body"])
16271623

16281624
# We should only see the one device (from the login in `prepare`)
@@ -1638,28 +1634,24 @@ def test_logout(self):
16381634
request, channel = self.make_request(
16391635
"GET", "devices", b"{}", access_token=puppet_token
16401636
)
1641-
self.render(request)
16421637
self.assertEqual(200, int(channel.result["code"]), msg=channel.result["body"])
16431638

16441639
# Logout with the puppet token
16451640
request, channel = self.make_request(
16461641
"POST", "logout", b"{}", access_token=puppet_token
16471642
)
1648-
self.render(request)
16491643
self.assertEqual(200, int(channel.result["code"]), msg=channel.result["body"])
16501644

16511645
# The puppet token should no longer work
16521646
request, channel = self.make_request(
16531647
"GET", "devices", b"{}", access_token=puppet_token
16541648
)
1655-
self.render(request)
16561649
self.assertEqual(401, int(channel.result["code"]), msg=channel.result["body"])
16571650

16581651
# .. but the real user's tokens should still work
16591652
request, channel = self.make_request(
16601653
"GET", "devices", b"{}", access_token=self.other_user_tok
16611654
)
1662-
self.render(request)
16631655
self.assertEqual(200, int(channel.result["code"]), msg=channel.result["body"])
16641656

16651657
def test_user_logout_all(self):
@@ -1673,28 +1665,24 @@ def test_user_logout_all(self):
16731665
request, channel = self.make_request(
16741666
"GET", "devices", b"{}", access_token=puppet_token
16751667
)
1676-
self.render(request)
16771668
self.assertEqual(200, int(channel.result["code"]), msg=channel.result["body"])
16781669

16791670
# Logout all with the real user token
16801671
request, channel = self.make_request(
16811672
"POST", "logout/all", b"{}", access_token=self.other_user_tok
16821673
)
1683-
self.render(request)
16841674
self.assertEqual(200, int(channel.result["code"]), msg=channel.result["body"])
16851675

16861676
# The puppet token should still work
16871677
request, channel = self.make_request(
16881678
"GET", "devices", b"{}", access_token=puppet_token
16891679
)
1690-
self.render(request)
16911680
self.assertEqual(200, int(channel.result["code"]), msg=channel.result["body"])
16921681

16931682
# .. but the real user's tokens shouldn't
16941683
request, channel = self.make_request(
16951684
"GET", "devices", b"{}", access_token=self.other_user_tok
16961685
)
1697-
self.render(request)
16981686
self.assertEqual(401, int(channel.result["code"]), msg=channel.result["body"])
16991687

17001688
def test_admin_logout_all(self):
@@ -1708,28 +1696,24 @@ def test_admin_logout_all(self):
17081696
request, channel = self.make_request(
17091697
"GET", "devices", b"{}", access_token=puppet_token
17101698
)
1711-
self.render(request)
17121699
self.assertEqual(200, int(channel.result["code"]), msg=channel.result["body"])
17131700

17141701
# Logout all with the admin user token
17151702
request, channel = self.make_request(
17161703
"POST", "logout/all", b"{}", access_token=self.admin_user_tok
17171704
)
1718-
self.render(request)
17191705
self.assertEqual(200, int(channel.result["code"]), msg=channel.result["body"])
17201706

17211707
# The puppet token should no longer work
17221708
request, channel = self.make_request(
17231709
"GET", "devices", b"{}", access_token=puppet_token
17241710
)
1725-
self.render(request)
17261711
self.assertEqual(401, int(channel.result["code"]), msg=channel.result["body"])
17271712

17281713
# .. but the real user's tokens should still work
17291714
request, channel = self.make_request(
17301715
"GET", "devices", b"{}", access_token=self.other_user_tok
17311716
)
1732-
self.render(request)
17331717
self.assertEqual(200, int(channel.result["code"]), msg=channel.result["body"])
17341718

17351719
@unittest.override_config(

0 commit comments

Comments
 (0)