open-orchestra/open-orchestra-cms-bundle

View on GitHub
BackofficeBundle/Tests/DependencyInjection/OpenOrchestraBackofficeExtensionTest.php

Summary

Maintainability
A
3 hrs
Test Coverage

Method testDefaultConfig has 95 lines of code (exceeds 25 allowed). Consider refactoring.
Open

    public function testDefaultConfig()
    {
        $container = $this->loadContainerFromFile('empty');

        $defaultLanguage = array(

The method testDefaultConfig() has 103 lines of code. Current threshold is set to 100. Avoid really long methods.
Open

    public function testDefaultConfig()
    {
        $container = $this->loadContainerFromFile('empty');

        $defaultLanguage = array(

Call to method getParameter from undeclared class \Symfony\Component\DependencyInjection\ContainerBuilder
Open

        $this->assertEquals($defaultLanguage, $container->getParameter('open_orchestra_backoffice.orchestra_choice.front_language'));

Call to undeclared method \OpenOrchestra\BackofficeBundle\DependencyInjection\OpenOrchestraBackofficeExtensionTest::assertCount
Open

        $this->assertCount(11, $fields);

Call to undeclared method \OpenOrchestra\BackofficeBundle\DependencyInjection\OpenOrchestraBackofficeExtensionTest::assertEquals
Open

        $this->assertEquals($fixedAttributes, $container->getParameter('open_orchestra_backoffice.block.fixed_attributes'));

Call to undeclared method \OpenOrchestra\BackofficeBundle\DependencyInjection\OpenOrchestraBackofficeExtensionTest::assertEquals
Open

        $this->assertEquals($configurationRoles, $container->getParameter('open_orchestra_backoffice.configuration.roles'));

Call to undeclared method \OpenOrchestra\BackofficeBundle\DependencyInjection\OpenOrchestraBackofficeExtensionTest::assertArrayHasKey
Open

        $this->assertArrayHasKey('fake_field', $fields);

Reference to constant NAME from undeclared class \OpenOrchestra\DisplayBundle\DisplayBlock\Strategies\ContentStrategy (Did you mean class \OpenOrchestra\Backoffice\BusinessRules\Strategies\ContentStrategy or class \OpenOrchestra\Backoffice\GenerateForm\Strategies\ContentStrategy)
Open

        ContentStrategy::NAME,

Call to undeclared method \OpenOrchestra\BackofficeBundle\DependencyInjection\OpenOrchestraBackofficeExtensionTest::assertEquals
Open

        $this->assertEquals($choice_frequence, $container->getParameter('open_orchestra_backoffice.choice.frequence'));

Call to method getParameter from undeclared class \Symfony\Component\DependencyInjection\ContainerBuilder
Open

        $this->assertEquals($defaultLanguage, $container->getParameter('open_orchestra_backoffice.orchestra_choice.front_language'));

Call to method getParameter from undeclared class \Symfony\Component\DependencyInjection\ContainerBuilder
Open

        $fields = $container->getParameter('open_orchestra_backoffice.field_types');

Return type of loadContainerFromFile() is undeclared type \Symfony\Component\DependencyInjection\ContainerBuilder
Open

    private function loadContainerFromFile($file)

Call to method __construct from undeclared class \Symfony\Component\DependencyInjection\ContainerBuilder
Open

        $container = new ContainerBuilder();

Class extends undeclared class \OpenOrchestra\BaseBundle\Tests\AbstractTest\AbstractBaseTestCase
Open

class OpenOrchestraBackofficeExtensionTest extends AbstractBaseTestCase

Reference to constant NAME from undeclared class \OpenOrchestra\DisplayBundle\DisplayBlock\Strategies\ContentListStrategy (Did you mean class \OpenOrchestra\Backoffice\GenerateForm\Strategies\ContentListStrategy)
Open

        ContentListStrategy::NAME,

Reference to constant NAME from undeclared class \OpenOrchestra\DisplayBundle\DisplayBlock\Strategies\VideoStrategy (Did you mean class \OpenOrchestra\Backoffice\GenerateForm\Strategies\VideoStrategy)
Open

        VideoStrategy::NAME,

Reference to constant NAME from undeclared class \OpenOrchestra\DisplayBundle\DisplayBlock\Strategies\ContactStrategy (Did you mean class \OpenOrchestra\Backoffice\GenerateForm\Strategies\ContactStrategy)
Open

        ContactStrategy::NAME,

Call to undeclared method \OpenOrchestra\BackofficeBundle\DependencyInjection\OpenOrchestraBackofficeExtensionTest::assertEquals
Open

        $this->assertEquals($defaultLanguage, $container->getParameter('open_orchestra_backoffice.orchestra_choice.front_language'));

Call to method getParameter from undeclared class \Symfony\Component\DependencyInjection\ContainerBuilder
Open

        $this->assertEquals($fixedAttributes, $container->getParameter('open_orchestra_backoffice.block.fixed_attributes'));

Call to undeclared method \OpenOrchestra\BackofficeBundle\DependencyInjection\OpenOrchestraBackofficeExtensionTest::assertCount
Open

        $this->assertCount(15, $options);

Call to undeclared method \OpenOrchestra\BackofficeBundle\DependencyInjection\OpenOrchestraBackofficeExtensionTest::assertArrayHasKey
Open

        $this->assertArrayHasKey('fake_option', $options);

Call to method setParameter from undeclared class \Symfony\Component\DependencyInjection\ContainerBuilder
Open

        $container->setParameter('kernel.environment', 'prod');

Call to method getParameter from undeclared class \Symfony\Component\DependencyInjection\ContainerBuilder
Open

        $this->assertEquals('OpenOrchestraBackofficeBundle::form.html.twig', $container->getParameter('open_orchestra_user.form_template'));

Call to method getParameter from undeclared class \Symfony\Component\DependencyInjection\ContainerBuilder
Open

        $this->assertEquals($fixedAttributes, $container->getParameter('open_orchestra_backoffice.block.fixed_attributes'));

Reference to constant NAME from undeclared class \OpenOrchestra\DisplayBundle\DisplayBlock\Strategies\ConfigurableContentStrategy (Did you mean class \OpenOrchestra\Backoffice\GenerateForm\Strategies\ConfigurableContentStrategy)
Open

        ConfigurableContentStrategy::NAME,

Call to undeclared method \OpenOrchestra\BackofficeBundle\DependencyInjection\OpenOrchestraBackofficeExtensionTest::assertEquals
Open

        $this->assertEquals($blocks, $container->getParameter('open_orchestra.blocks'));

Call to method setParameter from undeclared class \Symfony\Component\DependencyInjection\ContainerBuilder
Open

        $container->setParameter('kernel.debug', false);

Call to method getParameter from undeclared class \Symfony\Component\DependencyInjection\ContainerBuilder
Open

        $this->assertEquals($choice_frequence, $container->getParameter('open_orchestra_backoffice.choice.frequence'));

Call to method compile from undeclared class \Symfony\Component\DependencyInjection\ContainerBuilder
Open

        $container->compile();

Call to undeclared method \OpenOrchestra\BackofficeBundle\DependencyInjection\OpenOrchestraBackofficeExtensionTest::assertEquals
Open

        $this->assertEquals($fixedAttributes, $container->getParameter('open_orchestra_backoffice.block.fixed_attributes'));

Call to method getParameter from undeclared class \Symfony\Component\DependencyInjection\ContainerBuilder
Open

        $this->assertEquals($configurationRoles, $container->getParameter('open_orchestra_backoffice.configuration.roles'));

Call to method getParameter from undeclared class \Symfony\Component\DependencyInjection\ContainerBuilder
Open

        $this->assertEquals($this->defaultBlocks, $container->getParameter('open_orchestra.blocks'));

Call to undeclared method \OpenOrchestra\BackofficeBundle\DependencyInjection\OpenOrchestraBackofficeExtensionTest::assertEquals
Open

        $this->assertEquals($defaultLanguage, $container->getParameter('open_orchestra_backoffice.orchestra_choice.front_language'));

Call to method load from undeclared class \Symfony\Component\DependencyInjection\Loader\YamlFileLoader
Open

        $loader->load($file . '.yml');

Call to undeclared method \OpenOrchestra\BackofficeBundle\DependencyInjection\OpenOrchestraBackofficeExtensionTest::assertEquals
Open

        $this->assertEquals($this->defaultBlocks, $container->getParameter('open_orchestra.blocks'));

Call to method getCompilerPassConfig from undeclared class \Symfony\Component\DependencyInjection\ContainerBuilder
Open

        $container->getCompilerPassConfig()->setRemovingPasses(array());

Call to method getParameter from undeclared class \Symfony\Component\DependencyInjection\ContainerBuilder
Open

        $this->assertEquals('OpenOrchestraBackofficeBundle::layout.html.twig', $container->getParameter('open_orchestra_user.base_layout'));

Call to undeclared method \OpenOrchestra\BackofficeBundle\DependencyInjection\OpenOrchestraBackofficeExtensionTest::assertEquals
Open

        $this->assertEquals('OpenOrchestraBackofficeBundle::layout.html.twig', $container->getParameter('open_orchestra_user.base_layout'));

Call to undeclared method \OpenOrchestra\BackofficeBundle\DependencyInjection\OpenOrchestraBackofficeExtensionTest::assertEquals
Open

        $this->assertEquals('OpenOrchestraBackofficeBundle::form.html.twig', $container->getParameter('open_orchestra_user.form_template'));

Call to method getParameter from undeclared class \Symfony\Component\DependencyInjection\ContainerBuilder
Open

        $options = $container->getParameter('open_orchestra_backoffice.options');

Call to method getCompilerPassConfig from undeclared class \Symfony\Component\DependencyInjection\ContainerBuilder
Open

        $container->getCompilerPassConfig()->setOptimizationPasses(array());

Reference to constant NAME from undeclared class \OpenOrchestra\DisplayBundle\DisplayBlock\Strategies\TinyMCEWysiwygStrategy (Did you mean class \OpenOrchestra\Backoffice\DisplayBlock\Strategies\TinyMCEWysiwygStrategy or class \OpenOrchestra\Backoffice\GenerateForm\Strategies\TinyMCEWysiwygStrategy)
Open

        TinyMCEWysiwygStrategy::NAME,

Call to method setParameter from undeclared class \Symfony\Component\DependencyInjection\ContainerBuilder
Open

        $container->setParameter('kernel.cache_dir', '/tmp');

Call to method __construct from undeclared class \Symfony\Component\Config\FileLocator
Open

        $locator = new FileLocator(__DIR__ . '/Fixtures/config/');

Call to method __construct from undeclared class \Symfony\Component\DependencyInjection\Loader\YamlFileLoader
Open

        $loader = new YamlFileLoader($container, $locator);

Call to method getParameter from undeclared class \Symfony\Component\DependencyInjection\ContainerBuilder
Open

        $this->assertEquals($blocks, $container->getParameter('open_orchestra.blocks'));

Call to method registerExtension from undeclared class \Symfony\Component\DependencyInjection\ContainerBuilder
Open

        $container->registerExtension(new OpenOrchestraBackofficeExtension());

Line exceeds 120 characters; contains 124 characters
Open

        $this->assertEquals($configurationRoles, $container->getParameter('open_orchestra_backoffice.configuration.roles'));

Line exceeds 120 characters; contains 124 characters
Open

        $this->assertEquals($fixedAttributes, $container->getParameter('open_orchestra_backoffice.block.fixed_attributes'));

Line exceeds 120 characters; contains 124 characters
Open

        $this->assertEquals($fixedAttributes, $container->getParameter('open_orchestra_backoffice.block.fixed_attributes'));

Line exceeds 120 characters; contains 140 characters
Open

        $this->assertEquals('OpenOrchestraBackofficeBundle::layout.html.twig', $container->getParameter('open_orchestra_user.base_layout'));

Line exceeds 120 characters; contains 140 characters
Open

        $this->assertEquals('OpenOrchestraBackofficeBundle::form.html.twig', $container->getParameter('open_orchestra_user.form_template'));

Line exceeds 120 characters; contains 133 characters
Open

        $this->assertEquals($defaultLanguage, $container->getParameter('open_orchestra_backoffice.orchestra_choice.front_language'));

Line exceeds 120 characters; contains 133 characters
Open

        $this->assertEquals($defaultLanguage, $container->getParameter('open_orchestra_backoffice.orchestra_choice.front_language'));

Closing brace indented incorrectly; expected 4 spaces, found 3
Open

   }

Line indented incorrectly; expected 4 spaces, found 3
Open

   }

The variable $choice_frequence is not named in camelCase.
Open

    public function testDefaultConfig()
    {
        $container = $this->loadContainerFromFile('empty');

        $defaultLanguage = array(

CamelCaseVariableName

Since: 0.2

It is considered best practice to use the camelCase notation to name variables.

Example

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

Source

The variable $choice_frequence is not named in camelCase.
Open

    public function testDefaultConfig()
    {
        $container = $this->loadContainerFromFile('empty');

        $defaultLanguage = array(

CamelCaseVariableName

Since: 0.2

It is considered best practice to use the camelCase notation to name variables.

Example

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

Source

There are no issues that match your filters.

Category
Status