Shamaseen\Repository\Tests\Feature\UngenerateFilesTest

Complexity

Total Complexity

3

Size/Duplication

Total Lines	23
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	testUngenerate()	0	8	2
A	__construct()	0	3	1

<?php

namespace Shamaseen\Repository\Tests\Feature;

use Shamaseen\Repository\PathResolver;
use Shamaseen\Repository\Tests\TestCase;

class UngenerateFilesTest extends TestCase
{
    private array $filesToGenerate = ['Controller', 'Repository', 'Model', 'Request', 'Resource', 'Collection', 'Policy', 'Test'];

    protected string $modelName = 'Test';
    protected string $userPath = 'Tests';

    /**
     * @param string $dataName
     */
    public function __construct(?string $name = null, array $data = [], $dataName = '')
    {
        parent::__construct($name, $data, $dataName);

The call to PHPUnit\Framework\TestCase::__construct() has too many arguments starting with $data.

It seems like $name can also be of type null; however, parameter $name of PHPUnit\Framework\TestCase::__construct() does only seem to accept string, maybe add an additional type check?

    }

    public function testUngenerate()
    {
        $this->artisan("ungenerate:repository $this->userPath/$this->modelName")
            ->expectsConfirmation('This will delete Test files and folder, Do you want to continue ?', 'yes');

        foreach ($this->filesToGenerate as $type) {
            $outputPath = $this->generator->absolutePath($this->pathResolver->outputPath($type));
            $this->assertFileDoesNotExist($outputPath);
        }
    }
}