[prev in list] [next in list] [prev in thread] [next in thread] 

List:       wine-devel
Subject:    [PATCH] schedsvc/tests: Check the entire entry list to find the new job
From:       Alex Henrie <alexhenrie24 () gmail ! com>
Date:       2018-08-31 3:49:50
Message-ID: 20180831034950.760-1-alexhenrie24 () gmail ! com
[Download RAW message or body]

Signed-off-by: Alex Henrie <alexhenrie24@gmail.com>
---
This looks like it was a typo: The code loops through all the retrieved
jobs, but it only retrieved the first one.

This patch should fix the tests that usually fail on Debian, as well as
any intermittent failures on Windows.
---
 dlls/schedsvc/tests/atsvcapi.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/dlls/schedsvc/tests/atsvcapi.c b/dlls/schedsvc/tests/atsvcapi.c
index 66f237ecab..d7d48a122c 100644
--- a/dlls/schedsvc/tests/atsvcapi.c
+++ b/dlls/schedsvc/tests/atsvcapi.c
@@ -118,7 +118,7 @@ START_TEST(atsvcapi)
         container.EntriesRead = 0;
         container.Buffer = NULL;
         total = start_index = 0;
-        ret = NetrJobEnum(server_name, &container, 1, &total, &start_index);
+        ret = NetrJobEnum(server_name, &container, -1, &total, &start_index);
         if (ret == ERROR_ACCESS_DENIED)
         {
             win_skip("NetrJobEnum: Access denied, skipping the tests\n");
-- 
2.18.0




[prev in list] [next in list] [prev in thread] [next in thread]