Fixed bug that PostgresSQL can't work when create connection timed ou…

…t. (#5327)



Co-authored-by: 李铭昕 <[email protected]>

src/Connectors/PostgresSqlSwooleExtConnector.php

@@ -57,7 +57,7 @@ public function createConnection(array $config): PostgreSQL
         ));
 
         if ($result === false) {
-            throw new Exception($connection->error);
+            throw new Exception($connection->error ?? 'Create connection failed, Please check the database configuration.');
         }
 
         return $connection;

tests/Cases/PostgreSqlSwooleExtConnectionTest.php

@@ -11,6 +11,7 @@
  */
 namespace HyperfTest\Database\PgSQL\Cases;
 
+use Exception;
 use Hyperf\Database\Connection;
 use Hyperf\Database\Connectors\ConnectionFactory;
 use Hyperf\Database\Exception\QueryException;
@@ -89,4 +90,25 @@ public function testAffectingStatementWithWrongSql()
 
         $connection->affectingStatement('UPDATE xx SET x = 1 WHERE id = 1');
     }
+
+    public function testCreateConnectionTimedOut()
+    {
+        if (SWOOLE_MAJOR_VERSION < 5) {
+            $this->markTestSkipped('PostgreSql requires Swoole version >= 5.0.0');
+        }
+
+        $connection = $this->connectionFactory->make([
+            'driver' => 'pgsql-swoole',
+            'host' => 'non-existent-host.internal',
+            'port' => 5432,
+            'database' => 'postgres',
+            'username' => 'postgres',
+            'password' => 'postgres',
+        ]);
+
+        $this->expectException(Exception::class);
+        $this->expectExceptionMessage('Create connection failed, Please check the database configuration.');
+
+        $connection->affectingStatement('UPDATE xx SET x = 1 WHERE id = 1');
+    }
 }