fluentassertions / fluentassertions / 9099434365

97%

main: 97%

push

github

Allow event monitoring to ignore failing event accessors (#2629)

* Removed event monitor builder
Moved methods config options to `EventMonitorOptions`

* Removed unnecessary generic type from EventMonitorOptions

* fixed build errors due to removing generic from options

* Update api checks

* Update Src/FluentAssertions/Events/EventMonitorOptions.cs

Fixed wrong Timestamp

* Include reviewer feedback

* Added more reviewer suggestions.

* Update Src/FluentAssertions/Events/EventMonitor.cs

Co-authored-by: Jonas Nyrup <jnyrup@users.noreply.github.com>

* Update Src/FluentAssertions/Events/EventMonitorOptions.cs

Co-authored-by: Jonas Nyrup <jnyrup@users.noreply.github.com>

* - removed the exception when event recorder could not be removed from the dictionary
- Added option to still receive events even when add accessor threw an exception

* cleanups

* Fixed Qodana complaints

* Removed boolean parameter and added method for recording events with broken accessor

* Updated api tests for new RecordEventsWithBrokenAccessor method.

Updated descriptions to match current the actual behavior

* Update documentation

Co-authored-by: IT-VBFK <49762557+IT-VBFK@users.noreply.github.com>

* Update release notes

* Add suggestions to 3 files

---------

Co-authored-by: Andreas Pazureck <andreas.pazureck.ext@zeiss.com>
Co-authored-by: Andreas Pazureck <andreas@pazureck.de>
Co-authored-by: Jonas Nyrup <jnyrup@users.noreply.github.com>
Co-authored-by: Jonas Nyrup <jnyrup@gmail.com>
Co-authored-by: IT-VBFK <49762557+IT-VBFK@users.noreply.github.com>

3697 of 3906 branches covered (94.65%)

Branch coverage included in aggregate %.

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

12054 of 12238 relevant lines covered (98.5%)

39980.5 hits per line