maestriam / samurai

tests/Feature/Facade/CreateIncluderTest.php

<?php

namespace Maestriam\Samurai\Tests\Feature\Facade;

use Maestriam\Samurai\Exceptions\DirectiveExistsException;
use Maestriam\Samurai\Support\Samurai;
use Maestriam\Samurai\Tests\TestCase;

class CreateIncluderTest extends TestCase
{
    public function testCreateIncluder()
    {
        $theme = 'bands/rainbow';
        $name  = 'musics/lady-of-the-lake';

        $this->theme($theme)->findOrCreate();

        $includer = Samurai::theme($theme)->include($name)->create();

The method theme() does not exist on Maestriam\Samurai\Support\Samurai. Since you implemented __callStatic, consider adding a @method annotation.

        $this->assertValidIncluder($includer);
    }

    public function testExistingIncluder()
    {
        $theme = 'bands/rainbow';
        $name  = 'musics/lady-of-the-lake';

        $this->theme($theme)->findOrCreate();

        $this->expectException(DirectiveExistsException::class);

        $includer1 = Samurai::theme($theme)->include($name)->create();

The assignment to $includer1 is dead and can be removed.

        $includer2 = Samurai::theme($theme)->include($name)->create();

The assignment to $includer2 is dead and can be removed.

    }
}