Add tests for DirectoryScopeDeterminer and PhpFileFinder

Author: navarr

src/Controller/WhyBlockCommandController.php

@@ -42,8 +42,6 @@
 
 class WhyBlockCommandController extends Command
 {
-    private const ALL_DEPS = 'include-all-dependencies';
-    private const ROOT_DEPS = 'include-root-dependencies';
     private const LEGACY_ANNOTATION = 'include-legacy-annotations';
     private const FAIL_ON_ERROR = 'fail-on-error';
     private const OUTPUT_FORMAT = 'format';

tests/ScopeDeterminer/DirectoryScopeDeterminerTest.php

@@ -0,0 +1,44 @@
+<?php
+
+/**
+ * @copyright 2021 Navarr Barnier. All Rights Reserved.
+ */
+
+declare(strict_types=1);
+
+namespace Navarr\Depends\Test\ScopeDeterminer;
+
+use Navarr\Depends\ScopeDeterminer\DirectoryScopeDeterminer;
+use Navarr\Depends\ScopeDeterminer\PhpFileFinder;
+use PHPUnit\Framework\TestCase;
+
+class DirectoryScopeDeterminerTest extends TestCase
+{
+    public function testDirectoryIsGivenToPhpFileFinder(): void
+    {
+        $dir = uniqid();
+        $finder = $this->createMock(PhpFileFinder::class);
+        $finder->expects($this->once())
+            ->method('findAll')
+            ->with($dir)
+            ->willReturn([]);
+
+        $determiner = new DirectoryScopeDeterminer($finder, $dir);
+        $determiner->getFiles();
+    }
+
+    public function testResultOfPhpFileFinderIsProvidedBack(): void
+    {
+        $results = [
+            uniqid(),
+            uniqid()
+        ];
+
+        $finder = $this->createMock(PhpFileFinder::class);
+        $finder->method('findAll')
+            ->willReturn($results);
+
+        $determiner = new DirectoryScopeDeterminer($finder, uniqid());
+        $this->assertEquals($results, $determiner->getFiles());
+    }
+}

tests/ScopeDeterminer/PhpFileFinderTest.php

@@ -0,0 +1,51 @@
+<?php
+
+/**
+ * @copyright 2021 Navarr Barnier. All Rights Reserved.
+ */
+
+declare(strict_types=1);
+
+namespace Navarr\Depends\Test\ScopeDeterminer;
+
+use DI\Container;
+use Navarr\Depends\ScopeDeterminer\PhpFileFinder;
+use PHPUnit\Framework\TestCase;
+
+class PhpFileFinderTest extends TestCase
+{
+    public function testFindAll()
+    {
+        $directory = implode(
+            DIRECTORY_SEPARATOR,
+            [
+                __DIR__,
+                '..',
+                '_data',
+                'phpFileFinder',
+            ]
+        );
+
+        $files = array_map(
+            'realpath',
+            [
+                $directory . DIRECTORY_SEPARATOR . 'file1.php',
+                $directory . DIRECTORY_SEPARATOR . 'recursion1/file4.php',
+                $directory . DIRECTORY_SEPARATOR . 'recursion1/anotherFile.php',
+                $directory . DIRECTORY_SEPARATOR . 'recursion1/recursion2/file3.php',
+                $directory . DIRECTORY_SEPARATOR . 'recursion1/recursion2/recursion3/file2.php',
+            ]
+        );
+
+        $container = new Container();
+        $finder = $container->get(PhpFileFinder::class);
+        $results = $finder->findAll($directory);
+
+        $this->assertIsArray($results);
+        $this->assertCount(5, $results);
+
+        foreach ($files as $file) {
+            $this->assertContains($file, $results);
+        }
+    }
+}

tests/_data/phpFileFinder/file1.php

@@ -0,0 +1,3 @@
+<?php
+
+// a php file in the base directory searched

tests/_data/phpFileFinder/recursion1/anotherFile.php

@@ -0,0 +1,3 @@
+<?php
+
+// another file in the first directory deep

tests/_data/phpFileFinder/recursion1/file4.php

@@ -0,0 +1,3 @@
+<?php
+
+// a file one directory deep

tests/_data/phpFileFinder/recursion1/recursion2/file3.php

@@ -0,0 +1,3 @@
+<?php
+
+// a file two directories deep

tests/_data/phpFileFinder/recursion1/recursion2/recursion3/file2.php

@@ -0,0 +1,3 @@
+<?php
+
+// a file several directories deep