aboutsummaryrefslogtreecommitdiffhomepage
diff options
context:
space:
mode:
-rw-r--r--src/main/java/com/google/devtools/build/lib/Constants.java1
-rw-r--r--src/main/java/com/google/devtools/build/lib/runtime/TestResultAnalyzer.java14
2 files changed, 1 insertions, 14 deletions
diff --git a/src/main/java/com/google/devtools/build/lib/Constants.java b/src/main/java/com/google/devtools/build/lib/Constants.java
index 2db7d323a1..8f74ed3ae5 100644
--- a/src/main/java/com/google/devtools/build/lib/Constants.java
+++ b/src/main/java/com/google/devtools/build/lib/Constants.java
@@ -30,7 +30,6 @@ public class Constants {
public static final ImmutableList<String> DEFAULT_PACKAGE_PATH = ImmutableList.of("%workspace%");
public static final String MAIN_RULE_CLASS_PROVIDER =
"com.google.devtools.build.lib.bazel.rules.BazelRuleClassProvider";
- public static final ImmutableList<String> IGNORED_TEST_WARNING_PREFIXES = ImmutableList.of();
public static final String NATIVE_DEPS_LIB_SUFFIX = "_nativedeps";
diff --git a/src/main/java/com/google/devtools/build/lib/runtime/TestResultAnalyzer.java b/src/main/java/com/google/devtools/build/lib/runtime/TestResultAnalyzer.java
index e9dbd9bbb2..61f3d3d2a2 100644
--- a/src/main/java/com/google/devtools/build/lib/runtime/TestResultAnalyzer.java
+++ b/src/main/java/com/google/devtools/build/lib/runtime/TestResultAnalyzer.java
@@ -16,7 +16,6 @@ package com.google.devtools.build.lib.runtime;
import com.google.common.base.Preconditions;
import com.google.common.collect.Sets;
import com.google.common.eventbus.EventBus;
-import com.google.devtools.build.lib.Constants;
import com.google.devtools.build.lib.actions.Artifact;
import com.google.devtools.build.lib.analysis.ConfiguredTarget;
import com.google.devtools.build.lib.analysis.TransitiveInfoCollection;
@@ -232,17 +231,6 @@ public class TestResultAnalyzer {
}
}
- List<String> filtered = new ArrayList<>();
- warningLoop: for (String warning : result.getData().getWarningList()) {
- for (String ignoredPrefix : Constants.IGNORED_TEST_WARNING_PREFIXES) {
- if (warning.startsWith(ignoredPrefix)) {
- continue warningLoop;
- }
- }
-
- filtered.add(warning);
- }
-
List<Path> passed = new ArrayList<>();
if (result.getData().hasPassedLog()) {
passed.add(result.getTestAction().getTestLog().getPath().getRelative(
@@ -258,7 +246,7 @@ public class TestResultAnalyzer {
.addTestTimes(result.getData().getTestTimesList())
.addPassedLogs(passed)
.addFailedLogs(failed)
- .addWarnings(filtered)
+ .addWarnings(result.getData().getWarningList())
.collectFailedTests(result.getData().getTestCase())
.setRanRemotely(result.getData().getIsRemoteStrategy());