Merge pull request #1623 from zoonfafer/scalafmt-fixer

Add 'scalafmt' fixer for Scala files

2 files changed, 69 insertions, 0 deletions

diff --git a/test/command_callback/scala_paths/dummy.scala b/test/command_callback/scala_paths/dummy.scala
new file mode 100644
index 00000000..e69de29b
--- /dev/null
+++ b/test/command_callback/scala_paths/dummy.scala
diff --git a/test/fixers/test_scalafmt_fixer_callback.vader b/test/fixers/test_scalafmt_fixer_callback.vader
new file mode 100644
index 00000000..d82fda43
--- /dev/null
+++ b/test/fixers/test_scalafmt_fixer_callback.vader
@@ -0,0 +1,69 @@
+Before:
+  Save g:ale_scala_scalafmt_executable
+  Save g:ale_scala_scalafmt_options
+
+  " Use an invalid global executable, so we don't match it.
+  let g:ale_scala_scalafmt_executable = 'xxxinvalid'
+  let g:ale_scala_scalafmt_options = ''
+
+  call ale#test#SetDirectory('/testplugin/test/fixers')
+  silent cd ..
+  silent cd command_callback
+  let g:dir = getcwd()
+
+After:
+  Restore
+
+  call ale#test#RestoreDirectory()
+
+Execute(The scalafmt callback should return the correct default values):
+  call ale#test#SetFilename('scala_paths/dummy.scala')
+
+  AssertEqual
+  \ {
+  \   'read_temporary_file': 1,
+  \   'command': ale#Escape(g:ale_scala_scalafmt_executable)
+  \     . ' %t',
+  \ },
+  \ ale#fixers#scalafmt#Fix(bufnr(''))
+
+Execute(The scalafmt callback should use ng with scalafmt automatically):
+  let g:ale_scala_scalafmt_executable = 'ng'
+  call ale#test#SetFilename('scala_paths/dummy.scala')
+
+  AssertEqual
+  \ {
+  \   'read_temporary_file': 1,
+  \   'command': ale#Escape('ng')
+  \     . ' scalafmt'
+  \     . ' %t',
+  \ },
+  \ ale#fixers#scalafmt#Fix(bufnr(''))
+
+Execute(The scalafmt callback should include custom scalafmt options):
+  let g:ale_scala_scalafmt_options = '--diff'
+  call ale#test#SetFilename('scala_paths/dummy.scala')
+
+  AssertEqual
+  \ {
+  \   'read_temporary_file': 1,
+  \   'command': ale#Escape(g:ale_scala_scalafmt_executable)
+  \     . ' --diff'
+  \     . ' %t',
+  \ },
+  \ ale#fixers#scalafmt#Fix(bufnr(''))
+
+Execute(The scalafmt callback should include custom scalafmt options and use ng with scalafmt):
+  let g:ale_scala_scalafmt_options = '--diff'
+  let g:ale_scala_scalafmt_executable = 'ng'
+  call ale#test#SetFilename('scala_paths/dummy.scala')
+
+  AssertEqual
+  \ {
+  \   'read_temporary_file': 1,
+  \   'command': ale#Escape('ng')
+  \     . ' scalafmt'
+  \     . ' --diff'
+  \     . ' %t',
+  \ },
+  \ ale#fixers#scalafmt#Fix(bufnr(''))