Use waithandle instead of Thread.Sleep. Fixes a race condition in run…

…ning tests on Mono
joeyespo · Jun 23, 2011 · 68518a8 · 68518a8
1 parent d1c827f
commit 68518a8
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 4 deletions.
diff --git a/.gitignore b/.gitignore
@@ -8,3 +8,5 @@ _ReSharper.*
 *.suo
 *.userprefs
 test-results
+*.pidb
+TestResult.xml
diff --git a/src/Fleck.Tests/ReceiverTests.cs b/src/Fleck.Tests/ReceiverTests.cs
@@ -13,15 +13,18 @@ public class ReceiverTests
         private bool _wasClosed;
         private Mock<ISocket> _mockSocket;
         private Receiver _receiver;
-        private string _message;
+        private EventWaitHandle _closeHandle;
 
         [SetUp]
         public void Setup()
         {
-            _message = null;
             _wasClosed = false;
             _mockSocket = new Mock<ISocket>();
-            _receiver = new Receiver(_mockSocket.Object,s => _message = s, () => _wasClosed = true);
+            _closeHandle = new EventWaitHandle(false,EventResetMode.ManualReset);
+            _receiver = new Receiver(_mockSocket.Object,s => {}, () => {
+              _wasClosed = true;
+              _closeHandle.Set();
+            });
         }
 
         [Test]
@@ -52,7 +55,7 @@ public void ShouldCloseOnException()
                 .Throws<Exception>()
                 .Verifiable();
             _receiver.Receive();
-            Thread.Sleep(100);
+            _closeHandle.WaitOne();
             _mockSocket.Verify();
             Assert.IsTrue(_wasClosed);
         }