Bug-1452249: Removed redundant lines in test

Open moijes12 opened this issue 1 month ago • 2 comments

As part of this we remove the lines that check for job_log status at the end of test test_ingest_twice_log_parsing_status_changed. No asserts are being called there and as per @camd , the test has already achieved it's objective before that line.

Nov 12 '25 12:11 moijes12

Update: Moved PR to draft and rebased changes onto master. Beginning work on fixing failing tests now.

Dec 09 '25 10:12 moijes12

@camd Moving this back to review. I did not have to correct any tests. All I did was rebase my changes onto master and pushed the changes to my branch. The pipeline ran the tests and all tests passed. Please can you review them again.

Dec 09 '25 10:12 moijes12

@camd @Archaeopteryx Please can you review this now that the tests pass

Dec 22 '25 11:12 moijes12