Problem/Motivation

ie: https://git.drupalcode.org/issue/drupal-3403162/-/pipelines/53807/test_r... (and click on any category)

We can see the full path like: /builds/issue/drupal-3403162/core/tests/Drupal/Tests/Core/Command/GenerateThemeTest.php

The link's href value is: https://git.drupalcode.org/issue/drupal-3403162/-/blob/7aaa539cb4bf26e9266cdc6a764e969695e50aae/builds/issue/drupal-3403162/core/tests/Drupal/Tests/Core/Command/GenerateThemeTest.php

And it should be: https://git.drupalcode.org/issue/drupal-3403162/-/blob/7aaa539cb4bf26e9266cdc6a764e969695e50aae/core/tests/Drupal/Tests/Core/Command/GenerateThemeTest.php

Steps to reproduce

Open any test report detail.

Proposed resolution

Fix the path so the link shown is correct and takes you to the file.

Remaining tasks

MR

User interface changes

API changes

Data model changes

Release notes snippet

Issue fork drupal-3403420

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

fjgarlin created an issue. See original summary.

fjgarlin opened merge request !5506

fjgarlin’s picture

fjgarlin commented
Issue summary: View changes
fjgarlin’s picture

fjgarlin commented
Status: Active » Needs review

We can see the difference already within this MR. See here: https://git.drupalcode.org/issue/drupal-3403420/-/pipelines/53911/test_r... (and click on any category).

You will see that filenames are now relative and that the link to the file works as expected.

smustgrave’s picture

smustgrave commented
Status: Needs review » Reviewed & tested by the community

Seems like a simple update.

  • longwave committed 89944826 on 10.2.x 
    Issue #3403420 by fjgarlin: Test reports contain absolute path and link...

  • longwave committed 1cc37927 on 11.x 
    Issue #3403420 by fjgarlin: Test reports contain absolute path and link...
longwave’s picture

longwave
he/him
Primary language English
Location UK
commented
Status: Reviewed & tested by the community » Fixed

Thanks for fixing this, this has annoyed me once so far but not enough to post an issue yet!

Committed and pushed 1cc37927f8 to 11.x and 8994482633 to 10.2.x. Thanks!

alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
commented
Version: 11.x-dev » 10.2.x-dev

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.