Merge pull request #3257 from jhlink/fix_astyle_config_detection_with_source

Fix Astyle Project Option Detection when Placed with Source Files

autoload/ale/fixers/astyle.vim

@@ -49,7 +49,7 @@ endfunction
 function! ale#fixers#astyle#Fix(buffer) abort
     let l:executable = ale#fixers#astyle#Var(a:buffer, 'executable')
     let l:proj_options = ale#fixers#astyle#FindProjectOptions(a:buffer)
-    let l:command = ' --stdin='
+    let l:command = ' --stdin=' . ale#Escape(expand('#' . a:buffer))
 
     return {
     \   'command': ale#Escape(l:executable)

test/fixers/test_astyle_fixer_callback.vader

@@ -20,11 +20,12 @@ Execute(The astyle callback should return the correct default values):
   " Because this file doesn't exist, no astylrc config
   " exists near it. Therefore, project_options is empty.
   call ale#test#SetFilename('../c_files/testfile.c')
+  let targetfile = bufname(bufnr('%'))
 
   AssertEqual
   \ {
   \   'command': ale#Escape(g:ale_c_astyle_executable)
-  \     . ' --stdin='
+  \     . ' --stdin=' . ale#Escape(targetfile)
   \ },
   \ ale#fixers#astyle#Fix(bufnr(''))
 
@@ -33,46 +34,63 @@ Execute(The astyle callback should support cpp files):
   " exists near it. Therefore, project_options is empty.
   call ale#test#SetFilename('../cpp_files/dummy.cpp')
   set filetype=cpp " The test fails without this
+  let targetfile = bufname(bufnr('%'))
 
   AssertEqual
   \ {
   \   'command': ale#Escape(g:ale_cpp_astyle_executable)
-  \     . ' --stdin='
+  \     . ' --stdin=' . ale#Escape(targetfile)
   \ },
   \ ale#fixers#astyle#Fix(bufnr(''))
 
 Execute(The astyle callback should support cpp files with option file set):
   call ale#test#SetFilename('../cpp_files/dummy.cpp')
   let g:ale_cpp_astyle_project_options = '.astylerc_cpp'
+  let targetfile = bufname(bufnr('%'))
   set filetype=cpp " The test fails without this
 
   AssertEqual
   \ {
   \   'command': ale#Escape('invalidpp')
   \     . ' --project=' . g:ale_cpp_astyle_project_options
-  \     . ' --stdin='
+  \     . ' --stdin=' . ale#Escape(targetfile)
   \ },
   \ ale#fixers#astyle#Fix(bufnr(''))
 
-Execute(The astyle callback should return the correct default values with an option file set):
+Execute(The astyle callback should return the correct default values with a specified option file):
   call ale#test#SetFilename('../c_files/testfile.c')
   let g:ale_c_astyle_project_options = '.astylerc_c'
+  let targetfile = bufname(bufnr('%'))
 
   AssertEqual
   \ {
   \   'command': ale#Escape('xxxinvalid')
   \     . ' --project=' . g:ale_c_astyle_project_options
-  \     . ' --stdin='
+  \     . ' --stdin=' . ale#Escape(targetfile)
   \ },
   \ ale#fixers#astyle#Fix(bufnr(''))
 
 Execute(The astyle callback should find nearest default option file _astylrc):
   call ale#test#SetFilename('../test_c_projects/makefile_project/subdir/file.c')
+  let targetfile = bufname(bufnr('%'))
 
   AssertEqual
   \ {
   \   'command': ale#Escape('xxxinvalid')
   \     . ' --project=_astylerc'
-  \     . ' --stdin='
+  \     . ' --stdin=' . ale#Escape(targetfile)
+  \ },
+  \ ale#fixers#astyle#Fix(bufnr(''))
+
+Execute(The astyle callback should find .astylrc in the same directory as src):
+  call ale#test#SetFilename('../test_cpp_project/dummy.cpp')
+  set filetype=cpp " The test fails without this
+  let targetfile = bufname(bufnr('%'))
+
+  AssertEqual
+  \ {
+  \   'command': ale#Escape('invalidpp')
+  \     . ' --project=.astylerc'
+  \     . ' --stdin=' . ale#Escape(targetfile)
   \ },
   \ ale#fixers#astyle#Fix(bufnr(''))