BridgeTest - Code Metrics - Inspection of "[Composer][Travis] Drop PHP version 5.5, travis im..." - SuperdeskWebPublisher/SWPBridgeBundle - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 0708d4...99f0e4 )

by Rafał

created 2016-06-01 10:09 UTC

BridgeTest A

↳ Parent: Project

Complexity

Total Complexity

Size/Duplication

Total Lines	32
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	2

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
wmc	4
c	1
b	0
f	0
lcom	0
cbo	2
dl	0
loc	32
rs	10

4 Methods

Rating	Name	Size	Complexity
A	testIndexForCallToInvalidEndpoints()	6	1
A	getConfigs()	6	1
A	setUpBeforeClass()	4	1
A	tearDownAfterClass()	4	1

<?php

/**
 * This file is part of the Superdesk Web Publisher Bridge Bundle.
 *
 * Copyright 2016 Sourcefabric z.u. and contributors.
 *
 * For the full copyright and license information, please see the
 * AUTHORS and LICENSE files distributed with this source code.
 *
 * @copyright 2016 Sourcefabric z.ú.
 * @license http://www.superdesk.org/license
 */
namespace SWP\Bundle\BridgeBundle\Tests\Functional;

class BridgeTest extends WebTestCase
{
    const TEST_CASE_NAME = 'Bridge';

    /**
     * @expectedException \Superdesk\ContentApiSdk\Exception\ContentApiException
     * @dataProvider getConfigs
     */
    public function testIndexForCallToInvalidEndpoints($config)
    {
        $client = $this->createClient(array('test_case' => self::TEST_CASE_NAME, 'root_config' => $config));

        $client->request('GET', '/bridge/invalid_endpoint/');
    }

    public function getConfigs()
    {
        return array(
            array('config.yml'),
        );
    }

    public static function setUpBeforeClass()
    {
        parent::deleteTmpDir(self::TEST_CASE_NAME);
class Daddy
{
    protected function getFirstName()
    {
        return "Eidur";
    }

    protected function getSurName()
    {
        return "Gudjohnsen";
    }
}

class Son
{
    public function getFirstName()
    {
        return parent::getSurname();
    }
}
    }

    public static function tearDownAfterClass()
    {
        parent::deleteTmpDir(self::TEST_CASE_NAME);
class Daddy
{
    protected function getFirstName()
    {
        return "Eidur";
    }

    protected function getSurName()
    {
        return "Gudjohnsen";
    }
}

class Son
{
    public function getFirstName()
    {
        return parent::getSurname();
    }
}
    }
}


1			<?php
2
3			/**
4			* This file is part of the Superdesk Web Publisher Bridge Bundle.
5			*
6			* Copyright 2016 Sourcefabric z.u. and contributors.
7			*
8			* For the full copyright and license information, please see the
9			* AUTHORS and LICENSE files distributed with this source code.
10			*
11			* @copyright 2016 Sourcefabric z.ú.
12			* @license http://www.superdesk.org/license
13			*/
14			namespace SWP\Bundle\BridgeBundle\Tests\Functional;
15
16			class BridgeTest extends WebTestCase
17			{
18			const TEST_CASE_NAME = 'Bridge';
19
20			/**
21			* @expectedException \Superdesk\ContentApiSdk\Exception\ContentApiException
22			* @dataProvider getConfigs
23			*/
24			public function testIndexForCallToInvalidEndpoints($config)
25			{
26			$client = $this->createClient(array('test_case' => self::TEST_CASE_NAME, 'root_config' => $config));
27
28			$client->request('GET', '/bridge/invalid_endpoint/');
29			}
30
31			public function getConfigs()
32			{
33			return array(
34			array('config.yml'),
35			);
36			}
37
38			public static function setUpBeforeClass()
39			{
40			parent::deleteTmpDir(self::TEST_CASE_NAME);
			0 ignored issues – show Comprehensibility Bug introduced 2016-06-01 10:12 UTC by Report Bug Copy Issue Report It seems like you call parent on a different method (`deleteTmpDir()` instead of `setUpBeforeClass()`). Are you sure this is correct? If so, you might want to change this to `$this->deleteTmpDir()`. This check looks for a call to a parent method whose name is different than the method from which it is called. Consider the following code: class Daddy { protected function getFirstName() { return "Eidur"; } protected function getSurName() { return "Gudjohnsen"; } } class Son { public function getFirstName() { return parent::getSurname(); } } The `getFirstName()` method in the `Son` calls the wrong method in the parent class. Loading history...
41			}
42
43			public static function tearDownAfterClass()
44			{
45			parent::deleteTmpDir(self::TEST_CASE_NAME);
			0 ignored issues – show Comprehensibility Bug introduced 2016-06-01 10:12 UTC by Report Bug Copy Issue Report It seems like you call parent on a different method (`deleteTmpDir()` instead of `tearDownAfterClass()`). Are you sure this is correct? If so, you might want to change this to `$this->deleteTmpDir()`. This check looks for a call to a parent method whose name is different than the method from which it is called. Consider the following code: class Daddy { protected function getFirstName() { return "Eidur"; } protected function getSurName() { return "Gudjohnsen"; } } class Son { public function getFirstName() { return parent::getSurname(); } } The `getFirstName()` method in the `Son` calls the wrong method in the parent class. Loading history...
46			}
47			}
48

SuperdeskWebPublisher / SWPBridgeBundle

Push — master ( 0708d4...99f0e4 )

BridgeTest A

Complexity

Size/Duplication

Coupling/Cohesion

Importance

4 Methods

Duplication Side-by-Side

Filter issues like