From edb32d04fe669b3fba4bbedbaf5ba24718db5c15 Mon Sep 17 00:00:00 2001 From: Prem Palanisamy Date: Tue, 26 May 2026 11:26:54 +0100 Subject: [PATCH] Trim apiKey assertion comment --- tests/e2e/Services/Migrations/MigrationsBase.php | 7 +------ 1 file changed, 1 insertion(+), 6 deletions(-) diff --git a/tests/e2e/Services/Migrations/MigrationsBase.php b/tests/e2e/Services/Migrations/MigrationsBase.php index 4b363b0d3e..d9f5b0b41e 100644 --- a/tests/e2e/Services/Migrations/MigrationsBase.php +++ b/tests/e2e/Services/Migrations/MigrationsBase.php @@ -225,12 +225,7 @@ trait MigrationsBase $this->assertEquals('Appwrite', $response['source']); $this->assertEquals('Appwrite', $response['destination']); - // A fresh test project only intrinsically has its auth apiKey — every other - // supported resource type has zero data and gets pruned by Transfer::getStatusCounters. - // The auth apiKey itself is name-matched against the destination's auth apiKey on - // import (createApiKey in the destination handler), so it ends up skipped rather - // than successfully created. Either outcome means the resource was processed; an - // error is not. + // Auth apiKey name-collides with destination's own and gets skipped — treat success or skip as processed. $counts = $response['statusCounters'][Resource::TYPE_API_KEY]; $this->assertEquals([Resource::TYPE_API_KEY], array_keys($response['statusCounters'])); $this->assertEquals(0, $counts['error']);