From 784200bb49025adf48b02bdb8bea4b5ece3b1e20 Mon Sep 17 00:00:00 2001 From: Nicola Soranzo Date: Tue, 26 Sep 2023 18:22:57 +0100 Subject: [PATCH] Don't copy collection elements in ``test_dataset_collection_hide_originals`` Follow-up to https://github.com/galaxyproject/galaxy/pull/16717 . Minimal performance gain but helps document the parameter use after the change in default value to True. --- lib/galaxy_test/api/test_history_contents.py | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/lib/galaxy_test/api/test_history_contents.py b/lib/galaxy_test/api/test_history_contents.py index 74be58604d08..cf5f76c52d5f 100644 --- a/lib/galaxy_test/api/test_history_contents.py +++ b/lib/galaxy_test/api/test_history_contents.py @@ -495,7 +495,7 @@ def test_jobs_summary_implicit_hdca(self, history_id): def test_dataset_collection_hide_originals(self, history_id): payload = self.dataset_collection_populator.create_pair_payload( - history_id, type="dataset_collection", direct_upload=False + history_id, type="dataset_collection", direct_upload=False, copy_elements=False ) payload["hide_source_items"] = True @@ -503,9 +503,7 @@ def test_dataset_collection_hide_originals(self, history_id): self.__check_create_collection_response(dataset_collection_response) contents_response = self._get(f"histories/{history_id}/contents") - datasets = [ - d for d in contents_response.json() if d["history_content_type"] == "dataset" and d["hid"] in [1, 2] - ] + datasets = [d for d in contents_response.json() if d["history_content_type"] == "dataset"] # Assert two datasets in source were hidden. assert len(datasets) == 2 assert not datasets[0]["visible"]