[PATCH v3 1/3] ntoskrnl.exe/tests: Move driver testing helpers to a separate file.

Rémi Bernon rbernon at codeweavers.com
Tue Aug 31 02:10:59 CDT 2021


I know this is a bit ugly (I assume the PARENTSRC part) but I don't see 
very clean options otherwise.


1) Adding dinput(8) tests to ntoskrnl.exe doesn't sound great to me. 
Even more because I think that most, if not all, dinput tests should be 
done with a controlled driver. Otherwise only mouse and keyboard tests 
are going to be run often, and the rest will just quickly bitrot or be 
useless.

Even for hid, it really seems weird to me to have these tests live in 
ntoskrnl.exe, they aren't testing wdm or the hid driver at all, but 
instead most of it is about hid.dll, and a few tests about hidclass.sys.


2) Duplicating the code everywhere sounds even worse, especially 
"ntoskrnl.exe/tests/utils.h" and the driver test signing and loading 
code, which are non-trivial and meant to work together.

(The HID test driver too, as I intend to make it completely controllable 
and usable for other modules, but I can see reasons to fork it.)

Moving "ntoskrnl.exe/tests/utils.h" to "include/wine/ktest.h", may be a 
way around that, but the driver loading code is still an issue that I 
don't really know how to solve.


Cheers,
-- 
Rémi Bernon <rbernon at codeweavers.com>



More information about the wine-devel mailing list