You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@ignite.apache.org by ir...@apache.org on 2020/07/09 20:19:04 UTC
[ignite-teamcity-bot] 03/04: New tests list doesn't contain ignored
and muted tests. Fix visa text. #169
This is an automated email from the ASF dual-hosted git repository.
irakov pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/ignite-teamcity-bot.git
commit 177bd1e7bcf0b7fc3290328e249e5b68d73a8de2
Author: sergeyuttsel <ut...@gmail.com>
AuthorDate: Thu Jul 9 23:16:13 2020 +0300
New tests list doesn't contain ignored and muted tests. Fix visa text. #169
Signed-off-by: Ivan Rakov <iv...@gmail.com>
---
.../org/apache/ignite/ci/tcbot/visa/TcBotTriggerAndSignOffService.java | 2 +-
.../main/java/org/apache/ignite/tcbot/engine/pr/PrChainsProcessor.java | 2 +-
.../src/main/java/org/apache/ignite/tcbot/engine/ui/DsChainUi.java | 2 +-
3 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/ignite-tc-helper-web/src/main/java/org/apache/ignite/ci/tcbot/visa/TcBotTriggerAndSignOffService.java b/ignite-tc-helper-web/src/main/java/org/apache/ignite/ci/tcbot/visa/TcBotTriggerAndSignOffService.java
index c4de5ca..4523a57 100644
--- a/ignite-tc-helper-web/src/main/java/org/apache/ignite/ci/tcbot/visa/TcBotTriggerAndSignOffService.java
+++ b/ignite-tc-helper-web/src/main/java/org/apache/ignite/ci/tcbot/visa/TcBotTriggerAndSignOffService.java
@@ -962,7 +962,7 @@ public class TcBotTriggerAndSignOffService {
cnt++;
if (cnt > 10) {
- newTests.append("... and ").append(totalNewTests - cnt).append(" tests blockers\\n");
+ newTests.append("... and ").append(totalNewTests - cnt).append(" new tests\\n");
break;
}
diff --git a/tcbot-engine/src/main/java/org/apache/ignite/tcbot/engine/pr/PrChainsProcessor.java b/tcbot-engine/src/main/java/org/apache/ignite/tcbot/engine/pr/PrChainsProcessor.java
index 0f93b99..698a8b7 100644
--- a/tcbot-engine/src/main/java/org/apache/ignite/tcbot/engine/pr/PrChainsProcessor.java
+++ b/tcbot-engine/src/main/java/org/apache/ignite/tcbot/engine/pr/PrChainsProcessor.java
@@ -393,7 +393,7 @@ public class PrChainsProcessor {
.map((ctx) -> {
List<ShortTestUi> missingTests = ctx.getFilteredTests(test -> {
IRunHistory history = test.history(tcIgnited, baseBranchId, null);
- return history == null;
+ return history == null && !test.isMutedOrIgored();
})
.stream()
.map(occurrence -> new ShortTestUi().initFrom(occurrence, occurrence.isPassed()))
diff --git a/tcbot-engine/src/main/java/org/apache/ignite/tcbot/engine/ui/DsChainUi.java b/tcbot-engine/src/main/java/org/apache/ignite/tcbot/engine/ui/DsChainUi.java
index ec47896..cf2147e 100644
--- a/tcbot-engine/src/main/java/org/apache/ignite/tcbot/engine/ui/DsChainUi.java
+++ b/tcbot-engine/src/main/java/org/apache/ignite/tcbot/engine/ui/DsChainUi.java
@@ -302,7 +302,7 @@ public class DsChainUi {
.map((suite) -> {
List<ShortTestUi> missingTests = suite.getFilteredTests(test -> {
IRunHistory history = test.history(tcIgnited, baseBranchId, null);
- return history == null;
+ return history == null && !test.isMutedOrIgored();
})
.stream()
.map(occurrence -> {