busterjs / buster

Abandoned - A powerful suite of automated test tools for JavaScript.
http://docs.busterjs.org
Other
448 stars 37 forks source link

glob file excludes should work in buster config resources option #397

Closed GCheung55 closed 10 years ago

GCheung55 commented 10 years ago

The resources option doesn't take a !files/to/exclude/*.js path.

The error I get is: '!files/to/exclude/*.js' matched no files.

GCheung55 commented 10 years ago

I think it has to do with https://github.com/busterjs/buster-configuration/blob/master/lib/group.js#L146-L167.

By following the code, I determined that addResource doesn't call resolvePathsAndAddMissingResources, but appendLoad does.

resolvePathsAndAddMissingResources eventually invokes expand which does the exclusions.

dwittner commented 10 years ago

Implemented by 6745fc6.