advpack: Open the INF file if the RSC_FLAG_INF flag is specified

Alexandre Julliard julliard at winehq.org
Wed Mar 29 06:06:30 CST 2006


"James Hawkins" <truiken at gmail.com> writes:

> @@ -72,10 +72,7 @@ static void test_RunSetupCommand()
>      /* try to run an exe with the RSC_FLAG_INF flag */
>      hexe = (HANDLE)0xdeadbeef;
>      hr = pRunSetupCommand(NULL, "winver.exe", "Install", "c:\\windows\\system32", "Title", &hexe, RSC_FLAG_INF, NULL);
> -    todo_wine
> -    {
> -        ok(hr == SPAPI_E_WRONG_INF_STYLE, "Expected SPAPI_E_WRONG_INF_STYLE, got %ld\n", hr);
> -    }
> +    ok(hr & SPAPI_E_EXPECTED_SECTION_NAME, "Expected a setupapi error, got %ld\n", hr);

This test doesn't make sense. If you want to test certain bits you
have to write that explicitly, not pick some random constant that
happens to have the right value. And in any case a simple AND won't do
what you want.

-- 
Alexandre Julliard
julliard at winehq.org



More information about the wine-devel mailing list