Updated all FileSystemWatcher events to be virtual, to make them possibl... #62

Merged
merged 1 commit into from Mar 2, 2014

Conversation

Projects
None yet
2 participants
@KallDrexx
Contributor

KallDrexx commented Feb 23, 2014

...e to raise via mocks.

I realized that without making the events virtual there is no way for mocks to raise them. While I'm not a fan of virtual events (as you can shoot yourself in the foot by overriding events) I could not figure out a better way to allow mocks to raise events without changing the FileSystemWatcherBase to be an interface methods publicly.

tathamoddie added a commit that referenced this pull request Mar 2, 2014

Merge pull request #62 from KallDrexx/VirtualEventsForFileSystemWatch…
…erBase

Updated all FileSystemWatcher events to be virtual, to make them possibl...

@tathamoddie tathamoddie merged commit 4257b55 into tathamoddie:master Mar 2, 2014

1 check passed

default Finished TeamCity Build System.IO.Abstractions :: Build : Tests passed: 183
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment