Merge pull request #197 from wayofdev/feat/testing-helpers

Author: lotyp

src/Bridge/Laravel/Providers/CycleServiceProvider.php

@@ -41,10 +41,13 @@ public function boot(): void
 
     public function register(): void
     {
-        $this->mergeConfigFrom(
-            __DIR__ . '/../../../../config/cycle.php',
-            Registrator::CFG_KEY
-        );
+        // @phpstan-ignore-next-line
+        if (! $this->app->configurationIsCached()) {
+            $this->mergeConfigFrom(
+                __DIR__ . '/../../../../config/cycle.php',
+                Registrator::CFG_KEY
+            );
+        }
 
         $registrators = [
             Registrators\RegisterConfigs::class,

src/Testing/Concerns/InteractsWithDatabase.php

@@ -5,6 +5,7 @@
 namespace WayOfDev\Cycle\Testing\Concerns;
 
 use Cycle\Database\DatabaseProviderInterface;
+use Illuminate\Support\Facades\File;
 use PHPUnit\Framework\Constraint\Constraint;
 use PHPUnit\Framework\Constraint\LogicalNot as ReverseConstraint;
 use WayOfDev\Cycle\Testing\Constraints\CountInDatabase;
@@ -79,4 +80,12 @@ protected function assertDatabaseEmpty($table, $connection = null): static
 
         return $this;
     }
+
+    protected function cleanupMigrations(string $pathGlob): void
+    {
+        $files = File::glob($pathGlob);
+        foreach ($files as $file) {
+            File::delete($file);
+        }
+    }
 }

tests/src/TestCase.php

@@ -7,9 +7,7 @@
 use Faker\Factory as FakerFactory;
 use Faker\Generator;
 use Illuminate\Contracts\Console\Kernel;
-use Illuminate\Database\Eloquent\Factories\Factory;
 use Illuminate\Support\Facades\Artisan;
-use Illuminate\Support\Facades\File;
 use Orchestra\Testbench\TestCase as OrchestraTestCase;
 use Spatie\LaravelRay\RayServiceProvider;
 use WayOfDev\Cycle\Bridge\Laravel\Providers\CycleServiceProvider;
@@ -29,6 +27,8 @@ class TestCase extends OrchestraTestCase
     use InteractsWithDatabase;
     use RefreshDatabase;
 
+    protected ?string $migrationsPath = null;
+
     final protected static function faker(string $locale = 'en_US'): Generator
     {
         /** @var array<string, Generator> $fakers */
@@ -45,27 +45,24 @@ protected function setUp(): void
     {
         parent::setUp();
 
-        $this->cleanupMigrations();
+        $this->migrationsPath = __DIR__ . '/../app/database/migrations/cycle';
+        $this->cleanupMigrations($this->migrationsPath . '/*.php');
         $this->refreshDatabase();
 
-        Factory::guessFactoryNamesUsing(
-            static fn (string $modelName) => 'WayOfDev\\Laravel\\Cycle\\Database\\Factories\\' . class_basename($modelName) . 'Factory'
-        );
-
         if (app()->environment() === 'testing') {
             config()->set([
                 'cycle.tokenizer.directories' => array_merge(
                     config('cycle.tokenizer.directories'),
                     [__DIR__ . '/../app/Entities']
                 ),
-                'cycle.migrations.directory' => __DIR__ . '/../app/database/migrations/cycle',
+                'cycle.migrations.directory' => $this->migrationsPath,
             ]);
         }
     }
 
     protected function tearDown(): void
     {
-        $this->cleanupMigrations();
+        $this->cleanupMigrations($this->migrationsPath . '/*.php');
         $this->refreshDatabase();
 
         parent::tearDown();
@@ -105,14 +102,4 @@ protected function getPackageProviders($app): array
             RayServiceProvider::class,
         ];
     }
-
-    protected function cleanupMigrations(): void
-    {
-        $path = __DIR__ . '/../app/database/migrations/cycle/*.php';
-
-        $files = File::glob($path);
-        foreach ($files as $file) {
-            File::delete($file);
-        }
-    }
 }