[PATCH 1/2] wininet/tests: Moved a test outside a loop to make numbers of tests run independent on the numbers of entries in the cache.

Morten Rønne morten.roenne at tdcadsl.dk
Mon Apr 9 03:55:44 CDT 2012


Makes it easier to see that the same number of tests have been run on 
different platforms.

Morten Rønne
-------------- next part --------------
>From 681ec4b9104a06ca18079e13ea1e7e822ecdecf6 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Morten=20R=C3=B8nne?= <morten.roenne at tdcadsl.dk>
Date: Mon, 9 Apr 2012 10:44:58 +0200
Subject: [PATCH 1/2] wininet/tests: Moved a test outside a loop to make
 numbers of tests run independent on the numbers of
 entries in the cache. Makes it easier to see that the
 same number of tests have been run on different
 platforms.

---
 dlls/wininet/tests/urlcache.c |    3 ++-
 1 files changed, 2 insertions(+), 1 deletions(-)

diff --git a/dlls/wininet/tests/urlcache.c b/dlls/wininet/tests/urlcache.c
index 93ad333..02d8d28 100644
--- a/dlls/wininet/tests/urlcache.c
+++ b/dlls/wininet/tests/urlcache.c
@@ -69,6 +69,7 @@ static void test_find_url_cache_entriesA(void)
         if (!strcmp(lpCacheEntryInfo->lpszSourceUrlName, TEST_URL))
         {
             found = TRUE;
+            ret = TRUE;
             break;
         }
         SetLastError(0xdeadbeef);
@@ -83,10 +84,10 @@ static void test_find_url_cache_entriesA(void)
                 ret = FindNextUrlCacheEntry(hEnumHandle, lpCacheEntryInfo, &cbCacheEntryInfo);
             }
         }
-        ok(ret, "FindNextUrlCacheEntry failed with error %d\n", GetLastError());
         if (!ret)
             break;
     }
+    ok(ret, "FindNextUrlCacheEntry failed with error %d\n", GetLastError());
     ok(found, "Committed url cache entry not found during enumeration\n");
 
     ret = FindCloseUrlCache(hEnumHandle);
-- 
1.7.5.4



More information about the wine-patches mailing list