spec\Slick\Event\Application\CallableEventListenerSpec

Complexity

Total Complexity

4

Size/Duplication

Total Lines	22
Duplicated Lines	0 %

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	it_is_initializable()	0	3	1
A	it_call_callable_on_execute()	0	5	2
A	let()	0	7	1

<?php

/**
 * This file is part of slick/event package
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 */

namespace spec\Slick\Event\Application;

use Slick\Event\Application\CallableEventListener;
use PhpSpec\ObjectBehavior;
use Slick\Event\Event;

/**
 * CallableEventListenerSpec specs
 *
 * @package spec\Slick\Event\Application
 */
class CallableEventListenerSpec extends ObjectBehavior
{
    private bool $called = false;
    public function let()
    {
        $callable = function (object $event) {
            $this->called = true;
            return $event;
        };
        $this->beConstructedWith($callable);
    }

    function it_is_initializable()

It is generally recommended to explicitly declare the visibility for methods.

    {
        $this->shouldHaveType(CallableEventListener::class);
    }

    function it_call_callable_on_execute(Event $event)
    {
        $this->handle($event)->shouldBe($event);

The method handle() does not exist on spec\Slick\Event\Application\CallableEventListenerSpec. Since you implemented __call, consider adding a @method annotation.

        if (!$this->called) {
            throw new \Exception("Callable listener not called");
        }
    }
}