Skip to content

Commit

Permalink
Catch (and test for) a warning about deprecated 'field'
Browse files Browse the repository at this point in the history
  • Loading branch information
danielballan committed Feb 23, 2024
1 parent 5fa30b3 commit 88c1e18
Showing 1 changed file with 6 additions and 3 deletions.
9 changes: 6 additions & 3 deletions tiled/_tests/test_dataframe.py
Original file line number Diff line number Diff line change
Expand Up @@ -193,10 +193,13 @@ def test_redundant_query_parameters(context):
}

# It is OK to include query parameter 'column' OR 'field'
for param in ("field", "column"):
params = original_params.copy()
params.pop(param)
params = original_params.copy()
params.pop("column")
with pytest.warns(DeprecationWarning):
context.http_client.get(url_path, params=params).raise_for_status()
params = original_params.copy()
params.pop("field")
context.http_client.get(url_path, params=params).raise_for_status()

# It is an error to include query parameter 'column' AND 'field'
with fail_with_status_code(400) as response:
Expand Down

0 comments on commit 88c1e18

Please sign in to comment.