Skip to content

Commit e11597d

Browse files
committed
fix up the tests
1 parent c9a70a4 commit e11597d

File tree

1 file changed

+3
-4
lines changed

1 file changed

+3
-4
lines changed

test_rules_scala.sh

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -178,7 +178,7 @@ test_scala_library_expect_failure_on_missing_direct_java() {
178178
dependency_target='//test_expect_failure/missing_direct_deps/internal_deps:transitive_dependency'
179179
test_target='//test_expect_failure/missing_direct_deps/internal_deps:transitive_dependency_java_user'
180180

181-
expected_message=".*$test_target"
181+
expected_message=".*$dependency_target"
182182

183183
test_expect_failure_or_warning_on_missing_direct_deps_with_expected_message "${expected_message}" $test_target "--strict_java_deps=error"
184184
}
@@ -192,7 +192,7 @@ test_scala_library_expect_failure_on_java_in_src_jar_when_disabled() {
192192
}
193193

194194
test_scala_library_expect_better_failure_message_on_missing_transitive_dependency_labels_from_other_jvm_rules() {
195-
transitive_target='.*transitive_dependency_ijar.jar'
195+
transitive_target='.*transitive_dependency-ijar.jar'
196196
direct_target='//test_expect_failure/missing_direct_deps/internal_deps:direct_java_provider_dependency'
197197
test_target='//test_expect_failure/missing_direct_deps/internal_deps:dependent_on_some_java_provider'
198198

@@ -203,10 +203,9 @@ test_scala_library_expect_better_failure_message_on_missing_transitive_dependenc
203203

204204
test_scala_library_expect_failure_on_missing_direct_deps_warn_mode_java() {
205205
dependency_target='//test_expect_failure/missing_direct_deps/internal_deps:transitive_dependency'
206-
dependency_file='test_expect_failure/missing_direct_deps/internal_deps/transitive_dependency_ijar.jar'
207206
test_target='//test_expect_failure/missing_direct_deps/internal_deps:transitive_dependency_java_user'
208207

209-
local expected_message="$dependency_file.*$test_target"
208+
local expected_message=".*$dependency_target"
210209

211210
test_expect_failure_or_warning_on_missing_direct_deps_with_expected_message "${expected_message}" ${test_target} "--strict_java_deps=warn" "ne"
212211
}

0 commit comments

Comments
 (0)