Re: [PATCH v3 2/2] wbemprox/tests: Add some tests for SystemRestore class.

[Dmitry Timoshkov]

Hans Leidekker <hans(a)codeweavers.com> wrote:
> On Fri, 2020-12-18 at 13:21 +0300, Dmitry Timoshkov wrote:
> > +    hr = IWbemServices_ExecMethod( services, class, method, 0, NULL, in, &out, NULL );
> > +    ok( hr == S_OK || broken(hr == WBEM_E_ACCESS_DENIED), "failed to execute method %08x\n", hr );
>
> I would prefer to accept both results and print a message when access is
> denied, like in test_SystemSecurity().

Sure.
> Should we disable monitoring when this method succeeds?

What do you mean by disabling monitoring?

-- 
Dmitry.