fix(Spanner): Make System Tests Work & Work Faster (#9401)

* test(spanner): add IF NOT EXISTS to PostgreSQL DDL statements

test(spanner): consolidate test database provisioning

fix(spanner): fix PgReadTest index creation on shared database

test(spanner): Consolidate DDL operations into test setup suite

test(spanner): fix schema mismatches for partitionedDml and PgQueryTest

test(spanner): rename PgQueryTest_2 back to PgQueryTest

test(spanner): fix PostgreSQL column name mismatches for test tables

* test(spanner): BackupTest reliability and optimization

test(spanner): Handle DEADLINE_EXCEEDED in BackupTest

test(spanner): fix BackupTest timeout and PgQueryTest schema collision

test(spanner): add deadline exceeded polling to testCreateBackup2

refactor(spanner): DRY up extended polling loop in BackupTest

* chore: enable Spanner system tests in CI

* test(spanner): avoid ID collisions by using self::randId() with a larger range

test(spanner): restore seedTable in BatchTest to fix test coverage

test(spanner): fix array_rand bug and batch mutations in seedTable

test(spanner): fix fundamentally broken partitionRead test logic in BatchTest

fix: move seedTable back to its original location

fix(cs): fix style issues in BatchTest.php

fix(cs): format multi-line args

* fix(spanner): fix test database initialization and teardown on persistent databases, and apply code review fixes for tests

* test(spanner): rollback open transaction in ReadTest

* test(spanner): clear stale backups before running BackupTest

* address PR comments

* test(spanner): refactor backup tests (remove backup2) and add ApiException catching

Refactors backup tests for clarity and adds catching of ApiException where appropriate. Also adds debug logging to system tests for better observability.

* feat(spanner): enable parallel test execution

* fix: add break to prevent infinite loop in BackupTest

* fix: reload backup object before asserting state

* test: add Database::STATE_READY assertion in testRestoreToNewDatabase

* chore(Spanner): setting instance processing units to 1000

* fix(Spanner): fixing bugs in UniverseDomainTest and BackupTest

* fix(Spanner): fix premature setup flag assignment in system tests

* doc(Spanner): Readme clarifications

* docs(Spanner): add paratest to dependencies and clarify system test README

Author: cy-yun

Core/src/Testing/System/SystemTestCase.php

@@ -76,7 +76,7 @@ public static function processQueue()
      */
     public static function randId()
     {
-        return rand(1, 9999999);
+        return rand(1, 999999999);
     }
 
     /**

Spanner/composer.json

@@ -22,7 +22,8 @@
         "dg/bypass-finals": "^1.7",
         "dms/phpunit-arraysubset-asserts": "^0.5.0",
         "symfony/process": "^6.4",
-        "nikic/php-parser": "^5.0"
+        "nikic/php-parser": "^5.0",
+        "brianium/paratest": "^6.11"
     },
     "suggest": {
         "ext-protobuf": "Provides a significant increase in throughput over the pure PHP protobuf implementation. See https://cloud.google.com/php/grpc for installation instructions.",

Spanner/tests/System/AdminTest.php

@@ -68,7 +68,7 @@ public function testInstance()
         $this->assertEquals(Instance::STATE_READY, $instance->state());
 
         $displayName = uniqid(self::TESTING_PREFIX);
-        $processingUnits = 500;
+        $processingUnits = 1000;
         $op = $instance->update([
             'displayName' => $displayName,
             'processingUnits' => $processingUnits,