moved None comparison higher for Fail-fast principle purposes

Author: ShaharKatz

src/google/adk/evaluation/local_eval_service.py

@@ -271,15 +271,6 @@ async def _evaluate_single_inference_result(
         else 'test_user_id'
     )
 
-    if inference_result.status == InferenceStatus.FAILURE:
-      eval_case_result = await self._build_not_evaluated_eval_case_result(
-          inference_result=inference_result,
-          eval_case=eval_case,
-          user_id=user_id,
-          reason='Inference status indicates failure',
-      )
-      return (inference_result, eval_case_result)
-
     if inference_result.inferences is None:
       logger.error(
           'Evaluation attempted on failed inference for eval case `%s`.'
@@ -295,6 +286,15 @@ async def _evaluate_single_inference_result(
       )
       return (inference_result, eval_case_result)
 
+    if inference_result.status == InferenceStatus.FAILURE:
+      eval_case_result = await self._build_not_evaluated_eval_case_result(
+          inference_result=inference_result,
+          eval_case=eval_case,
+          user_id=user_id,
+          reason='Inference status indicates failure',
+      )
+      return (inference_result, eval_case_result)
+
     if eval_case.conversation_scenario is None and len(
         inference_result.inferences
     ) != len(eval_case.conversation):