Skip to content

Commit f164990

Browse files
author
Alexandru Bucur
authored
Merge pull request #158 from lucasmichot/feature/class-notation
Always use ::class notation.
2 parents 5a41bec + 5911958 commit f164990

File tree

2 files changed

+26
-13
lines changed

2 files changed

+26
-13
lines changed

tests/Integration/BaseTest.php

Lines changed: 24 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,18 @@
33
namespace Tests\Integration;
44

55
use PHPUnit\Framework\TestCase;
6+
use MessageBird\Resources\Chat\Contact;
7+
use MessageBird\Resources\Chat\Channel;
8+
use MessageBird\Resources\Chat\Platform;
9+
use MessageBird\Resources\Chat\Message;
10+
use MessageBird\Resources\Verify;
11+
use MessageBird\Resources\VoiceMessage;
12+
use MessageBird\Resources\Groups;
13+
use MessageBird\Resources\Contacts;
14+
use MessageBird\Resources\Messages;
15+
use MessageBird\Resources\Hlr;
16+
use MessageBird\Resources\Balance;
17+
use MessageBird\Common\HttpClient;
618

719
class BaseTest extends TestCase
820
{
@@ -16,7 +28,7 @@ protected function setUp()
1628
{
1729
parent::setUp();
1830

19-
$this->mockClient = $this->getMockBuilder("\MessageBird\Common\HttpClient")->setConstructorArgs(["fake.messagebird.dev"])->getMock();
31+
$this->mockClient = $this->getMockBuilder(HttpClient::class)->setConstructorArgs(["fake.messagebird.dev"])->getMock();
2032
$this->client = new \MessageBird\Client('YOUR_ACCESS_KEY', $this->mockClient);
2133
}
2234

@@ -33,17 +45,17 @@ protected function doAssertionToNotBeConsideredRiskyTest()
3345
public function testClientConstructor()
3446
{
3547
$messageBird = new \MessageBird\Client('YOUR_ACCESS_KEY');
36-
$this->assertInstanceOf('MessageBird\Resources\Balance', $messageBird->balance);
37-
$this->assertInstanceOf('MessageBird\Resources\Hlr', $messageBird->hlr);
38-
$this->assertInstanceOf('MessageBird\Resources\Messages', $messageBird->messages);
39-
$this->assertInstanceOf('MessageBird\Resources\Contacts', $messageBird->contacts);
40-
$this->assertInstanceOf('MessageBird\Resources\Groups', $messageBird->groups);
41-
$this->assertInstanceOf('MessageBird\Resources\VoiceMessage', $messageBird->voicemessages);
42-
$this->assertInstanceOf('MessageBird\Resources\Verify', $messageBird->verify);
43-
$this->assertInstanceOf('MessageBird\Resources\Chat\Message', $messageBird->chatMessages);
44-
$this->assertInstanceOf('MessageBird\Resources\Chat\Platform', $messageBird->chatPlatforms);
45-
$this->assertInstanceOf('MessageBird\Resources\Chat\Channel', $messageBird->chatChannels);
46-
$this->assertInstanceOf('MessageBird\Resources\Chat\Contact', $messageBird->chatContacts);
48+
$this->assertInstanceOf(Balance::class, $messageBird->balance);
49+
$this->assertInstanceOf(Hlr::class, $messageBird->hlr);
50+
$this->assertInstanceOf(Messages::class, $messageBird->messages);
51+
$this->assertInstanceOf(Contacts::class, $messageBird->contacts);
52+
$this->assertInstanceOf(Groups::class, $messageBird->groups);
53+
$this->assertInstanceOf(VoiceMessage::class, $messageBird->voicemessages);
54+
$this->assertInstanceOf(Verify::class, $messageBird->verify);
55+
$this->assertInstanceOf(Message::class, $messageBird->chatMessages);
56+
$this->assertInstanceOf(Platform::class, $messageBird->chatPlatforms);
57+
$this->assertInstanceOf(Channel::class, $messageBird->chatChannels);
58+
$this->assertInstanceOf(Contact::class, $messageBird->chatContacts);
4759
}
4860

4961
public function testHttpClientMock()

tests/Integration/Contacts/ContactTest.php

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,7 @@
33
namespace Tests\Integration\Contacts;
44

55
use Tests\Integration\BaseTest;
6+
use MessageBird\Objects\Message;
67

78
class ContactTest extends BaseTest
89
{
@@ -121,7 +122,7 @@ public function testContactGetMessages()
121122

122123
$messages = $this->client->contacts->getMessages("contact_id");
123124
foreach($messages->items as $message) {
124-
$this->assertInstanceOf('\MessageBird\Objects\Message', $message);
125+
$this->assertInstanceOf(Message::class, $message);
125126
}
126127
}
127128
}

0 commit comments

Comments
 (0)