Skip to content

Commit 3cd5ac5

Browse files
authored
Merge pull request #48 from clue-labs/coverage
Update CI setup to ensure 100% code coverage
2 parents 8f64659 + 074745f commit 3cd5ac5

File tree

1 file changed

+8
-2
lines changed

1 file changed

+8
-2
lines changed

.github/workflows/ci.yml

Lines changed: 8 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -29,10 +29,16 @@ jobs:
2929
php-version: ${{ matrix.php }}
3030
coverage: xdebug
3131
- run: composer install
32-
- run: vendor/bin/phpunit --coverage-text
32+
- run: vendor/bin/phpunit --coverage-text --coverage-clover=clover.xml
3333
if: ${{ matrix.php >= 7.3 }}
34-
- run: vendor/bin/phpunit --coverage-text -c phpunit.xml.legacy
34+
- run: vendor/bin/phpunit --coverage-text --coverage-clover=clover.xml -c phpunit.xml.legacy
3535
if: ${{ matrix.php < 7.3 }}
36+
- name: Check 100% code coverage
37+
shell: php {0}
38+
run: |
39+
<?php
40+
$metrics = simplexml_load_file('clover.xml')->project->metrics;
41+
exit((int) $metrics['statements'] === (int) $metrics['coveredstatements'] ? 0 : 1);
3642
3743
PHPUnit-hhvm:
3844
name: PHPUnit (HHVM)

0 commit comments

Comments
 (0)