try4 [1/3] kernel32/tests: Add tests for GetVolumeInformationA and QueryDosDevice

[Guy Albertelli]

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