You cannot select more than 25 topics
Topics must start with a letter or number, can include dashes ('-') and can be up to 35 characters long.
65 lines
2.1 KiB
Plaintext
65 lines
2.1 KiB
Plaintext
7 years ago
|
Before:
|
||
|
|
let g:ale_javascript_eslint_executable = 'eslint_d'
|
||
|
|
|
||
|
|
call ale#test#SetDirectory('/testplugin/test')
|
||
|
|
|
||
|
|
runtime ale_linters/javascript/eslint.vim
|
||
|
|
|
||
|
|
After:
|
||
|
|
let g:ale_has_override = {}
|
||
|
|
let g:ale_javascript_eslint_executable = 'eslint'
|
||
|
|
let g:ale_javascript_eslint_use_global = 0
|
||
|
|
|
||
|
|
call ale#test#RestoreDirectory()
|
||
|
|
call ale#linter#Reset()
|
||
|
|
|
||
|
|
Execute(create-react-app directories should be detected correctly):
|
||
|
|
call ale#test#SetFilename('eslint-test-files/react-app/subdir/testfile.js')
|
||
|
|
|
||
|
|
AssertEqual
|
||
|
|
\ g:dir . '/eslint-test-files/react-app/node_modules/eslint/bin/eslint.js',
|
||
|
|
\ ale#handlers#eslint#GetExecutable(bufnr(''))
|
||
|
|
|
||
|
|
Execute(use-global should override create-react-app detection):
|
||
|
|
let g:ale_javascript_eslint_use_global = 1
|
||
|
|
|
||
|
|
call ale#test#SetFilename('eslint-test-files/react-app/subdir/testfile.js')
|
||
|
|
|
||
|
|
AssertEqual
|
||
|
|
\ 'eslint_d',
|
||
|
|
\ ale#handlers#eslint#GetExecutable(bufnr(''))
|
||
|
|
|
||
|
|
Execute(other app directories should be detected correctly):
|
||
|
|
call ale#test#SetFilename('eslint-test-files/other-app/subdir/testfile.js')
|
||
|
|
|
||
|
|
AssertEqual
|
||
|
|
\ g:dir . '/eslint-test-files/node_modules/.bin/eslint',
|
||
|
|
\ ale#handlers#eslint#GetExecutable(bufnr(''))
|
||
|
|
|
||
|
|
Execute(use-global should override other app directories):
|
||
|
|
let g:ale_javascript_eslint_use_global = 1
|
||
|
|
|
||
|
|
call ale#test#SetFilename('eslint-test-files/other-app/subdir/testfile.js')
|
||
|
|
|
||
|
|
AssertEqual
|
||
|
|
\ 'eslint_d',
|
||
|
|
\ ale#handlers#eslint#GetExecutable(bufnr(''))
|
||
|
|
|
||
|
|
Execute(eslint_d should be detected correctly):
|
||
|
|
call ale#test#SetFilename('eslint-test-files/app-with-eslint-d/testfile.js')
|
||
|
|
|
||
|
|
AssertEqual
|
||
|
|
\ g:dir . '/eslint-test-files/app-with-eslint-d/node_modules/.bin/eslint_d',
|
||
|
|
\ ale#handlers#eslint#GetExecutable(bufnr(''))
|
||
|
|
|
||
|
|
Execute(eslint.js executables should be run with node on Windows):
|
||
|
|
call ale#test#SetFilename('eslint-test-files/react-app/subdir/testfile.js')
|
||
|
|
let g:ale_has_override['win32'] = 1
|
||
|
|
|
||
|
|
" We have to execute the file with node.
|
||
|
|
AssertEqual
|
||
|
|
\ ale#Escape('node.exe') . ' '
|
||
|
|
\ . ale#Escape(g:dir . '/eslint-test-files/react-app/node_modules/eslint/bin/eslint.js')
|
||
|
|
\ . ' -f unix --stdin --stdin-filename %s',
|
||
|
|
\ ale#handlers#eslint#GetCommand(bufnr(''))
|