Skip to content

Fix test failure detection

Tomasz Maczukin requested to merge fix-test-failure-detection into master

What does this MR do?

Fixes a regression introduced in our CI tests with !928 (merged).

Why was this MR needed?

A bug in scripts/go_test_with_coverage_report prevents a failure if failing test is not executed in the last group. This MR fixes the problem.

Are there points in the code the reviewer needs to double check?

After merging this MR should be ~"Pick into 11.2" since it's a regression introduced before %11.2 feature freeze.

Does this MR meet the acceptance criteria?

  • Documentation created/updated
  • Tests
    • Added for this feature/bug
    • All builds are passing
  • Branch has no merge conflicts with master (if you do - rebase it please)

What are the relevant issue numbers?

Merge request reports