VimPlug/ale
1
0
Fork 0
mirror of https://github.com/dense-analysis/ale.git synced 2025-12-06 12:44:23 +08:00
Code Issues Actions Packages Projects Releases Wiki Activity

Fix #1132 - Parse react error codes again for ESLint

Browse Source
This commit is contained in:
w0rp
2017-11-16 09:34:24 +00:00
parent 1d65e5692f
commit 71d34fc0c6
2 changed files with 19 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

20
test/handler/test_eslint_handler.vader
View File

@@ -3,13 +3,12 @@ Before:
Save g:ale_javascript_eslint_suppress_missing_config
let g:ale_javascript_eslint_suppress_eslintignore = 0
let b:ale_javascript_eslint_suppress_missing_config = 0
unlet! b:ale_javascript_eslint_suppress_missing_config
let g:ale_javascript_eslint_suppress_missing_config = 0
After:
Restore
unlet! b:ale_javascript_eslint_suppress_eslintignore
unlet! b:ale_javascript_eslint_suppress_missing_config
unlet! g:config_error_lines
@@ -351,3 +350,18 @@ Execute(eslint should not warn about ignored files when explicitly disabled):
\ ale#handlers#eslint#Handle(bufnr(''), [
\ '/path/to/some/ignored.js:0:0: File ignored because of a matching ignore pattern. Use "--no-ignore" to override. [Warning]',
\ ])
Execute(eslint should handle react errors correctly):
AssertEqual
\ [
\ {
\ 'lnum': 59,
\ 'col': 9,
\ 'type': 'E',
\ 'text': 'Property should be placed on the same line as the component declaration',
\ 'code': 'react/jsx-first-prop-new-line',
\ },
\ ],
\ ale#handlers#eslint#Handle(bufnr(''), [
\ '/path/editor-help.jsx:59:9: Property should be placed on the same line as the component declaration [Error/react/jsx-first-prop-new-line]',
\ ])