Skip to content

Commit 005b1f9

Browse files
committed
Fix runtime coverage save (use /tmp for write permissions)
1 parent e1747c1 commit 005b1f9

File tree

2 files changed

+2
-2
lines changed

2 files changed

+2
-2
lines changed

api/web/start.py

+1-1
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@
88
# http://coverage.readthedocs.io/en/coverage-4.2/faq.html
99
if os.environ.get("SCITRAN_RUNTIME_COVERAGE") == "true": # pragma: no cover - oh, the irony
1010
import coverage
11-
cov = coverage.coverage(source=["api"], data_suffix="integration-tests")
11+
cov = coverage.coverage(source=["api"], data_file="/tmp/.coverage.integration-tests")
1212

1313
class CoverageSaveHandler(webapp2.RequestHandler):
1414
def save_coverage(self):

tests/bin/docker-tests.sh

+1-1
Original file line numberDiff line numberDiff line change
@@ -106,7 +106,7 @@ clean_up() {
106106

107107
# Save integration test coverage
108108
docker exec core-test-service python -c 'import requests; requests.post("http://localhost/api/save-coverage")'
109-
docker cp core-test-service:/src/core/.coverage.integration-tests ./ 2>/dev/null
109+
docker cp core-test-service:/tmp/.coverage.integration-tests ./ 2>/dev/null
110110

111111
# Combine unit/integ coverage and report/grenerate html
112112
docker run --rm \

0 commit comments

Comments
 (0)