Bug 1639158: Fix Python coverage testing

mozilla · May 19, 2020 · ddfe3fb · ddfe3fb
1 parent ac87990
commit ddfe3fb
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/Makefile b/Makefile
@@ -178,7 +178,7 @@ rust-coverage: ## Generate code coverage information for Rust code
 .PHONY: rust-coverage
 
 python-coverage: build-python ## Generate a code coverage report for Python
-	$(GLEAN_PYENV)/bin/python3 -m coverage run --parallel-mode -m pytest
+	GLEAN_COVERAGE=1 $(GLEAN_PYENV)/bin/python3 -m coverage run --parallel-mode -m pytest
 	$(GLEAN_PYENV)/bin/python3 -m coverage combine
 	$(GLEAN_PYENV)/bin/python3 -m coverage html
 .PHONY: python-coverage
diff --git a/glean-core/python/glean/_subprocess/_process_dispatcher_helper.py b/glean-core/python/glean/_subprocess/_process_dispatcher_helper.py
@@ -18,7 +18,7 @@
     import sys
 
     # Run coverage in the subprocess if necessary
-    if "COVERAGE_PROCESS_START" in os.environ:
+    if "GLEAN_COVERAGE" in os.environ and "COVERAGE_PROCESS_START" in os.environ:
         import coverage  # type: ignore
 
         config_path = os.environ.get("COVERAGE_PROCESS_START")