Appwrite/appwrite
1
0
Fork 0
mirror of https://github.com/appwrite/appwrite.git synced 2026-05-26 13:51:13 +00:00
Code Issues Packages Projects Releases Wiki Activity

fix: Make Storage cursorAfter test parallel-safe

Browse Source 
Use assertGreaterThanOrEqual and find bucket by ID instead of
assuming exact count and position after cursor query.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
This commit is contained in:
Jake Barnby
2026-02-06 20:42:43 +13:00
co-authored by Claude Opus 4.5
parent 07c9a274b1
commit fd8e8fc736
1 changed files with 5 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files
tests/e2e/Services/Storage/StorageCustomServerTest.php
+5 -3
View File
@@ -204,9 +204,11 @@ class StorageCustomServerTest extends Scope
$this->assertEquals(200, $response['headers']['status-code']);
$this->assertNotEmpty($response['body']);
$this->assertNotEmpty($response['body']['buckets']);
$this->assertCount(1, $response['body']['buckets']);
$this->assertEquals('bucket1', $response['body']['buckets'][0]['$id']);
// In parallel execution, there may be more buckets after the cursor
$this->assertGreaterThanOrEqual(1, count($response['body']['buckets']));
// Find bucket1 by ID (may not be first in parallel execution)
$bucketIds = array_column($response['body']['buckets'], '$id');
$this->assertContains('bucket1', $bucketIds, 'bucket1 should exist in bucket list after cursor');
}
public function testGetBucket(): void