it seems that the only thing preventing this from being compatible with the Jasmine test framework is the

this.timeout(30000); 

...in index.js. If I comment out that line and use within a -spec, the test reports a failure with the jshint errors on output.

Consider adding a test on that function?

Need to use the upgraded version of jshint to resolve security vulnerability.

otherwise, node_modules will be hinted.

Hey, this is a really handy utility.

I'm running into an issue, however, where the JSHint output is being incorrectly formatted:

Any ideas what's going on?

Test repo

https://github.com/williscool/code_gym/

Directory where /tests dir is
https://github.com/williscool/code_gym/tree/master/javascript

If you run mocha --recursive there it will not properly find the files to fail jshint

but if you go to the root and run it like this

mocha javascript/test/ --grep jshint

It will find the files.

Removed files should be ignored from any jshint run

Example:

• I deleted ErrorEmailDecorator.spec.js
• I run jshint and get error log. Expected - removed file is never mentioned again

jshint,
[21:54:40] service-router is started
ERROR: Can't open client/static/src/shared/utils/decorators/EmailErrorDecorator.js
ERROR: Can't open client/tests/jasmine.unittests/shared/utils/EmailErrorDecorator.spec.js
    ✓ should pass for working directory (2034ms)

from this:

$ jshint reassemble-spec.js
reassemble-spec.js: line 7, col 30, Missing space after 'function'.
reassemble-spec.js: line 10, col 31, Missing space after 'function'.
reassemble-spec.js: line 11, col 63, Missing space after 'function'.

to this:

$ jshint --verbose reassemble-spec.js
reassemble-spec.js: line 7, col 30, Missing space after 'function'. (W013)
reassemble-spec.js: line 10, col 31, Missing space after 'function'. (W013)
reassemble-spec.js: line 11, col 63, Missing space after 'function'. (W013)

this allows us to selectively disable warning in .jshintrc

{
    "-W013": true
}

I tried
--reporter spec
and other possible reporter options but doesn't see to make difference. Also It will be nice if we can passed the customized reporter code as an option field to overwrite the default.

Thanks