Refactored per gemini suggestions.

Author: quad2524

src/google/adk/cli/cli_tools_click.py

@@ -792,12 +792,7 @@ def cli_eval(
     for eval_result in eval_results:
       eval_result: EvalCaseResult
 
-      should_print = (
-          eval_result.final_eval_status != EvalStatus.PASSED
-          or print_detailed_results_on_success
-      )
-      
-      if should_print:
+    if eval_result.final_eval_status != EvalStatus.PASSED or print_detailed_results_on_success:
         click.echo(
             "********************************************************************"
         )

src/google/adk/evaluation/agent_evaluator.py

@@ -683,15 +683,15 @@ def _process_metrics_and_get_failures(
 
       # Gather all the failures.
       if overall_eval_status != EvalStatus.PASSED:
-        should_print = print_detailed_results
         failures.append(
             f"{metric_name} for {agent_module} Failed. Expected {threshold},"
             f" but got {overall_score}."
         )
-      else:
-        should_print = (
-            print_detailed_results and print_detailed_results_on_success
-        )
+
+      should_print = print_detailed_results and (
+          overall_eval_status != EvalStatus.PASSED
+          or print_detailed_results_on_success
+      )
 
       if should_print:
         AgentEvaluator._print_details(