FirephpWrapperTest::setUp() - Code Metrics - mpalourdio/MpaFirephpWrapper - Measure and Improve Code Quality continuously with Scrutinizer

FirephpWrapperTest::setUp() A
last analyzed 2017-02-04 09:40 UTC

↳ Parent: FirephpWrapperTest

Complexity

Conditions	1
Paths	1

Size

Total Lines	4
Code Lines	2

Duplication

Lines	0
Ratio	0 %

Importance

Changes

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

<?php
/*
 * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
 * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY, FITNESS
 * FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE AUTHORS OR
 * COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER
 * IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN
 * CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.
 */

namespace MpaFirephpWrapperTest\Service;

use MpaFirephpWrapper\Options\FirephpWrapperOptions;
use MpaFirephpWrapper\Service\FirephpWrapper;
use MpaFirephpWrapperTest\Util\ServiceManagerFactory;
use PHPUnit\Framework\TestCase;

class FirephpWrapperTest extends TestCase
{
    protected $serviceManager;

    protected function setUp()
    {
        $this->serviceManager = ServiceManagerFactory::getServiceManager();
    }

    public function testFirephpWrapperReallyWrapsFirePHP()
    {
        $wrapper = (new FirephpWrapper($this->serviceManager->get(FirephpWrapperOptions::class)))->getFirephp();
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
        $this->assertInstanceOf('FirePHP', $wrapper);
    }

    public function testNoConfigProvidedIsNotAProblem()
    {
        $config = ServiceManagerFactory::getApplicationConfig();
        unset($config['module_listener_options']['config_glob_paths'][0]);

        $wrapper = (
            new FirephpWrapper(
                ServiceManagerFactory::getServiceManager($config)->get(FirephpWrapperOptions::class)
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
            )
        )
            ->getFirephp();
        $this->assertInstanceOf('FirePHP', $wrapper);
    }
}


1			<?php
2			/*
3			* THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
4			* IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY, FITNESS
5			* FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE AUTHORS OR
6			* COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER
7			* IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN
8			* CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.
9			*/
10
11			namespace MpaFirephpWrapperTest\Service;
12
13			use MpaFirephpWrapper\Options\FirephpWrapperOptions;
14			use MpaFirephpWrapper\Service\FirephpWrapper;
15			use MpaFirephpWrapperTest\Util\ServiceManagerFactory;
16			use PHPUnit\Framework\TestCase;
17
18			class FirephpWrapperTest extends TestCase
19			{
20			protected $serviceManager;
21
22			protected function setUp()
23			{
24			$this->serviceManager = ServiceManagerFactory::getServiceManager();
25			}
26
27			public function testFirephpWrapperReallyWrapsFirePHP()
28			{
29			$wrapper = (new FirephpWrapper($this->serviceManager->get(FirephpWrapperOptions::class)))->getFirephp();
			0 ignored issues – show Documentation introduced 2015-12-05 23:13 UTC by Report Bug Copy Issue Report `$this->serviceManager->g...pWrapperOptions::class)` is of type `object\|array`, but the function expects a `object<MpaFirephpWrapper...\FirephpWrapperOptions>`. It seems like the type of the argument is not accepted by the function/method which you are calling. In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug. We suggest to add an explicit type cast like in the following example: function acceptsInteger($int) { } $x = '123'; // string "123" // Instead of acceptsInteger($x); // we recommend to use acceptsInteger((integer) $x); Loading history...
30			$this->assertInstanceOf('FirePHP', $wrapper);
31			}
32
33			public function testNoConfigProvidedIsNotAProblem()
34			{
35			$config = ServiceManagerFactory::getApplicationConfig();
36			unset($config['module_listener_options']['config_glob_paths'][0]);
37
38			$wrapper = (
39			new FirephpWrapper(
40			ServiceManagerFactory::getServiceManager($config)->get(FirephpWrapperOptions::class)
			0 ignored issues – show Documentation introduced 2015-12-05 23:13 UTC by Report Bug Copy Issue Report `\MpaFirephpWrapperTest\U...pWrapperOptions::class)` is of type `object\|array`, but the function expects a `object<MpaFirephpWrapper...\FirephpWrapperOptions>`. It seems like the type of the argument is not accepted by the function/method which you are calling. In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug. We suggest to add an explicit type cast like in the following example: function acceptsInteger($int) { } $x = '123'; // string "123" // Instead of acceptsInteger($x); // we recommend to use acceptsInteger((integer) $x); Loading history...
41			)
42			)
43			->getFirephp();
44			$this->assertInstanceOf('FirePHP', $wrapper);
45			}
46			}
47

mpalourdio / MpaFirephpWrapper

GitHub Access Token became invalid

FirephpWrapperTest::setUp() A last analyzed 2017-02-04 09:40 UTC

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like

FirephpWrapperTest::setUp() A
last analyzed 2017-02-04 09:40 UTC