Fix setTimeout tests

Author: ig0ist

src/Client.php

@@ -164,7 +164,7 @@ public function addQueryDegeneration(Degeneration $degeneration)
      *
      * @return bool
      */
-    public function enableQueryConditions()
+    public function enableQueryConditions(): bool
     {
         return $this->transport->addQueryDegeneration(new Conditions());
     }
@@ -174,18 +174,20 @@ public function enableQueryConditions()
      *
      * @param string $host
      */
-    public function setHost($host)
+    public function setHost($host): void
     {
         $this->connectHost = $host;
         $this->transport()->setHost($host);
     }
 
     /**
+     * max_execution_time , in int value (seconds)
+     *
      * @return Settings
      */
-    public function setTimeout(int $timeout)
+    public function setTimeout(int|float $timeout):Settings
     {
-        return $this->settings()->max_execution_time($timeout);
+        return $this->settings()->max_execution_time(intval($timeout));
     }
 
     /**
@@ -199,7 +201,7 @@ public function getTimeout(): int
     /**
      * ConnectTimeOut in seconds ( support 1.5 = 1500ms )
      */
-    public function setConnectTimeOut(float $connectTimeOut)
+    public function setConnectTimeOut(float $connectTimeOut): void
     {
         $this->transport()->setConnectTimeOut($connectTimeOut);
     }
@@ -215,7 +217,7 @@ public function getConnectTimeOut(): float
     /**
      * @return Http
      */
-    public function transport()
+    public function transport(): Http
     {
         if (!$this->transport) {
             throw  new \InvalidArgumentException('Empty transport class');
@@ -227,23 +229,23 @@ public function transport()
     /**
      * @return string
      */
-    public function getConnectHost()
+    public function getConnectHost(): string
     {
         return $this->connectHost;
     }
 
     /**
      * @return string
      */
-    public function getConnectPassword()
+    public function getConnectPassword(): string
     {
         return $this->connectPassword;
     }
 
     /**
      * @return string
      */
-    public function getConnectPort()
+    public function getConnectPort(): string
     {
         return $this->connectPort;
     }

tests/FormatQueryTest.php

@@ -62,7 +62,12 @@ public function testClientTimeoutSettings()
         $this->client->database('default');
 
         $timeout = 0.55; // un support, "clickhouse source - Seconds"
-        $this->client->setTimeout($timeout);      // 1500 ms
+        $this->client->setTimeout($timeout);      // 550 ms
+        $this->client->select('SELECT 123,123 as ping ')->rows();
+        $this->assertSame(intval($timeout), intval($this->client->getTimeout()));
+
+        $timeout = 2.55; // un support, "clickhouse source - Seconds"
+        $this->client->setTimeout($timeout);      // 550 ms
         $this->client->select('SELECT 123,123 as ping ')->rows();
         $this->assertSame(intval($timeout), intval($this->client->getTimeout()));
 
@@ -76,7 +81,10 @@ public function testClientTimeoutSettings()
         $timeout = 5.14;
         $this->client->setConnectTimeOut($timeout);      // 5 seconds
         $this->client->select('SELECT 123,123 as ping ')->rows();
+
+
         $this->assertSame(5.14, $this->client->getConnectTimeOut());
+
     }