setupapi/tests: Use todo_wine_if() in tests.
Signed-off-by: Frédéric Delanoy <frederic.delanoy@gmail.com> Signed-off-by: Alexandre Julliard <julliard@winehq.org>
This commit is contained in:
parent
a8a6f0bd86
commit
5c2cff30aa
|
@ -152,14 +152,7 @@ static void test_invalid_files(void)
|
|||
if (invalid_files[i].error) /* should fail */
|
||||
{
|
||||
ok( hinf == INVALID_HANDLE_VALUE, "file %u: Open succeeded\n", i );
|
||||
if (invalid_files[i].todo) todo_wine
|
||||
{
|
||||
ok( err == invalid_files[i].error, "file %u: Bad error %u/%u\n",
|
||||
i, err, invalid_files[i].error );
|
||||
ok( err_line == invalid_files[i].err_line, "file %u: Bad error line %d/%d\n",
|
||||
i, err_line, invalid_files[i].err_line );
|
||||
}
|
||||
else
|
||||
todo_wine_if (invalid_files[i].todo)
|
||||
{
|
||||
ok( err == invalid_files[i].error, "file %u: Bad error %u/%u\n",
|
||||
i, err, invalid_files[i].error );
|
||||
|
|
|
@ -115,20 +115,11 @@ static void test_invalid_parametersA(void)
|
|||
ret = SetupIterateCabinetA(invalid_parameters[i].CabinetFile, 0,
|
||||
invalid_parameters[i].MsgHandler, NULL);
|
||||
ok(!ret, "[%d] Expected SetupIterateCabinetA to return 0, got %d\n", i, ret);
|
||||
if (invalid_parameters[i].todo_lasterror)
|
||||
{
|
||||
todo_wine
|
||||
todo_wine_if (invalid_parameters[i].todo_lasterror)
|
||||
ok(GetLastError() == invalid_parameters[i].expected_lasterror,
|
||||
"[%d] Expected GetLastError() to return %u, got %u\n",
|
||||
i, invalid_parameters[i].expected_lasterror, GetLastError());
|
||||
}
|
||||
else
|
||||
{
|
||||
ok(GetLastError() == invalid_parameters[i].expected_lasterror,
|
||||
"[%d] Expected GetLastError() to return %u, got %u\n",
|
||||
i, invalid_parameters[i].expected_lasterror, GetLastError());
|
||||
}
|
||||
}
|
||||
|
||||
SetLastError(0xdeadbeef);
|
||||
ret = SetupIterateCabinetA("", 0, NULL, NULL);
|
||||
|
@ -190,20 +181,11 @@ static void test_invalid_parametersW(void)
|
|||
ret = SetupIterateCabinetW(invalid_parameters[i].CabinetFile, 0,
|
||||
invalid_parameters[i].MsgHandler, NULL);
|
||||
ok(!ret, "[%d] Expected SetupIterateCabinetW to return 0, got %d\n", i, ret);
|
||||
if (invalid_parameters[i].todo_lasterror)
|
||||
{
|
||||
todo_wine
|
||||
todo_wine_if (invalid_parameters[i].todo_lasterror)
|
||||
ok(GetLastError() == invalid_parameters[i].expected_lasterror,
|
||||
"[%d] Expected GetLastError() to return %u, got %u\n",
|
||||
i, invalid_parameters[i].expected_lasterror, GetLastError());
|
||||
}
|
||||
else
|
||||
{
|
||||
ok(GetLastError() == invalid_parameters[i].expected_lasterror,
|
||||
"[%d] Expected GetLastError() to return %u, got %u\n",
|
||||
i, invalid_parameters[i].expected_lasterror, GetLastError());
|
||||
}
|
||||
}
|
||||
|
||||
SetLastError(0xdeadbeef);
|
||||
ret = SetupIterateCabinetW(NULL, 0, NULL, NULL);
|
||||
|
|
Loading…
Reference in New Issue