Elastica\Test\Connection\Strategy\SimpleTest

Complexity

Total Complexity

7

Size/Duplication

Total Lines	100
Duplicated Lines	57 %

Coupling/Cohesion

Components	1
Dependencies	4

Importance

Changes

0

6 Methods

Rating	Name	Duplication	Size	Complexity
A	testConnection()	0	9	1
A	testFailConnection()	11	11	1
A	testWithOneFailConnection()	23	23	1
A	testWithNoValidConnection()	22	22	2
A	_checkStrategy()	0	6	1
A	_checkResponse()	0	4	1

<?php

namespace Elastica\Test\Connection\Strategy;

use Elastica\Client;
use Elastica\Connection;
use Elastica\Connection\Strategy\Simple;
use Elastica\Exception\ConnectionException;
use Elastica\Response;
use Elastica\Test\Base;

/**
 * Description of SimplyTest.
 *
 * @author chabior
 *
 * @internal
 */
class SimpleTest extends Base
{
    /**
     * @var int Number of seconds to wait before timeout is called. Is set low for tests to have fast tests.
     */
    protected $_timeout = 1;

    /**
     * @group functional
     */
    public function testConnection(): void
    {
        $client = $this->_getClient();
        $response = $client->request('_aliases');

        $this->_checkResponse($response);

        $this->_checkStrategy($client);
    }

    /**
     * @group functional
     */
    public function testFailConnection(): void

This method seems to be duplicated in your project.

    {
        $this->expectException(\Elastica\Exception\ConnectionException::class);

        $config = ['host' => '255.255.255.0', 'timeout' => $this->_timeout];
        $client = $this->_getClient($config);

        $this->_checkStrategy($client);

        $client->request('_aliases');
    }

    /**
     * @group functional
     */
    public function testWithOneFailConnection(): void

This method seems to be duplicated in your project.

    {
        $connections = [
            new Connection(['host' => '255.255.255.0', 'timeout' => $this->_timeout]),
            new Connection(['host' => $this->_getHost(), 'timeout' => $this->_timeout]),
        ];

        $count = 0;
        $callback = static function ($connection, $exception, $client) use (&$count): void {

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

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

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

            ++$count;
        };

        $client = $this->_getClient([], $callback);
        $client->setConnections($connections);

        $response = $client->request('_aliases');

        $this->_checkResponse($response);

        $this->_checkStrategy($client);

        $this->assertLessThan(\count($connections), $count);
    }

    /**
     * @group functional
     */
    public function testWithNoValidConnection(): void

This method seems to be duplicated in your project.

    {
        $connections = [
            new Connection(['host' => '255.255.255.0', 'timeout' => $this->_timeout]),
            new Connection(['host' => '45.45.45.45', 'port' => '80', 'timeout' => $this->_timeout]),
            new Connection(['host' => '10.123.213.123', 'timeout' => $this->_timeout]),
        ];

        $count = 0;
        $client = $this->_getClient([], static function () use (&$count): void {
            ++$count;
        });

        $client->setConnections($connections);

        try {
            $client->request('_aliases');
            $this->fail('Should throw exception as no connection valid');
        } catch (ConnectionException $e) {
            $this->assertEquals(\count($connections), $count);
        }
    }

    protected function _checkStrategy(Client $client): void
    {
        $strategy = $client->getConnectionStrategy();

        $this->assertInstanceOf(Simple::class, $strategy);
    }

    protected function _checkResponse(Response $response): void
    {
        $this->assertTrue($response->isOk());
    }
}