Fix handling mutable IF conditions

pekkaklarck · pekkaklarck · commit 764219f64f60 · 2026-04-01T11:59:37.000+03:00
Fixes #5645.
diff --git a/atest/robot/running/if/if_else.robot b/atest/robot/running/if/if_else.robot
@@ -39,6 +39,10 @@ If failing in keyword
 If failing in else keyword
     Check Test Case    ${TESTNAME}
 
+Mutable condition
+    ${tc} =    Check Test Case    ${TESTNAME}
+    Check Log Message    ${tc[1, 0, 1, 0]}    Should be executed!
+
 Expression evaluation time is included in elapsed time
     ${tc} =    Check Test Case    ${TESTNAME}
     Elapsed Time Should Be Valid    ${tc[0].elapsed_time}       minimum=0.2
diff --git a/atest/testdata/running/if/if_else.robot b/atest/testdata/running/if/if_else.robot
@@ -66,6 +66,18 @@ If failing in else keyword
     [Documentation]    FAIL expected
     Failing else keyword
 
+Mutable condition
+    VAR    @{condition}    bar    baz
+    IF    $condition
+        Call Method    ${condition}    clear
+        Log    Should be executed!
+    ELSE
+        Fail    Should not be executed!
+    END
+    IF    $condition
+        Fail    Should not be executed!
+    END
+
 Expression evaluation time is included in elapsed time
     IF    ${{time.sleep(0.1)}}
         Fail    Not run
diff --git a/src/robot/running/bodyrunner.py b/src/robot/running/bodyrunner.py
@@ -597,8 +597,8 @@ def run(self, data, result):
                         ):
                             self._run = False
                     except ExecutionStatus as err:
-                        error = err
                         self._run = False
+                        error = err
                 if error:
                     raise error
 
@@ -630,8 +630,8 @@ def _run_if_branch(self, data, result, recursive_dry_run=False, syntax_error=Non
             try:
                 run_branch = self._should_run_branch(data, context, recursive_dry_run)
             except DataError as err:
-                error = err
                 run_branch = False
+                error = err
         with StatusReporter(data, result, context, run_branch):
             runner = BodyRunner(context, run_branch, self._templated)
             if not recursive_dry_run:
@@ -648,14 +648,16 @@ def _should_run_branch(self, data, context, recursive_dry_run=False):
         if data.condition is None:
             return True
         try:
-            return evaluate_expression(
+            condition = evaluate_expression(
                 data.condition,
                 context.variables.current,
                 resolve_variables=True,
             )
         except Exception:
             msg = get_error_message()
             raise DataError(f"Invalid {data.type} condition: {msg}")
+        else:
+            return bool(condition)
 
 
 class TryRunner:
