Improve Fleet-Ops server test coverage#277
Draft
roncodes wants to merge 45 commits into
Draft
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Continues the Fleet-Ops backend coverage push toward a 100% server-side coverage gate for
./server.This branch wires Composer coverage reporting, adds a 100% fail-under coverage command, and expands Pest coverage across backend contracts that can be exercised inside the package CI harness. Production changes remain limited to issues uncovered while making the backend tests executable in CI.
Latest Coverage
The Fleet-Ops server workflow now installs Xdebug and is configured to upload
coverage/clover.xml, but the current remote PR head is still failing before the coverage-generation step. The latest failing server run stops inserver/tests/FuelProviderResourceContractsTest.php, so no trustworthy branch coverage percentage has been produced yet. I will update this section with the actual line, method, and class coverage percentages as soon as the backend build reaches the coverage baseline step.Coverage Reporting
composer coverage:baselineto generate Clover coverage and print a summary.composer coverage:checkto generate Clover coverage and fail below 100% line coverage.scripts/coverage-summary.phpsupport for--fail-under=100.coverage/clover.xmlas an artifact.Coverage Added
Production Fixes
TelematicRateLimitExceededExceptioninheritance so it extends the existing telematics provider exception contract.Illuminate\Foundation\Bus\Dispatchabletrait instead of the missingIlluminate\Foundation\Events\Dispatchablenamespace in this package CI matrix.Current CI Status
FuelProviderResourceContractsTestrequest-context/resource fixture mismatch before the coverage step.Validation
php -lon every added or touched backend PHP test/source file.git diff --checkandgit diff --cached --checkfor each committed slice.gh.Notes
This remains a progressive draft PR. The next local batch is being held back until it represents a more meaningful coverage increase or a necessary CI-unblocking bundle, rather than pushing every small test slice individually.