diff options
author | Massimiliano Culpo <massimiliano.culpo@gmail.com> | 2018-01-10 17:14:17 +0100 |
---|---|---|
committer | Adam J. Stewart <ajstewart426@gmail.com> | 2018-01-10 10:14:17 -0600 |
commit | 596d463714d46929f81db62b171c5a772679bb12 (patch) | |
tree | c6b15a9fcfc07af79db3cd686c391094146d6215 | |
parent | d17a10c6ac07f372bb27ef61b25edb0407c08b83 (diff) | |
download | spack-596d463714d46929f81db62b171c5a772679bb12.tar.gz spack-596d463714d46929f81db62b171c5a772679bb12.tar.bz2 spack-596d463714d46929f81db62b171c5a772679bb12.tar.xz spack-596d463714d46929f81db62b171c5a772679bb12.zip |
Travis: use --concurrency=multiprocessing only on build tests (#6872)
On a local workstation, it seems that tracking multiple processes during
coverage may result in malformed coverage reports for unit tests and not
for build tests.
Given that multiple processes make a difference in coverage mainly for
build tests, try to disable the tracking for unit tests to see if we get
more stable coverage results.
-rw-r--r-- | .coveragerc | 1 | ||||
-rwxr-xr-x | share/spack/qa/setup.sh | 6 |
2 files changed, 5 insertions, 2 deletions
diff --git a/.coveragerc b/.coveragerc index 0201a4b502..224c472fa2 100644 --- a/.coveragerc +++ b/.coveragerc @@ -2,7 +2,6 @@ # .coveragerc to control coverage.py [run] parallel = True -concurrency = multiprocessing branch = True source = lib omit = diff --git a/share/spack/qa/setup.sh b/share/spack/qa/setup.sh index 98c79a0457..4b3f98e330 100755 --- a/share/spack/qa/setup.sh +++ b/share/spack/qa/setup.sh @@ -11,10 +11,14 @@ SPACK_ROOT="$QA_DIR/../../.." . "$SPACK_ROOT/share/spack/setup-env.sh" # Set up some variables for running coverage tests. -if [[ "$COVERAGE" == true ]]; then +if [[ "$COVERAGE" == "true" && "$TEST_SUITE" == "unit" ]]; then coverage=coverage coverage_run="coverage run" coverage_combine="coverage combine" +elif [[ "$COVERAGE" == "true" && "$TEST_SUITE" == "build" ]]; then + coverage=coverage + coverage_run="coverage run --concurrency=multiprocessing" + coverage_combine="coverage combine" else coverage="" coverage_run="" |