Merge pull request #12 from naxvog/wildcard-ns

Allow usage of namespace selection using wildcards

phpunit.xml

@@ -12,7 +12,6 @@
          processIsolation="false"
          stopOnError="false"
          stopOnFailure="false"
-         cacheDirectory=".phpunit.cache"
          backupStaticProperties="false"
          displayDetailsOnIncompleteTests="true"
          displayDetailsOnSkippedTests="true"

src/Factories/LayerFactory.php

@@ -60,7 +60,9 @@ public function make(LayerOptions $options, string $name, bool $onlyUserDefinedU
             return $object;
         }, $this->objectsStorage->allByNamespace($name, $onlyUserDefinedUses));
 
-        $layer = Layer::fromBase($objects)->leaveByNameStart($name);
+        $layer = str_contains($name, '*')
+            ? Layer::fromBase($objects)->leaveByNameRegex('/'.strtr(preg_quote($name, '/'), ['\\*' => '[^\\\\]+']).'/')
+            : Layer::fromBase($objects)->leaveByNameStart($name);
 
         foreach ($options->exclude as $exclude) {
             $layer = $layer->excludeByNameStart($exclude);

src/Layer.php

@@ -6,14 +6,14 @@
 
 use IteratorAggregate;
 use PHPUnit\Architecture\Elements\Layer\Layer as BaseLayer;
+use PHPUnit\Architecture\Elements\Layer\LayerLeave;
 use PHPUnit\Architecture\Elements\ObjectDescription;
 use Traversable;
 
 /**
  * @method Layer assertDoesNotDependOn(string ...$objects)
  * @method Layer excludeByNameStart(string $name)
- * @method Layer exclude(callable $callback)
- * @method Layer leaveByNameStart(string $name)
+ * @mixin LayerLeave
  *
  * @implements IteratorAggregate<int, ObjectDescription>
  */

src/Repositories/ObjectsRepository.php

@@ -103,7 +103,7 @@ public function allByNamespace(string $namespace, bool $onlyUserDefinedUses = tr
 
             $objectsPerPrefix = array_values(array_filter(array_reduce($directories, fn (array $files, string $fileOrDirectory): array => array_merge($files, array_values(array_map(
                 static fn (SplFileInfo $file): ?ObjectDescription => ObjectDescriptionFactory::make($file->getPathname(), $onlyUserDefinedUses),
-                is_dir($fileOrDirectory) ? iterator_to_array(Finder::create()->files()->in($fileOrDirectory)->name('*.php')) : [new SplFileInfo($fileOrDirectory)],
+                is_dir($fileOrDirectory) || str_contains($fileOrDirectory, '*') ? iterator_to_array(Finder::create()->files()->in($fileOrDirectory)->name('*.php')) : [new SplFileInfo($fileOrDirectory)],
             ))), [])));
 
             $objects = [...$objects, ...$this->cachedObjectsPerPrefix[$prefix][(int) $onlyUserDefinedUses] = $objectsPerPrefix];
@@ -151,13 +151,15 @@ private function directoriesByNamespace(string $name): array
 
                 $directoriesByNamespace[$name] = [...$directoriesByNamespace[$name] ?? [], ...array_values(array_filter(array_map(static function (string $directory) use ($prefix): string {
                     $fileOrDirectory = $directory.DIRECTORY_SEPARATOR.$prefix;
-
                     if (is_dir($fileOrDirectory)) {
                         return $fileOrDirectory;
                     }
+                    if (str_contains($fileOrDirectory, '*')) {
+                        return $fileOrDirectory;
+                    }
 
                     return $fileOrDirectory.'.php';
-                }, $directories), static fn (string $fileOrDirectory): bool => is_dir($fileOrDirectory) || file_exists($fileOrDirectory)))];
+                }, $directories), static fn (string $fileOrDirectory): bool => is_dir($fileOrDirectory) || str_contains($fileOrDirectory, '*') || file_exists($fileOrDirectory)))];
             }
         }
 

tests/Fixtures/Domains/A/Contracts/Models/Bazable.php

@@ -0,0 +1,10 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Tests\Fixtures\Domains\A\Contracts\Models;
+
+interface Bazable
+{
+    // ...
+}

tests/Fixtures/Domains/A/Models/Article.php

@@ -0,0 +1,12 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Tests\Fixtures\Domains\A\Models;
+
+use Tests\Fixtures\Domains\A\Contracts\Models\Bazable;
+
+final class Article implements Bazable
+{
+    // ...
+}

tests/Fixtures/Domains/B/Contracts/Models/Bazable.php

@@ -0,0 +1,10 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Tests\Fixtures\Domains\B\Contracts\Models;
+
+interface Bazable
+{
+    // ...
+}

tests/Fixtures/Domains/B/Models/Article.php

@@ -0,0 +1,12 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Tests\Fixtures\Domains\B\Models;
+
+use Tests\Fixtures\Domains\B\Contracts\Models\Bazable;
+
+final class Article implements Bazable
+{
+    // ...
+}

tests/Layer.php

@@ -1,5 +1,9 @@
 <?php
 
+use Tests\Fixtures\Domains\A\Contracts\Models\Bazable as BazableDomainA;
+use Tests\Fixtures\Domains\A\Models\Article as ArticleDomainA;
+use Tests\Fixtures\Domains\B\Contracts\Models\Bazable as BazableDomainB;
+use Tests\Fixtures\Domains\B\Models\Article as ArticleDomainB;
 use Tests\Fixtures\Enums\Color;
 use Tests\Fixtures\Enums\ColorThatDependsOnColor;
 use Tests\Fixtures\Misc\DependOnGlobalFunctions;
@@ -23,6 +27,24 @@
         ->toOnlyUse('Pest\Support\Str');
 });
 
+it('loads namespaces', function () {
+    expect('Tests\Fixtures\Domains\A\Models')
+        ->getTargets()
+        ->toBe([ArticleDomainA::class]);
+});
+
+it('loads namespaces using wildcards', function () {
+    expect('Tests\Fixtures\Domains\*\Models')
+        ->getTargets()
+        ->toBe([ArticleDomainA::class, ArticleDomainB::class]);
+});
+
+it('loads namespaces using multiple wildcards', function () {
+    expect('Tests\Fixtures\Domains\*\*\Models')
+        ->getTargets()
+        ->toBe([BazableDomainA::class, BazableDomainB::class]);
+});
+
 it('does support enums', function () {
     expect(Color::class)->toUseNothing()
         ->and(ColorThatDependsOnColor::class)->toUse([Color::class]);

tests/Pest.php

@@ -1,6 +1,8 @@
 <?php
 
 use Pest\Arch\Exceptions\ArchExpectationFailedException;
+use Pest\Arch\Expectations\Targeted;
+use Pest\Arch\Objects\ObjectDescription;
 
 uses()->beforeEach(function () {
     $this->arch()->ignore([
@@ -23,3 +25,20 @@
             ->toBe($line);
     });
 });
+
+expect()->extend('getTargets', function () {
+    $classes = [];
+    Targeted::make(
+        $this,
+        function (ObjectDescription $object) use (&$classes): bool {
+            $classes[] = $object->name;
+
+            return true;
+        },
+        '',
+        fn ($path) => 0,
+    );
+    $this->value = $classes;
+
+    return $this;
+});