[Lldb-commits] [lldb] [lldb] Improving dotest script flags. (PR #143797)

[David Spickett via lldb-commits]

================
@@ -665,17 +668,23 @@ def iter_filters():
         filtered = True
         print("adding filter spec %s to module %s" % (filterspec, repr(module)))
         tests = unittest.defaultTestLoader.loadTestsFromName(filterspec, module)
-        configuration.suite.addTests(tests)
+        # Add the test -# times.
+        for _ in range(configuration.count):
+            configuration.suite.addTests(tests)
 
     # Forgo this module if the (base, filterspec) combo is invalid
     if configuration.filters and not filtered:
         return
 
     if not filtered:
-        # Add the entire file's worth of tests since we're not filtered.
-        # Also the fail-over case when the filterspec branch
-        # (base, filterspec) combo doesn't make sense.
-        configuration.suite.addTests(unittest.defaultTestLoader.loadTestsFromName(base))
+        # Add the test -# times.
+        for _ in range(configuration.count):
+            # Add the entire file's worth of tests since we're not filtered.
+            # Also the fail-over case when the filterspec branch
+            # (base, filterspec) combo doesn't make sense.
----------------
DavidSpickett wrote:

I have zero idea what this refers to. Maybe it is missing a verb?

"Also add the fail-over case" ?

It's still mysterious even so.

https://github.com/llvm/llvm-project/pull/143797