jshint/jshint — Issue #317: Tests are broken as of 3c37587
Author: goatslacker
Created: 2011-10-26 21:25:48+00:00
State: open

Here's the commit:

https://github.com/jshint/jshint/commit/3c37587bda31d97f5ab48a2c872956cf2df316ca

Here's the output from `expresso tests/*.js`

core.js checkRhino: AssertionError:
TestRun: jshint-rhino
Errors thrown by JSHINT, but not defined in test run
Line 13 Missing space after 'function'.
Line 13 Missing space after ')'.
Line 14 Missing space after ')'.
Line 16 Missing space after ')'.
Line 43 Move the invocation into the parens that contain the function.
Line 53 Move the invocation into the parens that contain the function.
Line 57 Missing space after 'function'.
Line 57 Missing space after ')'.
Line 67 Unexpected use of '++'.

at Object.test (/home/josh/Development/jshint/tests/testhelper:108:20)
at Test.fn (/home/josh/Development/jshint/tests/core.js:40:29)
at Test.runParallel (/usr/local/lib/node_modules/expresso/bin/expresso:959:10)
at Test.run (/usr/local/lib/node_modules/expresso/bin/expresso:924:18)
at next (/usr/local/lib/node_modules/expresso/bin/expresso:867:22)
at Test.run (/usr/local/lib/node_modules/expresso/bin/expresso:925:13)
at next (/usr/local/lib/node_modules/expresso/bin/expresso:867:22)
at runSuite (/usr/local/lib/node_modules/expresso/bin/expresso:875:6)
at check (/usr/local/lib/node_modules/expresso/bin/expresso:814:12)
at runFile (/usr/local/lib/node_modules/expresso/bin/expresso:819:6)

Failures: 1

Now there's nothing wrong with that particular style of coding (although IMHO it should be consistent with the rest of the file) but the tests are failing regardless.

Either jshint ships with a `.jshintrc` file in the project directory and turns off these options, we "fix" the syntax, or we add the options at the top of the file.