ShopwarePlugins\Connect\Tests\AbstractConnectUnitTest

Complexity

Total Complexity

4

Size/Duplication

Total Lines	34
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	1

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	setUp()	0	4	1
A	tearDown()	0	4	1
A	killShopwareFunctionBefore()	0	5	1
A	restoreShopwareFunctionAfter()	0	4	1

<?php
/**
 * (c) shopware AG <info@shopware.com>
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 */

namespace ShopwarePlugins\Connect\Tests;

/**
 * Every unit test should depend on this abstract class. It is only used to prevent a call to the Shopware() singleton.
 */
abstract class AbstractConnectUnitTest extends \PHPUnit_Framework_TestCase
{
    /**
     * @var \Shopware|null
     */
    private static $appState = null;

    protected function setUp()
    {
        @trigger_error('Please use @before annotation');

It seems like you do not handle an error condition here. This can introduce security issues, and is generally not recommended.

    }

    protected function tearDown()
    {
        @trigger_error('Please use @after annotation');

It seems like you do not handle an error condition here. This can introduce security issues, and is generally not recommended.

    }

    /**
     * @before
     */
    public function killShopwareFunctionBefore()
    {
        self::$appState = Shopware();
        Shopware(new EmptyShopwareApplication());
    }

    /**
     * @after
     */
    public function restoreShopwareFunctionAfter()
    {
        Shopware(self::$appState);
    }
}

class EmptyShopwareApplication

PSR1 recommends that each class should be in its own file to aid autoloaders.

{
    public function __call($name, $arguments)
    {
        throw new RestrictedCallException('Restricted to call ' . $name . ' because you should not have a test kernel in this test case.');
    }
}

class RestrictedCallException extends \RuntimeException

PSR1 recommends that each class should be in its own file to aid autoloaders.

{
}