yarsay tests fail when running with --cache flag about nyc HOT 9 CLOSED

Worked on my machine with 5.1.1 yargs/yarsay#6

from nyc.

@jamestalmage mind trying it with a linked copy of nyc on master. I thought we'd fixed this, but I can't seem to get it not to explode.

from nyc.

yep - it explodes with a linked copy.

from nyc.

@jamestalmage for the life of me I can not seem to isolate the change that is causing this exception in yarsay -- it almost seems like a race condition:

1. I rolled nyc back to 5.1.1 and linked in, was able to successfully output an lcov report a few times.
2. I moved forward to the next commit, ran tests, and the report again started throwing an exception.
3. I pulled in your most recent fix, and continue to have problems.

I have not seen things fail unless the --cache flag is set, really confused.

from nyc.

I moved forward to the next commit, ran tests, and the report again started throwing an exception.

718a319 definitely broke things.

I pulled in your most recent fix, and continue to have problems.

#127? That should have fixe what 718a319 broke

from nyc.

make sure to clear node_modules/.cache and the self coverage files.

from nyc.

Oh, didn't know to remove self coverage will give that a shot.

On Saturday, January 2, 2016, James Talmage [email protected]
wrote:

make sure to clear node_modules/.cache and the self coverage files.

—
Reply to this email directly or view it on GitHub
https://github.com/bcoe/nyc/issues/124#issuecomment-168447601.

from nyc.

Regenerating self-coverage should also work. We automatically use self-coverage if it exists. If you pull in a change, and try to use it immediately before generating new self-coverage via node ./build-self-coverage.js (which is part of npm test), then you are actually using covered code from before the merge.

I usually just do npm run clean in nyc to delete the self coverage. Self coverage shouldn't create problems as long as it is up to date, but I'm just paranoid.

It's also annoying when hunting a bug while linked (your changes to the uncovered code will not be used if *.covered.js exists), so my process is usually to just delete it.

from nyc.

fixed by: #127

from nyc.