Merge pull request #34 from redpanda-data/boquard/fix-ok-to-fail-fips

CORE-14 Fixed ok-to-fail-fips to correctly return args
redpanda-data · Mar 28, 2024 · cad0aea · cad0aea
2 parents 386057f + 44c3280
commit cad0aea
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/ducktape/mark/_mark.py b/ducktape/mark/_mark.py
@@ -503,12 +503,13 @@ def running_fips() -> bool:
 
         return False
 
-    if len(args) == 1 and len(kwargs) == 0 and running_fips():
+    if len(args) == 1 and len(kwargs) == 0:
         # this corresponds to the usage of the decorator with no arguments
         # @ok_to_fail_fips
         # def test_function:
         #  ...
-        Mark.mark(args[0], OkToFailFIPS())
+        if running_fips():
+            Mark.mark(args[0], OkToFailFIPS())
         return args[0]