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

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

XMLWordPrintable

    • Icon: Bug Bug
    • Resolution: Fixed
    • Icon: Minor Minor
    • junit-plugin
    • None

      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.

          [JENKINS-48196] Stage/branch names in junit test displays should skip redundant names

          Andrew Bayer added a comment -

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

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

          Andrew Bayer added a comment -

          Fixed in upcoming 1.23 release.

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

          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/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.

          Andrew Bayer added a comment -

          Releasing as part of 1.23 as we speak.

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

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

              Created:
              Updated:
              Resolved: