summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorbutlerx <butlerx@notthe.cloud>2017-12-09 13:52:15 +0000
committerbutlerx <butlerx@notthe.cloud>2017-12-09 14:25:35 +0000
commit0700c2d90071883f1003b51e3f6d126f69d7f6f5 (patch)
treec79605b0510af7018b1ec6b90ee5972276faaed9
parentfcfd1025cc8fa6fca6771871820267d65fe9bf45 (diff)
downloadale-0700c2d90071883f1003b51e3f6d126f69d7f6f5.zip
add google-java-format fixer
-rw-r--r--README.md2
-rw-r--r--autoload/ale/fix/registry.vim5
-rw-r--r--autoload/ale/fixers/google_java_format.vim23
-rw-r--r--doc/ale-java.txt19
-rw-r--r--doc/ale.txt1
-rw-r--r--test/fixers/test_goofle_java_format_fixer_callback.vader27
6 files changed, 76 insertions, 1 deletions
diff --git a/README.md b/README.md
index b978c160..d2c4c5e1 100644
--- a/README.md
+++ b/README.md
@@ -108,7 +108,7 @@ formatting.
| Haskell | [brittany](https://github.com/lspitzner/brittany), [ghc](https://www.haskell.org/ghc/), [stack-ghc](https://haskellstack.org/), [stack-build](https://haskellstack.org/) !!, [ghc-mod](https://github.com/DanielG/ghc-mod), [stack-ghc-mod](https://github.com/DanielG/ghc-mod), [hlint](https://hackage.haskell.org/package/hlint), [hdevtools](https://hackage.haskell.org/package/hdevtools), [hfmt](https://github.com/danstiner/hfmt) |
| HTML | [HTMLHint](http://htmlhint.com/), [proselint](http://proselint.com/), [tidy](http://www.html-tidy.org/), [write-good](https://github.com/btford/write-good) |
| Idris | [idris](http://www.idris-lang.org/) |
-| Java | [checkstyle](http://checkstyle.sourceforge.net), [javac](http://www.oracle.com/technetwork/java/javase/downloads/index.html) |
+| Java | [checkstyle](http://checkstyle.sourceforge.net), [javac](http://www.oracle.com/technetwork/java/javase/downloads/index.html), [google-java-format](https://github.com/google/google-java-format) |
| JavaScript | [eslint](http://eslint.org/), [flow](https://flowtype.org/), [jscs](http://jscs.info/), [jshint](http://jshint.com/), [prettier](https://github.com/prettier/prettier), prettier-eslint >= 4.2.0, prettier-standard, [standard](http://standardjs.com/), [xo](https://github.com/sindresorhus/xo)
| JSON | [jsonlint](http://zaa.ch/jsonlint/), [prettier](https://github.com/prettier/prettier) |
| Kotlin | [kotlinc](https://kotlinlang.org) !!, [ktlint](https://ktlint.github.io) !! see `:help ale-integration-kotlin` for configuration instructions |
diff --git a/autoload/ale/fix/registry.vim b/autoload/ale/fix/registry.vim
index b4e18c47..53df7ccb 100644
--- a/autoload/ale/fix/registry.vim
+++ b/autoload/ale/fix/registry.vim
@@ -154,6 +154,11 @@ let s:default_registry = {
\ 'suggested_filetypes': ['sh'],
\ 'description': 'Fix sh files with shfmt.',
\ },
+\ 'google_java_format': {
+\ 'function': 'ale#fixers#google_java_format#Fix',
+\ 'suggested_filetypes': ['java'],
+\ 'description': 'Fix Java files with google-java-format.',
+\ },
\}
" Reset the function registry to the default entries.
diff --git a/autoload/ale/fixers/google_java_format.vim b/autoload/ale/fixers/google_java_format.vim
new file mode 100644
index 00000000..92632e84
--- /dev/null
+++ b/autoload/ale/fixers/google_java_format.vim
@@ -0,0 +1,23 @@
+" Author: butlerx <butlerx@notthe,cloud>
+" Description: Integration of Google-java-format with ALE.
+
+call ale#Set('google_java_format_executable', 'google-java-format')
+call ale#Set('google_java_format_use_global', 0)
+call ale#Set('google_java_format_options', '')
+
+function! ale#fixers#google_java_format#Fix(buffer) abort
+ let l:options = ale#Var(a:buffer, 'google_java_format_options')
+ let l:executable = ale#Var(a:buffer, 'google_java_format_executable')
+
+ if !executable(l:executable)
+ return 0
+ endif
+
+ return {
+ \ 'command': ale#Escape(l:executable)
+ \ . ' ' . (empty(l:options) ? '' : ' ' . l:options)
+ \ . ' --replace'
+ \ . ' %t',
+ \ 'read_temporary_file': 1,
+ \}
+endfunction
diff --git a/doc/ale-java.txt b/doc/ale-java.txt
index 13decb41..ce47542d 100644
--- a/doc/ale-java.txt
+++ b/doc/ale-java.txt
@@ -34,4 +34,23 @@ g:ale_java_javac_options *g:ale_java_javac_options*
===============================================================================
+google-java-format *ale-java-google-java-format*
+
+
+g:ale_java_google_java_format_executable *g:ale_java_google_java_format_executable*
+ *b:ale_java_google_java_format_executable*
+ Type: |String|
+ Default: `'google-java-format'`
+
+ See |ale-integrations-local-executables|
+
+
+g:ale_java_google_java_format_options *g:ale_java_google_java_format_options*
+ *b:ale_java_google_java_format_options*
+ Type: |String|
+ Default: `''`
+
+ This variable can be set to pass additional options
+
+===============================================================================
vim:tw=78:ts=2:sts=2:sw=2:ft=help:norl:
diff --git a/doc/ale.txt b/doc/ale.txt
index db725c08..f527b61c 100644
--- a/doc/ale.txt
+++ b/doc/ale.txt
@@ -93,6 +93,7 @@ CONTENTS *ale-contents*
java..................................|ale-java-options|
checkstyle..........................|ale-java-checkstyle|
javac...............................|ale-java-javac|
+ google-java-format..................|ale-java-google-java-format|
javascript............................|ale-javascript-options|
eslint..............................|ale-javascript-eslint|
flow................................|ale-javascript-flow|
diff --git a/test/fixers/test_goofle_java_format_fixer_callback.vader b/test/fixers/test_goofle_java_format_fixer_callback.vader
new file mode 100644
index 00000000..d64e2788
--- /dev/null
+++ b/test/fixers/test_goofle_java_format_fixer_callback.vader
@@ -0,0 +1,27 @@
+Before:
+ Save g:ale_google_java_format_executable
+
+ " Use an invalid global executable, so we don't match it.
+ let g:ale_google_java_format_executable = 'xxxinvalid'
+
+ call ale#test#SetDirectory('/testplugin/test/fixers')
+
+After:
+ Restore
+
+ call ale#test#RestoreDirectory()
+
+Execute(The google-java-format callback should return 0 when the executable isn't executable):
+ AssertEqual
+ \ 0,
+ \ ale#fixers#google_java_format#Fix(bufnr(''))
+
+Execute(The google-java-format callback should run the command when the executable test passes):
+ let g:ale_google_java_format_executable = has('win32') ? 'cmd' : 'echo'
+
+ AssertEqual
+ \ {
+ \ 'read_temporary_file': 1,
+ \ 'command': ale#Escape(ale_google_java_format_executable) . ' --replace %t'
+ \ },
+ \ ale#fixers#google_java_format#Fix(bufnr(''))