summaryrefslogtreecommitdiff
path: root/test
diff options
context:
space:
mode:
authorJon Parise <jon@indelible.org>2021-03-23 06:04:51 -0700
committerGitHub <noreply@github.com>2021-03-23 22:04:51 +0900
commiteb0ebe622102cc6da3d7e943a3b739db7b6ed216 (patch)
tree886dd0321e4e95e7dbf73b468330b22393a5bce4 /test
parentcdac7a830e78b1a1d2b3e7da135f2d92669dc3e3 (diff)
downloadale-eb0ebe622102cc6da3d7e943a3b739db7b6ed216.zip
Switch to using buildifier's -path option (#3640)
Buildifier offers a -path command line option: > Buildifier's reformatting depends in part on the path to the file > relative to the workspace directory. Normally buildifier deduces > that path from the file names given, but the path can be given > explicitly with the -path argument. This is especially useful when > reformatting standard input, or in scripts that reformat a temporary > copy of a file. This lets us drop our `-type`-guessing logic. For the moment, we rely on the buffer filename being "relative to the workspace directory", which it generally is, but in a future change, we should introduce logic that will locate the WORKSPACE file and adjusts the filename relative to that directory. This could be complicated based on the working directory in which `buildifier` is executed, so a little more research is necessary to implement that logic correctly.
Diffstat (limited to 'test')
-rw-r--r--test/fixers/test_buildifier_fixer_callback.vader26
1 files changed, 6 insertions, 20 deletions
diff --git a/test/fixers/test_buildifier_fixer_callback.vader b/test/fixers/test_buildifier_fixer_callback.vader
index db9f1e55..f82e8e6e 100644
--- a/test/fixers/test_buildifier_fixer_callback.vader
+++ b/test/fixers/test_buildifier_fixer_callback.vader
@@ -11,7 +11,9 @@ Execute(The buildifier callback should return the correct default values):
AssertFixer
\ {
\ 'command': ale#Escape(g:ale_bazel_buildifier_executable)
- \ . ' -mode fix -lint fix -type workspace -'
+ \ . ' -mode fix -lint fix -path '
+ \ . ale#Escape(ale#test#GetFilename('../test-files/bazel/WORKSPACE'))
+ \ . ' -'
\ }
Execute(The buildifier callback should include any additional options):
@@ -21,23 +23,7 @@ Execute(The buildifier callback should include any additional options):
AssertFixer
\ {
\ 'command': ale#Escape(g:ale_bazel_buildifier_executable)
- \ . ' -mode fix -lint fix -type workspace --some-option -',
- \ }
-
-Execute(The buildifier callback should recognize BUILD files):
- call ale#test#SetFilename('../test-files/bazel/BUILD')
-
- AssertFixer
- \ {
- \ 'command': ale#Escape(g:ale_bazel_buildifier_executable)
- \ . ' -mode fix -lint fix -type build -'
- \ }
-
-Execute(The buildifier callback should recognize .bzl files):
- call ale#test#SetFilename('../test-files/bazel/defs.bzl')
-
- AssertFixer
- \ {
- \ 'command': ale#Escape(g:ale_bazel_buildifier_executable)
- \ . ' -mode fix -lint fix -type bzl -'
+ \ . ' -mode fix -lint fix -path '
+ \ . ale#Escape(ale#test#GetFilename('../test-files/bazel/WORKSPACE'))
+ \ . ' --some-option -'
\ }