Paul Vriens : quartz/tests: Fix the tests on several platforms.

Alexandre Julliard julliard at winehq.org
Tue Sep 9 05:51:03 CDT 2008


Module: wine
Branch: master
Commit: ee14852ea9d064fe8249edce602239cf528d6575
URL:    http://source.winehq.org/git/wine.git/?a=commit;h=ee14852ea9d064fe8249edce602239cf528d6575

Author: Paul Vriens <paul.vriens.wine at gmail.com>
Date:   Tue Sep  9 10:50:51 2008 +0200

quartz/tests: Fix the tests on several platforms.

---

 dlls/quartz/tests/avisplitter.c |   17 ++++++-----------
 1 files changed, 6 insertions(+), 11 deletions(-)

diff --git a/dlls/quartz/tests/avisplitter.c b/dlls/quartz/tests/avisplitter.c
index 82fd4cf..f4c8157 100644
--- a/dlls/quartz/tests/avisplitter.c
+++ b/dlls/quartz/tests/avisplitter.c
@@ -206,11 +206,8 @@ static void test_threads()
         return;
     }
 
-    /* Before doing anything */
+    /* Before doing anything (number of threads at the start differs per OS) */
     baselevel = count_threads();
-    expected = 1;
-    ok(baselevel == expected,
-        "Basic amount of threads should be %d, not %d!\n", expected, baselevel);
 
     file = CreateFileW(wfile, GENERIC_READ, FILE_SHARE_READ|FILE_SHARE_WRITE,
         NULL, OPEN_EXISTING, 0, NULL);
@@ -374,11 +371,7 @@ static void test_threads()
     IBaseFilter_GetState(pavi, INFINITE, &state);
 
     curlevel = count_threads();
-    /* On a 2 stream filter, there are 4 or 5 threads (seems to be 5)
-     * One is the thread we are in. That leaves 3 or 4 for other dark purposes
-     * Wine is 1 thread short!
-     */
-    ok(curlevel == expected || curlevel == expected + 1,
+    ok(curlevel == expected,
         "Amount of threads should be %d not %d\n", expected, curlevel);
 
     IBaseFilter_Pause(pavi);
@@ -452,8 +445,10 @@ fail:
     if (pfile)
         IUnknown_Release(pfile);
 
-    ok(baselevel == 1,
-        "Basic amount of threads should be %d, not %d!\n", 1, baselevel);
+    curlevel = count_threads();
+    todo_wine
+    ok(curlevel == baselevel,
+        "Amount of threads should be %d not %d\n", baselevel, curlevel);
 }
 
 START_TEST(avisplitter)




More information about the wine-cvs mailing list