Skip to content

Commit

Permalink
add make report-nyc task as we observed that most examples (with co…
Browse files Browse the repository at this point in the history
…de coverage when FULL_CODE_COVERAGE env.var. is set) are run after the `test-nyc` make target, hence we added `report-nyc` as a final target to the `make all` a.k.a. `make` build process. **WARNING: as now all jison compiles and examples' runs are code coveraged as well, this may take a while: ~30mins this evening on this box here ;-) ) -- anyway, the key is that we now are able to collect a lot of code coverage data and have the `report-nyc` task report on those statistics.
  • Loading branch information
GerHobbelt committed Dec 23, 2017
1 parent 2108456 commit ecaec7c
Showing 1 changed file with 8 additions and 2 deletions.
10 changes: 8 additions & 2 deletions Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@ endif



all: clean-nyc build test test-nyc examples-test
all: clean-nyc build test test-nyc examples-test report-nyc

everything: \
clean \
Expand Down Expand Up @@ -97,6 +97,12 @@ test-nyc:
cd packages/json2jison && make test-nyc
cd packages/jison2json && make test-nyc
$(NYC) --reporter=lcov --reporter=text --exclude 'examples/issue-lex*.js' -- $(MOCHA) --timeout 18000 --check-leaks --globals assert --recursive tests/
-rm -rf ./coverage/
# report PRELIMINARY collective coverage results:
$(NYC) report --reporter=html

report-nyc:
-rm -rf ./coverage/
# report collective coverage results:
$(NYC) report --reporter=html

Expand Down Expand Up @@ -600,5 +606,5 @@ superclean: clean clean-site
compile-site clean-site \
publish subpackages-publish \
npm-update subpackages-npm-update \
test-nyc clean-nyc
test-nyc clean-nyc report-nyc

0 comments on commit ecaec7c

Please sign in to comment.