Re: [3/3] advapi32/tests: Add preliminary event tracing tests.

[Ztirfe Elgnid]

On 06/08/2015 01:54 AM, Nikolay Sivov wrote:
> On 08.06.2015 6:56, Zebediah Figura wrote:
> > +    /* test ERROR_DISK_FULL */
> > +    properties->MaximumFileSize = 1000000000000; /* 1 EB */
> > +    SetLastError(0xdeadbeef);
> > +    ret = StartTraceA(&handle, sessionname, properties);
> > +    todo_wine
> > +    {
> > +    ok(ret != ERROR_SUCCESS, "Expected failure\n");
> > +    ok(GetLastError() == ERROR_DISK_FULL, "Expected ERROR_DISK_FULL, got %d\n", GetLastError());
> > +    }
> > +    properties->MaximumFileSize = 0;
> > Patches look fine to me, but I have to ask - is it possible that this 
> > test will actually try to write that much?
> > That's... actually probably a reasonable fear. I'll remove that line.