Re: [PATCH v6 07/11] iotests: add findtests.py

2021-01-14 Thread Kevin Wolf
Am 14.01.2021 um 08:38 hat Vladimir Sementsov-Ogievskiy geschrieben: > 12.01.2021 19:42, Kevin Wolf wrote: > > > +def find_tests(self, groups: Optional[List[str]] = None, > > > + exclude_groups: Optional[List[str]] = None, > > > + tests: Optional[List[str]]

Re: [PATCH v6 07/11] iotests: add findtests.py

2021-01-13 Thread Vladimir Sementsov-Ogievskiy
12.01.2021 19:42, Kevin Wolf wrote: +def find_tests(self, groups: Optional[List[str]] = None, + exclude_groups: Optional[List[str]] = None, + tests: Optional[List[str]] = None, group and tests seem to be read-only, so this can be simplified to 'groups:

Re: [PATCH v6 07/11] iotests: add findtests.py

2021-01-13 Thread Eric Blake
On 1/9/21 6:26 AM, Vladimir Sementsov-Ogievskiy wrote: > Add python script with new logic of searching for tests: > > Current ./check behavior: > - tests are named [0-9][0-9][0-9] > - tests must be registered in group file (even if test doesn't belong >to any group, like 142) > > > This

Re: [PATCH v6 07/11] iotests: add findtests.py

2021-01-13 Thread Kevin Wolf
Am 13.01.2021 um 11:37 hat Vladimir Sementsov-Ogievskiy geschrieben: > 12.01.2021 19:42, Kevin Wolf wrote: > > Am 09.01.2021 um 13:26 hat Vladimir Sementsov-Ogievskiy geschrieben: > > > +def __init__(self, test_dir: Optional[str] = None) -> None: > > > +self.groups = defaultdict(set) >

Re: [PATCH v6 07/11] iotests: add findtests.py

2021-01-13 Thread Vladimir Sementsov-Ogievskiy
12.01.2021 19:42, Kevin Wolf wrote: Am 09.01.2021 um 13:26 hat Vladimir Sementsov-Ogievskiy geschrieben: Add python script with new logic of searching for tests: Current ./check behavior: - tests are named [0-9][0-9][0-9] - tests must be registered in group file (even if test doesn't

Re: [PATCH v6 07/11] iotests: add findtests.py

2021-01-12 Thread Kevin Wolf
Am 09.01.2021 um 13:26 hat Vladimir Sementsov-Ogievskiy geschrieben: > Add python script with new logic of searching for tests: > > Current ./check behavior: > - tests are named [0-9][0-9][0-9] > - tests must be registered in group file (even if test doesn't belong >to any group, like 142)

[PATCH v6 07/11] iotests: add findtests.py

2021-01-09 Thread Vladimir Sementsov-Ogievskiy
Add python script with new logic of searching for tests: Current ./check behavior: - tests are named [0-9][0-9][0-9] - tests must be registered in group file (even if test doesn't belong to any group, like 142) Behavior of findtests.py: - group file is dropped - tests are all files in