chore: require to specify test coverage (#162)

Author: simPod

phpunit.xml.dist

@@ -8,6 +8,8 @@
     bootstrap="tests/bootstrap.php"
     colors="true"
     verbose="true"
+    forceCoversAnnotation="true"
+    failOnRisky="true"
 >
     <coverage>
         <include>

src/Exception/ServerError.php

@@ -11,6 +11,6 @@ final class ServerError extends Exception
 {
     public static function fromResponse(ResponseInterface $response): self
     {
-        return new self((string) $response->getBody());
+        return new self($response->getBody()->__toString());
     }
 }

tests/Client/InsertTest.php

@@ -11,6 +11,14 @@
 use SimPod\ClickHouseClient\Tests\TestCaseBase;
 use SimPod\ClickHouseClient\Tests\WithClient;
 
+/**
+ * @covers \SimPod\ClickHouseClient\Client\Http\RequestFactory
+ * @covers \SimPod\ClickHouseClient\Client\PsrClickHouseClient
+ * @covers \SimPod\ClickHouseClient\Exception\CannotInsert
+ * @covers \SimPod\ClickHouseClient\Exception\ServerError
+ * @covers \SimPod\ClickHouseClient\Format\JsonEachRow
+ * @covers \SimPod\ClickHouseClient\Format\JsonCompact
+ */
 final class InsertTest extends TestCaseBase
 {
     use WithClient;
@@ -30,8 +38,7 @@ public function testInsert(string $tableSql): void
         $output = $this->client->select(
             <<<'CLICKHOUSE'
 SELECT * FROM UserActivity
-CLICKHOUSE
-            ,
+CLICKHOUSE,
             new JsonEachRow()
         );
 
@@ -62,8 +69,7 @@ public function testInsertUseColumns(string $tableSql): void
         $output = $this->client->select(
             <<<'CLICKHOUSE'
 SELECT * FROM UserActivity
-CLICKHOUSE
-            ,
+CLICKHOUSE,
             new JsonEachRow()
         );
 
@@ -91,8 +97,7 @@ public function testInsertEscaping(): void
         $output = $this->client->select(
             <<<'CLICKHOUSE'
 SELECT * FROM a
-CLICKHOUSE
-            ,
+CLICKHOUSE,
             new JsonCompact()
         );
 

tests/Client/SelectAsyncTest.php

@@ -11,6 +11,13 @@
 use SimPod\ClickHouseClient\Tests\TestCaseBase;
 use SimPod\ClickHouseClient\Tests\WithClient;
 
+/**
+ * @covers \SimPod\ClickHouseClient\Client\Http\RequestFactory
+ * @covers \SimPod\ClickHouseClient\Client\PsrClickHouseAsyncClient
+ * @covers \SimPod\ClickHouseClient\Exception\ServerError
+ * @covers \SimPod\ClickHouseClient\Format\JsonEachRow
+ * @covers \SimPod\ClickHouseClient\Format\TabSeparated
+ */
 final class SelectAsyncTest extends TestCaseBase
 {
     use WithClient;
@@ -24,7 +31,8 @@ public function testAsyncSelect(): void
 CLICKHOUSE;
 
         /** @var JsonEachRow<array{number: string}> $format */
-        $format   = new JsonEachRow();
+        $format = new JsonEachRow();
+
         $promises = [
             $client->select($sql, $format),
             $client->select($sql, $format),

tests/Client/SelectTest.php

@@ -11,6 +11,14 @@
 use SimPod\ClickHouseClient\Tests\TestCaseBase;
 use SimPod\ClickHouseClient\Tests\WithClient;
 
+/**
+ * @covers \SimPod\ClickHouseClient\Client\Http\RequestFactory
+ * @covers \SimPod\ClickHouseClient\Client\PsrClickHouseClient
+ * @covers \SimPod\ClickHouseClient\Exception\ServerError
+ * @covers \SimPod\ClickHouseClient\Format\Json
+ * @covers \SimPod\ClickHouseClient\Format\JsonEachRow
+ * @covers \SimPod\ClickHouseClient\Format\JsonCompact
+ */
 final class SelectTest extends TestCaseBase
 {
     use WithClient;

tests/Logger/LoggerChainTest.php

@@ -4,11 +4,12 @@
 
 namespace SimPod\ClickHouseClient\Tests\Logger;
 
-use PHPUnit\Framework\TestCase;
 use SimPod\ClickHouseClient\Logger\LoggerChain;
 use SimPod\ClickHouseClient\Logger\SqlLogger;
+use SimPod\ClickHouseClient\Tests\TestCaseBase;
 
-final class LoggerChainTest extends TestCase
+/** @covers \SimPod\ClickHouseClient\Logger\LoggerChain */
+final class LoggerChainTest extends TestCaseBase
 {
     public function testLog(): void
     {

tests/Output/JsonEachRowTest.php

@@ -0,0 +1,25 @@
+<?php
+
+declare(strict_types=1);
+
+namespace SimPod\ClickHouseClient\Tests\Output;
+
+use SimPod\ClickHouseClient\Output\JsonEachRow;
+use SimPod\ClickHouseClient\Tests\TestCaseBase;
+
+/** @covers \SimPod\ClickHouseClient\Output\JsonEachRow */
+final class JsonEachRowTest extends TestCaseBase
+{
+    public function testPropertiesAreSet(): void
+    {
+        $format = new JsonEachRow(
+            <<<'JSON'
+{"number":"0"}
+{"number":"1"}
+
+JSON
+        );
+
+        self::assertSame([['number' => '0'], ['number' => '1']], $format->data);
+    }
+}