Re: [PATCH 0/6] Fix one corner case when parsing 'mdevctl' output

2023-08-24 Thread Kristina Hanicova 
On Thu, Aug 24, 2023 at 10:57 AM Michal Privoznik 
wrote:

> See 2/6 for explanation.
>
> Michal Prívozník (6):
>   nodedevmdevctltest: Rename mdevctl-list-empty test case
>   nodeDeviceParseMdevctlJSON: Accept empty string
>   nodedevmdevctltest: Introduce a test case for empty mdevctl output
>   node_device_driver: Deduplicate mediated devices listing
>   virMdevctlList: Don't check for !output
>   virjsontest: Introduce a test case for an empty array
>
>  src/node_device/node_device_driver.c  | 40 ++-
>  .../mdevctl-list-empty-array.json |  1 +
>  .../mdevctl-list-empty-array.out.xml  |  0
>  .../mdevctl-list-empty.json   |  1 -
>  tests/nodedevmdevctltest.c|  1 +
>  tests/virjsontest.c   |  1 +
>  6 files changed, 16 insertions(+), 28 deletions(-)
>  create mode 100644 tests/nodedevmdevctldata/mdevctl-list-empty-array.json
>  create mode 100644
> tests/nodedevmdevctldata/mdevctl-list-empty-array.out.xml


Reviewed-by: Kristina Hanicova 
Kristina

[PATCH 0/6] Fix one corner case when parsing 'mdevctl' output

2023-08-24 Thread Michal Privoznik 
See 2/6 for explanation.

Michal Prívozník (6):
  nodedevmdevctltest: Rename mdevctl-list-empty test case
  nodeDeviceParseMdevctlJSON: Accept empty string
  nodedevmdevctltest: Introduce a test case for empty mdevctl output
  node_device_driver: Deduplicate mediated devices listing
  virMdevctlList: Don't check for !output
  virjsontest: Introduce a test case for an empty array

 src/node_device/node_device_driver.c  | 40 ++-
 .../mdevctl-list-empty-array.json |  1 +
 .../mdevctl-list-empty-array.out.xml  |  0
 .../mdevctl-list-empty.json   |  1 -
 tests/nodedevmdevctltest.c|  1 +
 tests/virjsontest.c   |  1 +
 6 files changed, 16 insertions(+), 28 deletions(-)
 create mode 100644 tests/nodedevmdevctldata/mdevctl-list-empty-array.json
 create mode 100644 tests/nodedevmdevctldata/mdevctl-list-empty-array.out.xml

-- 
2.41.0