Use different reporter to support older versions of jscs (#737)

* Use different reporter to support older versions of jscs

* Add test and make more consistent with other code

* Add documentation for jscs

* Add more test coverage
This commit is contained in:
Jonathan Boudreau
2017-07-08 13:37:21 -04:00
committed by w0rp
parent dd5806662a
commit 7def00d5a9
4 changed files with 126 additions and 3 deletions

View File

@@ -0,0 +1,25 @@
Execute(jscs should parse lines correctly):
AssertEqual
\ [
\ {
\ 'lnum': 1,
\ 'col': 7,
\ 'text': 'disallowVar: Variable declarations should use `let` or `const` not `var`',
\ },
\ {
\ 'lnum': 3,
\ 'col': 21,
\ 'text': 'disallowTrailingWhitespace: Illegal trailing whitespace',
\ },
\ {
\ 'lnum': 5,
\ 'col': 9,
\ 'text': 'disallowUnusedVariables: Variable `hello` is not used',
\ },
\ ],
\ ale_linters#javascript#jscs#Handle(347, [
\ 'foobar.js: line 1, col 7, disallowVar: Variable declarations should use `let` or `const` not `var`',
\ 'foobar.js: line 3, col 21, disallowTrailingWhitespace: Illegal trailing whitespace',
\ 'foobar.js: line 5, col 9, disallowUnusedVariables: Variable `hello` is not used',
\ ])