From 128ecfa21613837f2492dd5984614270be6d9aa8 Mon Sep 17 00:00:00 2001
From: Martin Blanchard <martin.blanchard@codethink.co.uk>
Date: Tue, 19 Mar 2019 16:21:39 +0000
Subject: Generate and publish HTML coverage report

https://gitlab.com/BuildStream/buildstream/issues/629
---
 .gitlab-ci.yml | 7 ++++++-
 tox.ini        | 5 +++--
 2 files changed, 9 insertions(+), 3 deletions(-)

diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml
index 99bde6066..2d3a24700 100644
--- a/.gitlab-ci.yml
+++ b/.gitlab-ci.yml
@@ -8,6 +8,7 @@ cache:
 stages:
   - test
   - post
+  - publish
 
 variables:
   PYTEST_ADDOPTS: "--color=yes"
@@ -344,17 +345,21 @@ coverage:
 # Deploy, only for merges which land on master branch.
 #
 pages:
-  stage: post
+  stage: publish
   dependencies:
+  - coverage
   - docs
   variables:
     ACME_DIR: public/.well-known/acme-challenge
+    COVERAGE_DIR: public/coverage
   script:
   - mkdir -p ${ACME_DIR}
     # Required to finish the creation of the Let's Encrypt certificate,
     # which allows using https://docs.buildstream.build/ for accessing
     # the documentation.
   - echo ${ACME_CHALLENGE} > ${ACME_DIR}/$(echo ${ACME_CHALLENGE} | cut -c1-43)
+  - mkdir -p ${COVERAGE_DIR}
+  - cp -a ./coverage-report/ ${COVERAGE_DIR}
   artifacts:
     paths:
     - public/
diff --git a/tox.ini b/tox.ini
index 6cfe51818..29ea10ffb 100644
--- a/tox.ini
+++ b/tox.ini
@@ -61,8 +61,9 @@ whitelist_externals =
 #
 [testenv:coverage]
 commands =
-    - coverage combine --rcfile={toxinidir}/.coveragerc {toxinidir}/.coverage-reports/
-    coverage report --rcfile={toxinidir}/.coveragerc -m
+    coverage combine --rcfile={toxinidir}/.coveragerc {toxinidir}/.coverage-reports/
+    coverage html --rcfile={toxinidir}/.coveragerc --directory={toxinidir}/.coverage-reports/
+    coverage report --rcfile={toxinidir}/.coveragerc --show-missing
 deps =
     -rrequirements/requirements.txt
     -rrequirements/dev-requirements.txt
-- 
cgit v1.2.1