DbConnection::check时，验证闲置时间是否超过最大值 (swoft-cloud/swoft-component#189)

* 验证空闲时间是否超过最大值

* 验证空闲时间是否超过最大值

* 验证空闲时间是否超过最大值

* 完善对应单测

* 抽离验证空闲时间是否超过最大值的逻辑

* 代码格式化

* 抽离验证空闲时间是否超过最大值的逻辑

* Update AbstractDbConnection.php

Author: limingxinleo

src/AbstractDbConnection.php

@@ -97,4 +97,14 @@ protected function pushSqlToStack(string $sql)
 
         RequestContext::setContextDataByKey($contextSqlKey, $stack);
     }
+
+    /**
+     * Verify whether the idle time exceeds the maximum value.
+     */
+    protected function isIdleTimeOut(): bool
+    {
+        $idleTime = time() - $this->getLastTime();
+        $maxIdleTime = $this->getPool()->getPoolConfig()->getMaxIdleTime();
+        return $idleTime > $maxIdleTime;
+    }
 }

src/Driver/Mysql/MysqlConnection.php

@@ -205,6 +205,9 @@ public function reconnect()
      */
     public function check(): bool
     {
+        if ($this->isIdleTimeOut()) {
+            return false;
+        }
         return $this->connection->connected;
     }
 

src/Driver/Mysql/SyncMysqlConnection.php

@@ -124,6 +124,10 @@ public function reconnect()
      */
     public function check(): bool
     {
+        if ($this->isIdleTimeOut()) {
+            return false;
+        }
+        
         try {
             $this->connection->getAttribute(\PDO::ATTR_SERVER_INFO);
         } catch (\Throwable $e) {

test/Cases/PoolTest.php

@@ -14,6 +14,7 @@
 use SwoftTest\Db\Testing\Pool\DbPptPoolConfig;
 use SwoftTest\Db\Testing\Pool\DbSlaveEnvPoolConfig;
 use SwoftTest\Db\Testing\Pool\DbSlavePptConfig;
+use SwoftTest\Db\Testing\Pool\OtherDbPool;
 
 /**
  * PoolTest
@@ -89,4 +90,37 @@ public function testDbSlaveEnv()
         $this->assertEquals($pConfig->isUseProvider(), false);
         $this->assertEquals($pConfig->getMaxWait(), 10);
     }
+
+    public function testMaxIdleTime()
+    {
+        $pool = App::getPool('idle.master');
+        $connection = $pool->getConnection();
+        $this->assertTrue($connection->check());
+        $connection->release(true);
+        $connection2 = $pool->getConnection();
+        $this->assertTrue($connection2->check());
+        $connection2->release(true);
+    }
+
+    public function testMaxIdleTimeByCo()
+    {
+        go(function () {
+            $this->testMaxIdleTime();
+
+            $pool = App::getPool('idle.master');
+            $connection = $pool->getConnection();
+            $this->assertTrue($connection->check());
+            $connection->release(true);
+
+            \co::sleep(2);
+
+            $connection2 = $pool->getConnection();
+            $this->assertFalse($connection2->check());
+            $connection2->release(true);
+
+            $connection3 = $pool->getConnection();
+            $this->assertTrue($connection3->check());
+            $connection2->release(true);
+        });
+    }
 }

test/Testing/Pool/MaxIdleTimeDbConfig.php

@@ -0,0 +1,33 @@
+<?php
+/**
+ * This file is part of Swoft.
+ *
+ * @link     https://swoft.org
+ * @document https://doc.swoft.org
+ * @contact  group@swoft.org
+ * @license  https://github.com/swoft-cloud/swoft/blob/master/LICENSE
+ */
+namespace SwoftTest\Db\Testing\Pool;
+
+use Swoft\Bean\Annotation\Bean;
+use Swoft\Bean\Annotation\Value;
+use Swoft\Db\Driver\Driver;
+use Swoft\Db\Pool\Config\DbPoolProperties;
+
+/**
+ * OtherDbConfig
+ *
+ * @Bean()
+ */
+class MaxIdleTimeDbConfig extends OtherDbConfig
+{
+    /**
+     * @var string
+     */
+    protected $name = 'idle.master';
+
+    /**
+     * @var int
+     */
+    protected $maxIdleTime = 1;
+}

test/Testing/Pool/MaxIdleTimeDbPool.php

@@ -0,0 +1,28 @@
+<?php
+/**
+ * This file is part of Swoft.
+ *
+ * @link     https://swoft.org
+ * @document https://doc.swoft.org
+ * @contact  group@swoft.org
+ * @license  https://github.com/swoft-cloud/swoft/blob/master/LICENSE
+ */
+namespace SwoftTest\Db\Testing\Pool;
+
+use Swoft\Bean\Annotation\Inject;
+use Swoft\Bean\Annotation\Pool;
+use Swoft\Db\Pool\DbPool;
+
+/**
+ * OtherDbPool
+ *
+ * @Pool("idle.master")
+ */
+class MaxIdleTimeDbPool extends DbPool
+{
+    /**
+     * @Inject()
+     * @var MaxIdleTimeDbConfig
+     */
+    protected $poolConfig;
+}