tests refactoring

Author: Triplkrypl

test/Factory/Symfony/ResponseFactoryTest.php

@@ -11,10 +11,9 @@
 final class ResponseFactoryTest extends TestCase
 {
     /**
-     * @dataProvider dataProviderMakeArray
-     *
      * @param iterable<mixed> $streamedData
      */
+    #[DataProvider('dataProviderMakeArray')]
     public function testMakeArray(string $expectedBody, iterable $streamedData): void
     {
         $response = ResponseFactory::makeArray($streamedData);

test/Model/Client/RequestTest.php

@@ -11,7 +11,7 @@ final class RequestTest extends TestCase
     public function testWithJson(): void
     {
         $request = new Request('POST', '/test');
-        $request = $request->withBaseUrl('http://host');
+        $request = $request->withBaseUrl('https://host');
         $requestWithJson = $request->withJson(['test' => 'test'], null);
 
         $factory = new HttpFactory();
@@ -23,13 +23,13 @@ public function testWithJson(): void
     public function testWithBaseUrl(): void
     {
         $request = new Request('GET', '/test', ['test' => 'test'], null, ['test' => 'test']);
-        $requestWithBaseUrl = $request->withBaseUrl('http://host');
+        $requestWithBaseUrl = $request->withBaseUrl('https://host');
         $psrRequest = $requestWithBaseUrl->createPsr(new HttpFactory());
 
         self::assertSame(false, $request->hasBaseUrl());
         self::assertSame(true, $requestWithBaseUrl->hasBaseUrl());
         self::assertSame('GET', $psrRequest->getMethod());
-        self::assertSame('http://host/test?test=test', (string) $psrRequest->getUri());
+        self::assertSame('https://host/test?test=test', (string) $psrRequest->getUri());
         self::assertSame('', $psrRequest->getBody()->getContents());
         self::assertSame(['test'], $psrRequest->getHeader('test'));
     }

test/Model/Server/HeaderRuleTest.php

@@ -2,15 +2,14 @@
 
 declare(strict_types=1);
 
+use PHPUnit\Framework\Attributes\CoversClass;
 use PHPUnit\Framework\TestCase;
 use SimpleAsFuck\ApiToolkit\Model\Server\HeaderRule;
 use SimpleAsFuck\Validator\Factory\UnexpectedValueException;
 use SimpleAsFuck\Validator\Model\Validated;
 use SimpleAsFuck\Validator\Rule\String\StringRule;
 
-/**
- * @covers \SimpleAsFuck\ApiToolkit\Model\Server\HeaderRule
- */
+#[CoversClass(HeaderRule::class)]
 final class HeaderRuleTest extends TestCase
 {
     private HeaderRule $rule;

test/Model/Server/QueryRuleTest.php

@@ -2,15 +2,13 @@
 
 declare(strict_types=1);
 
-
+use PHPUnit\Framework\Attributes\CoversClass;
 use PHPUnit\Framework\TestCase;
 use SimpleAsFuck\ApiToolkit\Model\Server\QueryRule;
-use SimpleAsFuck\Validator\Factory\UnexpectedValueException as UnexpectedValueException;
-use SimpleAsFuck\Validator\Model\Validated as Validated;
+use SimpleAsFuck\Validator\Factory\UnexpectedValueException;
+use SimpleAsFuck\Validator\Model\Validated;
 
-/**
- * @covers \SimpleAsFuck\ApiToolkit\Model\Server\QueryRule
- */
+#[CoversClass(QueryRule::class)]
 final class QueryRuleTest extends TestCase
 {
     private QueryRule $rule;

test/Service/Client/DeprecationsLoggerTest.php

@@ -3,24 +3,23 @@
 declare(strict_types=1);
 
 use GuzzleHttp\Psr7\Response;
+use PHPUnit\Framework\Attributes\CoversClass;
+use PHPUnit\Framework\Attributes\DataProvider;
 use PHPUnit\Framework\TestCase;
 use Psr\Http\Message\ResponseInterface;
 use Psr\Log\LoggerInterface;
 use SimpleAsFuck\ApiToolkit\Model\Client\Request;
 use SimpleAsFuck\ApiToolkit\Service\Client\Config;
 use SimpleAsFuck\ApiToolkit\Service\Client\DeprecationsLogger;
 
-/**
- * @covers \SimpleAsFuck\ApiToolkit\Service\Client\DeprecationsLogger
- */
+#[CoversClass(DeprecationsLogger::class)]
 final class DeprecationsLoggerTest extends TestCase
 {
     /**
-     * @dataProvider dataProviderLogDeprecation
-     *
      * @param non-empty-string|null $expectedLogMessage
      * @param array<non-empty-string, string> $expectedContext
      */
+    #[DataProvider('dataProviderLogDeprecation')]
     public function testLogDeprecation(?string $expectedLogMessage, array $expectedContext, ResponseInterface $response): void
     {
         $psrLogger = $this->createMock(LoggerInterface::class);

test/Service/Server/ExceptionTransformerTest.php

@@ -2,17 +2,15 @@
 
 declare(strict_types=1);
 
+use PHPUnit\Framework\Attributes\CoversClass;
+use PHPUnit\Framework\Attributes\DataProvider;
 use PHPUnit\Framework\TestCase;
 use SimpleAsFuck\ApiToolkit\Service\Server\ExceptionTransformer;
 
-/**
- * @covers \SimpleAsFuck\ApiToolkit\Service\Server\ExceptionTransformer
- */
+#[CoversClass(ExceptionTransformer::class)]
 final class ExceptionTransformerTest extends TestCase
 {
-    /**
-     * @dataProvider dataProviderToApi
-     */
+    #[DataProvider('dataProviderToApi')]
     public function testToApi(string $expectedMessage, bool $debug): void
     {
         $config = $this->createMock(SimpleAsFuck\ApiToolkit\Service\Config\Repository::class);

test/Service/Webhook/LaravelMysqlRepositoryTest.php

@@ -6,15 +6,15 @@
 use Illuminate\Database\ConnectionResolver;
 use Illuminate\Database\MySqlConnection;
 use Illuminate\Support\Facades\DB;
+use PHPUnit\Framework\Attributes\CoversClass;
+use PHPUnit\Framework\Attributes\DataProvider;
 use PHPUnit\Framework\TestCase;
 use SimpleAsFuck\ApiToolkit\Model\Webhook\Params;
 use SimpleAsFuck\ApiToolkit\Model\Webhook\Priority;
 use SimpleAsFuck\ApiToolkit\Model\Webhook\Webhook;
 use SimpleAsFuck\ApiToolkit\Service\Webhook\LaravelMysqlRepository;
 
-/**
- * @covers \SimpleAsFuck\ApiToolkit\Service\Webhook\LaravelMysqlRepository
- */
+#[CoversClass(LaravelMysqlRepository::class)]
 final class LaravelMysqlRepositoryTest extends TestCase
 {
     private static ConnectionResolver $connectionResolver;
@@ -61,12 +61,11 @@ public static function tearDownAfterClass(): void
     }
 
     /**
-     * @dataProvider dataProviderLoadForDispatching
-     *
      * @param array<non-empty-string> $expectedUrls
      * @param non-empty-string $type
      * @param array<non-empty-string, non-empty-string> $attributes
      */
+    #[DataProvider('dataProviderLoadForDispatching')]
     public function testLoadForDispatching(array $expectedUrls, string $type, array $attributes): void
     {
         $repository = new LaravelMysqlRepository(
@@ -159,10 +158,9 @@ public static function dataProviderLoadForDispatching(): array
     }
 
     /**
-     * @dataProvider dataProviderSaveAndDelete
-     *
      * @param non-empty-string $type
      */
+    #[DataProvider('dataProviderSaveAndDelete')]
     public function testSave(string $type, Params $params): void
     {
         $repository = new LaravelMysqlRepository(
@@ -208,10 +206,9 @@ public function testSave(string $type, Params $params): void
     }
 
     /**
-     * @dataProvider dataProviderSaveAndDelete
-     *
      * @param non-empty-string $type
      */
+    #[DataProvider('dataProviderSaveAndDelete')]
     public function testDelete(string $type, Params $params): void
     {
         $connection = self::$connectionResolver->connection('default');