dinput/tests: Initial tests for Action Mapping. Added some tests for the creation of a DirectInput8 Interface and a dummy call of EnumDevicesBySemantics.

Vitaliy Margolen wine-devel at kievinfo.com
Tue Apr 26 08:54:30 CDT 2011


On 04/25/2011 05:14 PM, Austin English wrote:
> On Sun, Apr 24, 2011 at 18:12, Lucas Fialho Zawacki<lfzawacki at gmail.com>  wrote:
>
> Howdy Lucas,
>
>> +    hr = CoCreateInstance(&CLSID_DirectInput8, 0, 1,&IID_IDirectInput8A, (LPVOID*)&pDI);
>> +    if (hr == DIERR_OLDDIRECTINPUTVERSION ||
>> +        hr == DIERR_BETADIRECTINPUTVERSION ||
>> +        hr == REGDB_E_CLASSNOTREG)
>> +    {
>> +        skip("ActionMapping requires dinput8\n");
>> +        return;
>> +    }
>> +    ok(SUCCEEDED(hr), "DirectInput8 Create failed: hr=%08x\n", hr);
>> +    if (FAILED(hr)) return;
>> +
>> +    hr = IDirectInput8_Initialize(pDI,hinst, 0x0800 /* DirectInput8 */ );
>> +    if (hr == DIERR_OLDDIRECTINPUTVERSION || hr == DIERR_BETADIRECTINPUTVERSION)
>> +    {
>> +        skip("ActionMapping requires dinput8\n");
>> +        return;
>> +    }
>
> You probably should use win_skip here, Wine shouldn't have missing
> dinput8 support.

IMHO these tests should go into dinput8 not dinput if they are testing 
dinput8 functionality.

Vitaliy.



More information about the wine-devel mailing list