Uploaded image for project: 'Jenkins'
  1. Jenkins
  2. JENKINS-48196

Stage/branch names in junit test displays should skip redundant names

    XMLWordPrintable

    Details

    • Similar Issues:

      Description

      Declarative parallel stages result in an actual stage inside an identically named parallel branch. That results in the display name for a CaseResult being named like Test / Chrome / Chrome / testCase. We should look for those cases and squash them down, so that the display name is actually just Test / Chrome / testCase.

        Attachments

          Issue Links

            Activity

            Show
            abayer Andrew Bayer added a comment - PR up at https://github.com/jenkinsci/junit-plugin/pull/92
            Hide
            abayer Andrew Bayer added a comment -

            Fixed in upcoming 1.23 release.

            Show
            abayer Andrew Bayer added a comment - Fixed in upcoming 1.23 release.
            Hide
            scm_issue_link SCM/JIRA link daemon added a comment -

            Code changed in jenkins
            User: Andrew Bayer
            Path:
            src/main/java/hudson/tasks/junit/pipeline/JUnitResultsStepExecution.java
            src/test/java/hudson/tasks/junit/pipeline/JUnitResultsStepTest.java
            http://jenkins-ci.org/commit/junit-plugin/d49e21a8b27578a01182c00df2a102d923c64756
            Log:
            [FIXED JENKINS-48196] Remove redundant branch names from case displays

            If we've got a suite with a stage inside a parallel branch where the
            stage and parallel branch have the same name, we should only include
            that name once in the suite/case/etc display name - it's redundant to
            display it twice.

            There is a potential gotcha here if you've got multiple stages inside
            a branch and one of those stages has the same name as the branch, but
            I think that's an edge case we can live with for now.

            Show
            scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Andrew Bayer Path: src/main/java/hudson/tasks/junit/pipeline/JUnitResultsStepExecution.java src/test/java/hudson/tasks/junit/pipeline/JUnitResultsStepTest.java http://jenkins-ci.org/commit/junit-plugin/d49e21a8b27578a01182c00df2a102d923c64756 Log: [FIXED JENKINS-48196] Remove redundant branch names from case displays If we've got a suite with a stage inside a parallel branch where the stage and parallel branch have the same name, we should only include that name once in the suite/case/etc display name - it's redundant to display it twice. There is a potential gotcha here if you've got multiple stages inside a branch and one of those stages has the same name as the branch, but I think that's an edge case we can live with for now.
            Hide
            abayer Andrew Bayer added a comment -

            Releasing as part of 1.23 as we speak.

            Show
            abayer Andrew Bayer added a comment - Releasing as part of 1.23 as we speak.

              People

              • Assignee:
                abayer Andrew Bayer
                Reporter:
                abayer Andrew Bayer
              • Votes:
                0 Vote for this issue
                Watchers:
                3 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: