Yiisoft\Widget\Tests\ContentDecoratorTest

Complexity

Total Complexity

1

Size/Duplication

Total Lines	41
Duplicated Lines	0 %

Importance

Changes	2
Bugs	0	Features	0

1 Method

Rating	Name	Duplication	Size	Complexity
A	testShouldTriggerInitEvent()	0	36	1

<?php

namespace Yiisoft\Widget\Tests;

use Yiisoft\Tests\TestCase;
use Yiisoft\Widget\ContentDecorator;
use Yiisoft\Widget\Event\BeforeRun;

/**
 * @group widgets
 */
class ContentDecoratorTest extends TestCase
{
    /**
     * @see https://github.com/yiisoft/yii2/issues/15536
     */
    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());
    }
}