t/strided: check_result() has no return value
authorVincent Fu <vincent.fu@samsung.com>
Fri, 29 Dec 2023 03:12:47 +0000 (22:12 -0500)
committerVincent Fu <vincent.fu@samsung.com>
Fri, 29 Dec 2023 03:19:44 +0000 (22:19 -0500)
check_result() erroneously returned True or False. Drop the return value
to be consistent with the parent and related classes.

Signed-off-by: Vincent Fu <vincent.fu@samsung.com>
t/strided.py

index b8396aef69b302c678d2fbab140f43d489bb11d2..75c429e454cf6c5fff4518674a0be891263b0f2c 100755 (executable)
@@ -99,7 +99,7 @@ class StridedTest(FioJobCmdTest):
             offset = int(tokens[4])
             if offset < zonestart or offset >= zonestart + self.fio_opts['zonerange']:
                 print(f"Offset {offset} outside of zone starting at {zonestart}")
-                return False
+                return
 
             # skip next section if norandommap is enabled with no
             # random_generator or with a random_generator != lfsr
@@ -117,17 +117,15 @@ class StridedTest(FioJobCmdTest):
             block = (offset - zonestart) / self.fio_opts['bs']
             if block in zoneset:
                 print(f"Offset {offset} in zone already touched")
-                return False
+                return
 
             zoneset.add(block)
             if iosperzone % iosperrange == 0:
                 if len(zoneset) != iosperrange:
                     print(f"Expected {iosperrange} blocks in zone but only saw {len(zoneset)}")
-                    return False
+                    return
                 zoneset = set()
 
-        return True
-
 
 TEST_LIST = [   # randommap enabled
     {