Skip to content

Commit 646739e

Browse files
committed
Align ordering of test cases, improve wording of comments
1 parent a9cff16 commit 646739e

File tree

1 file changed

+5
-5
lines changed

1 file changed

+5
-5
lines changed

tests/base/test_handlers.py

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -89,12 +89,12 @@ async def test_jupyter_handler_auth_permissive(jp_serverapp, jp_fetch):
8989
],
9090
)
9191

92-
# should allow access by default when no authentication rules are set up
93-
res = await jp_fetch("no-rules", method="OPTIONS", headers={"Authorization": ""})
92+
# should always permit access when `@allow_unauthenticated` is used
93+
res = await jp_fetch("permissive", method="OPTIONS", headers={"Authorization": ""})
9494
assert res.code == 200
9595

96-
# should allow access by default when `@allow_unauthenticated` is used
97-
res = await jp_fetch("permissive", method="OPTIONS", headers={"Authorization": ""})
96+
# should allow access when no authentication rules are set up
97+
res = await jp_fetch("no-rules", method="OPTIONS", headers={"Authorization": ""})
9898
assert res.code == 200
9999

100100

@@ -111,7 +111,7 @@ async def test_jupyter_handler_auth_required(jp_serverapp, jp_fetch):
111111
],
112112
)
113113

114-
# should permit access when `@allow_unauthenticated` is used
114+
# should always permit access when `@allow_unauthenticated` is used
115115
res = await jp_fetch("permissive", method="OPTIONS", headers={"Authorization": ""})
116116
assert res.code == 200
117117

0 commit comments

Comments
 (0)