Add test for trailing slash

Author: dantleech

src/TextUI/Configuration/README.md

@@ -44,6 +44,8 @@ The Factory:
 - create a new PHPUnit `Iterator` passing a recursive iterator iterator
   directory iterator... (the iterator that iterates over the directories) that
   **follows symlinks and skips dots**.
+- the iterator also Excludes the excluded `<directory>` elements **but does
+  not take into account the prefixes or suffixes**.
 
 The Iterator:
 

src/TextUI/Configuration/SourceFilter.php

@@ -63,7 +63,7 @@ public function __construct(Source $source)
     public function includes(string $path): bool
     {
         $included = false;
-        $dirPath = dirname($path) . '/';
+        $dirPath = rtrim(dirname($path), '/') . '/';
         $filename = basename($path);
         foreach ($this->source->includeFiles() as $file) {
             if ($file->path() === $path) {
@@ -100,12 +100,12 @@ public function includes(string $path): bool
      */
     public static function toGlob(FilterDirectory $directory): string
     {
-        $path = $directory->path();
+        $path = rtrim($directory->path(), '/');
 
         return sprintf(
             '{(%s)|(%s)}',
-            Glob::toRegEx(sprintf('%s/**/*', $directory->path()), 0, ''),
-            Glob::toRegEx(sprintf('%s/*', $directory->path()), 0, ''),
+            Glob::toRegEx(sprintf('%s/**/*', $path), 0, ''),
+            Glob::toRegEx(sprintf('%s/*', $path), 0, ''),
         );
     }
 

tests/unit/TextUI/SourceFilterTest.php

@@ -80,6 +80,18 @@ public static function provider(): array
                     ),
                 ),
             ],
+            'file included using directory with trailing slash' => [
+                [
+                    self::fixturePath('a/PrefixSuffix.php') => true,
+                ],
+                self::createSource(
+                    includeDirectories: FilterDirectoryCollection::fromArray(
+                        [
+                            new FilterDirectory(self::fixturePath() . '/', '', '.php'),
+                        ],
+                    ),
+                ),
+            ],
             'file included using directory, but excluded using file' => [
                 [
                     self::fixturePath('a/PrefixSuffix.php') => false,