This is an automated email from the ASF dual-hosted git repository.

dpavlov pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/ignite-teamcity-bot.git

commit cf804672fde5217ab0f854ce95522b21ca4baf50
Author: Дмитрий Рябов <somefire...@gmail.com>
AuthorDate: Sat Nov 10 13:00:52 2018 +0300

    IGNITE-10190 Failed tests don't count as blockers (if created in the PR) - 
Fixes #62.
    
    Signed-off-by: Dmitriy Pavlov <dpav...@apache.org>
---
 .../java/org/apache/ignite/ci/web/model/current/TestFailure.java  | 7 +++++--
 ignite-tc-helper-web/src/main/webapp/js/testfails-2.1.js          | 8 ++++----
 2 files changed, 9 insertions(+), 6 deletions(-)

diff --git 
a/ignite-tc-helper-web/src/main/java/org/apache/ignite/ci/web/model/current/TestFailure.java
 
b/ignite-tc-helper-web/src/main/java/org/apache/ignite/ci/web/model/current/TestFailure.java
index dbee4f7..c3c28a3 100644
--- 
a/ignite-tc-helper-web/src/main/java/org/apache/ignite/ci/web/model/current/TestFailure.java
+++ 
b/ignite-tc-helper-web/src/main/java/org/apache/ignite/ci/web/model/current/TestFailure.java
@@ -235,11 +235,14 @@ import static 
org.apache.ignite.ci.web.model.current.SuiteCurrentStatus.branchFo
      * @return {@code True} if this failure is appeared in the current branch.
      */
     public boolean isNewFailedTest() {
-        FailureSummary recent = histBaseBranch.recent;
-
         if (!Strings.isNullOrEmpty(webIssueUrl))
             return false;
 
+        if (histBaseBranch.latestRuns == null)
+            return true;
+
+        FailureSummary recent = histBaseBranch.recent;
+
         boolean lowFailureRate = recent != null && recent.failureRate != null 
&&
             Float.valueOf(recent.failureRate.replace(',', '.')) < 4.;
 
diff --git a/ignite-tc-helper-web/src/main/webapp/js/testfails-2.1.js 
b/ignite-tc-helper-web/src/main/webapp/js/testfails-2.1.js
index 037a720..8262276 100644
--- a/ignite-tc-helper-web/src/main/webapp/js/testfails-2.1.js
+++ b/ignite-tc-helper-web/src/main/webapp/js/testfails-2.1.js
@@ -693,12 +693,12 @@ function showSuiteData(suite, settings) {
  * @returns {boolean} True - if test is new. False - otherwise.
  */
 function isNewFailedTest(testFail) {
-    if(!isDefinedAndFilled(testFail.histBaseBranch))
-        return true;
-
     if (isDefinedAndFilled(testFail.webIssueUrl))
         return false;
 
+    if (!isDefinedAndFilled(testFail.histBaseBranch) || 
!isDefinedAndFilled(testFail.histBaseBranch.latestRuns))
+        return true;
+
     var hist = testFail.histBaseBranch;
 
     if (!isDefinedAndFilled(hist.recent))
@@ -995,4 +995,4 @@ function initMoreInfo() {
             });
         });
     });
-}
\ No newline at end of file
+}

Reply via email to