Revert "Check that scan is running before checking if it finished"

[mirekdlugosz]

This reverts commit 527a7d5, which was a part of PR #432 .

The risk I saw turned out to be true - since this commit was merged, all pipeline runs have one or two failed tests. Earlier we had failures only once in couple of runs.

From what I see, usually the problem is that we expect scan to be running, but it is already completed. It doesn't sound right - looks like scans complete in just few seconds. I find it more likely that we check wrong scan, before it even started.

But either way, we need to go back to drawing board and think about this problem a little more. I'll create a jira ticket, as this turns out to require more time.

[codecov]

Codecov Report

Merging #433 (aecacc1) into main (527a7d5) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #433   +/-   ##
=======================================
  Coverage   79.11%   79.11%           
=======================================
  Files           5        5           
  Lines         249      249           
=======================================
  Hits          197      197           
  Misses         52       52           

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

[infinitewarp]

🤞 🍀 🙏