From 2e061e2f2bb1fefebe9a9fab029eb3781e27e143 Mon Sep 17 00:00:00 2001 From: Jake Barnby Date: Thu, 5 Feb 2026 01:33:01 +1300 Subject: [PATCH] fix: make testCreatePhoneVerification self-contained Remove @depends annotation and set up phone number within the test to ensure it works with PHPUnit 12 and paratest parallel execution. Co-Authored-By: Claude Opus 4.5 --- tests/e2e/Services/GraphQL/AccountTest.php | 16 +++++++++++++++- 1 file changed, 15 insertions(+), 1 deletion(-) diff --git a/tests/e2e/Services/GraphQL/AccountTest.php b/tests/e2e/Services/GraphQL/AccountTest.php index 9dd224f1a1..76f98ce02f 100644 --- a/tests/e2e/Services/GraphQL/AccountTest.php +++ b/tests/e2e/Services/GraphQL/AccountTest.php @@ -116,7 +116,6 @@ class AccountTest extends Scope } /** - * @depends testUpdateAccountPhone * @return array * @throws \Exception */ @@ -124,6 +123,21 @@ class AccountTest extends Scope { $projectId = $this->getProject()['$id']; + // Ensure phone is set up for this test to be self-contained + $phoneQuery = $this->getQuery(self::UPDATE_ACCOUNT_PHONE); + $phonePayload = [ + 'query' => $phoneQuery, + 'variables' => [ + 'phone' => '+123456789', + 'password' => 'password', + ] + ]; + + $this->client->call(Client::METHOD_POST, '/graphql', \array_merge([ + 'content-type' => 'application/json', + 'x-appwrite-project' => $projectId, + ], $this->getHeaders()), $phonePayload); + $query = $this->getQuery(self::CREATE_PHONE_VERIFICATION); $graphQLPayload = [ 'query' => $query,