SqsQueueStoreConnection::connect() - Code Metrics - Inspection of "100% code coverage again!!!" - 2amigos/mailer-library - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#10)

by resu

created 2016-01-15 19:45 UTC

SqsQueueStoreConnection::connect() A

↳ Parent: SqsQueueStoreConnection

Complexity

Conditions	2
Paths	2

Size

Total Lines	17
Code Lines	11

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	13
CRAP Score	2

Importance

Changes	2
Bugs	0	Features	0

Metric	Value
c	2
b	0
f	0
dl	0
loc	17
ccs	13
cts	13
cp	1
rs	9.4285
cc	2
eloc	11
nc	2
nop	0
crap	2

<?php
namespace Da\Mailer\Queue\Backend\Sqs;

use Da\Mailer\Queue\Backend\AbstractQueueStoreConnection;
use Aws\Sqs\SqsClient;

class SqsQueueStoreConnection extends AbstractQueueStoreConnection
{
    /**
     * SqsQueueStoreConnection constructor.
     *
     * @param array $configuration
     */
    public function __construct(array $configuration = [])
    {
        parent::__construct($configuration);
    }

    /**
     * @inheritdoc
     */
    public function connect()
    {
        if ($this->instance !== null) {
            $this->instance = null; // close previous connection
        }
        $key = $this->getConfigurationValue('key');
        $secret = $this->getConfigurationValue('secret');
        $region = $this->getConfigurationValue('region');

        $this->instance = SqsClient::factory([
            'key' => $key,
            'secret' => $secret,
            'region' => $region,
        ]);

        return $this;
    }

    /**
     * Returns the connection instance.
     *
     * @return SqsClient|null
     */
    public function getInstance()
    {
        if ($this->instance === null) {
            $this->connect();
        }

        return $this->instance;
    }
}


1		<?php
2		namespace Da\Mailer\Queue\Backend\Sqs;
3
4		use Da\Mailer\Queue\Backend\AbstractQueueStoreConnection;
5		use Aws\Sqs\SqsClient;
6
7		class SqsQueueStoreConnection extends AbstractQueueStoreConnection
8		{
9		/**
10		* SqsQueueStoreConnection constructor.
11		*
12		* @param array $configuration
13		*/
14	2	public function __construct(array $configuration = [])
15		{
16	2	parent::__construct($configuration);
17	2	}
18
19		/**
20		* @inheritdoc
21		*/
22	1	public function connect()
23		{
24	1	if ($this->instance !== null) {
25	1	$this->instance = null; // close previous connection
26	1	}
27	1	$key = $this->getConfigurationValue('key');
28	1	$secret = $this->getConfigurationValue('secret');
29	1	$region = $this->getConfigurationValue('region');
30
31	1	$this->instance = SqsClient::factory([
32	1	'key' => $key,
33	1	'secret' => $secret,
34	1	'region' => $region,
35	1	]);
36
37	1	return $this;
38		}
39
40		/**
41		* Returns the connection instance.
42		*
43		* @return SqsClient\|null
44		*/
45	1	public function getInstance()
46		{
47	1	if ($this->instance === null) {
48	1	$this->connect();
49	1	}
50
51	1	return $this->instance;
52		}
53		}
54

2amigos / mailer-library

Pull Request — master (#10)

SqsQueueStoreConnection::connect() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like