Tests: Wave 7 — reachable kernel + module remainders (coverage 70.0% → 74.4%)#69
Merged
Conversation
… 74.4%) Four parallel agents mopping up reachable-but-uncovered branches (committed incrementally after a session-limit restart). 177 tests, combined suite green. Moved: Tiger_Backup 47→82, Controller_Plugin_Authorization 43→88, Cms_Renderer 69→100, Location 79→95, Code_Runtime 82→90, Vendor 80→87, Module_Installer 68→74, Acl 89→93, License_Checker 82→91; Media cluster 40→75 (Storage 100, Filesystem 97, Azure/Gcs/ClamAv/ Rekognition unlocked from 0 — exercised but not CoversClass-attributed); Tiger_Application 60→74; Vendor_Environment 0→84, Update_Composer 0→57, Google_Analytics 29→47, Compat→100; Seo/Code/Backup module services →95-97. Network classes covered WITHOUT network: dead-localhost curl (instant ECONNREFUSED → fail-soft branches), primed caches, probe subclasses exposing build/parse helpers, _run driven with echo/exit. No behavioral bugs. CI floor MIN_COVERAGE 66 → 72. The path from here to 80% (~800 lines) is the in-process app-boot harness + the last scattered branches; the rest is uploads/composer/live-network (functional-test territory), documented in COVERAGE-PLAN §9. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
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
1 participant
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.
Four parallel agents mopped up the reachable-but-uncovered branches across the kernel and modules. 177 tests, combined suite green. Overall line coverage 70.0% → 74.4%.
Moved (line %)
Tiger_Backup47→82 ·Controller_Plugin_Authorization43→88 ·Cms_Renderer69→100 ·Location79→95 ·Code_Runtime82→90 ·Vendor80→87 ·Module_Installer68→74 ·Acl89→93 ·License_Checker82→91 · Media cluster 40→75 (Storage 100, Filesystem 97, and the Azure/Gcs/ClamAv/Rekognition adapters unlocked from 0% — they were exercised but not#[CoversClass]-attributed) ·Tiger_Application60→74 ·Vendor_Environment0→84 ·Update_Composer0→57 ·Google_Analytics29→47 ·Compat→100 · Seo/Code/Backup module services →95-97.Network covered without network
Dead-localhost curl (
http://127.0.0.1:1→ instant ECONNREFUSED → the fail-soft branches), primed report caches, probe subclasses exposing the build/parse helpers, and_rundriven withecho/exit 3for the composer proc_open plumbing — all with no real network and nocomposer update/vendorswap. No behavioral bugs.Minor finding (characterized, not fixed)
Tiger_Media_Storage_Azure::put()/write()surface a raw\Error(not the friendlyRuntimeExceptionS3/GCS give) when the Azure SDK is absent — it constructs the SDK class before its guarded try/catch. A small consistency fix is available.The path from 74.4% to 80%
The remaining ~3,670 uncovered lines are now dominated by genuinely-hard I/O: kernel boot orchestration (
Tiger_Application_Bootstrap::_init*, ~200-400 lines — the one big reachable chunk, needs an in-process app-boot harness),is_uploaded_file()uploads, composer/vendor-swap (unsafe to run), and live network (GA/authority). CI floorMIN_COVERAGE66 → 72. Wave 8 builds the boot harness; below that is functional/live-test territory, honestly out of unit/integration scope.🤖 Generated with Claude Code