Skip to content

Test report link fix

Juan J. Ramirez requested to merge test-report-link-fix into master

What does this MR do?

We recently shipped a new feature that allows users to open test reports from the MR widget. (!29729 (merged)) Apparently the way the pipeline-path was being passed was returning an empty string and therefore the button wasn't being shown.

This MR fixes this by passing an actual valid path for the button.

Screenshots

image

Does this MR meet the acceptance criteria?

Conformity

Availability and Testing

Security

If this MR contains changes to processing or storing of credentials or tokens, authorization and authentication methods and other items described in the security review guidelines:

  • Label as security and @ mention @gitlab-com/gl-security/appsec
  • The MR includes necessary changes to maintain consistency between UI, API, email, or other methods
  • Security reports checked/validated by a reviewer from the AppSec team
Edited by Juan J. Ramirez

Merge request reports