ClientTest::testExchange() - Code Metrics - Inspection of "First running version with unit tests" - thecodingmachine/oo-amqp-client - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — 1.0 ( 96ad2d...e7718b )

by David

created 2016-07-07 16:20 UTC

ClientTest::testExchange() A

↳ Parent: ClientTest

Complexity

Conditions	1
Paths	1

Size

Total Lines	4
Code Lines	2

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
c	1
b	0
f	0
dl	0
loc	4
rs	10
cc	1
eloc	2
nc	1
nop	0

<?php

namespace Mouf\AmqpClient\Objects;

use Mouf\AmqpClient\Client;
use Mouf\AmqpClient\Consumer;
use Mouf\AmqpClient\ConsumerService;
use PhpAmqpLib\Message\AMQPMessage;

class ClientTest extends \PHPUnit_Framework_TestCase
{
    /**
     * @var Client
     */
    private $client;
    private $exchange;
    private $queue;
    private $msgReceived;

    protected function setUp()
    {
        global $rabbitmq_host;
function myFunction($a, $b) {
    // Do something
}
        global $rabbitmq_port;
function myFunction($a, $b) {
    // Do something
}
        global $rabbitmq_user;
function myFunction($a, $b) {
    // Do something
}
        global $rabbitmq_password;
function myFunction($a, $b) {
    // Do something
}

        $this->client = new Client($rabbitmq_host, $rabbitmq_port, $rabbitmq_user, $rabbitmq_password);
        $this->client->setPrefetchCount(1);

        $this->exchange = new Exchange($this->client, 'test_exchange', 'fanout');
        $this->queue = new Queue($this->client, 'test_queue', [
            new Consumer(function(AMQPMessage $msg) {
                $this->msgReceived = $msg;
            })
        ]);
        $this->queue->setDurable(true);

        $binding = new Binding($this->exchange, $this->queue);
        $this->client->register($binding);
    }



    public function testExchange()
    {
        $this->exchange->publish(new Message('my message'), 'key');
    }

    /**
     * @depends testExchange
     */
    public function testQueue()
    {
        $consumerService = new ConsumerService($this->client, [
            $this->queue
        ]);

        $consumerService->run(true);

        $this->assertEquals('my message', $this->msgReceived->getBody());
    }
}


1			<?php
2
3			namespace Mouf\AmqpClient\Objects;
4
5			use Mouf\AmqpClient\Client;
6			use Mouf\AmqpClient\Consumer;
7			use Mouf\AmqpClient\ConsumerService;
8			use PhpAmqpLib\Message\AMQPMessage;
9
10			class ClientTest extends \PHPUnit_Framework_TestCase
11			{
12			/**
13			* @var Client
14			*/
15			private $client;
16			private $exchange;
17			private $queue;
18			private $msgReceived;
19
20			protected function setUp()
21			{
22			global $rabbitmq_host;
			0 ignored issues – show Compatibility Best Practice introduced 2016-07-07 16:27 UTC by Report Bug Copy Issue Report Use of `global` functionality is not recommended; it makes your code harder to test, and less reusable. Instead of relying on `global` state, we recommend one of these alternatives: 1. Pass all data via parameters function myFunction($a, $b) { // Do something } 2. Create a class that maintains your state class MyClass { private $a; private $b; public function __construct($a, $b) { $this->a = $a; $this->b = $b; } public function myFunction() { // Do something } } Loading history...
23			global $rabbitmq_port;
			0 ignored issues – show Compatibility Best Practice introduced 2016-07-07 16:27 UTC by Report Bug Copy Issue Report Use of `global` functionality is not recommended; it makes your code harder to test, and less reusable. Instead of relying on `global` state, we recommend one of these alternatives: 1. Pass all data via parameters function myFunction($a, $b) { // Do something } 2. Create a class that maintains your state class MyClass { private $a; private $b; public function __construct($a, $b) { $this->a = $a; $this->b = $b; } public function myFunction() { // Do something } } Loading history...
24			global $rabbitmq_user;
			0 ignored issues – show Compatibility Best Practice introduced 2016-07-07 16:27 UTC by Report Bug Copy Issue Report Use of `global` functionality is not recommended; it makes your code harder to test, and less reusable. Instead of relying on `global` state, we recommend one of these alternatives: 1. Pass all data via parameters function myFunction($a, $b) { // Do something } 2. Create a class that maintains your state class MyClass { private $a; private $b; public function __construct($a, $b) { $this->a = $a; $this->b = $b; } public function myFunction() { // Do something } } Loading history...
25			global $rabbitmq_password;
			0 ignored issues – show Compatibility Best Practice introduced 2016-07-07 16:27 UTC by Report Bug Copy Issue Report Use of `global` functionality is not recommended; it makes your code harder to test, and less reusable. Instead of relying on `global` state, we recommend one of these alternatives: 1. Pass all data via parameters function myFunction($a, $b) { // Do something } 2. Create a class that maintains your state class MyClass { private $a; private $b; public function __construct($a, $b) { $this->a = $a; $this->b = $b; } public function myFunction() { // Do something } } Loading history...
26
27			$this->client = new Client($rabbitmq_host, $rabbitmq_port, $rabbitmq_user, $rabbitmq_password);
28			$this->client->setPrefetchCount(1);
29
30			$this->exchange = new Exchange($this->client, 'test_exchange', 'fanout');
31			$this->queue = new Queue($this->client, 'test_queue', [
32			new Consumer(function(AMQPMessage $msg) {
33			$this->msgReceived = $msg;
34			})
35			]);
36			$this->queue->setDurable(true);
37
38			$binding = new Binding($this->exchange, $this->queue);
39			$this->client->register($binding);
40			}
41
42
43
44			public function testExchange()
45			{
46			$this->exchange->publish(new Message('my message'), 'key');
47			}
48
49			/**
50			* @depends testExchange
51			*/
52			public function testQueue()
53			{
54			$consumerService = new ConsumerService($this->client, [
55			$this->queue
56			]);
57
58			$consumerService->run(true);
59
60			$this->assertEquals('my message', $this->msgReceived->getBody());
61			}
62			}
63

thecodingmachine / oo-amqp-client

Push — 1.0 ( 96ad2d...e7718b )

ClientTest::testExchange() A

Complexity

Size

Duplication

Importance

1. Pass all data via parameters

2. Create a class that maintains your state

1. Pass all data via parameters

2. Create a class that maintains your state

1. Pass all data via parameters

2. Create a class that maintains your state

1. Pass all data via parameters

2. Create a class that maintains your state

Duplication Side-by-Side

Filter issues like