build,test: fail coverage target if tests fail

PR-URL: nodejs#25432 Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: Michael Dawson <michael_dawson@ca.ibm.com>
Trott · Jan 15, 2019 · f216d5b · f216d5b
1 parent 2f1ae9e
commit f216d5b
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/Makefile b/Makefile
@@ -227,7 +227,7 @@ coverage-test: coverage-build
 	$(RM) out/$(BUILDTYPE)/obj.target/node_lib/gen/*.gcda
 	$(RM) out/$(BUILDTYPE)/obj.target/node_lib/src/*.gcda
 	$(RM) out/$(BUILDTYPE)/obj.target/node_lib/src/tracing/*.gcda
-	-$(MAKE) $(COVTESTS)
+	$(MAKE) $(COVTESTS)
 	mv lib lib__
 	mv lib_ lib
 	mkdir -p coverage .cov_tmp