DaveLiddament\StaticAnalysisResultsBaseliner\Tests\Integration\TestDirectoryTrait::createTestDirectory()

Complexity

Conditions	1
Paths	1

Size

Total Lines	9
Code Lines	7

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

<?php
    private function createTestDirectory(): void
    {
        $dateTimeFolderName = date('Ymd_His');
        $random = random_int(10000, 99999);
        $testDirectory = __DIR__."/../scratchpad/{$dateTimeFolderName}{$random}";
        $this->fileSystem->mkdir($testDirectory);
        $cwd = getcwd();
        $this->assertNotFalse($cwd);

It seems like assertNotFalse() must be provided by classes using this trait. How about adding it as abstract method to this trait?

        $this->projectRoot = ProjectRoot::fromProjectRoot($testDirectory, $cwd);

The property projectRoot does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly.

    }