ConsumeContext

Complexity

Total Complexity

19

Size/Duplication

Total Lines	175
Duplicated Lines	8 %

Coupling/Cohesion

Components	1
Dependencies	10

Importance

Changes

0

15 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	7	1
A	theQueueContainsTheTextMessage()	0	9	1
A	theQueueContainsTheJsonMessage()	0	12	1
A	iConsumeAllTheMessagesInTheQueue()	0	14	1
A	process()	0	4	1
A	iHaveConsumedMessage()	0	4	1
A	theMessageIsATextOne()	7	7	1
A	theMessageIsAJsonOne()	7	7	1
A	theMessageContainsTheJson()	0	4	1
A	theMessageContains()	0	6	1
A	oneOfTheMessagesIsATextOne()	0	4	1
A	oneOfTheMessagesIsAJsonOne()	0	4	1
A	oneOfTheMessagesIs()	0	16	3
A	oneOfTheMessagesContainsTheJson()	0	4	1
A	oneOfTheMessagesContains()	0	15	3

<?php

use Behat\Behat\Tester\Exception\PendingException;
use Puzzle\AMQP\Messages\Message;
use Puzzle\AMQP\Consumers\Insomniac;
use Puzzle\AMQP\Workers\Worker;
use Puzzle\AMQP\Workers\WorkerContext;
use Puzzle\AMQP\ReadableMessage;
use Psr\Log\LoggerAwareTrait;
use Psr\Log\NullLogger;
use Puzzle\AMQP\Workers\ProcessorInterfaceAdapter;
use Puzzle\AMQP\Messages\ContentType;
use Puzzle\AMQP\Messages\Bodies\Json;

class ConsumeContext extends AbstractRabbitMQContext implements Worker

PSR1 recommends that each class must be in a namespace of at least one level to avoid collisions.

{
    use LoggerAwareTrait;
    
    private
        $consumedMessages;

The visibility should be declared for property $consumedMessages.

    
    public function __construct($path)
    {
        parent::__construct($path);
        
        $this->logger = new NullLogger();

Equals sign not aligned with surrounding assignments; expected 11 spaces but found 1 space

        $this->consumedMessages = [];
    }
    
    /**
     * @Given The queue :queue contains the text message :bodyContent
     */
    public function theQueueContainsTheTextMessage($bodyContent, $queue)

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

    {
        // FIXME Use RabbitMQCTL instead

Comment refers to a FIXME task "Use RabbitMQCTL instead"

        
        $message = new Message(self::TEXT_ROUTING_KEY);
        $message->setText($bodyContent);
        
        $this->client->publish($this->exchange, $message);
    }

    /**
     * @Given The queue :queue contains the json message :bodyContent
     */
    public function theQueueContainsTheJsonMessage($bodyContent, $queue)

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

    {
        // FIXME Use RabbitMQCTL instead

Comment refers to a FIXME task "Use RabbitMQCTL instead"

        
        $message = new Message(self::JSON_ROUTING_KEY);
        
        $body = new Json();
        $body->changeContentWithJson($bodyContent);
        $message->setBody($body);
        
        $this->client->publish($this->exchange, $message);
    }
    
    /**
     * @When I consume all the messages in the queue :queue
     */
    public function iConsumeAllTheMessagesInTheQueue($queue)
    {
        $this->consumedMessages = [];
        $workerContext = new WorkerContext(

Equals sign not aligned with surrounding assignments; expected 10 spaces but found 1 space

            function() {
                return $this;
            },
            $consumer = new Insomniac(),
            $queue
        );
        
        $processor = new ProcessorInterfaceAdapter($workerContext);
        $consumer->consume($processor, $this->client, $workerContext);
    }
    
    public function process(ReadableMessage $message)
    {
        $this->consumedMessages[] = $message;
    }
    
    /**
     * @Then /I have consumed (\d+) messages?/
     */
    public function iHaveConsumedMessage($nbMessages)
    {
        \PHPUnit_Framework_Assert::assertSame((int) $nbMessages, count($this->consumedMessages));
    }
    
    /**
     * @Then the message is a text one
     */
    public function theMessageIsATextOne()

This method seems to be duplicated in your project.

    {
        $firstMessage = $this->consumedMessages[0];
        
        \PHPUnit_Framework_Assert::assertSame($firstMessage->getRoutingKeyFromHeader(), self::TEXT_ROUTING_KEY);
        \PHPUnit_Framework_Assert::assertSame($firstMessage->getContentType(), ContentType::TEXT);
    }
    
    /**
     * @Then the message is a json one
     */
    public function theMessageIsAJsonOne()

This method seems to be duplicated in your project.

    {
        $firstMessage = $this->consumedMessages[0];
        
        \PHPUnit_Framework_Assert::assertSame($firstMessage->getRoutingKeyFromHeader(), self::JSON_ROUTING_KEY);
        \PHPUnit_Framework_Assert::assertSame($firstMessage->getContentType(), ContentType::JSON);
    }
    
    /**
     * @Then the message contains the json :jsonString
     */
    public function theMessageContainsTheJson($jsonString)
    {
        $this->theMessageContains(json_decode($jsonString, true));
    }
    
    /**
     * @Then the message contains :bodyContent
     */
    public function theMessageContains($bodyContent)
    {
        $firstMessage = $this->consumedMessages[0];
        
        \PHPUnit_Framework_Assert::assertSame($firstMessage->getBodyInOriginalFormat(), $bodyContent);
    }

    /**
     * @Then one of the messages is a text one
     */
    public function oneOfTheMessagesIsATextOne()
    {
        $this->oneOfTheMessagesIs(ContentType::TEXT, self::TEXT_ROUTING_KEY);
    }

    /**
     * @Then one of the messages is a json one
     */
    public function oneOfTheMessagesIsAJsonOne()
    {
        $this->oneOfTheMessagesIs(ContentType::JSON, self::JSON_ROUTING_KEY);
    }
    
    private function oneOfTheMessagesIs($contentType, $routingKey)
    {
        $found = null;
        
        foreach($this->consumedMessages as $message)
        {
            if($message->getContentType() === $contentType)
            {
                $found = $message;
                break;
            }
        }
        
        \PHPUnit_Framework_Assert::assertNotNull($found);
        \PHPUnit_Framework_Assert::assertSame($routingKey, $found->getRoutingKeyFromHeader());
    }
    
    /**
     * @Then one of the messages contains the json :jsonString
     */
    public function oneOfTheMessagesContainsTheJson($jsonString)
    {
        $this->oneOfTheMessagesContains(json_decode($jsonString, true));
    }
    
    /**
     * @Then one of the messages contains :bodyContent
     */
    public function oneOfTheMessagesContains($bodyContent)
    {
        $found = false;
        
        foreach($this->consumedMessages as $message)
        {
            if($message->getBodyInOriginalFormat() === $bodyContent)
            {
                $found = true;
                break;
            }
        }
        
        \PHPUnit_Framework_Assert::assertTrue($found);
    }
}