[PATCH] advapi32/tests: Use unique names for each created service.
Zebediah Figura
z.figura12 at gmail.com
Sat Oct 19 15:31:28 CDT 2019
Signed-off-by: Zebediah Figura <z.figura12 at gmail.com>
---
dlls/advapi32/tests/service.c | 38 ++++++-----------------------------
1 file changed, 6 insertions(+), 32 deletions(-)
diff --git a/dlls/advapi32/tests/service.c b/dlls/advapi32/tests/service.c
index f91ea185ae..6c78900bd4 100644
--- a/dlls/advapi32/tests/service.c
+++ b/dlls/advapi32/tests/service.c
@@ -199,7 +199,7 @@ static void test_create_delete_svc(void)
CHAR username[UNLEN + 1], domain[MAX_PATH];
DWORD user_size = UNLEN + 1;
CHAR account[UNLEN + 3];
- static const CHAR servicename [] = "Winetest";
+ static const CHAR servicename [] = "winetest_create_delete";
static const CHAR pathname [] = "we_dont_care.exe";
static const CHAR empty [] = "";
static const CHAR password [] = "secret";
@@ -428,12 +428,6 @@ static void test_create_delete_svc(void)
CloseServiceHandle(svc_handle1);
CloseServiceHandle(scm_handle);
- /* Wait a while. One of the following tests also does a CreateService for the
- * same servicename and this would result in an ERROR_SERVICE_MARKED_FOR_DELETE
- * error if we do this too quickly. Vista seems more picky than the others.
- */
- Sleep(1000);
-
/* And a final NULL check */
SetLastError(0xdeadbeef);
ret = DeleteService(NULL);
@@ -453,7 +447,7 @@ static void test_get_displayname(void)
static const WCHAR spoolerW[] = {'S','p','o','o','l','e','r',0};
static const WCHAR deadbeefW[] = {'D','e','a','d','b','e','e','f',0};
static const WCHAR abcW[] = {'A','B','C',0};
- static const CHAR servicename[] = "Winetest";
+ static const CHAR servicename[] = "winetest_displayname";
static const CHAR pathname[] = "we_dont_care.exe";
/* Having NULL for the size of the buffer will crash on W2K3 */
@@ -725,9 +719,6 @@ static void test_get_displayname(void)
CloseServiceHandle(svc_handle);
CloseServiceHandle(scm_handle);
-
- /* Wait a while. Just in case one of the following tests does a CreateService again */
- Sleep(1000);
}
static void test_get_servicekeyname(void)
@@ -1861,7 +1852,7 @@ static void test_sequence(void)
BOOL ret, is_nt4;
QUERY_SERVICE_CONFIGA *config;
DWORD given, needed;
- static const CHAR servicename [] = "Winetest";
+ static const CHAR servicename [] = "winetest_sequence";
static const CHAR displayname [] = "Winetest dummy service";
static const CHAR displayname2[] = "Winetest dummy service (2)";
static const CHAR pathname [] = "we_dont_care.exe";
@@ -2029,10 +2020,6 @@ static void test_sequence(void)
ret = DeleteService(svc_handle);
ok(ret, "Expected success, got error %u\n", GetLastError());
CloseServiceHandle(svc_handle);
-
- /* Wait a while. The following test does a CreateService again */
- Sleep(1000);
-
CloseServiceHandle(scm_handle);
HeapFree(GetProcessHeap(), 0, config);
}
@@ -2046,7 +2033,7 @@ static void test_queryconfig2(void)
LPSERVICE_DESCRIPTIONA pConfig = (LPSERVICE_DESCRIPTIONA)buffer;
LPSERVICE_DESCRIPTIONW pConfigW = (LPSERVICE_DESCRIPTIONW)buffer;
SERVICE_PRESHUTDOWN_INFO preshutdown_info;
- static const CHAR servicename [] = "Winetest";
+ static const CHAR servicename [] = "winetest_query_config2";
static const CHAR displayname [] = "Winetest dummy service";
static const CHAR pathname [] = "we_dont_care.exe";
static const CHAR dependencies[] = "Master1\0Master2\0+MasterGroup1\0";
@@ -2304,12 +2291,7 @@ static void test_queryconfig2(void)
cleanup:
DeleteService(svc_handle);
-
CloseServiceHandle(svc_handle);
-
- /* Wait a while. The following test does a CreateService again */
- Sleep(1000);
-
CloseServiceHandle(scm_handle);
}
@@ -2512,7 +2494,7 @@ static void test_start_stop(void)
BOOL ret;
SC_HANDLE scm_handle, svc_handle;
DWORD le, is_nt4;
- static const char servicename[] = "Winetest";
+ static const char servicename[] = "winetest_start_stop";
char cmd[MAX_PATH+20];
const char* displayname;
@@ -2592,17 +2574,13 @@ cleanup:
DeleteService(svc_handle);
CloseServiceHandle(svc_handle);
}
-
- /* Wait a while. The following test does a CreateService again */
- Sleep(1000);
-
CloseServiceHandle(scm_handle);
}
static void test_refcount(void)
{
SC_HANDLE scm_handle, svc_handle1, svc_handle2, svc_handle3, svc_handle4, svc_handle5;
- static const CHAR servicename [] = "Winetest";
+ static const CHAR servicename [] = "winetest_refcount";
static const CHAR pathname [] = "we_dont_care.exe";
BOOL ret;
@@ -2681,10 +2659,6 @@ static void test_refcount(void)
/* Delete the service */
ret = DeleteService(svc_handle5);
ok(ret, "Expected success (err=%d)\n", GetLastError());
-
- /* Wait a while. Just in case one of the following tests does a CreateService again */
- Sleep(1000);
-
CloseServiceHandle(svc_handle5);
CloseServiceHandle(scm_handle);
}
--
2.20.1
More information about the wine-devel
mailing list