Skip to content

Commit

Permalink
fix: one more test assertion
Browse files Browse the repository at this point in the history
  • Loading branch information
alexeagle committed Feb 23, 2024
1 parent 64bb23f commit 607d0b6
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 2 deletions.
2 changes: 2 additions & 0 deletions format/defs.bzl
Original file line number Diff line number Diff line change
Expand Up @@ -51,6 +51,8 @@ def format_multirun(name, **kwargs):

# Target name should be human-readable since multirun 'print_targets' will echo it, e.g.
# Running @@//tools/format:format_Protocol_Buffer_with_buf
# Ideally multirun could print a message like "Formatting Protocol Buffer with Buf", see
# https://github.com/keith/rules_multirun/issues/31
target_name = "_".join([name, lang.replace(" ", "_"), "with", toolname])

for mode in ["check", "fix"]:
Expand Down
4 changes: 2 additions & 2 deletions format/test/format_test.bats
Original file line number Diff line number Diff line change
Expand Up @@ -22,8 +22,8 @@ bats_load_library "bats-assert"
run bazel run //format/test:format_JavaScript_with_prettier README.md example/.eslintrc.cjs
assert_success

assert_output --partial "Formatting JavaScript with Prettier..."
refute_output --partial "Formatting TypeScript with Prettier..."
assert_output --partial "README.md"
refute_output --partial "file.ts"
}

@test "should run buildozer on starlark" {
Expand Down

0 comments on commit 607d0b6

Please sign in to comment.