Changeset 6547

Timestamp:

07/08/11 01:26:46 (13 years ago)

Author:

swagner

Message:

Changed ThreadSafeLogTest as testing the race condition with the unsafe log resulted in strange exceptions sometimes (#1545)

File:

• trunk/sources/HeuristicLab/3.3/Tests/ThreadSafeLogTest.cs (modified) (2 diffs)

trunk/sources/HeuristicLab/3.3/Tests/ThreadSafeLogTest.cs

@@ -20 +20 @@
 #endregion
 
-using System;
 using System.Linq;
 using System.Threading.Tasks;
@@ -32 +31 @@
     [TestMethod]
     public void ThreadSafeLogThreadSavetyTest() {
-      int count = 100000;
-      if (Environment.ProcessorCount > 2) {
-        Log l = new Log();
-        AddMessagesInParallel(count, l);
-        // with a normal log there are race conditions which should cause problems
-        // actually we might geht lucky here but this should happen only seldomly
-        Assert.AreNotEqual(count, l.Messages.Count());
-      }
+      int count = 10000;
+      ThreadSafeLog log = new ThreadSafeLog();
 
-      ThreadSafeLog safeLog = new ThreadSafeLog();
-      AddMessagesInParallel(count, safeLog);
-      // the thread safe log should work like a charm
-      Assert.AreEqual(count, safeLog.Messages.Count());
-    }
+      Parallel.For(0, count, (i) => {
+        log.LogMessage("Message " + i); // write something
+        log.Messages.Count(); // iterate over all messages
+      });
 
-    private void AddMessagesInParallel(int count, ILog l) {
-      Parallel.For(0, count, (i) => {
-        l.LogMessage("Message " + i); // write something
-        l.Messages.Count(); // iterate over all messages
-      });
+      Assert.AreEqual(count, log.Messages.Count());
     }
   }