From 6c4314a8dbf1ef45d629536f03760f4ce0d7daa9 Mon Sep 17 00:00:00 2001 From: davelopez <46503462+davelopez@users.noreply.github.com> Date: Mon, 14 Jun 2021 19:01:00 +0200 Subject: [PATCH] Force PUT request headers to JSON in API test Aparentely FastAPI no longer assumes JSON by default. --- lib/galaxy_test/api/test_datasets.py | 5 ++++- test/integration/test_tool_data_delete.py | 5 ++++- 2 files changed, 8 insertions(+), 2 deletions(-) diff --git a/lib/galaxy_test/api/test_datasets.py b/lib/galaxy_test/api/test_datasets.py index 514b2981ab26..a69bfbfa457a 100644 --- a/lib/galaxy_test/api/test_datasets.py +++ b/lib/galaxy_test/api/test_datasets.py @@ -128,7 +128,10 @@ def test_tag_change(self): 'item_class': 'HistoryDatasetAssociation', 'item_tags': ['cool:tag_a', 'cool:tag_b', 'tag_c', 'name:tag_d', '#tag_e'], }) - put_response = self._put("tags", payload) + + # TODO remove the headers here and add json parameter to _put method + put_response = self._put("tags", data=payload, headers={'Content-Type': 'application/json'}) + self._assert_status_code_is_ok(put_response) updated_hda = self._get( f"histories/{self.history_id}/contents/{hda_id}").json() diff --git a/test/integration/test_tool_data_delete.py b/test/integration/test_tool_data_delete.py index 0b883178c482..ee0bb16a35b0 100644 --- a/test/integration/test_tool_data_delete.py +++ b/test/integration/test_tool_data_delete.py @@ -61,7 +61,10 @@ def test_admin_delete_data_table_entry(self): self.assertEquals(len(updated_fields), original_count + 1) new_field = updated_fields[-1] url = self._api_url(f"tool_data/testbeta?key={self.galaxy_interactor.api_key}") - delete_response = delete(url, data=json.dumps({"values": "\t".join(new_field)})) + + # TODO remove the headers here, use the interactor _delete method and add json parameter to it + delete_response = delete(url, data=json.dumps({"values": "\t".join(new_field)}), headers={'Content-Type': 'application/json'}) + delete_response.raise_for_status() time.sleep(2) show_response = self._get("tool_data/testbeta")