One of the functionality implemented into qemuxmlconftest is to detect
unused test files. This generalizes the idea and implements it into two
other tests (networkxmlconftest and cputest). But if we find this
useful, it can be implemented into the rest of the tests.

Now, I was contemplating an idea of turning `existingTestCases` variable
into a global one (just like we have testDebug, testVerbose, etc.). It
would make the changes a lot cleaner but also it's a global variable
after all. What do you think?

Michal Prívozník (6):
  testutils: Introduce unused file detection
  qemuxmlconftest: Switch to virTestEnumerateTestCases()
  networkxmlconfdata: Remove passthrough-pf.conf
  networkxmlconftest: Detect unused files
  cputestdata: Drop unused files
  cputest: Detect unused files

 tests/cputest.c                               | 103 ++++++++++++++----
 tests/cputestdata/ppc64-guest-host-model.xml  |   3 -
 .../ppc64-host+guest-host-model.xml           |   3 -
 tests/cputestdata/x86_64-bogus-vendor.xml     |   4 -
 tests/networkxmlconfdata/passthrough-pf.conf  |  11 --
 tests/networkxmlconftest.c                    |  35 +++++-
 tests/qemuxmlconftest.c                       |  82 ++++----------
 tests/testutils.c                             |  58 ++++++++++
 tests/testutils.h                             |  14 +++
 9 files changed, 207 insertions(+), 106 deletions(-)
 delete mode 100644 tests/cputestdata/ppc64-guest-host-model.xml
 delete mode 100644 tests/cputestdata/ppc64-host+guest-host-model.xml
 delete mode 100644 tests/cputestdata/x86_64-bogus-vendor.xml
 delete mode 100644 tests/networkxmlconfdata/passthrough-pf.conf

-- 
2.53.0

Reply via email to