Skip to content

Commit

Permalink
Remove superfluous check of request.method from test_http_url_limit_b…
Browse files Browse the repository at this point in the history
…ypass()
  • Loading branch information
padraic-shafer committed Feb 17, 2024
1 parent 1b2e9d8 commit e69ed02
Showing 1 changed file with 1 addition and 6 deletions.
7 changes: 1 addition & 6 deletions tiled/_tests/test_dataframe.py
Original file line number Diff line number Diff line change
Expand Up @@ -163,6 +163,7 @@ def test_http_url_limit_bypass(context, http_method, link):
"GET requests beyond the URL length limit should become POST requests."
if http_method not in ("GET", "POST"):
pytest.fail(reason="HTTP method {http_method} is not expected.")

client = from_context(context)
url_path = client["wide"].item["links"][link]
original_df = tree["wide"].read()
Expand All @@ -188,12 +189,6 @@ def test_http_url_limit_bypass(context, http_method, link):
requests = list(request for request in history.requests)
assert len(requests) == 1

request_methods = list(request.method for request in requests)
if http_method == "POST":
assert "POST" in request_methods # At least one POST request
elif http_method == "GET":
assert "POST" not in request_methods # No POST request


def test_deprecated_query_parameter(context):
"HTTP route /table/partition: 'field' is a deprecated query parameter"
Expand Down

0 comments on commit e69ed02

Please sign in to comment.