Skip to content

Commit

Permalink
Add Label() to archive_contents_test marco (#2618)
Browse files Browse the repository at this point in the history
Since this macro is not specific to any test or target under test in
rule_apple, Label() helps resolve the script form externally defined
targets and tests.

There are many more uses of the `apple_verification_test` rule, but they
are specific internally and I chose not to add across the board.
  • Loading branch information
erikkerber authored Dec 9, 2024
1 parent 5f7d38d commit 348f5a4
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion test/starlark_tests/rules/common_verification_tests.bzl
Original file line number Diff line number Diff line change
Expand Up @@ -221,7 +221,7 @@ def archive_contents_test(
"TEXT_FILE_NOT_CONTAINS": text_file_not_contains,
},
target_under_test = target_under_test,
verifier_script = "//test/starlark_tests:verifier_scripts/archive_contents_test.sh",
verifier_script = Label("//test/starlark_tests:verifier_scripts/archive_contents_test.sh"),
**kwargs
)

Expand Down

0 comments on commit 348f5a4

Please sign in to comment.