[Lldb-commits] [lldb] ce075df - [lldb] Fix output for unconditionally decorated tests

[Jonas Devlieghere via lldb-commits]

Author: Jonas Devlieghere
Date: 2022-08-11T16:20:15-07:00
New Revision: ce075dfa190454f6f85920ad4ffe6928980868af

URL: https://github.com/llvm/llvm-project/commit/ce075dfa190454f6f85920ad4ffe6928980868af
DIFF: https://github.com/llvm/llvm-project/commit/ce075dfa190454f6f85920ad4ffe6928980868af.diff

LOG: [lldb] Fix output for unconditionally decorated tests

A missing call to `format` resulted in curly braces getting printed in
the reason a test was modified by a decorator. For example it would
print "{} unconditionally" instead of "skipping unconditionally" for
tests that were marked as such.

Added: 
    

Modified: 
    lldb/packages/Python/lldbsuite/test/decorators.py

Removed: 
    


################################################################################
diff  --git a/lldb/packages/Python/lldbsuite/test/decorators.py b/lldb/packages/Python/lldbsuite/test/decorators.py
index 6d6591c3a9f20..477f4a7209820 100644
--- a/lldb/packages/Python/lldbsuite/test/decorators.py
+++ b/lldb/packages/Python/lldbsuite/test/decorators.py
@@ -248,7 +248,7 @@ def fn(self):
                 reason_str = "{} due to the following parameter(s): {}".format(
                     mode_str, reason_str)
             else:
-                reason_str = "{} unconditionally"
+                reason_str = "{} unconditionally".format(mode_str)
             if bugnumber is not None and not six.callable(bugnumber):
                 reason_str = reason_str + " [" + str(bugnumber) + "]"
         return reason_str