From fa7ebbffa8126a05ff4c2ac4dbf99f6ad5129324 Mon Sep 17 00:00:00 2001 From: Jake Barnby Date: Thu, 12 Feb 2026 18:12:54 +1300 Subject: [PATCH] Fix void return in testListProjectKey and increase spatial test timeout - Remove `return $data` from testListProjectKey() which is declared void, causing fatal PHP error that crashed Projects and Abuse test suites - Increase waitForAllAttributes timeout to 10min for spatial column test to handle MariaDB slowness under parallel CI load Co-Authored-By: Claude Opus 4.6 --- tests/e2e/Services/Databases/DatabasesBase.php | 4 ++-- tests/e2e/Services/Projects/ProjectsConsoleClientTest.php | 2 -- 2 files changed, 2 insertions(+), 4 deletions(-) diff --git a/tests/e2e/Services/Databases/DatabasesBase.php b/tests/e2e/Services/Databases/DatabasesBase.php index 6137c59cea..411c069b22 100644 --- a/tests/e2e/Services/Databases/DatabasesBase.php +++ b/tests/e2e/Services/Databases/DatabasesBase.php @@ -9132,8 +9132,8 @@ trait DatabasesBase $this->assertEquals(202, $poly['headers']['status-code']); - // Wait for attributes to be available - $this->waitForAllAttributes($databaseId, $colId); + // Wait for attributes to be available (longer timeout for MariaDB spatial columns under parallel load) + $this->waitForAllAttributes($databaseId, $colId, 600000); // Create a new document without spatial data to test default values $newDocument = $this->client->call(Client::METHOD_POST, $this->getRecordUrl($databaseId, $colId), array_merge([ diff --git a/tests/e2e/Services/Projects/ProjectsConsoleClientTest.php b/tests/e2e/Services/Projects/ProjectsConsoleClientTest.php index 7345a1a6f5..4cc935a091 100644 --- a/tests/e2e/Services/Projects/ProjectsConsoleClientTest.php +++ b/tests/e2e/Services/Projects/ProjectsConsoleClientTest.php @@ -3249,8 +3249,6 @@ class ProjectsConsoleClientTest extends Scope ]); $this->assertEquals(400, $response['headers']['status-code']); - - return $data; }