RFC: detecting wine drivers in the audio tests
Maarten Lankhorst
m.b.lankhorst at gmail.com
Tue Apr 29 23:31:41 CDT 2008
Hello Robert,
2008/4/29 Robert Reif <reif at earthlink.net>:
> The returned result of some audio functions on windows may be inconsistent
> because a driver may actually supply the returned value.
>
> This presents a problem for the wine regression tests because a buggy
> driver may return an unexpected result which causes the test to fail. One
> way around this is to accept known failures as OK but that reduces the
> usefulness of the test for wine because it may allow wine bugs to slip in.
> I'm proposing the we determine if we are running on wine by defining a wine
> manufactures id and checking for that id in the test. If a wine driver is
> found, don't accept a failure as OK. The returned result should be well
> defined and any failure is unacceptable. However, if the driver is not a
> wine driver and there are known buggy windows drivers that return specific
> errors, we can check for that and not fail the test. This should be easier
> than maintaining a database of known broken drivers and black listing them.
>
> I am concerned that requiring 100% wine regression test success on windows
> is not practical when there are broken windows drivers out there. Accepting
> the failures as success is not good for wine because it may allow buggy wine
> drivers to also pass. I think we should hold wine audio drivers to higher
> standards than the typical audio card manufacture.
>
> I am suppling a minimal patch to the alsa driver and a single wave test to
> illustrate this concept. I hope this allows valid tests to remain in spite
> of buggy windows drivers.
I don't know, the general idea is that whatever windows lets you do is
fine for wine, so I'm not 100% sure that testing for this is a good
idea, because it means in border cases windows programs can't rely on
those to return a correct or incorrect value either.
By the way you also used a C++ style comment, wine doesn't allow that.
Cheers,
Maarten
More information about the wine-devel
mailing list