Skip to content

Commit

Permalink
Switch Bazel over to proguard 6.2.2 in the tests
Browse files Browse the repository at this point in the history
Last little but of cleanup after switching Bazel over to Progard 6.2.2.

Closes bazelbuild#12929.

PiperOrigin-RevId: 354904720
  • Loading branch information
Bencodes authored and copybara-github committed Feb 1, 2021
1 parent bb6af8b commit 165c1a0
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 8 deletions.
11 changes: 5 additions & 6 deletions src/test/shell/bazel/bazel_java_tools_dist_test.sh
Original file line number Diff line number Diff line change
Expand Up @@ -147,14 +147,13 @@ function test_java_tools_has_jacocoagent() {
expect_path_in_java_tools "third_party/asm/asm-8.0-sources.jar"
}

# TODO(bencodes) This test should assert against a 6.2.2 after proguard is switched over
function test_java_tools_has_proguard() {
expect_path_in_java_tools "third_party/java/proguard"
expect_path_in_java_tools "third_party/java/proguard/proguard.*"
expect_path_in_java_tools "third_party/java/proguard/proguard.*/bin"
expect_path_in_java_tools "third_party/java/proguard/proguard.*/buildscripts"
expect_path_in_java_tools "third_party/java/proguard/proguard.*/src"
expect_path_in_java_tools "third_party/java/proguard/proguard.*/src/proguard"
expect_path_in_java_tools "third_party/java/proguard/proguard6.2.2"
expect_path_in_java_tools "third_party/java/proguard/proguard6.2.2/bin"
expect_path_in_java_tools "third_party/java/proguard/proguard6.2.2/buildscripts"
expect_path_in_java_tools "third_party/java/proguard/proguard6.2.2/src"
expect_path_in_java_tools "third_party/java/proguard/proguard6.2.2/src/proguard"
}

run_suite "Java tools archive tests"
3 changes: 1 addition & 2 deletions src/test/shell/bazel/bazel_java_tools_test.sh
Original file line number Diff line number Diff line change
Expand Up @@ -187,10 +187,9 @@ function test_java_tools_has_jacocoagent() {
expect_path_in_java_tools "java_tools/third_party/java/jacoco/LICENSE"
}

# TODO(bencodes) This test should assert against a 6.2.2 after proguard is switched over
function test_java_tools_has_proguard() {
expect_path_in_java_tools "java_tools/third_party/java/proguard/proguard.jar"
expect_path_in_java_tools "java_tools/third_party/java/proguard/GPL.*"
expect_path_in_java_tools "java_tools/third_party/java/proguard/GPL.md"
}

function test_java_tools_toolchain_builds() {
Expand Down

0 comments on commit 165c1a0

Please sign in to comment.