Skip to content

Commit abebfcf

Browse files
authored
Merge pull request #674 from averevki/fix-plain-identity
Fix plain identity test
2 parents 0828885 + 27ea9b7 commit abebfcf

1 file changed

Lines changed: 4 additions & 5 deletions

File tree

testsuite/tests/singlecluster/authorino/identity/plain/test_plain_identity.py

Lines changed: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -22,11 +22,11 @@
2222
scope="module",
2323
params=[
2424
pytest.param(
25-
{"path": "context.nonexistent.path", "code": 401, "identity": None},
25+
{"path": "context.nonexistent.path", "code": 401},
2626
id="non-existent AuthJson path to retrieve identity from",
2727
),
2828
pytest.param(
29-
{"path": "context.request.http.method", "code": 200, "identity": "GET"},
29+
{"path": "context.request.http.method", "code": 200},
3030
id="existing AuthJson path to retrieve identity from",
3131
),
3232
],
@@ -58,6 +58,5 @@ def test_plain_identity(client, plain_identity):
5858
response = client.get("/get")
5959
assert response.status_code == plain_identity["code"]
6060

61-
identity = extract_response(response)
62-
# identity should be populated with the value retrieved from AuthJson in the specified path
63-
assert identity % None == plain_identity["identity"]
61+
if plain_identity["code"] == 200:
62+
assert extract_response(response) % None == "GET"

0 commit comments

Comments
 (0)