Massively reduce the amount of code needed for linter tests

1 files changed, 13 insertions, 26 deletions

diff --git a/test/command_callback/test_reason_ols_callbacks.vader b/test/command_callback/test_reason_ols_callbacks.vader
index 5fb39afe..53ef7fab 100644
--- a/test/command_callback/test_reason_ols_callbacks.vader
+++ b/test/command_callback/test_reason_ols_callbacks.vader
@@ -1,54 +1,41 @@
 Before:
-  Save &filetype
-  Save g:ale_reason_ols_executable
-  Save g:ale_reason_ols_use_global
+  call ale#assert#SetUpLinterTest('reason', 'ols')
 
+  Save &filetype
   let &filetype = 'reason'
-  unlet! g:ale_reason_ols_executable
-  unlet! g:ale_reason_ols_use_global
-
-  runtime ale_linters/reason/ols.vim
-
-  call ale#test#SetDirectory('/testplugin/test/command_callback')
 
 After:
-  Restore
-
-  call ale#test#RestoreDirectory()
-  call ale#linter#Reset()
+  call ale#assert#TearDownLinterTest()
 
 Execute(The language string should be correct):
-  AssertEqual 'reason', ale#handlers#ols#GetLanguage(bufnr(''))
+  AssertLSPLanguage 'reason'
 
 Execute(The default executable should be correct):
-  AssertEqual 'ocaml-language-server', ale#handlers#ols#GetExecutable(bufnr(''))
-  AssertEqual
-  \ ale#Escape('ocaml-language-server') . ' --stdio',
-  \ ale#handlers#ols#GetCommand(bufnr(''))
+  AssertLinter 'ocaml-language-server',
+  \ ale#Escape('ocaml-language-server') . ' --stdio'
 
 Execute(The project root should be detected correctly):
-  AssertEqual '', ale#handlers#ols#GetProjectRoot(bufnr(''))
+  AssertLSPProject ''
 
   call ale#test#SetFilename('ols_paths/file.re')
 
-  AssertEqual
-  \ ale#path#Simplify(g:dir . '/ols_paths'),
-  \ ale#handlers#ols#GetProjectRoot(bufnr(''))
+  AssertLSPProject ale#path#Simplify(g:dir . '/ols_paths')
 
 Execute(The local executable should be used when available):
   call ale#test#SetFilename('ols_paths/file.re')
 
-  AssertEqual
+  AssertLinter
   \ ale#path#Simplify(g:dir . '/ols_paths/node_modules/.bin/ocaml-language-server'),
-  \ ale#handlers#ols#GetExecutable(bufnr(''))
+  \ ale#Escape(ale#path#Simplify(g:dir . '/ols_paths/node_modules/.bin/ocaml-language-server')) . ' --stdio'
 
 Execute(The gloabl executable should always be used when use_global is set):
   let g:ale_reason_ols_use_global = 1
   call ale#test#SetFilename('ols_paths/file.re')
 
-  AssertEqual 'ocaml-language-server', ale#handlers#ols#GetExecutable(bufnr(''))
+  AssertLinter 'ocaml-language-server',
+  \ ale#Escape('ocaml-language-server') . ' --stdio'
 
 Execute(The executable should be configurable):
   let g:ale_reason_ols_executable = 'foobar'
 
-  AssertEqual 'foobar', ale#handlers#ols#GetExecutable(bufnr(''))
+  AssertLinter 'foobar', ale#Escape('foobar') . ' --stdio'