Skip to content

Commit

Permalink
Merge pull request #19 from sjaensch/fix-bool-query-param-test
Browse files Browse the repository at this point in the history
boolean query param values are lowercased in the newest bravado library
  • Loading branch information
sjaensch committed Aug 6, 2018
2 parents 2ccf5c4 + c815a4f commit 62dc29d
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion testing/integration_server.py
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,7 @@ async def login(request):
if not (
request.query.get('username') == 'asyncio'
and request.query.get('password') == 'p%s&wörd?'
and request.query.get('invalidate_sessions') == 'True'
and request.query.get('invalidate_sessions') in ('True', 'true')
):
return web.HTTPBadRequest()

Expand Down
2 changes: 1 addition & 1 deletion tests/integration/conftest.py
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,7 @@ def integration_server():
server_port = ephemeral_port_reserve.reserve()
server = subprocess.Popen(
['python', script_name, '-p', str(server_port)],
stdin=None, stdout=subprocess.DEVNULL, stderr=None,
stdin=None, stdout=None, stderr=None,
)
wait_unit_service_starts('http://localhost:{port}'.format(port=server_port))

Expand Down

0 comments on commit 62dc29d

Please sign in to comment.