Use ale#Escape in c_clangformat_style_option (#4949)

* Use ale#Escape in c_clangformat_style_option
* Update Vader test

autoload/ale/fixers/clangformat.vim

@@ -22,7 +22,7 @@ function! ale#fixers#clangformat#Fix(buffer) abort
     let l:use_local_file = ale#Var(a:buffer, 'c_clangformat_use_local_file')
 
     if l:style_option isnot# ''
-        let l:style_option = '-style=' . "'" . l:style_option . "'"
+        let l:style_option = '-style=' . ale#Escape(l:style_option)
     endif
 
     if l:use_local_file

autoload/ale/linter.vim

@@ -11,6 +11,7 @@ let s:linters = {}
 " NOTE: Update the g:ale_linter_aliases documentation when modifying this.
 let s:default_ale_linter_aliases = {
 \   'Dockerfile': 'dockerfile',
+\   'bash': 'sh',
 \   'csh': 'sh',
 \   'javascriptreact': ['javascript', 'jsx'],
 \   'plaintex': 'tex',

doc/ale.txt

@@ -1735,6 +1735,7 @@ g:ale_linter_aliases
 
   {
   \   'Dockerfile': 'dockerfile',
+  \   'bash': 'sh',
   \   'csh': 'sh',
   \   'javascriptreact': ['javascript', 'jsx'],
   \   'plaintex': 'tex',

test/fixers/test_clangformat_fixer_callback.vader

@@ -45,7 +45,8 @@ Execute(The clangformat callback should include style options as well):
   \ {
   \   'command': ale#Escape(g:ale_c_clangformat_executable)
   \     . ' --assume-filename=' . ale#Escape(bufname(bufnr('')))
-  \     . ' --some-option' . " -style='{BasedOnStyle: Microsoft, ColumnLimit:80,}'",
+  \     . ' --some-option'
+  \     . ' -style=' . ale#Escape(g:ale_c_clangformat_style_option)
   \ },
   \ ale#fixers#clangformat#Fix(bufnr(''))