Skip to content

Commit fbd9914

Browse files
committed
address review comments
Signed-off-by: Swati Mukund Bagal <sbagal@redhat.com>
1 parent a087bdb commit fbd9914

2 files changed

Lines changed: 1 addition & 5 deletions

File tree

tests/model_serving/maas_billing/maas_subscription/test_api_key_authorization.py

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -153,8 +153,4 @@ def test_non_admin_cannot_search_by_other_username(
153153
f"Expected 403 when free user searches by another user's username, "
154154
f"got {list_resp.status_code}: {list_resp.text[:200]}"
155155
)
156-
<<<<<<< HEAD
157-
LOGGER.info(f"[authz] Free user correctly received 403 when searching by admin username='{admin_username}'")
158-
=======
159156
LOGGER.info("[authz] Free user correctly received 403 when searching by another user's username")
160-
>>>>>>> 626a88d (test: update API key authorization tests)

tests/model_serving/maas_billing/maas_subscription/utils.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -300,7 +300,7 @@ def resolve_api_key_username(
300300
f"Expected 200 on GET /v1/api-keys/{key_id}, got {get_resp.status_code}: {get_resp.text[:200]}"
301301
)
302302
username = get_body.get("username") or get_body.get("owner")
303-
assert username, f"Expected 'username' or 'owner' field in GET response: {get_body}"
303+
assert username, "Expected 'username' or 'owner' field in GET response"
304304
return username
305305

306306

0 commit comments

Comments
 (0)