Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Enable code coverage #1194

Merged
merged 7 commits into from
Apr 28, 2024
Merged

Enable code coverage #1194

Changes from all commits
Commits
Show all changes
7 commits
Select commit Hold shift + click to select a range
b9ad9c1
Use Devel::Cover in CI
oalders Apr 27, 2024
0429f55
Upload coverage without using orb
oalders Apr 27, 2024
35a9cbd
Ensure some CircleCI env vars are available inside container before t…
oalders Apr 28, 2024
f26395a
Switch CircleCI resource class from medium to large
oalders Apr 28, 2024
c82120d
Upload coverage report from outside of the container
oalders Apr 28, 2024
0e2cb0e
Ignore more files when running coverage
oalders Apr 28, 2024
74eb5a4
Don't build coverage report twice
oalders Apr 28, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
32 changes: 29 additions & 3 deletions .circleci/config.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,4 +1,6 @@
# Use the latest 2.1 version of CircleCI pipeline process engine. See: https://circleci.com/docs/2.0/configuration-reference
---
# Use the latest 2.1 version of CircleCI pipeline process engine. See:
# https://circleci.com/docs/2.0/configuration-reference
version: 2.1
# Orchestrate or schedule a set of jobs
workflows:
Expand All @@ -8,7 +10,7 @@ workflows:
jobs:
build-and-test:
machine: true
resource_class: medium
resource_class: large
steps:
- run:
name: docker-compose version
Expand Down Expand Up @@ -44,10 +46,34 @@ jobs:
pushd metacpan-docker
./src/metacpan-api/wait-for-es.sh http://localhost:9200 elasticsearch_test --
name: wait for ES
# We are running coverage inside the container, but creating the report
# from outside of the container, so Devel::Cover is installed twice. We
# are relying on environment variables from the host to be available when
# we publish the report, so we publish from the host rather than trying
# to propagate env variables to the container. There is a CircleCI orb
# that can publish the report for us, but it is hard to debug. I had to
# set up the following config in order to debug the orb, so we may as
# well keep it around for now.
- run:
command: |
pushd metacpan-docker
docker-compose exec -T api_test prove -lr --jobs 2 t
docker-compose exec -T api_test cpm install -g Devel::Cover
name: install Devel::Cover
- run:
command: |
pushd metacpan-docker
docker-compose exec -T api_test env HARNESS_PERL_SWITCHES="-MDevel::Cover=+ignore,^t/|^test-data/|^etc/" prove -lr --jobs 2 t
name: run tests with coverage
# The report ends up being owned by root. We need to chown it so that
# the cover command can make changes to it.
- run:
command: |
pushd metacpan-docker/src/metacpan-api
sudo apt install cpanminus
sudo cpanm --notest Devel::Cover Devel::Cover::Report::Codecov Devel::Cover::Report::Codecovbash
sudo chown -R circleci.circleci cover_db
cover -report codecov
name: upload coverage report
- run:
command: |
pushd metacpan-docker
Expand Down