[BUGFIX] Add support for multiple -i args for cover command

gotwarlost / istanbul

Yet another JS code coverage tool that computes statement, line, function and branch coverage with module loader hooks to transparently add coverage when running tests. Supports all JS coverage use cases including unit tests, server side functional tests and browser tests. Built for scale.

Other

8.7k stars 787 forks source link

[BUGFIX] Add support for multiple -i args for cover command #756

Open golyshevd opened 7 years ago

golyshevd commented 7 years ago

Hi! istanbul cover does not support multiple -i arguments, because glob used incorrectly

istanbul cover -i "express/**/*.js" node_modules/.bin/_mocha
#ok, report written to disk

istanbul cover -i "express/**/*.js" -i "utils/**/*.js" node_modules/.bin/_mocha
#no coverage information collected

coveralls commented 7 years ago

Coverage decreased (-0.07%) to 97.454% when pulling aed046563ebd78e73c8bd016ad12cd7340efa5f3 on golyshevd:fix-cover into 89e338fcb1c8a7dea3b9e8f851aa55de2bc3abee on gotwarlost:master.

golyshevd commented 7 years ago

ping! :(