fixers/xo: refactor to handlers

autoload/ale/fixers/xo.vim

@@ -1,20 +1,9 @@
 " Author: Albert Marquez - https://github.com/a-marquez
 " Description: Fixing files with XO.
 
-call ale#Set('javascript_xo_executable', 'xo')
-call ale#Set('javascript_xo_use_global', get(g:, 'ale_use_global_executables', 0))
-call ale#Set('javascript_xo_options', '')
-
-function! ale#fixers#xo#GetExecutable(buffer) abort
-    return ale#node#FindExecutable(a:buffer, 'javascript_xo', [
-    \   'node_modules/xo/cli.js',
-    \   'node_modules/.bin/xo',
-    \])
-endfunction
-
 function! ale#fixers#xo#Fix(buffer) abort
-    let l:executable = ale#fixers#xo#GetExecutable(a:buffer)
-    let l:options = ale#Var(a:buffer, 'javascript_xo_options')
+    let l:executable = ale#handlers#xo#GetExecutable(a:buffer, 'javascript')
+    let l:options = ale#handlers#xo#GetOptions(a:buffer, 'javascript')
 
     return {
     \   'command': ale#node#Executable(a:buffer, l:executable)

autoload/ale/handlers/xo.vim

@@ -8,6 +8,7 @@ call ale#Set('typescript_xo_options', '')
 
 function! ale#handlers#xo#GetExecutable(buffer, type) abort
     return ale#node#FindExecutable(a:buffer, a:type . '_xo', [
+    \   'node_modules/xo/cli.js',
     \   'node_modules/.bin/xo',
     \])
 endfunction

test/fixers/test_xo_fixer_callback.vader

@@ -1,5 +1,6 @@
 Before:
   call ale#assert#SetUpFixerTest('javascript', 'xo')
+  runtime autoload/ale/handlers/xo.vim
 
 After:
   call ale#assert#TearDownFixerTest()