diff options
-rw-r--r-- | src/main/java/com/google/devtools/build/lib/Constants.java | 1 | ||||
-rw-r--r-- | src/main/java/com/google/devtools/build/lib/runtime/TestResultAnalyzer.java | 14 |
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()); |