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

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

    XMLWordPrintable

Details

    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

          links to

          Web Link PR #92

          Activity

            Descending order - Click to sort in ascending order
            abayer Andrew Bayer added a comment -

            Releasing as part of 1.23 as we speak.

            abayer Andrew Bayer added a comment - Releasing as part of 1.23 as we speak.
            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.

            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.
            abayer Andrew Bayer added a comment -

            Fixed in upcoming 1.23 release.

            abayer Andrew Bayer added a comment - Fixed in upcoming 1.23 release.
            abayer Andrew Bayer added a comment -

            PR up at https://github.com/jenkinsci/junit-plugin/pull/92

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

            People

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

              Dates

                Created:
                Updated:
                Resolved: