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