Hi,
In this new version I added a handful of patches to make it easier to
review, as suggested by Greg in v1.
Thomas, the changes made aren't negligible so I didn't add your
Tested-by. You're more than welcome to give this version a try :)
The first 2 patches are needed to ease the burden of individual test
changes. It was really annoying to deal with changing the dir name every
time a test was being removed. Parametrizing the dir name (and anything
that has the dir in it) makes it trivial to change the dir name when
needed.
I kept the slow revert patch (7) because this series is passing the Gitlab
CI with it:
https://gitlab.com/danielhb/qemu/-/pipelines/1228813691
I'll leave to the 9p maintainers to decide whether we should go ahead with
it or not.
Changes from v1:
- patches 1 and 2 from v1: dropped
- patches 1 and 2 (new):
- parametrize 'new_dir' name
- patches 3, 4, 5, 6 (new):
- individual test consolidation
- v1 link: https://mail.gnu.org/archive/html/qemu-devel/2024-03/msg06146.html
Daniel Henrique Barboza (7):
qtest/virtio-9p-test.c: add '_path' in path vars
qtest/virtio-9p-test.c: parametrize 'dir' name in local tests
qtest/virtio-9p-test.c: consolidate create dir tests
qtest/virtio-9p-test.c: consolidate create file tests
qtest/virtio-9p-test.c: consolidate create symlink tests
qtest/virtio-9p-test.c: consolidate hardlink tests
qtest/virtio-9p-test.c: remove g_test_slow() gate
tests/qtest/virtio-9p-test.c | 228 +++++++++++++++--------------------
1 file changed, 96 insertions(+), 132 deletions(-)
--
2.44.0