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

Commit 844fb19

Browse files
committed
fix: correct test template to use args[1] for URL and keep_blank_values
1 parent 0d03d92 commit 844fb19

File tree

1 file changed

+3
-2
lines changed

1 file changed

+3
-2
lines changed

gapic/templates/tests/unit/gapic/%name_%version/%sub/test_macros.j2

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1201,9 +1201,10 @@ def test_{{ method_name }}_rest_required_fields(request_type={{ method.input.ide
12011201
{% endif %}
12021202
]
12031203
# Verify query params are correctly included in the URL
1204-
actual_url = req.call_args.args[0]
1204+
# Session.request is called as request(method, url, ...), so url is args[1]
1205+
actual_url = req.call_args.args[1]
12051206
parsed_url = urllib.parse.urlparse(actual_url)
1206-
actual_params = urllib.parse.parse_qsl(parsed_url.query)
1207+
actual_params = urllib.parse.parse_qsl(parsed_url.query, keep_blank_values=True)
12071208
assert set(expected_params).issubset(set(actual_params))
12081209

12091210

0 commit comments

Comments
 (0)