Skip to content

Commit

Permalink
Don't copy collection elements in ``test_dataset_collection_hide_orig…
Browse files Browse the repository at this point in the history
…inals``

Follow-up to galaxyproject#16717 .
Minimal performance gain but helps document the parameter use after
the change in default value to True.
  • Loading branch information
nsoranzo committed Sep 27, 2023
1 parent b180b79 commit 784200b
Showing 1 changed file with 2 additions and 4 deletions.
6 changes: 2 additions & 4 deletions lib/galaxy_test/api/test_history_contents.py
Original file line number Diff line number Diff line change
Expand Up @@ -495,17 +495,15 @@ 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
dataset_collection_response = self._post(f"histories/{history_id}/contents", payload, json=True)
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"]
Expand Down

0 comments on commit 784200b

Please sign in to comment.