From 6a3f32131f1ded60a2bf80077f6b806831176849 Mon Sep 17 00:00:00 2001 From: Alex Moses Date: Wed, 12 Nov 2025 12:02:01 +0000 Subject: [PATCH] Bug-1452249: Removed redundant lines in test Removing the lines that check for job_log status at the end of test `test_ingest_twice_log_parsing_status_changed` as no asserts are being called there and the test has already achieved it's objective before that line. --- tests/etl/test_job_ingestion.py | 2 -- 1 file changed, 2 deletions(-) diff --git a/tests/etl/test_job_ingestion.py b/tests/etl/test_job_ingestion.py index 3855fdf70a9..acf9aa1cc68 100644 --- a/tests/etl/test_job_ingestion.py +++ b/tests/etl/test_job_ingestion.py @@ -48,8 +48,6 @@ def test_ingest_twice_log_parsing_status_changed( job_data[0]["job"]["state"] = "completed" test_utils.do_job_ingestion(test_repository, job_data, sample_push) assert JobLog.objects.count() == 1 - for job_log in JobLog.objects.all(): - job_log.status == JobLog.FAILED @pytest.mark.parametrize("same_ingestion_cycle", [False, True])