dennisdoomen / pathy / 31405283827

96%

push

github

Add TypeConverter and opt-in JsonConverter support for ChainablePath (#160)

* Test

* Add TypeConverter and opt-in JsonConverter support for ChainablePath (#145)

- Add ChainablePathTypeConverter (unconditional, no extra dependency) so
  ChainablePath can be bound from configuration, command-line args, or
  MSBuild properties via System.ComponentModel.TypeConverter.
- Add ChainablePathJsonConverter gated behind a new opt-in
  PATHY_SYSTEM_TEXT_JSON compilation symbol, since System.Text.Json is not
  guaranteed to be available on every targeted framework (notably net47),
  and this is a source-only contentFiles package that must not force a new
  dependency onto every consumer.
- Apply [TypeConverter] unconditionally and [JsonConverter] only under
  PATHY_SYSTEM_TEXT_JSON on the ChainablePath struct, following the
  existing PATHY_PUBLIC conditional visibility pattern.
- Define PATHY_SYSTEM_TEXT_JSON for Pathy.csproj itself (and add a
  System.Text.Json package reference for non-net8.0 TFMs) so this repo's
  own build/tests actually exercise the JSON converter, without adding a
  System.Text.Json dependency to the packed contentFiles.
- Add unit tests covering TypeConverter CanConvertFrom/ConvertFrom/ConvertTo
  round-tripping and JSON serialize/deserialize round-tripping of a
  ChainablePath property.
- Update approved public API snapshots (PublicApiGenerator) for all four
  TFMs to include the new converter types and attributes.
- Document TypeConverter (works out of the box) and JsonConverter (opt-in
  via PATHY_SYSTEM_TEXT_JSON) support in README.md.

Fixes #145

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>

---------

Co-authored-by: Dennis Doomen <dennis.doomen@greenchoice.nl>
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>

115 of 126 branches covered (91.27%)

Branch coverage included in aggregate %.

23 of 25 new or added lines in 2 files covered. (92.0%)

284 of 294 relevant lines covered (96.6%)

141.92 hits per line