add --all functionality

[bcoe]

we need to figure out a way for --all to work with this instrumentation approach, we might be able to work backwards from the --all functionality in nyc.

[bcoe]

see also: dtinth/babel-plugin-__coverage__#38

[bcoe]

@gotwarlost @dtinth, any thoughts as to how we might approach adding blank coverage for all files, without messing with the babel cache?

Perhaps we could just walk all the files, and add fake entries in __coverage__, could we add an API method for this to lib-instrument?

[danez]

Do you think this would be possible todo in the babel-plugin itself? To me it looks more like that nyc should traverse all files and add blank coverage before running the tests. Because on a test-suite without tests, the plugin wouldn't be called at all, but you might still want to get an report telling you 0% right?

I would like to give this a shoot over the weekend, so if you have hints were to start let me know please.

[bcoe]

@danez I'm not 100% sure of the best solution; folks might be using different approaches to instrumenting code with babel, etc., so my temptation was to try to do it in the plugin layer -- the other reason being that the plugin knows about include/exclude rules -- which we want to make sure we adhere to.

my temptation was to think about logic similar to nyc: https://github.com/istanbuljs/nyc/blob/master/index.js#L220 perhaps this is a good starting point?

I'm not an expert with babel, so I'm sure your approach might be better, really I'm open to experimentation.

[danez]

I had a look, but I couldn't figure out an easy way to do this and spent most of the time understanding how nyc works :)
In general it feels wrong to me to put this logic in the babel-plugin itself, as these would be side-effects that are not really nice. And even if the plugin would do it and instrument all included files, then it would still need to communicate the list of files to nyc somehow.

Maybe shipping an instrumenter adapter with this repo for nyc (like the noop.js) that handles and knows how to load stuff that was processed by this plugin.

I won't have time to look into to this again in the next weeks.

[lughino]

Hi, any news about this enhancement?

[benhunsaker]

has there been any update on this?