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

email-ext - new failed test regression trigger

    XMLWordPrintable

    Details

    • Similar Issues:

      Description

      Curently regressionTrigger triggers email when THERE IS DIFFERENCE in failed tests, which doesn't cover fully regressions.
      Example: build #1 testA failed, test B ok
      build #2: testA ok, test B failed.
      In both cases 1 failed so for build #2 regressiontrigger won't trigger email.

      return build.getTestResultAction().getFailCount() >
      previousBuild.getTestResultAction().getFailCount();

      But in fact it is the regression because new test failed.

      So modify existing RegressionTrigger or add new 'NewFailedTestsRegressionTrigger' with
      public boolean trigger(AbstractBuild<?, ?> build, TaskListener listener) {
      ....
      for (test in build.getTestResultAction().getFailedTests()){
      if (test.getAge() == 1)

      { return true; }

      }
      return false;

        Attachments

          Activity

          Hide
          slide_o_mix Alex Earl added a comment -

          Added the code to check for age == 1 into the regression trigger.

          Show
          slide_o_mix Alex Earl added a comment - Added the code to check for age == 1 into the regression trigger.
          Hide
          scm_issue_link SCM/JIRA link daemon added a comment -

          Code changed in jenkins
          User: Alex Earl
          Path:
          src/main/java/hudson/plugins/emailext/plugins/trigger/RegressionTrigger.java
          http://jenkins-ci.org/commit/email-ext-plugin/de6ee279a67091fb58405193c9d9acc3a5e3ac89
          Log:
          Fix JENKINS-22041

          Added check to see if any tests failed this time that didn't fail last
          time

          Show
          scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Alex Earl Path: src/main/java/hudson/plugins/emailext/plugins/trigger/RegressionTrigger.java http://jenkins-ci.org/commit/email-ext-plugin/de6ee279a67091fb58405193c9d9acc3a5e3ac89 Log: Fix JENKINS-22041 Added check to see if any tests failed this time that didn't fail last time

            People

            • Assignee:
              slide_o_mix Alex Earl
              Reporter:
              tomaszbech Tomasz Bech
            • Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: