Skip to content

Commit 58aa0a3

Browse files
author
Hugo Osvaldo Barrera
committed
Simplify coverage submission
We submit separate coverage reports for each group of tests, but codecov doesn't give us proper tooling to analyse each one separately. When there's a percentage drop in one, there's no way to pinpoint where -- at least not without running locally and inspecting results separately. Treat all coverage as one. Analysis of coverage for each group of tests can be done offline and manually, but there's little value in keeping it in codecov. Should also help us transition a more simplified CI design (e.g.: running all tests together rather than in groups).
1 parent 21db254 commit 58aa0a3

File tree

2 files changed

+3
-12
lines changed

2 files changed

+3
-12
lines changed

.codecov.yml

Lines changed: 0 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -2,10 +2,3 @@ comment: false
22
coverage:
33
status:
44
patch: false
5-
project:
6-
unit:
7-
flags: unit
8-
system:
9-
flags: system
10-
storage:
11-
flags: storage

Makefile

Lines changed: 3 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -43,10 +43,8 @@ all:
4343

4444
ci-test:
4545
curl -s https://codecov.io/bash > $(CODECOV_PATH)
46-
$(PYTEST) tests/unit/
47-
bash $(CODECOV_PATH) -c -F unit
48-
$(PYTEST) tests/system/
49-
bash $(CODECOV_PATH) -c -F system
46+
$(PYTEST) --cov-append tests/unit/ tests/system/
47+
bash $(CODECOV_PATH) -c
5048
[ "$(ETESYNC_TESTS)" = "false" ] || make test-storage
5149

5250
ci-test-storage:
@@ -55,7 +53,7 @@ ci-test-storage:
5553
for server in $(DAV_SERVER); do \
5654
DAV_SERVER=$$server $(PYTEST) --cov-append tests/storage; \
5755
done
58-
bash $(CODECOV_PATH) -c -F storage
56+
bash $(CODECOV_PATH) -c
5957

6058
test:
6159
$(PYTEST)

0 commit comments

Comments
 (0)