Allow PHP 7.1 risky fixer

Author: pyrech

.php-cs-fixer.php

@@ -20,6 +20,7 @@
         '@Symfony' => true,
         '@Symfony:risky' => true,
         '@PHP71Migration' => true,
+        '@PHP71Migration:risky' => true,
         'header_comment' => array('header' => $header),
         'strict_param' => true,
         'array_syntax' => ['syntax' => 'short'],

src/Command/CheckerCommand.php

@@ -23,7 +23,7 @@ class CheckerCommand extends Command
 {
     protected static $defaultName = 'checker';
 
-    protected function configure()
+    protected function configure(): void
     {
         $this
             ->setDescription('Checks the patched file and displays a summary table.')

src/Command/GeneratePatchCommand.php

@@ -25,7 +25,7 @@ class GeneratePatchCommand extends Command
 {
     protected static $defaultName = 'spec:generate-patch';
 
-    protected function configure()
+    protected function configure(): void
     {
         $this
             ->setDescription('Generates the patch by comparing Slack\'s official sorted spec to the currently patched one.')

src/Command/UpdateSpecificationCommand.php

@@ -26,7 +26,7 @@ class UpdateSpecificationCommand extends Command
 {
     protected static $defaultName = 'spec:update';
 
-    protected function configure()
+    protected function configure(): void
     {
         $this
             ->setDescription('Downloads a new version of the official Slack OpenAPI file, and applies our patches to it.')

tests/ApiTest.php

@@ -20,7 +20,7 @@
 
 class ApiTest extends TestCase
 {
-    public function testItWorksOnTestSuccess()
+    public function testItWorksOnTestSuccess(): void
     {
         $client = ClientFactory::create('');
         $response = $client->apiTest();
@@ -29,7 +29,7 @@ public function testItWorksOnTestSuccess()
         self::assertTrue($response->getOk());
     }
 
-    public function testItThrowsExceptionOnTestError()
+    public function testItThrowsExceptionOnTestError(): void
     {
         $client = ClientFactory::create('');
 

tests/ClientTest.php

@@ -17,7 +17,7 @@
 
 class ClientTest extends SlackTokenDependentTest
 {
-    public function testItCanIterate()
+    public function testItCanIterate(): void
     {
         $client = $this->createClient();
 
@@ -33,7 +33,7 @@ public function testItCanIterate()
         self::assertInstanceOf(ObjsUser::class, $users[0]);
     }
 
-    public function testItThrowsExceptionOnUnknownIterate()
+    public function testItThrowsExceptionOnUnknownIterate(): void
     {
         $client = $this->createClient();
 
@@ -43,7 +43,7 @@ public function testItThrowsExceptionOnUnknownIterate()
         $client->iterateFooBar();
     }
 
-    public function testItThrowsExceptionOnUnknownMethod()
+    public function testItThrowsExceptionOnUnknownMethod(): void
     {
         $client = $this->createClient();
 

tests/JsonSorterTest.php

@@ -28,7 +28,7 @@ protected function setUp(): void
     /**
      * @dataProvider provideJsonStrings
      */
-    public function testItCanSortJsonString(string $source, string $expectedResult)
+    public function testItCanSortJsonString(string $source, string $expectedResult): void
     {
         $this->sorter->sort($source);
         $this->assertSame(

tests/ReadingTest.php

@@ -23,7 +23,7 @@
 
 class ReadingTest extends SlackTokenDependentTest
 {
-    public function testItWorksOnUserListWithCorrectToken()
+    public function testItWorksOnUserListWithCorrectToken(): void
     {
         $client = $this->createClient();
 
@@ -35,7 +35,7 @@ public function testItWorksOnUserListWithCorrectToken()
         self::assertGreaterThan(2, \count($response->getMembers()));
     }
 
-    public function testItThrowsExceptionOnUserListWithoutToken()
+    public function testItThrowsExceptionOnUserListWithoutToken(): void
     {
         $client = $this->createClient('');
 
@@ -45,7 +45,7 @@ public function testItThrowsExceptionOnUserListWithoutToken()
         $client->usersList();
     }
 
-    public function testItCanReadAConversationHistory()
+    public function testItCanReadAConversationHistory(): void
     {
         $client = $this->createClient();
 
@@ -73,7 +73,7 @@ public function testItCanReadAConversationHistory()
         $this->assertTrue($hadAFileMessage, 'We expect a message in File in the history, cf \JoliCode\Slack\Tests\WritingTest::testItCanUploadFile');
     }
 
-    public function testItCanGetTheImList()
+    public function testItCanGetTheImList(): void
     {
         $client = $this->createClient();
 
@@ -83,7 +83,7 @@ public function testItCanGetTheImList()
         self::assertNotEmpty($results->getChannels());
     }
 
-    public function testItCanReadConversationsAndHydrateThem()
+    public function testItCanReadConversationsAndHydrateThem(): void
     {
         $client = $this->createClient();
 
@@ -101,7 +101,7 @@ public function testItCanReadConversationsAndHydrateThem()
         }
     }
 
-    public function testItCanGetConversationLocale()
+    public function testItCanGetConversationLocale(): void
     {
         $client = $this->createClient();
 
@@ -118,7 +118,7 @@ public function testItCanGetConversationLocale()
         $this->assertNotNull($channel->getLocale());
     }
 
-    public function testItCanSearchMessages()
+    public function testItCanSearchMessages(): void
     {
         $client = $this->createClient();
 

tests/UserInfoTest.php

@@ -17,7 +17,7 @@
 
 class UserInfoTest extends SlackTokenDependentTest
 {
-    public function testItCanFetchUserInfo()
+    public function testItCanFetchUserInfo(): void
     {
         $client = $this->createClient();
 

tests/WritingTest.php

@@ -19,7 +19,7 @@
 
 class WritingTest extends SlackTokenDependentTest
 {
-    public function testItCanPostAttachment()
+    public function testItCanPostAttachment(): void
     {
         $client = $this->createClient();
 
@@ -42,7 +42,7 @@ public function testItCanPostAttachment()
         }
     }
 
-    public function testItCanPostMessageWithBlock()
+    public function testItCanPostMessageWithBlock(): void
     {
         $client = $this->createClient();
 
@@ -77,7 +77,7 @@ public function testItCanPostMessageWithBlock()
         self::assertNotEmpty($response->getMessage()->getBlocks());
     }
 
-    public function testItCanPostAMessageAndThenAThreadResponse()
+    public function testItCanPostAMessageAndThenAThreadResponse(): void
     {
         $client = $this->createClient();
 
@@ -98,7 +98,7 @@ public function testItCanPostAMessageAndThenAThreadResponse()
         $this->assertTrue($response2->getOk());
     }
 
-    public function testItCanUploadFile()
+    public function testItCanUploadFile(): void
     {
         $client = $this->createClient();
 
@@ -122,7 +122,7 @@ public function testItCanUploadFile()
         }
     }
 
-    public function testScheduleMessage()
+    public function testScheduleMessage(): void
     {
         $client = $this->createClient();
         $futureTs = (new \DateTime('+1 hour'))->getTimestamp();
@@ -136,7 +136,7 @@ public function testScheduleMessage()
         $this->assertTrue($response->getOk());
     }
 
-    public function testItCanMarkConversation()
+    public function testItCanMarkConversation(): void
     {
         $client = $this->createClient();