Set default connection settings for tests (#385)

Tigrov · web-flow · commit b9a325991288 · 2025-02-23T19:12:12.000+07:00
diff --git a/.github/workflows/build.yml b/.github/workflows/build.yml
@@ -93,12 +93,6 @@ jobs:
 
       - name: Run tests with phpunit with code coverage.
         run: vendor/bin/phpunit --coverage-clover=coverage.xml --colors=always
-        env:
-          YII_PGSQL_DATABASE: yiitest
-          YII_PGSQL_HOST: 127.0.0.1
-          YII_PGSQL_PORT: 5432
-          YII_PGSQL_USER: root
-          YII_PGSQL_PASSWORD: root
 
       - name: Upload coverage to Codecov.
         uses: codecov/codecov-action@v3
diff --git a/.github/workflows/mutation.yml b/.github/workflows/mutation.yml
@@ -79,8 +79,3 @@ jobs:
           vendor/bin/roave-infection-static-analysis-plugin --threads=2 --ignore-msi-with-no-mutations --only-covered
         env:
           STRYKER_DASHBOARD_API_KEY: ${{ secrets.STRYKER_DASHBOARD_API_KEY }}
-          YII_PGSQL_DATABASE: yiitest
-          YII_PGSQL_HOST: 127.0.0.1
-          YII_PGSQL_PORT: 5432
-          YII_PGSQL_USER: root
-          YII_PGSQL_PASSWORD: root
diff --git a/src/Schema.php b/src/Schema.php
@@ -7,6 +7,7 @@
 use JsonException;
 use Throwable;
 use Yiisoft\Db\Constant\ColumnType;
+use Yiisoft\Db\Constant\ReferentialAction;
 use Yiisoft\Db\Constraint\CheckConstraint;
 use Yiisoft\Db\Constraint\Constraint;
 use Yiisoft\Db\Constraint\DefaultValueConstraint;
@@ -832,13 +833,13 @@ private function loadTableConstraints(string $tableName, string $returnType): ar
         ORDER BY "a"."attnum" ASC, "fa"."attnum" ASC
         SQL;
 
-        /** @psalm-var string[] $actionTypes */
+        /** @psalm-var ReferentialAction::*[] $actionTypes */
         $actionTypes = [
-            'a' => 'NO ACTION',
-            'r' => 'RESTRICT',
-            'c' => 'CASCADE',
-            'n' => 'SET NULL',
-            'd' => 'SET DEFAULT',
+            'a' => ReferentialAction::NO_ACTION,
+            'r' => ReferentialAction::RESTRICT,
+            'c' => ReferentialAction::CASCADE,
+            'n' => ReferentialAction::SET_NULL,
+            'd' => ReferentialAction::SET_DEFAULT,
         ];
 
         $resolvedName = $this->resolveTableName($tableName);
diff --git a/tests/Support/TestTrait.php b/tests/Support/TestTrait.php
@@ -93,26 +93,26 @@ private function getDriver(): PdoDriverInterface
 
     private static function getDatabaseName(): string
     {
-        return getenv('YII_PGSQL_DATABASE') ?: '';
+        return getenv('YII_PGSQL_DATABASE') ?: 'yiitest';
     }
 
     private static function getHost(): string
     {
-        return getenv('YII_PGSQL_HOST') ?: '';
+        return getenv('YII_PGSQL_HOST') ?: '127.0.0.1';
     }
 
     private static function getPort(): string
     {
-        return getenv('YII_PGSQL_PORT') ?: '';
+        return getenv('YII_PGSQL_PORT') ?: '5432';
     }
 
     private static function getUsername(): string
     {
-        return getenv('YII_PGSQL_USER') ?: '';
+        return getenv('YII_PGSQL_USER') ?: 'root';
     }
 
     private static function getPassword(): string
     {
-        return getenv('YII_PGSQL_PASSWORD') ?: '';
+        return getenv('YII_PGSQL_PASSWORD') ?: 'root';
     }
 }

Original file line number	Diff line number	Diff line change
`@@ -93,26 +93,26 @@ private function getDriver(): PdoDriverInterface`
`93`	`93`
`94`	`94`	`private static function getDatabaseName(): string`
`95`	`95`	`{`
`96`		`- return getenv('YII_PGSQL_DATABASE') ?: '';`
	`96`	`+ return getenv('YII_PGSQL_DATABASE') ?: 'yiitest';`
`97`	`97`	`}`
`98`	`98`
`99`	`99`	`private static function getHost(): string`
`100`	`100`	`{`
`101`		`- return getenv('YII_PGSQL_HOST') ?: '';`
	`101`	`+ return getenv('YII_PGSQL_HOST') ?: '127.0.0.1';`
`102`	`102`	`}`
`103`	`103`
`104`	`104`	`private static function getPort(): string`
`105`	`105`	`{`
`106`		`- return getenv('YII_PGSQL_PORT') ?: '';`
	`106`	`+ return getenv('YII_PGSQL_PORT') ?: '5432';`
`107`	`107`	`}`
`108`	`108`
`109`	`109`	`private static function getUsername(): string`
`110`	`110`	`{`
`111`		`- return getenv('YII_PGSQL_USER') ?: '';`
	`111`	`+ return getenv('YII_PGSQL_USER') ?: 'root';`
`112`	`112`	`}`
`113`	`113`
`114`	`114`	`private static function getPassword(): string`
`115`	`115`	`{`
`116`		`- return getenv('YII_PGSQL_PASSWORD') ?: '';`
	`116`	`+ return getenv('YII_PGSQL_PASSWORD') ?: 'root';`
`117`	`117`	`}`
`118`	`118`	`}`