dennisdoomen / reflectify / 31405432405

98%

push

github

Expose nullable reference type metadata across all target frameworks (#174)

* Add Nullability metadata support for PropertyInfo, FieldInfo and ParameterInfo

Closes #162

Add a public/internal Nullability enum and GetNullability/IsNullableReference
extension methods for PropertyInfo, FieldInfo and ParameterInfo. On net6.0+
this delegates to System.Reflection.NullabilityInfoContext; on net47,
netstandard2.0 and netstandard2.1 it reads the compiler-emitted
NullableAttribute/NullableContextAttribute metadata directly via a shared
NullabilityMetadataReader. Value types (including Nullable<T>) are
short-circuited using the existing NullableOrActualType helper.

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

* Exclude NullabilityMetadataReader from net6.0-or-greater builds

The fallback attribute-reading logic is only used on frameworks without
System.Reflection.NullabilityInfoContext. It was previously compiled
unconditionally into every TFM, which meant it showed up as entirely
uncovered/unreachable code on net6.0+ builds where the fast path is used
instead, dragging down overall coverage. Wrapping the whole type in
#if !NET6_0_OR_GREATER excludes it from those builds so coverage tooling
no longer counts it there.

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>

298 of 312 branches covered (95.51%)

Branch coverage included in aggregate %.

61 of 61 new or added lines in 3 files covered. (100.0%)

557 of 562 relevant lines covered (99.11%)

3914612.91 hits per line