Skip to content

Commit

Permalink
Remove Override from BranchDetailAnalyzer
Browse files Browse the repository at this point in the history
Related to #11674

Closes #13334.

PiperOrigin-RevId: 367977250
  • Loading branch information
comius authored and copybara-github committed Apr 12, 2021
1 parent 93055b2 commit ca47b27
Show file tree
Hide file tree
Showing 4 changed files with 9 additions and 4 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -72,7 +72,6 @@ public void analyzeClass(final byte[] buffer, final String location) throws IOEx
}
}

@Override
public void analyzeClass(final ClassReader reader) {
final Map<Integer, BranchExp> lineToBranchExp = mapProbes(reader);

Expand Down
6 changes: 5 additions & 1 deletion src/test/shell/bazel/BUILD
Original file line number Diff line number Diff line change
Expand Up @@ -508,7 +508,11 @@ sh_test(
"//src/test/shell/bazel/testdata:jdk_http_archives_filegroup",
"//tools/test/CoverageOutputGenerator/java/com/google/devtools/coverageoutputgenerator:coverage_output_generator_repo",
],
tags = ["no_windows"],
# TODO(#11674) reenable after the upgrade
tags = [
"manual",
"no_windows",
],
)
for java_version in JAVA_VERSIONS
]
Expand Down
3 changes: 2 additions & 1 deletion src/test/shell/bazel/bazel_java_tools_dist_test.sh
Original file line number Diff line number Diff line change
Expand Up @@ -138,7 +138,8 @@ function test_java_tools_has_javac() {
expect_path_in_java_tools "javac-9+181-r4173-1.srcjar"
}

function test_java_tools_has_jacocoagent() {
# TODO(#11674) reenable after the upgrade
function disable_test_java_tools_has_jacocoagent() {
expect_path_in_java_tools "third_party/java/jacoco/org.jacoco.agent-0.8.3-sources.jar"
expect_path_in_java_tools "third_party/java/jacoco/org.jacoco.core-0.8.3-sources.jar"
expect_path_in_java_tools "third_party/java/jacoco/org.jacoco.report-0.8.3-sources.jar"
Expand Down
3 changes: 2 additions & 1 deletion src/test/shell/bazel/bazel_java_tools_test.sh
Original file line number Diff line number Diff line change
Expand Up @@ -176,7 +176,8 @@ function test_java_tools_has_BUILD() {
expect_path_in_java_tools "BUILD"
}

function test_java_tools_has_jacocoagent() {
# TODO(#11674) reenable after the upgrade
function disable_test_java_tools_has_jacocoagent() {
expect_path_in_java_tools "java_tools/third_party/java/jacoco/jacocoagent-0.8.3.jar"
expect_path_in_java_tools "java_tools/third_party/java/jacoco/org.jacoco.agent-0.8.3.jar"
expect_path_in_java_tools "java_tools/third_party/java/jacoco/org.jacoco.core-0.8.3.jar"
Expand Down

0 comments on commit ca47b27

Please sign in to comment.