Application::__clone() - Code Metrics - Inspection of "[WIP] Added PhantomJsDriverFactory to project" - minkphp/phpunit-mink - Measure and Improve Code Quality continuously with Scrutinizer

Failed Conditions

Pull Request — master (#75)

by Malte

created 2016-02-01 03:58 UTC

Application::__clone() A

↳ Parent: Application

Complexity

Conditions	1
Paths	1

Size

Total Lines	4
Code Lines	1

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	2

Importance

Changes	1
Bugs	0	Features	0

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

<?php
/**
 * This file is part of the phpunit-mink library.
 * For the full copyright and license information, please view
 * the LICENSE file that was distributed with this source code.
 *
 * @copyright Alexander Obuhovich <[email protected]>
 * @link      https://github.com/aik099/phpunit-mink
 */

namespace aik099\PHPUnit;


use aik099\PHPUnit\TestSuite\TestSuiteFactory;

/**
 * Main application class.
 *
 * @method \Mockery\Expectation shouldReceive(string $name)
 */
class Application
{

	/**
	 * Dependency injection container.
	 *
	 * @var DIContainer
	 */
	protected $container;

	/**
	 * Returns instance of strategy manager.
	 *
	 * @param DIContainer $container Dependency injection container.
	 *
	 * @return self
	 */
	public static function getInstance(DIContainer $container = null)
	{
		static $instance = null;

		if ( null === $instance ) {
			$instance = new static($container);
		}

		return $instance;
	}

	/**
	 * Prevents direct instantiation.
	 *
	 * @param DIContainer $container Dependency injection container.
	 */
	public function __construct(DIContainer $container = null)
	{
		if ( !isset($container) ) {
			$container = new DIContainer();
		}

		$this->container = $container;
		$this->container->setApplication($this);
class User
{
    private $email;

    public function getEmail()
    {
        return $this->email;
    }

    public function setEmail($email)
    {
        $this->email = $email;
    }
}
	}

	/**
	 * Returns test suite builder.
	 *
	 * @return TestSuiteFactory
	 * @see    BrowserTestCase::suite()
	 */
	public function getTestSuiteFactory()
	{
		return $this->getObject('test_suite_factory');
	}

	/**
	 * Returns object from the container.
	 *
	 * @param string $service_id Name of the object in the container.
	 *
	 * @return \stdClass
	 */
	public function getObject($service_id)
	{
		return $this->container[$service_id];
	}

	/**
	 * Replaces object in the container.
	 *
	 * @param string   $service_id Name of the object in the container.
	 * @param callable $callable   The callable that will return the object.
	 * @param boolean  $is_factory The callable should be considered as a factory.
	 *
	 * @return callable Previous service version.
	 * @throws \InvalidArgumentException When attempt is made to replace non-existing service.
	 */
	public function replaceObject($service_id, $callable, $is_factory = false)

	{
		if ( !isset($this->container[$service_id]) ) {
			throw new \InvalidArgumentException('Service "' . $service_id . '" not found');
		}

		$backup = $this->container->raw($service_id);
		unset($this->container[$service_id]);
		$this->container[$service_id] = $is_factory ? $this->container->factory($callable) : $callable;

		return $backup;
	}

	/**
	 * Prevents cloning.
	 *
	 * @return void
	 *
	 * @codeCoverageIgnore
	 */
	private function __clone()
	{

	}

}


1		<?php
2		/**
3		* This file is part of the phpunit-mink library.
4		* For the full copyright and license information, please view
5		* the LICENSE file that was distributed with this source code.
6		*
7		* @copyright Alexander Obuhovich <[email protected]>
8		* @link https://github.com/aik099/phpunit-mink
9		*/
10
11		namespace aik099\PHPUnit;
12
13
14		use aik099\PHPUnit\TestSuite\TestSuiteFactory;
15
16		/**
17		* Main application class.
18		*
19		* @method \Mockery\Expectation shouldReceive(string $name)
20		*/
21		class Application
22		{
23
24		/**
25		* Dependency injection container.
26		*
27		* @var DIContainer
28		*/
29		protected $container;
30
31		/**
32		* Returns instance of strategy manager.
33		*
34		* @param DIContainer $container Dependency injection container.
35		*
36		* @return self
37		*/
38	3	public static function getInstance(DIContainer $container = null)
39		{
40	3	static $instance = null;
41
42	3	if ( null === $instance ) {
43		$instance = new static($container);
44		}
45
46	3	return $instance;
47		}
48
49		/**
50		* Prevents direct instantiation.
51		*
52		* @param DIContainer $container Dependency injection container.
53		*/
54	19	public function __construct(DIContainer $container = null)
55		{
56	19	if ( !isset($container) ) {
57	19	$container = new DIContainer();
58	19	}
59
60	19	$this->container = $container;
61	19	$this->container->setApplication($this);
		1 ignored issue – show Unused Code introduced 2016-01-07 08:39 UTC by Report Bug Copy Issue Report The call to the method `aik099\PHPUnit\DIContainer::setApplication()` seems un-needed as the method has no side-effects. PHP Analyzer performs a side-effects analysis of your code. A side-effect is basically anything that might be visible after the scope of the method is left. Let’s take a look at an example: class User { private $email; public function getEmail() { return $this->email; } public function setEmail($email) { $this->email = $email; } } If we look at the `getEmail()` method, we can see that it has no side-effect. Whether you call this method or not, no future calls to other methods are affected by this. As such code as the following is useless: $user = new User(); $user->getEmail(); // This line could safely be removed as it has no effect. On the hand, if we look at the `setEmail()`, this method _has_ side-effects. In the following case, we could not remove the method call: $user = new User(); $user->setEmail('email@domain'); // This line has a side-effect (it changes an // instance variable). Loading history...
62	19	}
63
64		/**
65		* Returns test suite builder.
66		*
67		* @return TestSuiteFactory
68		* @see BrowserTestCase::suite()
69		*/
70	19	public function getTestSuiteFactory()
71	19	{
72	3	return $this->getObject('test_suite_factory');
73		}
74
75		/**
76		* Returns object from the container.
77		*
78		* @param string $service_id Name of the object in the container.
79		*
80		* @return \stdClass
81		*/
82	5	public function getObject($service_id)
83		{
84	5	return $this->container[$service_id];
85		}
86
87		/**
88		* Replaces object in the container.
89		*
90		* @param string $service_id Name of the object in the container.
91		* @param callable $callable The callable that will return the object.
92		* @param boolean $is_factory The callable should be considered as a factory.
93		*
94		* @return callable Previous service version.
95		* @throws \InvalidArgumentException When attempt is made to replace non-existing service.
96		*/
97	19	public function replaceObject($service_id, $callable, $is_factory = false)
		1 ignored issue – show introduced 2016-01-07 08:39 UTC by Report Bug Copy Issue Report Type hint "callable" missing for $callable Loading history...
98		{
99	2	if ( !isset($this->container[$service_id]) ) {
100	19	throw new \InvalidArgumentException('Service "' . $service_id . '" not found');
101	1	}
102
103	1	$backup = $this->container->raw($service_id);
104	19	unset($this->container[$service_id]);
105	1	$this->container[$service_id] = $is_factory ? $this->container->factory($callable) : $callable;
106
107	2	return $backup;
108		}
109
110		/**
111		* Prevents cloning.
112		*
113		* @return void
114		*
115		* @codeCoverageIgnore
116		*/
117		private function __clone()
118		{
119
120		}
121
122		}
123

minkphp / phpunit-mink

Pull Request — master (#75)

Application::__clone() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like