Re: [PATCH v4 9/9] iotests: rename and move 169 and 199 tests

[Vladimir Sementsov-Ogievskiy]

18.05.2020 19:12, Thomas Huth wrote:
> On 15/05/2020 23.15, Vladimir Sementsov-Ogievskiy wrote:
> > Rename bitmaps migration tests and move them to tests subdirectory to
> > demonstrate new human-friendly test naming.
> >
> > Signed-off-by: Vladimir Sementsov-Ogievskiy <address@hidden>
> > ---
> >   tests/qemu-iotests/{199 => tests/migrate-bitmaps-postcopy-test}   | 0
> >   .../{199.out => tests/migrate-bitmaps-postcopy-test.out}          | 0
> >   tests/qemu-iotests/{169 => tests/migrate-bitmaps-test}            | 0
> >   tests/qemu-iotests/{169.out => tests/migrate-bitmaps-test.out}    | 0
> >   4 files changed, 0 insertions(+), 0 deletions(-)
> >   rename tests/qemu-iotests/{199 => tests/migrate-bitmaps-postcopy-test} (100%)
> >   rename tests/qemu-iotests/{199.out => 
> > tests/migrate-bitmaps-postcopy-test.out} (100%)
> >   rename tests/qemu-iotests/{169 => tests/migrate-bitmaps-test} (100%)
> >   rename tests/qemu-iotests/{169.out => tests/migrate-bitmaps-test.out} (100%)
> >
> > diff --git a/tests/qemu-iotests/199 
> > b/tests/qemu-iotests/tests/migrate-bitmaps-postcopy-test
> > similarity index 100%
> > rename from tests/qemu-iotests/199
> > rename to tests/qemu-iotests/tests/migrate-bitmaps-postcopy-test
> > diff --git a/tests/qemu-iotests/199.out 
> > b/tests/qemu-iotests/tests/migrate-bitmaps-postcopy-test.out
> > similarity index 100%
> > rename from tests/qemu-iotests/199.out
> > rename to tests/qemu-iotests/tests/migrate-bitmaps-postcopy-test.out
> > diff --git a/tests/qemu-iotests/169 
> > b/tests/qemu-iotests/tests/migrate-bitmaps-test
> > similarity index 100%
> > rename from tests/qemu-iotests/169
> > rename to tests/qemu-iotests/tests/migrate-bitmaps-test
> > diff --git a/tests/qemu-iotests/169.out 
> > b/tests/qemu-iotests/tests/migrate-bitmaps-test.out
> > similarity index 100%
> > rename from tests/qemu-iotests/169.out
> > rename to tests/qemu-iotests/tests/migrate-bitmaps-test.out
>
> I like the idea ... but the path name + file names get now quite long.
> While you're at it, what about renaming the "qemu-iotests" directory to
> just "iotests" or even just "io" now?

I'm for, and I remember, someone else already said about it. OK, I'll add a 
patch if send a next version.


--
Best regards,
Vladimir