On Wed, 2009-04-15 at 08:18 +0200, Paul Vriens wrote:
> Guy Albertelli wrote:
> > Fixed memory leak and changed skip to win_skip in this patch.
> >
> > Changelog:
> > - Enhance QueryDosDevice test for attempting to return all devices.
> > - Add tests for GetVolumeInformationA testing the different root dir
> > formats.
> > ---
> > dlls/kernel32/tests/volume.c | 137
> > -
> > 1 files changed, 133 insertions(+), 4 deletions(-)
> >
> >
> >
> >
> >
> >
> One of the tests fails here:
>
> volume.c:290: Test failed: GetVolumeInformationA w/o '\' did not fail,
> last error 2
>
> I just ran the tests as your second mail/patch suggested it fixed
> something. That patch however didn't contain a change to the tests.
>
> If you add tests that fail on Wine and you don't include a fixed
> implementation in the same patch you should mark these with todo_wine().
> If you fix the implementation in a later patch you should remove the
> todo_wine() again if needed.
> Bottom line: after each patch all tests should still succeed.
The point was that [2/3] actually fixed that issue. Since they were
being rejected as a set, I tried to keep each patch to a single module.
Thanks, Guy