Fix cs

Author: polidog

src/Api/IncomingRequests.php

@@ -45,7 +45,7 @@ public function accept($requestId)
         );
     }
 
-    public function reject($requestId)
+    public function reject($requestId): void
     {
         $this->client->delete("incoming_requests/{$requestId}");
     }

src/Api/Rooms.php

@@ -17,8 +17,8 @@
 
 class Rooms
 {
-    const ACTION_TYPE_LEAVE = 'leave';
-    const ACTION_TYPE_DELETE = 'delete';
+    public const ACTION_TYPE_LEAVE = 'leave';
+    public const ACTION_TYPE_DELETE = 'delete';
 
     /**
      * @var ClientInterface
@@ -93,7 +93,7 @@ public function create(Room $room, MemberCollection $members)
      *
      * @param Room $room
      */
-    public function update(Room $room)
+    public function update(Room $room): void
     {
         $this->client->put(
             "rooms/{$room->roomId}",
@@ -109,7 +109,7 @@ public function update(Room $room)
      *
      * @throws InvalidArgumentException
      */
-    public function remove(Room $room, $actionType)
+    public function remove(Room $room, $actionType): void
     {
         if ('leave' !== $actionType && 'delete' !== $actionType) {
             throw new InvalidArgumentException('ActionType is only leave or delete');

src/Api/Rooms/Files.php

@@ -74,12 +74,12 @@ public function show($accountId = null)
     public function detail($id, $downloadLink = false)
     {
         return $this->factory->entity(
-          $this->client->get(
-              "rooms/{$this->roomId}/files/{$id}",
-              [
-                  'create_download_url' => (int) $downloadLink,
-              ]
-          )
+            $this->client->get(
+                "rooms/{$this->roomId}/files/{$id}",
+                [
+                    'create_download_url' => (int) $downloadLink,
+                ]
+            )
         );
     }
 }

src/Api/Rooms/Members.php

@@ -55,7 +55,7 @@ public function show()
     /**
      * @param MemberCollection $members
      */
-    public function update(MemberCollection $members)
+    public function update(MemberCollection $members): void
     {
         $options = [
             'members_admin_ids' => implode(',', $members->getAdminIds()),

src/Api/Rooms/Messages.php

@@ -82,7 +82,7 @@ public function detail($id, $force = false)
     /**
      * @param Message $message
      */
-    public function create(Message $message)
+    public function create(Message $message): void
     {
         $result = $this->client->post(
             "rooms/{$this->roomId}/messages",
@@ -98,7 +98,7 @@ public function create(Message $message)
      * @param Message $message
      * @param $id
      */
-    public function update(Message $message, $id)
+    public function update(Message $message, $id): void
     {
         $result = $this->client->put(
             "rooms/{$this->roomId}/messages/{$id}",
@@ -113,7 +113,7 @@ public function update(Message $message, $id)
     /**
      * @param $id
      */
-    public function delete($id)
+    public function delete($id): void
     {
         $result = $this->client->delete(
             "rooms/{$this->roomId}/messages/{$id}"

src/Client/Client.php

@@ -31,9 +31,7 @@ public function __construct(
         string $chatworkToken,
         string $apiVersion
     ) {
-        $httpClient->getConfig('handler')->push(Middleware::mapRequest(function (RequestInterface $request) use ($chatworkToken) {
-            return $request->withHeader('X-ChatWorkToken', $chatworkToken);
-        }));
+        $httpClient->getConfig('handler')->push(Middleware::mapRequest(fn (RequestInterface $request) => $request->withHeader('X-ChatWorkToken', $chatworkToken)));
         $this->apiVersion = $apiVersion;
         $this->httpClient = $httpClient;
     }

tests/Api/ContactsTest.php

@@ -1,4 +1,6 @@
 <?php
+
+declare(strict_types=1);
 /**
  * Created by PhpStorm.
  * User: polidog
@@ -19,7 +21,7 @@ class ContactsTest extends TestCase
     /**
      * @dataProvider providerResponseData
      */
-    public function testShow($apiResult)
+    public function testShow($apiResult): void
     {
         $client = $this->prophesize(ClientInterface::class);
         $client->get('contacts')

tests/Api/IncomingRequestsTest.php

@@ -1,9 +1,9 @@
 <?php
 
+declare(strict_types=1);
 
 namespace Polidog\Chatwork\Api;
 
-
 use PHPUnit\Framework\TestCase;
 use Polidog\Chatwork\Client\ClientInterface;
 use Polidog\Chatwork\Entity\Collection\EntityCollection;
@@ -15,7 +15,7 @@ class IncomingRequestsTest extends TestCase
     /**
      * @dataProvider providerIncomingRequests
      */
-    public function testShow($apiResults)
+    public function testShow($apiResults): void
     {
         $client = $this->prophesize(ClientInterface::class);
         $factory = new IncomingRequestsFactory();
@@ -24,7 +24,7 @@ public function testShow($apiResults)
             ->willReturn($apiResults);
 
         $api = new IncomingRequests($client->reveal(), $factory);
-        $incomingRequests =$api->show();
+        $incomingRequests = $api->show();
         $this->assertInstanceOf(EntityCollection::class, $incomingRequests);
         foreach ($incomingRequests as $incomingRequest) {
             $this->assertInstanceOf(IncomingRequest::class, $incomingRequest);
@@ -35,7 +35,7 @@ public function testShow($apiResults)
     /**
      * @dataProvider providerIncomingRequestPut
      */
-    public function testAccept($apiResults)
+    public function testAccept($apiResults): void
     {
         $requestId = 1;
 
@@ -46,7 +46,7 @@ public function testAccept($apiResults)
             ->willReturn($apiResults);
 
         $api = new IncomingRequests($client->reveal(), $factory);
-        $incomingRequest =$api->accept($requestId);
+        $incomingRequest = $api->accept($requestId);
         $this->assertInstanceOf(IncomingRequest::class, $incomingRequest);
 
     }

tests/Api/MeTest.php

@@ -1,5 +1,7 @@
 <?php
 
+declare(strict_types=1);
+
 namespace Polidog\Chatwork\Api;
 
 use PHPUnit\Framework\TestCase;
@@ -12,7 +14,7 @@ class MeTest extends TestCase
     /**
      * @dataProvider providerResponseData
      */
-    public function testShow($apiResult)
+    public function testShow($apiResult): void
     {
         $client = $this->prophesize(ClientInterface::class);
         $client->get('me')

tests/Api/My/StatusTest.php

@@ -1,5 +1,7 @@
 <?php
 
+declare(strict_types=1);
+
 namespace Polidog\Chatwork\Api\My;
 
 use PHPUnit\Framework\TestCase;
@@ -11,7 +13,7 @@ class StatusTest extends TestCase
     /**
      * @dataProvider providerResponseData
      */
-    public function testShow($apiResult)
+    public function testShow($apiResult): void
     {
         $client = $this->prophesize(ClientInterface::class);
         $client->get('my/status')