From d9051005bc7bc7c66bede80c151762ed6860fe8c Mon Sep 17 00:00:00 2001 From: davelopez <46503462+davelopez@users.noreply.github.com> Date: Fri, 9 Aug 2024 14:03:53 +0200 Subject: [PATCH] Remove unnecessary assert for file_size The file_size should be preserved in the database for traceability and other uses. --- test/integration/objectstore/_purged_handling.py | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/test/integration/objectstore/_purged_handling.py b/test/integration/objectstore/_purged_handling.py index 1437f6742ae5..882b162bdebb 100644 --- a/test/integration/objectstore/_purged_handling.py +++ b/test/integration/objectstore/_purged_handling.py @@ -63,9 +63,7 @@ def purge_while_job_running(dataset_populator: DatasetPopulator, extra_sleep=0): # Make sure that we don't revert state while finishing job assert output_details["purged"], f"expected output '{output_name}' to be purged, but it is not purged." assert not output_details.get("file_name") - assert ( - output_details["file_size"] == 0 - ), f"expected file_size for '{output_name}' to be 0, but it is {output_details['file_size']}." + for file_name in hda_filenames: # Make sure job didn't push to object store assert not os.path.exists(file_name), f"Expected {file_name} to be purged."