diff --git a/autoload/ale/fixers/shfmt.vim b/autoload/ale/fixers/shfmt.vim index 06e8da57..0eefc985 100644 --- a/autoload/ale/fixers/shfmt.vim +++ b/autoload/ale/fixers/shfmt.vim @@ -5,27 +5,13 @@ scriptencoding utf-8 call ale#Set('sh_shfmt_executable', 'shfmt') call ale#Set('sh_shfmt_options', '') -function! s:DefaultOption(buffer) abort - if getbufvar(a:buffer, '&expandtab') == 0 - " Tab is used by default - return '' - endif - - let l:tabsize = getbufvar(a:buffer, '&shiftwidth') - - if l:tabsize == 0 - let l:tabsize = getbufvar(a:buffer, '&tabstop') - endif - - return ' -i ' . l:tabsize -endfunction - function! ale#fixers#shfmt#Fix(buffer) abort let l:executable = ale#Var(a:buffer, 'sh_shfmt_executable') let l:options = ale#Var(a:buffer, 'sh_shfmt_options') return { \ 'command': ale#Escape(l:executable) - \ . (empty(l:options) ? s:DefaultOption(a:buffer) : ' ' . l:options) + \ . ' -filename=%s' + \ . (empty(l:options) ? '' : ' ' . l:options) \} endfunction diff --git a/test/fixers/test_shfmt_fixer_callback.vader b/test/fixers/test_shfmt_fixer_callback.vader index 99cb0987..02fce526 100644 --- a/test/fixers/test_shfmt_fixer_callback.vader +++ b/test/fixers/test_shfmt_fixer_callback.vader @@ -14,39 +14,6 @@ Execute(The shfmt callback should return 'shfmt' as default command): \ ale#fixers#shfmt#Fix(bufnr('')).command =~# '^' . ale#Escape('shfmt'), \ "Default command name is expected to be 'shfmt'" -Execute(The shfmt callback should return the command with no option as default when noexpandtab is set): - let g:ale_sh_shfmt_executable = 'shfmt' - let g:ale_sh_shfmt_options = '' - setlocal noexpandtab - AssertEqual - \ { - \ 'command': ale#Escape('shfmt'), - \ }, - \ ale#fixers#shfmt#Fix(bufnr('')) - -Execute(The shfmt callback should return the command specifying indent width by looking shiftwidth as default): - let g:ale_sh_shfmt_executable = 'shfmt' - let g:ale_sh_shfmt_options = '' - setlocal expandtab - setlocal shiftwidth=4 - AssertEqual - \ { - \ 'command': ale#Escape('shfmt') . ' -i 4', - \ }, - \ ale#fixers#shfmt#Fix(bufnr('')) - -Execute(The shfmt callback should return the command specifying indent width by looking tabstop when shiftwidth is 0 as default): - let g:ale_sh_shfmt_executable = 'shfmt' - let g:ale_sh_shfmt_options = '' - setlocal expandtab - setlocal shiftwidth=0 - setlocal tabstop=8 - AssertEqual - \ { - \ 'command': ale#Escape('shfmt') . ' -i 8', - \ }, - \ ale#fixers#shfmt#Fix(bufnr('')) - Execute(The shfmt executable and options should be configurable): let g:ale_sh_shfmt_executable = 'foobar' let g:ale_sh_shfmt_options = '--some-option' @@ -54,6 +21,7 @@ Execute(The shfmt executable and options should be configurable): AssertEqual \ { \ 'command': ale#Escape('foobar') + \ . ' -filename=%s' \ . ' --some-option', \ }, \ ale#fixers#shfmt#Fix(bufnr(''))