Permalink
Browse files

build,test: refacor `coverage-test` to implicit dependancy

  • Loading branch information...
refack committed Jan 10, 2019
1 parent 29d3d1e commit 6f9fc9057431e64a11864fd85b4caefc4cd139d4
Showing with 6 additions and 2 deletions.
  1. +6 −2 Makefile
@@ -197,7 +197,9 @@ coverage-clean:
# For C++ coverage reporting, this needs to be run in conjunction with configure
# --coverage. html coverage reports will be created under coverage/
# Related CI job: node-test-commit-linux-coverage
coverage: coverage-test ## Run the tests and generate a coverage report.
coverage: # Run the tests and generate a coverage report.
$(MAKE) coverage-build
$(MAKE) coverage-test

.PHONY: coverage-build
coverage-build: all
@@ -218,7 +220,9 @@ coverage-build: all
$(MAKE)

.PHONY: coverage-test
coverage-test: coverage-build
# Implicitly dependant on the node binary built with '--coverage'.
coverage-test:
$(NODE) -p "process.exit(!process.config.variables.coverage)"
$(RM) -r out/$(BUILDTYPE)/.coverage
$(RM) -r .cov_tmp
$(RM) out/$(BUILDTYPE)/obj.target/node/gen/*.gcda

0 comments on commit 6f9fc90

Please sign in to comment.