Yiisoft\Widget\Tests\ContentDecoratorTest::testShouldTriggerInitEvent()

Complexity

Conditions	1
Paths	1

Size

Total Lines	36
Code Lines	24

Duplication

Lines	0
Ratio	0 %

Importance

Changes	2
Bugs	0	Features	0

<?php
    public function testShouldTriggerInitEvent()
    {
        $initTriggered = false;

        // adding some listeners
        $this->listenerProvider->attach(function (BeforeRun $event) use (&$initTriggered) {

The parameter $event is not used and could be removed.

            $initTriggered = true;
        });

        ob_start();
        ob_implicit_flush(0);

        ContentDecorator::begin()
            ->viewFile($this->aliases->get('@view/layout.php'))

It seems like $this->aliases->get('@view/layout.php') can also be of type boolean; however, parameter $value of Yiisoft\Widget\ContentDecorator::viewFile() does only seem to accept string, maybe add an additional type check?

            ->params([])
            ->init();

        echo "\t\t<div class='left-column'>\n";
        echo "\t\t\t<p>This is a left bar!</p>\n";
        echo "\t\t</div>\n\n";
        echo "\t\t<div class='right-column'>\n";
        echo "\t\t\t<p>This is a right bar!</p>\n";
        echo "\t\t</div>\n";

        ContentDecorator::end();

        $this->assertTrue($initTriggered);

        $expected = "\t\t<div class='left-column'>\n" .
                    "\t\t\t<p>This is a left bar!</p>\n" .
                    "\t\t</div>\n\n" .
                    "\t\t<div class='right-column'>\n" .
                    "\t\t\t<p>This is a right bar!</p>\n" .
                    "\t\t</div>\n";

        $this->assertStringContainsString($expected, ob_get_clean());
    }