Exposing filename in JSON, doc, and json-stream reporters

[dfebs]

Description of the Change

This change makes the json, doc, and json-stream reporters output the test filenames in addition to their other attributes

Alternate Designs

No other designs were considered in the making of this pull request

Why should this be in core?

There currently is not a way to access filenames through reporters when running tests even though mocha has full access to it.

Benefits

People who want to see what file is associated with their test step will easily be able to see it in the json, doc, and json-stream reporters. This is especially useful for people with large code/test base where it may be difficult to find the test.

Possible Drawbacks

If we intend to encapsulate the test's filename, then this code change would be an issue

Applicable issues

This would be considered a minor release.

[jsf-clabot]

All committers have signed the CLA.

[outsideris]

If we add filename to JSON reporter, we should consider to add it to other reporters like JSONStream reporter.

[coveralls]

Coverage increased (+0.07%) to 93.133% when pulling bcb9c4c on Daniel0113:expose-test-filename into 6838eaf on mochajs:master.

[dfebs]

@outsideris I have added filename exposure to the doc and json-stream reporters (and updated tests for all the reporters I touched). Unless there are other reporters we have in mind, this is ready for a second look 😄

[boneskull]

Thanks for the PR! I note that this will break #4245, but that's my problem.

If there's a case in which test.file is undefined (I am unsure if this is possible), these reporters will print undefined. Might be worth while to check for test.file, and if it's falsy, output (no filename) or something instead. Not a big deal, and this could be merged w/o it.

[dfebs]

Given that I also can't think of a test that isn't located in a file, I'm good with merging as-is then 😄 . If anyone has strong opinions on the matter though I am open to that.