Re: [PATCH] iotests/testrunner.py: refactor test_field_width

2022-01-13 Thread Kevin Wolf
Am 10.12.2021 um 21:14 hat Vladimir Sementsov-Ogievskiy geschrieben: > A lot of Optional[] types doesn't make code beautiful. > test_field_width defaults to 8, but that is never used in the code. > > More over, if we want some default behavior for single call of > test_run(), it should just print

Re: [PATCH] iotests/testrunner.py: refactor test_field_width

2021-12-16 Thread John Snow
On Fri, Dec 10, 2021 at 3:15 PM Vladimir Sementsov-Ogievskiy < vsement...@virtuozzo.com> wrote: > A lot of Optional[] types doesn't make code beautiful. > test_field_width defaults to 8, but that is never used in the code. > > More over, if we want some default behavior for single call of > test_r

[PATCH] iotests/testrunner.py: refactor test_field_width

2021-12-10 Thread Vladimir Sementsov-Ogievskiy
A lot of Optional[] types doesn't make code beautiful. test_field_width defaults to 8, but that is never used in the code. More over, if we want some default behavior for single call of test_run(), it should just print the whole test name, not limiting or expanding its width, so 8 is bad default.