valkyrjaio / sindri-php / 27853338125
100%

DEFAULT BRANCH: 26.x
Ran
Jobs 1
Files 79
Run time 1min
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
27853338125

push

github

web-flow 
[Tests] Adding missing tests (#115)

# Description

Brings Sindri to 100% test coverage with the stronger guarantee that
every source file containing logic is covered 100% by its own dedicated
test in isolation — not merely as a side effect of other tests
exercising it.

Coverage was verified per file by running each test on its own with
coverage scoped to its target class (`phpunit <one-test-file>
--coverage-text`), so a class only counts as covered when its own test
fully exercises it. An audit script swept every unit test against its
target source to find files that passed only via cross-test coverage,
and those gaps were closed by extending the relevant dedicated test.

This is a test-only change — no behavior changes.

---

## Types of Changes

- [x] Improvement _(non-breaking change which improves code)_
- [ ] Bug fix _(non-breaking change which fixes an issue)_
- [ ] New feature _(non-breaking change which adds functionality)_
- [ ] Deprecation _(breaking change which removes functionality)_
- [ ] Breaking change _(fix or feature that would cause existing
functionality to change)_
- [ ] Documentation improvement

---

## Changes

- **`tests/Tests/Unit/Ast/Abstract/RouteAttributeReaderTest.php`** — new
dedicated test for the abstract `RouteAttributeReader`, covering
`updateHandler()` (the fallback, attribute-array, and non-handler-arg
branches) via a test-double subclass, instead of relying on the concrete
CLI/HTTP reader tests
- **`tests/Tests/Unit/Ast/Abstract/AstReaderTest.php`** — extended the
exposed-proxy subclass and added tests for the ~21 previously
subclass-only-covered helpers (`unwrapNamespace`, `buildUseMap`,
`findClass`, `indexMethods`, `findAttributesOnNode`, `getAttrArg`,
`parseClassFile`, the `extract*Arg` / `extractStringListFromArrayExpr`
helpers, and all `build*` AST builders), plus the `self::class` /
short-name and enum-case/handler-array branches
- **`tests/Tests/Unit/Ast/CliRouteParameterReaderTest.php`** — added
tests fo... (continued)

1544 of 1544 relevant lines covered (100.0%)

17.11 hits per line

Jobs
ID Job ID Ran Files Coverage
1 27853338125.1 79
100.0
 GitHub Action Run
Source Files on build 27853338125