NotificationChannels\FortySixElks\Test\TestNotification

Complexity

Total Complexity

2

Size/Duplication

Total Lines	10
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	3	1
A	to46Elks()	0	3	1

<?php

namespace NotificationChannels\FortySixElks\Test;

use Illuminate\Notifications\Notifiable;
use Illuminate\Notifications\Notification;
use Mockery;
use NotificationChannels\FortySixElks\FortySixElksChannel;
use NotificationChannels\FortySixElks\FortySixElksSMS;

class FortySixElksChannelTest extends \PHPUnit_Framework_TestCase
{
    protected $dispatcher;

    protected $channel;

    protected $notifiable;

    protected $notification;

    protected $smsMessage;

    public function setUp()
    {
        parent::setUp();
        $this->dispatcher = new \Illuminate\Events\Dispatcher();

        $this->channel = new FortySixElksChannel($this->dispatcher);

        $this->smsMessage = Mockery::mock(FortySixElksSMS::class);

        $this->notifiable = new TestNotifiable();

        $this->notification = new TestNotification($this->smsMessage);
    }

    public function testItCanBeInstantiatedTest()
    {
        $this->assertInstanceOf(FortySixElksChannel::class, $this->channel);
    }

    public function testCanSendNotification()
    {
        $this->smsMessage->shouldReceive('send')
            ->once()
            ->andReturn($this->smsMessage);

        $this->notification->to46Elks($this->notifiable);
        $response = $this->channel->send($this->notifiable, $this->notification);

        $this->assertInstanceOf(FortySixElksSMS::class, $response);
    }
}

class TestNotifiable
{
    use Notifiable;

The trait Illuminate\Notifications\Notifiable requires some properties which are not provided by NotificationChannels\FortySixElks\Test\TestNotifiable: $email, $phone_number

}

class TestNotification extends Notification
{
    public function __construct($message)
    {
        $this->message = $message;

The property message does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly.

    }

    public function to46Elks($notifiable)

The parameter $notifiable is not used and could be removed.

    {
        return $this->message;
    }
}