Merge pull request #53 from cleentfaar/fixed-tests

fixed tests before refactoring

tests/src/CL/Slack/Test/Model/ModelTrait.php

@@ -337,6 +337,7 @@ protected function createFile()
     protected function assertFile(array $expected, File $actual)
     {
         $this->assertNotEmpty($expected);
+
         Assert::assertInstanceOf('CL\Slack\Model\File', $actual);
         Assert::assertEquals($expected, [
             'id' => $actual->getId(),
@@ -586,12 +587,14 @@ protected function createTeam()
             'name' => 'My Team',
             'domain' => 'example.com',
             'email_domain' => 'example.com',
-            'image_34' => 'https:\/\/www.example.com/image34.jpg',
-            'image_44' => 'https:\/\/www.example.com/image44.jpg',
-            'image_68' => 'https:\/\/www.example.com/image68.jpg',
-            'image_88' => 'https:\/\/www.example.com/image88.jpg',
-            'image_102' => 'https:\/\/www.example.com/image102.jpg',
-            'image_132' => 'https:\/\/www.example.com/image132.jpg',
+            'icon' => [
+                'image_34' => 'https:\/\/www.example.com/image34.jpg',
+                'image_44' => 'https:\/\/www.example.com/image44.jpg',
+                'image_68' => 'https:\/\/www.example.com/image68.jpg',
+                'image_88' => 'https:\/\/www.example.com/image88.jpg',
+                'image_102' => 'https:\/\/www.example.com/image102.jpg',
+                'image_132' => 'https:\/\/www.example.com/image132.jpg',
+            ],
         ];
     }
 
@@ -606,12 +609,14 @@ protected function assertTeam(array $expected, Team $actual)
             'name' => $actual->getName(),
             'domain' => $actual->getDomain(),
             'email_domain' => $actual->getEmailDomain(),
-            'image_34' => $actual->getImage34(),
-            'image_44' => $actual->getImage44(),
-            'image_68' => $actual->getImage68(),
-            'image_88' => $actual->getImage88(),
-            'image_102' => $actual->getImage102(),
-            'image_132' => $actual->getImage132(),
+            'icon' => [
+                'image_34' => $actual->getIcon()['image_34'],
+                'image_44' => $actual->getIcon()['image_44'],
+                'image_68' => $actual->getIcon()['image_68'],
+                'image_88' => $actual->getIcon()['image_88'],
+                'image_102' => $actual->getIcon()['image_102'],
+                'image_132' => $actual->getIcon()['image_132'],
+            ],
         ]);
     }
 

tests/src/CL/Slack/Tests/Payload/GroupsInfoPayloadResponseTest.php

@@ -17,7 +17,7 @@
 /**
  * @author Travis Raup <[email protected]>
  */
-class GroupsInfoPayloadResponseTest extends AbstractPayloadResponseTest
+class GroupsInfoPayloadResponseTest extends AbstractPayloadResponseTestCase
 {
     /**
      * {@inheritdoc}

tests/src/CL/Slack/Tests/Payload/GroupsInfoPayloadTest.php

@@ -17,7 +17,7 @@
 /**
  * @author Travis Raup <[email protected]>
  */
-class GroupsInfoPayloadTest extends AbstractPayloadTest
+class GroupsInfoPayloadTest extends AbstractPayloadTestCase
 {
     /**
      * {@inheritdoc}

tests/src/CL/Slack/Tests/Payload/RtmStartPayloadResponseTest.php

@@ -17,7 +17,7 @@
 /**
  * @author Travis Raup <[email protected]>
  */
-class RtmStartPayloadResponseTest extends AbstractPayloadResponseTest
+class RtmStartPayloadResponseTest extends AbstractPayloadResponseTestCase
 {
     /**
      * {@inheritdoc}

tests/src/CL/Slack/Tests/Payload/RtmStartPayloadTest.php

@@ -17,7 +17,7 @@
 /**
  * @author Travis Raup <[email protected]>
  */
-class RtmStartPayloadTest extends AbstractPayloadTest
+class RtmStartPayloadTest extends AbstractPayloadTestCase
 {
     /**
      * {@inheritdoc}

tests/src/CL/Slack/Tests/Payload/TeamInfoPayloadResponseTest.php

@@ -17,7 +17,7 @@
 /**
  * @author Nic Malan <[email protected]>
  */
-class TeamInfoPayloadResponseTest extends AbstractPayloadResponseTest
+class TeamInfoPayloadResponseTest extends AbstractPayloadResponseTestCase
 {
     /**
      * {@inheritdoc}

tests/src/CL/Slack/Tests/Payload/TeamInfoPayloadTest.php

@@ -17,7 +17,7 @@
 /**
  * @author Nic Malan <[email protected]>
  */
-class TeamInfoPayloadTest extends AbstractPayloadTest
+class TeamInfoPayloadTest extends AbstractPayloadTestCase
 {
     /**
      * {@inheritdoc}

tests/src/CL/Slack/Tests/Payload/UsersAdminInvitePayloadResponseTest.php

@@ -17,7 +17,7 @@
 /**
  * @author Dries De Peuter <[email protected]>
  */
-class UsersAdminInvitePayloadResponseTest extends AbstractPayloadResponseTest
+class UsersAdminInvitePayloadResponseTest extends AbstractPayloadResponseTestCase
 {
     /**
      * {@inheritdoc}

tests/src/CL/Slack/Tests/Payload/UsersAdminInvitePayloadTest.php

@@ -17,7 +17,7 @@
 /**
  * @author Dries De Peuter <[email protected]>
  */
-class UsersAdminInvitePayloadTest extends AbstractPayloadTest
+class UsersAdminInvitePayloadTest extends AbstractPayloadTestCase
 {
     /**
      * {@inheritdoc}

tests/src/CL/Slack/Tests/Payload/UsersGetPresencePayloadResponseTest.php

@@ -17,7 +17,7 @@
 /**
  * @author Cas Leentfaar <[email protected]>
  */
-class UsersGetPresencePayloadResponseTest extends AbstractPayloadResponseTest
+class UsersGetPresencePayloadResponseTest extends AbstractPayloadResponseTestCase
 {
     /**
      * @inheritdoc

tests/src/CL/Slack/Tests/Payload/UsersGetPresencePayloadTest.php

@@ -17,7 +17,7 @@
 /**
  * @author Cas Leentfaar <[email protected]>
  */
-class UsersGetPresencePayloadTest extends AbstractPayloadTest
+class UsersGetPresencePayloadTest extends AbstractPayloadTestCase
 {
     /**
      * @inheritdoc