From 5986fde6107f49b7a2b7fb74252616f78050065d Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Giedrius=20Statkevi=C4=8Dius?= Date: Tue, 12 May 2020 17:45:04 +0300 Subject: [PATCH] Makefile: remove --coverage from test run MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Found out that there is some weird interaction between `jest --coverage` and `babel-plugin-istanbul`. Maybe related to: https://github.com/facebook/jest/issues/6827. From my testing, removing `--coverage` makes this work again. Probably worth investigating in the future why that happens. Also, this is really not needed during CI because we do not use the coverage data anywhere anyway. Signed-off-by: Giedrius Statkevičius --- Makefile | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Makefile b/Makefile index fbbe4df4c5..46a3f6ed53 100644 --- a/Makefile +++ b/Makefile @@ -166,7 +166,7 @@ react-app-lint-fix: .PHONY: react-app-test react-app-test: | $(REACT_APP_NODE_MODULES_PATH) react-app-lint @echo ">> running React app tests" - cd $(REACT_APP_PATH) && export CI=true && yarn test --no-watch --coverage + cd $(REACT_APP_PATH) && export CI=true && yarn test --no-watch .PHONY: react-app-start react-app-start: $(REACT_APP_NODE_MODULES_PATH)