[PATCH] schedsvc/tests: Check the entire entry list to find the new job

Alex Henrie alexhenrie24 at gmail.com
Thu Aug 30 22:49:50 CDT 2018


Signed-off-by: Alex Henrie <alexhenrie24 at 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




More information about the wine-devel mailing list