valkyrjaio / sindri-java / 30191283928 / 1

100%

26.x: 100%

push

github

[Sindri] Restore 100% line coverage with follow-up tests (#51)

# Description

The gRPC codegen work (merged and released as v26.3.0) added new
syntactic code paths — the `MiddlewareClassifier` ancestry walk, the
gRPC route/service readers, and the middleware source resolver in
`GenerateDataFromAst` — whose edge cases were not fully exercised,
dropping Sindri's line coverage from 100% to ~99%. This follow-up adds
the missing tests to bring it back to **100% line coverage** (methods
and classes 100% as well). It is a test-only change: no production
source is modified.

## 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

- **`SindriComponentProviderTest`** — assert `getGrpcProviders(app)` in
`exposesAllProviderLists`, covering the new provider-list accessor.
- **`ComponentProviderReaderTest` / `RouteProviderReaderTest` /
`ListenerProviderReaderTest` / `ServiceProviderReaderTest` /
`CliRouteAttributeReaderTest`** — add a
`readFile_noTypeDeclaration_throws` case each, covering the
`orElseThrow("No type found")` guard.
- **5 new no-type fixtures** under
`Fixtures/{Component,Http,Event,Container}/Provider/` and
`Fixtures/Cli/Controller/` — package-only source files that trigger the
"no type found" path.
- **`MiddlewareClassifierTest`** — 8 new cases covering the guard
branches: diamond ancestor visited once, resolved source with no
class/interface declaration, marker `@Middleware`, a non-`name`
annotation pair, default-package classification (class and method
forms), static-wildcard imports, and a non-annotation entry inside a
`@Middlewares` container.
- **`GenerateDataFromConfigCommandTest`** — add a
`@M... (continued)

524 of 525 branches covered (99.81%)

Branch coverage included in aggregate %.

1513 of 1513 relevant lines covered (100.0%)

4.38 hits per line