Improve the tests and prepare release 4.1.0 (#106)

* Add a test for #105, fix CS, and skip Slack tests when misconfigured

* Prepare release 4.1.0

* Fix tests for PHP 7.2 and lowest deps compatibility

CHANGELOG.md

@@ -2,7 +2,11 @@
 
 ## Unreleased
 
+## 4.1.0 (2021-02-26)
+
 * **Specification override** Fix some `timestamp` from integer to mixed string and integer to support old messages
+* **Specification override** Change some `latest` / `oldest` / `ts_from` / `ts_to` query parameters from integer to string
+* Improve carbon footprint by removing unnecessary files from git export 
 
 ## 4.0.0 (2020-12-28)
 

src/Checker/JsonSorter.php

@@ -21,7 +21,7 @@ public function sort(string $content, bool $prettyPrint = true): string
         $content = $this->mergeNodes($content);
         $content = $this->recursiveAlphabeticalSort($content);
 
-        return json_encode($content, $prettyPrint ? JSON_PRETTY_PRINT | JSON_UNESCAPED_SLASHES : JSON_UNESCAPED_SLASHES);
+        return json_encode($content, $prettyPrint ? \JSON_PRETTY_PRINT | \JSON_UNESCAPED_SLASHES : \JSON_UNESCAPED_SLASHES);
     }
 
     public function recursiveAlphabeticalSort($item)
@@ -44,7 +44,7 @@ public function recursiveAlphabeticalSort($item)
             }
 
             $item = $asArray;
-            ksort($item, SORT_STRING);
+            ksort($item, \SORT_STRING);
 
             return $item;
         } elseif (!\is_array($item)) {
@@ -58,7 +58,7 @@ public function recursiveAlphabeticalSort($item)
         $types = array_unique(array_map('getType', $item));
 
         if (1 === \count($types) && \in_array($types[0], ['string', 'int'], true)) {
-            sort($item, SORT_STRING);
+            sort($item, \SORT_STRING);
         }
 
         return $item;

tests/ReadingTest.php

@@ -25,6 +25,13 @@
 
 class ReadingTest extends TestCase
 {
+    protected function setUp(): void
+    {
+        if (!\array_key_exists('SLACK_TOKEN', $_SERVER)) {
+            $this->markTestSkipped('SLACK_TOKEN env var not present, skip the test.');
+        }
+    }
+
     public function testItWorksOnTestSuccess()
     {
         $client = ClientFactory::create('');
@@ -74,15 +81,26 @@ public function testItCanReadAConversationHistory()
 
         $results = $client->conversationsHistory([
             'channel' => $_SERVER['SLACK_TEST_CHANNEL'],
+            'oldest' => (string) strtotime('10 September 2000'), // test #105
         ]);
 
         self::assertInstanceOf(ConversationsHistoryGetResponse200::class, $results);
 
+        $hadAFileMessage = false;
         foreach ($results->getMessages() as $message) {
             if ($message->getFiles()) {
+                $hadAFileMessage = true;
                 self::assertInstanceOf(ObjsFile::class, $message->getFiles()[0]);
+
+                if (method_exists($this, 'assertIsString')) {
+                    self::assertIsString($message->getTs());
+                } else {
+                    self::assertInternalType('string', $message->getTs());
+                }
             }
         }
+
+        $this->assertTrue($hadAFileMessage, 'We expect a message in File in the history, cf \JoliCode\Slack\Tests\WritingTest::testItCanUploadFile');
     }
 
     public function testItCanGetTheImList()

tests/UserInfoTest.php

@@ -19,6 +19,13 @@
 
 class UserInfoTest extends TestCase
 {
+    protected function setUp(): void
+    {
+        if (!\array_key_exists('SLACK_TOKEN', $_SERVER)) {
+            $this->markTestSkipped('SLACK_TOKEN env var not present, skip the test.');
+        }
+    }
+
     public function testItCanFetchUserInfo()
     {
         $client = ClientFactory::create($_SERVER['SLACK_TOKEN']);

tests/WritingTest.php

@@ -21,6 +21,13 @@
 
 class WritingTest extends TestCase
 {
+    protected function setUp(): void
+    {
+        if (!\array_key_exists('SLACK_TOKEN', $_SERVER)) {
+            $this->markTestSkipped('SLACK_TOKEN env var not present, skip the test.');
+        }
+    }
+
     public function testItCanPostAttachment()
     {
         $client = ClientFactory::create($_SERVER['SLACK_TOKEN']);