valkyrjaio / valkyrja-php / 30325936160
100%

DEFAULT BRANCH: 26.x
Ran
Jobs 1
Files 1138
Run time 2min
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst
coverage: 100.0%. Remained the same
30325936160

push

github

web-flow 
[CI] Close reachable branch coverage gaps found by path coverage (#936)

# Description

Closes the reachable branch-coverage gaps surfaced by the path-coverage
harness from #933, and
records what turned out not to be reachable at all.

Ten branches across seven files now execute that previously did not.
Each was verified per-file
before and after by running that file's own shard under
`--path-coverage`:

| File | Before | After |
|------|--------|-------|
| `Container/Manager/NativeChildContainer.php` | 29/32 | **32/32** |
| `Type/Object/Support/Cls.php` | 8/10 | **10/10** |
| `Container/Manager/Container.php` | 51/52 | **52/52** |
| `Cli/Interaction/Output/Output.php` | 37/38 | **38/38** |
| `Type/Ulid/Factory/UlidFactory.php` | 45/46 | **46/46** |
| `View/Renderer/PhpRenderer.php` | 22/23 | **23/23** |
| `Dispatch/Factory/DispatchFactory.php` | 13/16 | 14/16 |

These are real tests, not coverage padding. Both `Cls` validators were
only ever exercised on their
throwing path, so nothing asserted they accept valid input; quiet
`Output` had no test proving a
failing exit code still writes its message; and
`NativeChildContainer::has()` was never called for a
service registered in neither the child nor the parent, so most of its
lookup chain never ran.

## The reported gap was substantially larger than the real one

The merged report claimed 41 missing branches. Checking each file
against its own shard — a single
process, so no merging — showed that **19 of those are already fully
covered**:

| File | Merged claimed missing | Actual, in its own shard |
|------|------------------------|--------------------------|
| `Http/Routing/Processor/Processor.php` | 9 | **43/43 — 100%** |
| `Api/Manager/Api.php` | 4 | **24/24 — 100%** |
| `Http/Message/Request/Request.php` | 4 | **32/32 — 100%** |
| `Cli/Server/Provider/CliServerServiceProvider.php` | 2 | **22/22 —
100%** |

The merged report counts 52 executable branches for `Processor.php`
where every individual sh... (continued)

12591 of 12591 relevant lines covered (100.0%)

23.53 hits per line

Jobs
ID Job ID Ran Files Coverage
1 30325936160.1 1138
100.0
 GitHub Action Run
Source Files on build 30325936160