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 Sites tests parallel-safe for --functional mode

Browse Source 
Add search filters to isolate test data in testListSites to prevent
count assertions from failing when other test methods create sites
during parallel execution.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
This commit is contained in:
Jake Barnby
2026-02-06 17:24:57 +13:00
co-authored by Claude Opus 4.5
parent 2d160747ca
commit 2a4ef06355
1 changed files with 10 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files
tests/e2e/Services/Sites/SitesCustomServerTest.php
+10 -5
View File
@@ -659,8 +659,9 @@ class SitesCustomServerTest extends Scope
$this->assertEquals($sites['headers']['status-code'], 200);
$this->assertCount(1, $sites['body']['sites']);
// Test pagination offset
// Test pagination offset with search filter (to only count our test site)
$sites = $this->listSites([
'search' => 'Test List Sites',
'queries' => [
Query::offset(1)->toString(),
],
@@ -669,8 +670,9 @@ class SitesCustomServerTest extends Scope
$this->assertEquals($sites['headers']['status-code'], 200);
$this->assertCount(0, $sites['body']['sites']);
// Test filter enabled
// Test filter enabled (with search to isolate our test site)
$sites = $this->listSites([
'search' => 'Test List Sites',
'queries' => [
Query::equal('enabled', [true])->toString(),
],
@@ -679,8 +681,9 @@ class SitesCustomServerTest extends Scope
$this->assertEquals($sites['headers']['status-code'], 200);
$this->assertCount(1, $sites['body']['sites']);
// Test filter disabled
// Test filter disabled (with search to isolate our test site)
$sites = $this->listSites([
'search' => 'Test List Sites',
'queries' => [
Query::equal('enabled', [false])->toString(),
],
@@ -698,9 +701,9 @@ class SitesCustomServerTest extends Scope
$this->assertCount(1, $sites['body']['sites']);
$this->assertEquals($sites['body']['sites'][0]['$id'], $siteId);
// Test search framework
// Test search by site ID (more specific than framework search)
$sites = $this->listSites([
'search' => 'other'
'search' => $siteId
]);
$this->assertEquals($sites['headers']['status-code'], 200);
@@ -733,6 +736,7 @@ class SitesCustomServerTest extends Scope
$this->assertEquals($sites['body']['sites'][1]['name'], 'Test List Sites 2');
$sites1 = $this->listSites([
'search' => 'Test List Sites',
'queries' => [
Query::cursorAfter(new Document(['$id' => $sites['body']['sites'][0]['$id']]))->toString(),
],
@@ -743,6 +747,7 @@ class SitesCustomServerTest extends Scope
$this->assertEquals($sites1['body']['sites'][0]['name'], 'Test List Sites 2');
$sites2 = $this->listSites([
'search' => 'Test List Sites',
'queries' => [
Query::cursorBefore(new Document(['$id' => $sites['body']['sites'][1]['$id']]))->toString(),
],