Skip to content

Commit

Permalink
Apply suggestions from code review
Browse files Browse the repository at this point in the history
Co-authored-by: Matthew Evans <7916000+ml-evs@users.noreply.github.com>
  • Loading branch information
JPBergsma and ml-evs committed May 9, 2022
1 parent 1d0356e commit 8a06903
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion tests/server/conftest.py
Original file line number Diff line number Diff line change
Expand Up @@ -117,7 +117,7 @@ def inner(
server: Union[str, OptimadeTestClient] = "regular",
):
response = get_good_response(request, server)
if type(response["data"]) == dict:
if isinstance(response["data"], dict):
response_ids = [response["data"]["id"]]
else:
response_ids = [struct["id"] for struct in response["data"]]
Expand Down
2 changes: 1 addition & 1 deletion tests/server/routers/test_structures.py
Original file line number Diff line number Diff line change
Expand Up @@ -69,7 +69,7 @@ def test_structures_endpoint_data(self):
assert "_exmpl_chemsys" in self.json_response["data"]["attributes"]


def TestCheckResponseSingleStructure(check_response):
def test_check_response_single_structure(check_response):
"""Tests whether check_response also handles single endpoint queries correctly."""

test_id = "mpf_1"
Expand Down

0 comments on commit 8a06903

Please sign in to comment.