[TASK] Add test for FileFinder

.phpunit.result.cache

@@ -0,0 +1 @@
+{"version":1,"defects":{"FileSystem\\FileFinderTest::test":8,"FileSystem\\FileFinderTest::testThatWeFindAllNonEmptyFilesInGivenDirectories":7,"FileSystem\\FileFinderTest::testThatWeFindAllNonEmptyFilesInGivenDirectoriesWithGivenExtensions":7,"FileSystem\\FileFinderTest::test_that_an_exception_is_thrown_when_directories_are_empty":7},"times":{"FileSystem\\FileFinderTest::test":0.006,"FileSystem\\FileFinderTest::testThatWeFindAllNonEmptyFilesInGivenDirectories":0.003,"FileSystem\\FileFinderTest::testThatWeFindAllNonEmptyFilesInGivenDirectoriesWithGivenExtensions":0.002,"FileSystem\\FileFinderTest::test_that_an_exception_is_thrown_when_directories_are_empty":0.003,"FileSystem\\FileFinderTest::test_that_we_find_all_non_empty_files_in_given_directories":0.004,"FileSystem\\FileFinderTest::test_that_we_find_all_files_with_given_extensions_in_given_directories":0.002,"FileSystem\\FileFinderTest::an_exception_is_thrown_when_directories_are_empty":0.003,"FileSystem\\FileFinderTest::find_all_non_empty_files_in_given_directories":0.003,"FileSystem\\FileFinderTest::find_all_files_with_given_extensions_in_given_directories":0.001,"FileSystem\\FileFinderTest::anExceptionIsThrownWhenDirectoriesAreEmpty":0.003,"FileSystem\\FileFinderTest::findAllNonEmptyFilesInGivenDirectories":0.003,"FileSystem\\FileFinderTest::findAllNonEmptyFilesInGivenDirectoriesWithGivenExtensions":0.002}}

fractor/composer.json

@@ -45,7 +45,7 @@
     },
     "scripts": {
         "analyze:php": "phpstan analyze",
-        "style:php": "ecs",
+        "style:php": "ecs --fix",
         "test:php": "phpunit tests/"
     }
 }

fractor/src/FileSystem/FileFinder.php

@@ -13,15 +13,18 @@ final class FileFinder
      */
     public function findFiles(array $directories, array $fileExtensions): array
     {
+        if ($directories === []) {
+            throw new \UnexpectedValueException('Directories must not be an empty array');
+        }
+
         $finder = Finder::create()
             ->files()
             // skip empty files
             ->size('> 0')
             ->in($directories);
 
-        if ($fileExtensions !== []) {
-            $pattern = sprintf('/(%s)/', implode('|', $fileExtensions));
-            $finder->name($pattern);
+        foreach ($fileExtensions as $fileExtension) {
+            $finder->name('*.' . $fileExtension);
         }
 
         $files = [];

fractor/tests/FileSystem/FileFinderTest.php

@@ -0,0 +1,39 @@
+<?php
+
+declare(strict_types=1);
+
+namespace FileSystem;
+
+use a9f\Fractor\FileSystem\FileFinder;
+use PHPUnit\Framework\Attributes\Test;
+use PHPUnit\Framework\TestCase;
+use UnexpectedValueException;
+
+final class FileFinderTest extends TestCase
+{
+    private FileFinder $subject;
+
+    protected function setUp(): void
+    {
+        $this->subject = new FileFinder();
+    }
+
+    #[Test]
+    public function anExceptionIsThrownWhenDirectoriesAreEmpty(): void
+    {
+        $this->expectException(UnexpectedValueException::class);
+        $this->subject->findFiles([], []);
+    }
+
+    #[Test]
+    public function findAllNonEmptyFilesInGivenDirectories(): void
+    {
+        self::assertCount(4, $this->subject->findFiles([__DIR__ . '/Fixture/DirectorToSearchIn'], []));
+    }
+
+    #[Test]
+    public function findAllNonEmptyFilesInGivenDirectoriesWithGivenExtensions(): void
+    {
+        self::assertCount(2, $this->subject->findFiles([__DIR__ . '/Fixture/DirectorToSearchIn'], ['txt', 'json']));
+    }
+}

fractor/tests/FileSystem/Fixture/DirectorToSearchIn/my_file.txt

@@ -0,0 +1 @@
+My file

fractor/tests/FileSystem/Fixture/DirectorToSearchIn/my_file.yaml

@@ -0,0 +1,2 @@
+yaml:
+  test: 1

fractor/tests/FileSystem/Fixture/DirectorToSearchIn/my_json.json

@@ -0,0 +1,3 @@
+{
+
+}

fractor/tests/FileSystem/Fixture/DirectorToSearchIn/my_txt.yaml

@@ -0,0 +1 @@
+My file