On Fri, Dec 15, 2017 at 08:01 PM +0100, John Ferlan <[email protected]> wrote: > On 12/12/2017 06:36 AM, Marc Hartmayer wrote: >> Enable testing for 'auth_pending' in the virnetdaemon test case. >> >> Signed-off-by: Marc Hartmayer <[email protected]> >> Reviewed-by: Boris Fiuczynski <[email protected]> >> --- >> .../input-data-client-auth-pending-failure.json | 44 ++++++++++++++ >> .../input-data-client-auth-pending.json | 70 >> ++++++++++++++++++++++ >> .../output-data-client-auth-pending.json | 70 >> ++++++++++++++++++++++ >> tests/virnetdaemontest.c | 2 + >> 4 files changed, 186 insertions(+) >> create mode 100644 >> tests/virnetdaemondata/input-data-client-auth-pending-failure.json >> create mode 100644 >> tests/virnetdaemondata/input-data-client-auth-pending.json >> create mode 100644 >> tests/virnetdaemondata/output-data-client-auth-pending.json >> > > I assume the -failure test case is what I pointed out in the previous > patch, right? That somehow the ExecRestart case was incorrect and that > the check should be there and not in the NewInternal helper.
Yep. > > In any case, for this patch... Thanks for the test it helps. > > Reviewed-by: John Ferlan <[email protected]> Thanks. > > John > -- Beste Grüße / Kind regards Marc Hartmayer IBM Deutschland Research & Development GmbH Vorsitzende des Aufsichtsrats: Martina Koederitz Geschäftsführung: Dirk Wittkopp Sitz der Gesellschaft: Böblingen Registergericht: Amtsgericht Stuttgart, HRB 243294 -- libvir-list mailing list [email protected] https://www.redhat.com/mailman/listinfo/libvir-list
