Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

File Name Coverage Issue #461

Closed
arb opened this issue Oct 4, 2015 · 2 comments
Closed

File Name Coverage Issue #461

arb opened this issue Oct 4, 2015 · 2 comments
Assignees
Labels
bug
Milestone

Comments

@arb
Copy link
Contributor

@arb arb commented Oct 4, 2015

If you try to require a file that has the word "test" in the file, coverage will not be run on that file.

|> project
|--index.js
|--test-server.js
|>  test
|------index.js

I think the problem is here. I think the end of that regex should read (?!test\/|node_modules\/) by default.

@Marsup

This comment has been minimized.

Copy link
Member

@Marsup Marsup commented Oct 4, 2015

Are you sure just a $ at the end wouldn't be enough ?

@arb

This comment has been minimized.

Copy link
Contributor Author

@arb arb commented Oct 4, 2015

Perhaps. You know what they say about regular expressions; if you solve a problem with a regular expression, now you've got two problems.

@geek geek added the bug label Oct 4, 2015
@geek geek added this to the 10.6.1 milestone May 19, 2016
@geek geek self-assigned this May 19, 2016
@geek geek closed this in #588 May 19, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants
You can’t perform that action at this time.