Drupal\mongodb\Tests\ClientFactoryTest

Complexity

Total Complexity

6

Size/Duplication

Total Lines	44
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	3

Importance

Changes	1
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	testGetHappy()	0	18	3
A	testGetSadBadAlias()	0	16	3

<?php

namespace Drupal\mongodb\Tests;

use Drupal\Component\Render\FormattableMarkup;
use Drupal\Core\Site\Settings;
use Drupal\KernelTests\KernelTestBase;
use Drupal\mongodb\ClientFactory;
use MongoDB\Driver\Exception\ConnectionTimeoutException;

/**
 * Class ClientFactoryTest.
 *
 * @group MongoDB
 */
class ClientFactoryTest extends MongoDbTestBase {
  /**
   * Test a normal client creation attempt.
   */
  public function testGetHappy() {
    $cf = new ClientFactory($this->settings);

Avoid variables with short names like $cf. Configured minimum length is 3.

    try {
      $client = $cf->get(static::CLIENT_TEST_ALIAS);
      // Force connection attempt by executing a command.
      $client->listDatabases();
    }
    catch (ConnectionTimeoutException $e) {

The class MongoDB\Driver\Exception\ConnectionTimeoutException does not exist. Did you forget a USE statement, or did you not list all dependencies?

      $this->fail(new FormattableMarkup("Could not connect to server on @uri. Enable one on @default or specify one in MONGODB_URI.", [

The string literal Could not connect to server on @uri. Enable one on @default or specify one in MONGODB_URI. does not require double quotes, as per coding-style, please use single quotes.

        '@default' => static::DEFAULT_URI,
        '@uri' => $this->uri,
      ]));
    }
    catch (\Exception $e) {
      $this->fail($e->getMessage());
    }
  }

  /**
   * Test an existing alias pointing to an invalid server.
   */
  public function testGetSadBadAlias() {
    $cf = new ClientFactory($this->settings);

Avoid variables with short names like $cf. Configured minimum length is 3.

    try {
      $client = $cf->get(static::CLIENT_BAD_ALIAS);
      // Force connection attempt by executing a command.
      $client->listDatabases();
      $this->fail("Should not have been able to connect to a non-server.");

The string literal Should not have been able to connect to a non-server. does not require double quotes, as per coding-style, please use single quotes.

    }
    catch (ConnectionTimeoutException $e) {

The class MongoDB\Driver\Exception\ConnectionTimeoutException does not exist. Did you forget a USE statement, or did you not list all dependencies?

      $this->assertTrue(TRUE, "Cannot create a client to a non-server.");

The string literal Cannot create a client to a non-server. does not require double quotes, as per coding-style, please use single quotes.

    }
    catch (\Exception $e) {
      $this->fail($e->getMessage());
    }
  }

}

As per coding style, files should not end with a newline character.