dennisdoomen / reflectify / 34030589688

98%

push

github

Add event and constructor lookup to complete member categories (#184)

* Add event and constructor lookup to complete member categories

Adds GetEvents/FindEvent (via the same Reflector hierarchy walk, visibility
rules and de-duplication treatment as properties, including the
EventInfo inherit bug workaround) and GetConstructors/FindConstructor/
HasDefaultConstructor to TypeMemberExtensions.

Closes #154

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

* Add missing specs for events, constructors and EventInfo extensions

Adds direct specs for the previously untested EventInfoExtensions, plus
hierarchy-walk, interface-reflection and cross-hierarchy precedence specs
for GetEvents, and edge-case specs for constructor lookup.

Also changes FindConstructor so that omitting the parameter types means the
parameterless constructor, matching Type.GetConstructor(Type.EmptyTypes).
Previously the empty array meant 'match any', which made FindConstructor
throw from SingleOrDefault on any type with more than one constructor.

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

* Honour MemberKind.Protected and MemberKind.Private for events

The preceding layer added Protected and Private visibility flags and wired
them into property and field filtering. Events were added independently, so
GetEvents(MemberKind.Protected) silently returned nothing while
GetProperties(MemberKind.Protected) worked. Adds IsProtected and IsPrivate to
EventInfoExtensions and applies both flags in the event visibility filter.

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

* Remove unreachable ordered member fallback branches

The ordered member collections keep their name maps and backing lists in sync.
The fallback add paths after a successful normal-over-explicit replacement
were unreachable and lowered coverage despite not representing real behavior.
Use FindIndex for the replacement instead and remove the dead... (continued)

462 of 492 branches covered (93.9%)

Branch coverage included in aggregate %.

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

875 of 878 relevant lines covered (99.66%)

2505727.1 hits per line