PhpFlo\Test\ComponentTestHelperTrait

Complexity

Total Complexity

8

Size/Duplication

Total Lines	81
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	0

Importance

Changes	1
Bugs	0	Features	0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	connectInPorts()	0	8	2
B	connectOutPorts()	0	28	2
A	getOutPortData()	0	10	3
A	wasCalled()	0	4	1

<?php
/*
 * This file is part of the phpflo/core package.
 *
 * (c) Marc Aschmann <maschmann@gmail.com>
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 */
namespace PhpFlo\Test;

use PhpFlo\Common\ComponentInterface;
use PhpFlo\Interaction\InternalSocket;

/**
 * Class ComponentTestHelperTrait
 *
 * @package PhpFlo\PhpFloBundle\Test
 * @author Marc Aschmann <maschmann@gmail.com>
 */
trait ComponentTestHelperTrait
{
    /**
     * @var array
     */
    private $outPortSockets;

    /**
     * Fake-connect socket to port.
     *
     * @param ComponentInterface $component
     * @return ComponentInterface
     */
    protected function connectInPorts(ComponentInterface $component)
    {
        foreach ($component->inPorts() as $alias => $inPort) {
            $inPort->attach($this->stub(InternalSocket::class));

It seems like stub() must be provided by classes using this trait. How about adding it as abstract method to this trait?

        }

        return $component;
    }

    /**
     * Fake-connect socket to port and add a storage for later value checks.
     *
     * @param ComponentInterface $component
     * @return ComponentInterface
     */
    protected function connectOutPorts(ComponentInterface $component)
    {
        $this->outPortSockets = [];
        foreach ($component->outPorts() as $port) {
            $socket = $this->stub(

It seems like stub() must be provided by classes using this trait. How about adding it as abstract method to this trait?

                InternalSocket::class,
                [
                    'isConnected' => true,
                ]
            );
            $socket->expects($this->any())

It seems like any() must be provided by classes using this trait. How about adding it as abstract method to this trait?

                ->method('send')
                ->willReturnCallback(
                    \Closure::bind(
                        function ($data) {
                            $this->data = $data;

The property data does not exist. Did you maybe forget to declare it?

                        },
                        $socket
                    )
                );
            $this->outPortSockets[$port->getName()] = $socket;
            $socket->from = [];
            $socket->to = [];
            $port->attach($socket);
        }

        return $component;
    }

    /**
     * @param string $port
     * @return array|mixed
     */
    protected function getOutPortData($port = '')
    {
        if ('' !== $port) {
            if (array_key_exists($port, $this->outPortSockets)) {
                return $this->outPortSockets[$port]->data;
            }
        }

        return $this->outPortSockets;
    }

    /**
     * @param string $port
     * @return bool
     */
    protected function wasCalled($port)
    {
        return !empty($this->outPortSockets[$port]);
    }
}