valkyrjaio / valkyrja-ts / 30237514553

100%

push

github

[Cli] Add routing argument/option combination tests (#89)

# Description

Adds argument/option binding combination tests for CLI routing,
mirroring the PHP
reference (valkyrjaio/valkyrja-php#924) and completing the framework
routing test effort
across PHP, Java, and TypeScript.

The existing `Router` tests covered default/array argument binding, an
empty default
argument, and option binding by name and short name. This adds the
remaining permutations a
developer relies on so they can trust that any command they define is
bound and validated
correctly. Test-only; no source 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/Cli/Routing/Dispatcher/Router.test.ts`** — added
binding tests through `dispatch`: multiple inputs collected into an
`ARRAY` option; an option parameter that matches zero inputs; a `NONE`
flag that receives a value throws; a missing `REQUIRED` option throws; a
`DEFAULT` option that receives more than one value throws; and a missing
`REQUIRED` argument throws.

1585 of 1585 branches covered (100.0%)

Branch coverage included in aggregate %.

3466 of 3466 relevant lines covered (100.0%)

23.24 hits per line