[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PATCH RFC WIP 0/6] iotests: delinting work-in-progress
From: |
John Snow |
Subject: |
[PATCH RFC WIP 0/6] iotests: delinting work-in-progress |
Date: |
Wed, 13 May 2020 17:41:24 -0400 |
I ran out of time, but I was briefly entertaining the idea of hitting
everything else in the iotests folder with the pylint and mypy beam.
This is just a draft of what I had at the time, in case someone gets
around to it before I do.
I forced all of the python scripts in this directory to Python3 and pass
the formatter check imposed by the pylintrc that we make iotests.py
adhere to, but there's no guarantee these scripts actually work. I was a
little rough around the edges.
qed.py and qcow2.py in particular abuse some python features that pylint
really doesn't like to see at all, and forcing these to pass a pylint
check was clearly fighting against the idioms in-use in those files,
with more than a few unsatisfactory hacks employed just to make pylint
shut up. They are not necessarily improvements to those scripts.
I created a Makefile check to hit the iotests themselves with the pylint
checker so we can identify when refactors to the testing infrastructure
invalidate tests we don't get to run that often; but didn't get around
to improving the quality of the tests themselves.
(Whether or not this is even worth our time is debatable, it depends on
how much effort it would take to bring them up to par. Maybe it's a lot,
I don't know.)
--js
John Snow (6):
iotests: type hint wip
Makefile: add delint WIP
nbd-fault-injector: delint
qed.py: delint
qcow2.py: delint
WIP: delint test files
tests/qemu-iotests/Makefile | 6 +
tests/qemu-iotests/iotests.py | 28 ++--
tests/qemu-iotests/nbd-fault-injector.py | 34 +++--
tests/qemu-iotests/pylintrc | 1 +
tests/qemu-iotests/qcow2.py | 156 +++++++++++++++--------
tests/qemu-iotests/qed.py | 46 ++++---
6 files changed, 173 insertions(+), 98 deletions(-)
--
2.21.1
- [PATCH RFC WIP 0/6] iotests: delinting work-in-progress,
John Snow <=
- [PATCH RFC WIP 1/6] iotests: type hint wip, John Snow, 2020/05/13
- [PATCH RFC WIP 2/6] Makefile: add delint WIP, John Snow, 2020/05/13
- [PATCH RFC WIP 3/6] nbd-fault-injector: delint, John Snow, 2020/05/13
- [PATCH RFC WIP 5/6] qcow2.py: delint, John Snow, 2020/05/13
- [PATCH RFC WIP 4/6] qed.py: delint, John Snow, 2020/05/13
- [PATCH RFC WIP 6/6] WIP: delint test files, John Snow, 2020/05/13
- Re: [PATCH RFC WIP 0/6] iotests: delinting work-in-progress, no-reply, 2020/05/13