tests: remove text monitor testing infrastructure

[libvirt] [PATCH 00/12] tests: remove text monitor testing infrastructure

Posted by Ján Tomko 4 years, 10 months ago

Ján Tomko (12):
  tests: assume JSON monitor in qemuMonitorTestNewSimple
  tests: always assume JSON in qemuMonitorTestNew
  tests: qemuMonitorTestAddErrorResponse: use VIR_AUTOFREE
  tests: qemuMonitorTestProcessCommandDefaultValidate: simplify
    condition
  tests: assume JSON in qemuMonitorTestIO
  tests: qemuMonitorReportError: use tmp variable properly
  tests: remove text monitor testing infrastructure
  tests: qemuMonitorReportError: use VIR_AUTOFREE
  tests: refactor qemuMonitorTestProcessCommandDefault
  tests: refactor qemuMonitorTestProcessCommandDefaultValidate
  tests: qemuMonitorTestAddUnexpectedErrorResponse: VIR_AUTOFREE
  tests: qemuMonitorTestAddInvalidCommandResponse: VIR_AUTOFREE

 tests/qemuhotplugtest.c      |   2 +-
 tests/qemumonitortestutils.c | 235 ++++++++++++-----------------------
 tests/qemumonitortestutils.h |   9 +-
 3 files changed, 84 insertions(+), 162 deletions(-)

-- 
2.20.1

--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list

Re: [libvirt] [PATCH 00/12] tests: remove text monitor testing infrastructure

Posted by Peter Krempa 4 years, 10 months ago

On Sat, Jun 15, 2019 at 13:53:58 +0200, Ján Tomko wrote:
> Ján Tomko (12):
>   tests: assume JSON monitor in qemuMonitorTestNewSimple
>   tests: always assume JSON in qemuMonitorTestNew
>   tests: qemuMonitorTestAddErrorResponse: use VIR_AUTOFREE
>   tests: qemuMonitorTestProcessCommandDefaultValidate: simplify
>     condition
>   tests: assume JSON in qemuMonitorTestIO
>   tests: qemuMonitorReportError: use tmp variable properly
>   tests: remove text monitor testing infrastructure
>   tests: qemuMonitorReportError: use VIR_AUTOFREE
>   tests: refactor qemuMonitorTestProcessCommandDefault
>   tests: refactor qemuMonitorTestProcessCommandDefaultValidate
>   tests: qemuMonitorTestAddUnexpectedErrorResponse: VIR_AUTOFREE
>   tests: qemuMonitorTestAddInvalidCommandResponse: VIR_AUTOFREE

ACK to all the patches I did not comment on.
--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list