-
Notifications
You must be signed in to change notification settings - Fork 25
Description
In #234 we added Behat code coverage support which works great except for a few minor issues.
See also this related Slack discussion.
Incorrect autoload require statement here:
wp-cli-tests/utils/generate-coverage.php
Lines 16 to 18 in 16ee1c9
if ( ! class_exists( 'SebastianBergmann\CodeCoverage\Filter' ) ) { | |
require "{$root_folder}/vendor/autoload.php"; | |
} |
The path is wrong.
Class mismatch
Sometimes SebastianBergmann\CodeCoverage\ProcessedCodeCoverageData
is loaded but SebastianBergmann\CodeCoverage\Data\ProcessedCodeCoverageData
is expected (or the other way around?). That means somehow some older versions of that class are loaded. Needs investigating.
Very slow tests
Behat tests with coverage reporting take much longer to process. In the case of entity-command the job is even cancelled after 6 hours because it's still not finished.
We should look into improving this. Maybe pcov is faster than xdebug. Or we split up the job into multiple ones to run in parallel (see #189)