From d13a564d5d3c483ebe0f4e95ad87ddd67e4a3a79 Mon Sep 17 00:00:00 2001
From: ArnabChatterjee20k <arnabchatterjee.ac.2@gmail.com>
Date: Thu, 21 May 2026 12:41:04 +0530
Subject: [PATCH] fix: update execution count assertion to include failures

---
 tests/e2e/General/UsageTest.php | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/tests/e2e/General/UsageTest.php b/tests/e2e/General/UsageTest.php
index 912ef4ef8e..9811c99c18 100644
--- a/tests/e2e/General/UsageTest.php
+++ b/tests/e2e/General/UsageTest.php
@@ -1656,7 +1656,9 @@ class UsageTest extends Scope
     {
         $functionId = $data['functionId'];
         $executionTime = $data['executionTime'];
-        $executions = $data['executions'];
+        // METRIC_EXECUTIONS counts every ExecutionCompleted event regardless of status,
+        // so the assertion has to compare against successes + failures, not successes alone.
+        $executions = $data['executions'] + $data['failures'];
 
         $this->assertEventually(function () use ($functionId, $executions, $executionTime) {
             $response = $this->client->call(