WebStream\Container\Test\ContainerTest

Complexity

Total Complexity

10

Size/Duplication

Total Lines	177
Duplicated Lines	100 %

Coupling/Cohesion

Components	1
Dependencies	3

Importance

Changes

0

10 Methods

Rating	Name	Duplication	Size	Complexity
A	okContainer()	6	6	1
A	okLength()	7	7	1
A	okRemove()	7	7	1
A	okRegisterPrimitive()	6	6	1
A	okRegisterClosure()	11	11	1
A	okRegisterAsDynamic()	11	11	1
A	okRegisterAsLazy()	12	12	1
A	okRegisterCached()	22	22	1
A	okRegisterUnCached()	20	20	1
A	okUnStrictContainer()	5	5	1

<?php
namespace WebStream\Container\Test;

require_once dirname(__FILE__) . '/../Container.php';
require_once dirname(__FILE__) . '/../ValueProxy.php';
require_once dirname(__FILE__) . '/../Test/Providers/ContainerProvider.php';
require_once dirname(__FILE__) . '/../Test/Modules/InvalidArgumentException.php';

use WebStream\Container\Container;
use WebStream\Container\ValueProxy;
use WebStream\Container\Test\Providers\ContainerProvider;

/**
 * ContainerTest
 * @author Ryuichi TANAKA.
 * @since 2016/08/20
 * @version 0.7
 */
class ContainerTest extends \PHPUnit_Framework_TestCase

This class seems to be duplicated in your project.

{
    use ContainerProvider;

    /**
     * 正常系
     * 値を格納して取り出せること
     * @test
     * @dataProvider valueLazyProvider
     */
    public function okContainer($value, $result)
    {
        $container = new Container();
        $container->test = $value;

The property test does not exist on object<WebStream\Container\Container>. Since you implemented __set, maybe consider adding a @property annotation.

        $this->assertEquals($container->test, $result);

The property test does not exist on object<WebStream\Container\Container>. Since you implemented __get, maybe consider adding a @property annotation.

    }

    /**
     * 正常系
     * 設定した値のカウントが取得できること
     * @test
     */
    public function okLength()
    {
        $container = new Container();
        $container->test1 = 1;

The property test1 does not exist on object<WebStream\Container\Container>. Since you implemented __set, maybe consider adding a @property annotation.

        $container->test2 = 2;

The property test2 does not exist on object<WebStream\Container\Container>. Since you implemented __set, maybe consider adding a @property annotation.

        $this->assertEquals($container->length(), 2);
    }

    /**
     * 正常系
     * 値を削除できること
     * 削除後の要素にアクセスすると例外が発生すること
     * @test
     * @expectedException WebStream\Exception\Extend\InvalidArgumentException
     */
    public function okRemove()
    {
        $container = new Container();
        $container->test1 = 1;

The property test1 does not exist on object<WebStream\Container\Container>. Since you implemented __set, maybe consider adding a @property annotation.

        $container->remove("test1");
        $this->assertNull($container->test1);

The property test1 does not exist on object<WebStream\Container\Container>. Since you implemented __get, maybe consider adding a @property annotation.

    }

    /**
     * 正常系
     * プリミティブ値を登録できること
     * 値はそのまま保存される
     * @test
     * @dataProvider valueDynamicProvider
     */
    public function okRegisterPrimitive($value, $result)
    {
        $container = new Container();
        $container->register("test", $value);
        $this->assertEquals($container->test, $result);

The property test does not exist on object<WebStream\Container\Container>. Since you implemented __get, maybe consider adding a @property annotation.

    }

    /**
     * 正常系
     * クロージャを登録できること
     * 値はそのまま保存される
     * @test
     */
    public function okRegisterClosure()
    {
        $func = function() {
            return "test";
        };
        $container = new Container();
        $container->register("test", $func);

$func is of type object<Closure>, but the function expects a string.

        $this->assertInternalType("object", $container->test);

The property test does not exist on object<WebStream\Container\Container>. Since you implemented __get, maybe consider adding a @property annotation.

        $result = $container->test;

The property test does not exist on object<WebStream\Container\Container>. Since you implemented __set, maybe consider adding a @property annotation.

        $this->assertEquals($result(), "test");
    }

    /**
     * 正常系
     * 即時実行状態で登録されること
     * @test
     */
    public function okRegisterAsDynamic()
    {
        $func = function() {
            echo "evaluated";
            return "test";
        };
        $container = new Container();
        $container->registerAsDynamic("test", $func);
        $this->expectOutputString("evaluated");
        $this->assertEquals($container->test, "test");

The property test does not exist on object<WebStream\Container\Container>. Since you implemented __get, maybe consider adding a @property annotation.

    }

    /**
     * 正常系
     * 遅延実行状態で登録されること
     * @test
     */
    public function okRegisterAsLazy()
    {
        $func = function() {
            echo "evaluated";
            return "test";
        };
        $container = new Container();
        $container->registerAsLazy("test", $func);
        $this->expectOutputString(null);
        $this->assertEquals($container->test, "test");

The property test does not exist on object<WebStream\Container\Container>. Since you implemented __get, maybe consider adding a @property annotation.

        $this->expectOutputString("evaluated");
    }

    /**
     * 正常系
     * コンテナから取得するとき、値はキャッシュされていること
     * @test
     */
    public function okRegisterCached()
    {
        $func = function() {
            echo "evaluated";
            return "test";
        };

        $container = new Container();
        // $container->registerAsLazyUnCached("test", $func);

73% of this comment could be valid code. Did you maybe forget this after debugging?

        ob_start();
        $container->registerAsDynamic("test", $func);
        $actual = ob_get_clean();
        $this->assertEquals($actual, "evaluated");
        $this->assertEquals($container->test, "test");

The property test does not exist on object<WebStream\Container\Container>. Since you implemented __get, maybe consider adding a @property annotation.

        ob_start();
        $container->registerAsLazy("test", $func);
        $this->assertEquals($container->test, "test");

The property test does not exist on object<WebStream\Container\Container>. Since you implemented __get, maybe consider adding a @property annotation.

        $actual = ob_get_clean();
        $this->assertEquals($actual, "evaluated"); // uncached
    }

    /**
     * 正常系
     * コンテナから取得するとき、値はキャッシュされていないこと
     * @test
     */
    public function okRegisterUnCached()
    {
        $func = function() {
            echo "evaluated";
            return "test";
        };

        $container = new Container();
        $container->registerAsLazyUnCached("test", $func);

        ob_start();
        $this->assertEquals($container->test, "test");

The property test does not exist on object<WebStream\Container\Container>. Since you implemented __get, maybe consider adding a @property annotation.

        $actual = ob_get_clean();
        $this->assertEquals($actual, "evaluated");

        ob_start();
        $this->assertEquals($container->test, "test");

The property test does not exist on object<WebStream\Container\Container>. Since you implemented __get, maybe consider adding a @property annotation.

        $actual = ob_get_clean();
        $this->assertEquals($actual, "evaluated"); // uncached
    }

    /**
     * 正常系
     * 未定義の値にアクセスしても例外が発生しないこと
     * @test
     */
    public function okUnStrictContainer()
    {
        $container = new Container(false);
        $this->assertNull($container->undefined);

The property undefined does not exist on object<WebStream\Container\Container>. Since you implemented __get, maybe consider adding a @property annotation.

    }
}