Skip to content

Conversation

@dennisdoomen
Copy link
Member

@dennisdoomen dennisdoomen commented Aug 20, 2017

Replaces the old thread-unsafe MonitorEvents API with a new Monitor extension method that will return a thread-safe monitoring scope that exposes methods like Should().Raise() and metadata such as OccurredEvents and MonitoredEvents.

  • Have EventMonitor use a static map
  • Ensure EventMonitor.registeredRecorders is thread-safe
  • Make EventRecordsMap thread-safe
  • Have IEventMonitor implement IDisposable so that it unsubscribes
  • Rewrite ShouldRaise as Should().Raise() by moving it to ReferenceTypeAssertions.
  • Do we need to support nested scopes?
  • Add the possibility to get the metadata for the monitored events
  • Add the possibility to get the occurred events

Closed #477 #348

@dennisdoomen dennisdoomen changed the title Adds a async/await safe alternative for monitoring events [WIP] Adds a async/await safe alternative for monitoring events Aug 22, 2017
@dennisdoomen dennisdoomen changed the base branch from develop to release-5.0 August 25, 2017 20:10
@dennisdoomen dennisdoomen force-pushed the SafeEventMonitoring branch 2 times, most recently from 96c009b to ae454a1 Compare September 3, 2017 10:07
@dennisdoomen dennisdoomen force-pushed the SafeEventMonitoring branch 5 times, most recently from b1155ca to f617b24 Compare September 17, 2017 19:12
@dennisdoomen dennisdoomen changed the title [WIP] Adds a async/await safe alternative for monitoring events Adds a async/await safe alternative for monitoring events Sep 17, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant