WebStream\Container\Test\ContainerTest

Complexity

Total Complexity

10

Size/Duplication

Total Lines	177
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	3

Importance

Changes

0

10 Methods

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

<?php

For compatibility and reusability of your code, PSR1 recommends that a file should introduce either new symbols (like classes, functions, etc.) or have side-effects (like outputting something, or including other files), but not both at the same time. The first symbol is defined on line 19 and the first side effect is on line 4.

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
{
    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.

    }
}