Skip to content

Improve test failure output

Tiffany Rea requested to merge 332121-improve-test-failure-output into master

What does this MR do and why?

Describe in detail what your merge request does and why.

Per test failure #332121 (closed) it was impossible to know which job status test was expecting at time of failure.

  • Implement aggregation block to capture all possible failures in one run.
  • Add proper check job status logic
  • Remove unused variable in test
  • Promoting test back to :smoke since the last failure seen was 3 weeks ago and the issue mentioned in the test when it was demoted hasn't surface in 5 months. Also trigger run against canary and passed https://ops.gitlab.net/gitlab-org/quality/canary/-/jobs/5409190

Screenshots or screen recordings

These are strongly recommended to assist reviewers and reduce the time to merge your change.

How to set up and validate locally

Numbered steps to set up and validate the change are strongly suggested.

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Related to #332121 (closed)

Edited by Tiffany Rea

Merge request reports