Skip to content

Junit test reports: Search for attachments in stderr, too. ~backend

Michael Zangl requested to merge michaelzangl/gitlab:michaelzangl-master-patch-88885 into master

What does this MR do and why?

Currently, only [[Attachment|...]] tags in stdout are parsed by gitlab.

There are cases in which it is easier to include the tags in stderr:

  • Some test reporters / report converters only add the test output as stderr.
  • One can add the attachment to the exception message

Screenshots or screen recordings

No UI changes.

How to set up and validate locally

  • Have a project that uses a CI pipeline with Junit test reports
  • Have a test that adds an attachment to stdout using the [[ATTACHMENT|...] syntax
  • Run the CI pipeline => the attachment is shown, as before
  • Change that test to not print to stdout, but use stderr instead
  • Run the CI pipeline => the attachment should still be shown

MR acceptance checklist

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

Merge request reports