ClientFactoryTest::setUp() - Code Metrics - Inspection of "Driver: ClientFactory 100% S0 coverage." - FGM/mongodb - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — 2762893-mongodb_tests ( 41f0fc )

by Frédéric G.

created 2016-07-08 12:45 UTC

ClientFactoryTest::setUp() B

↳ Parent: ClientFactoryTest

Complexity

Conditions	1
Paths	1

Size

Total Lines	24
Code Lines	16

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	1
eloc	16
nc	1
nop	0
dl	0
loc	24
rs	8.9713
c	0
b	0
f	0

<?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 KernelTestBase {
  const DEFAULT_URI = 'mongodb://localhost:27017';
  const BAD_ALIAS = 'bad';
  const TEST_ALIAS = 'test';

  public static $modules = ['mongodb'];

  /**
   * A test-specific instance of Settings.
   *
   * @var \Drupal\Core\Site\Settings
   */
  protected $settings;


  protected $uri;

  /**
   * {@inheritdoc}
   */
  public function setUp() {
    parent::setUp();
    $this->uri = $_SERVER['MONGODB_URI'] ?? static::DEFAUlT_URI;


    $this->settings = new Settings([
      'mongodb' => [
        'clients' => [
          static::BAD_ALIAS => [
            'uri' => 'mongodb://localhost:80',
            'uriOptions' => [],
            'driverOptions' => [],
          ],
          static::TEST_ALIAS => [
            'uri' => $this->uri,
            'uriOptions' => [],
            'driverOptions' => [],
          ],
        ],
        'databases' => [
          'default' => [static::TEST_ALIAS, $this->getDatabasePrefix()],
        ],
      ],
    ]);
  }

  /**
   * Test a normal client creation attempt.
   */
  public function testGetHappy() {
    $cf = new ClientFactory($this->settings);

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

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

$singleQuoted = 'Value';
$doubleQuoted = "\tSingle is $singleQuoted";

print $doubleQuoted;
        '@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);

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

$singleQuoted = 'Value';
$doubleQuoted = "\tSingle is $singleQuoted";

print $doubleQuoted;
    }
    catch (ConnectionTimeoutException $e) {

      $this->pass("Cannot create a client to a non-server.");
<?php

$singleQuoted = 'Value';
$doubleQuoted = "\tSingle is $singleQuoted";

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

}



1			<?php
2
3			namespace Drupal\mongodb\Tests;
4
5			use Drupal\Component\Render\FormattableMarkup;
6			use Drupal\Core\Site\Settings;
7			use Drupal\KernelTests\KernelTestBase;
8			use Drupal\mongodb\ClientFactory;
9			use MongoDB\Driver\Exception\ConnectionTimeoutException;
10
11			/**
12			* Class ClientFactoryTest.
13			*
14			* @group MongoDB
15			*/
16			class ClientFactoryTest extends KernelTestBase {
17			const DEFAULT_URI = 'mongodb://localhost:27017';
18			const BAD_ALIAS = 'bad';
19			const TEST_ALIAS = 'test';
20
21			public static $modules = ['mongodb'];
22
23			/**
24			* A test-specific instance of Settings.
25			*
26			* @var \Drupal\Core\Site\Settings
27			*/
28			protected $settings;
29
30
31			protected $uri;
32
33			/**
34			* {@inheritdoc}
35			*/
36			public function setUp() {
37			parent::setUp();
38			$this->uri = $_SERVER['MONGODB_URI'] ?? static::DEFAUlT_URI;
			1 ignored issue – show introduced 2016-07-08 12:48 UTC by Report Bug Copy Issue Report Expected 1 space after "?"; 0 found Loading history...
39
40			$this->settings = new Settings([
41			'mongodb' => [
42			'clients' => [
43			static::BAD_ALIAS => [
44			'uri' => 'mongodb://localhost:80',
45			'uriOptions' => [],
46			'driverOptions' => [],
47			],
48			static::TEST_ALIAS => [
49			'uri' => $this->uri,
50			'uriOptions' => [],
51			'driverOptions' => [],
52			],
53			],
54			'databases' => [
55			'default' => [static::TEST_ALIAS, $this->getDatabasePrefix()],
56			],
57			],
58			]);
59			}
60
61			/**
62			* Test a normal client creation attempt.
63			*/
64			public function testGetHappy() {
65			$cf = new ClientFactory($this->settings);
66
67			try {
68			$client = $cf->get(static::TEST_ALIAS);
69			// Force connection attempt by executing a command.
70			$client->listDatabases();
71			}
72			catch (ConnectionTimeoutException $e) {
			1 ignored issue – show Bug introduced 2016-07-08 12:48 UTC by Report Bug Copy Issue Report The class `MongoDB\Driver\Exception...nectionTimeoutException` does not exist. Did you forget a USE statement, or did you not list all dependencies? Scrutinizer analyzes your `composer.json`/`composer.lock` file if available to determine the classes, and functions that are defined by your dependencies. It seems like the listed class was neither found in your dependencies, nor was it found in the analyzed files in your repository. If you are using some other form of dependency management, you might want to disable this analysis. Loading history...
73			$this->fail(new FormattableMarkup("Could not connect to server on @uri. Enable one on @default or specify one in MONGODB_URI.", [
			0 ignored issues – show Coding Style Comprehensibility introduced 2016-07-08 12:48 UTC by Report Bug Copy Issue Report The string literal `Could not connect to ser...ify one in MONGODB_URI.` does not require double quotes, as per coding-style, please use single quotes. PHP provides two ways to mark string literals. Either with single quotes `'literal'` or with double quotes `"literal"`. The difference between these is that string literals in double quotes may contain variables with are evaluated at run-time as well as escape sequences. String literals in single quotes on the other hand are evaluated very literally and the only two characters that needs escaping in the literal are the single quote itself (`\'`) and the backslash (`\\`). Every other character is displayed as is. Double quoted string literals may contain other variables or more complex escape sequences. <?php $singleQuoted = 'Value'; $doubleQuoted = "\tSingle is $singleQuoted"; print $doubleQuoted; will print an indented: `Single is Value` If your string literal does not contain variables or escape sequences, it should be defined using single quotes to make that fact clear. For more information on PHP string literals and available escape sequences see the PHP core documentation. Loading history...
74			'@default' => static::DEFAUlT_URI,
75			'@uri' => $this->uri,
76			]));
77			}
78			catch (\Exception $e) {
79			$this->fail($e->getMessage());
80			}
81			}
82
83			/**
84			* Test an existing alias pointing to an invalid server.
85			*/
86			public function testGetSadBadAlias() {
87			$cf = new ClientFactory($this->settings);
88
89			try {
90			$client = $cf->get(static::BAD_ALIAS);
91			// Force connection attempt by executing a command.
92			$client->listDatabases();
93			$this->fail("Should not have been able to connect to a non-server.");
			0 ignored issues – show Coding Style Comprehensibility introduced 2016-07-08 12:48 UTC by Report Bug Copy Issue Report The string literal `Should not have been abl...onnect to a non-server.` does not require double quotes, as per coding-style, please use single quotes. PHP provides two ways to mark string literals. Either with single quotes `'literal'` or with double quotes `"literal"`. The difference between these is that string literals in double quotes may contain variables with are evaluated at run-time as well as escape sequences. String literals in single quotes on the other hand are evaluated very literally and the only two characters that needs escaping in the literal are the single quote itself (`\'`) and the backslash (`\\`). Every other character is displayed as is. Double quoted string literals may contain other variables or more complex escape sequences. <?php $singleQuoted = 'Value'; $doubleQuoted = "\tSingle is $singleQuoted"; print $doubleQuoted; will print an indented: `Single is Value` If your string literal does not contain variables or escape sequences, it should be defined using single quotes to make that fact clear. For more information on PHP string literals and available escape sequences see the PHP core documentation. Loading history...
94			}
95			catch (ConnectionTimeoutException $e) {
			1 ignored issue – show Bug introduced 2016-07-08 12:48 UTC by Report Bug Copy Issue Report The class `MongoDB\Driver\Exception...nectionTimeoutException` does not exist. Did you forget a USE statement, or did you not list all dependencies? Scrutinizer analyzes your `composer.json`/`composer.lock` file if available to determine the classes, and functions that are defined by your dependencies. It seems like the listed class was neither found in your dependencies, nor was it found in the analyzed files in your repository. If you are using some other form of dependency management, you might want to disable this analysis. Loading history...
96			$this->pass("Cannot create a client to a non-server.");
			0 ignored issues – show Coding Style Comprehensibility introduced 2016-07-08 12:48 UTC by Report Bug Copy Issue Report The string literal `Cannot create a client to a non-server.` does not require double quotes, as per coding-style, please use single quotes. PHP provides two ways to mark string literals. Either with single quotes `'literal'` or with double quotes `"literal"`. The difference between these is that string literals in double quotes may contain variables with are evaluated at run-time as well as escape sequences. String literals in single quotes on the other hand are evaluated very literally and the only two characters that needs escaping in the literal are the single quote itself (`\'`) and the backslash (`\\`). Every other character is displayed as is. Double quoted string literals may contain other variables or more complex escape sequences. <?php $singleQuoted = 'Value'; $doubleQuoted = "\tSingle is $singleQuoted"; print $doubleQuoted; will print an indented: `Single is Value` If your string literal does not contain variables or escape sequences, it should be defined using single quotes to make that fact clear. For more information on PHP string literals and available escape sequences see the PHP core documentation. Loading history...
97			}
98			catch (\Exception $e) {
99			$this->fail($e->getMessage());
100			}
101			}
102
103			}
			0 ignored issues – show Coding Style introduced 2016-07-08 12:48 UTC by Report Bug Copy Issue Report As per coding style, files should not end with a newline character. This check marks files that end in a newline character, i.e. an empy line. Loading history...
104

FGM / mongodb

Push — 2762893-mongodb_tests ( 41f0fc )

ClientFactoryTest::setUp() B

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like