Fix test_compiler_sources_integrity.sh on non-Mac

mbland · mbland · commit 4dff11b935b8 · 2025-07-24T18:55:24.000-04:00
The unescaped `{` in the expected failure message caused `test_fail_if_compiler_srcjar_object_is_empty` to fail on Linux and Windows. Rather than escape it, it works just as well to remove it entirely. See also the "Escape '{', '}' in `test/shell/test_bzlmod_macros`" message from bazel-contrib#1722 and commit 3f37e26.
diff --git a/test/shell/test_compiler_sources_integrity.sh b/test/shell/test_compiler_sources_integrity.sh
@@ -195,7 +195,7 @@ test_fail_if_compiler_srcjar_object_is_empty() {
   # scala_compiler_srcjar argument was empty. Therefore it's worth testing here.
   local expected_msg=(
     "scala_compiler_srcjar invalid, must be a dict with exactly one of"
-    '"label", "url" or "urls" keys, got: {'
+    '"label", "url" or "urls" keys, got: '
   )
 
   action_should_fail_with_message \

Original file line number	Diff line number	Diff line change
`@@ -195,7 +195,7 @@ test_fail_if_compiler_srcjar_object_is_empty() {`
`195`	`195`	`# scala_compiler_srcjar argument was empty. Therefore it's worth testing here.`
`196`	`196`	`local expected_msg=(`
`197`	`197`	`"scala_compiler_srcjar invalid, must be a dict with exactly one of"`
`198`		`- '"label", "url" or "urls" keys, got: {'`
	`198`	`+ '"label", "url" or "urls" keys, got: '`
`199`	`199`	`)`
`200`	`200`
`201`	`201`	`action_should_fail_with_message \`