Merge pull request #102 from TheDragonCode/3.x

Fixed trying to execute files other than PHP

Author: andrey-helldar

src/Processors/Processor.php

@@ -24,33 +24,33 @@ abstract class Processor
 {
     use Artisan;
 
-    abstract public function handle(): void;
-
     public function __construct(
-        protected Options $options,
-        protected InputInterface $input,
-        protected OutputStyle $output,
-        protected Config $config,
+        protected Options          $options,
+        protected InputInterface   $input,
+        protected OutputStyle      $output,
+        protected Config           $config,
         protected ActionRepository $repository,
-        protected Git $git,
-        protected File $file,
-        protected Migrator $migrator,
-        protected Notification $notification,
-        protected Dispatcher $events,
-        protected Sorter $sorter
+        protected Git              $git,
+        protected File             $file,
+        protected Migrator         $migrator,
+        protected Notification     $notification,
+        protected Dispatcher       $events,
+        protected Sorter           $sorter
     ) {
         $this->notification->setOutput($this->output, $this->options->silent);
         $this->repository->setConnection($this->options->connection);
         $this->migrator->setConnection($this->options->connection)->setOutput($this->output);
     }
 
+    abstract public function handle(): void;
+
     protected function getFiles(string $path, ?Closure $filter = null): array
     {
         $file = Str::finish($path, '.php');
 
         $files = $this->isFile($file) ? [$file] : $this->file->names($path, $filter, true);
 
-        $files = Arr::filter($files, fn (string $path) => ! Str::contains($path, $this->config->exclude()));
+        $files = Arr::filter($files, fn (string $path) => Str::endsWith($path, '.php') && ! Str::contains($path, $this->config->exclude()));
 
         return Arr::of($this->sorter->byValues($files))
             ->map(fn (string $value) => Str::before($value, '.php'))

tests/Commands/MigrateTest.php

@@ -652,4 +652,21 @@ public function testFileExclusion()
         $this->assertDatabaseMigrationDoesntLike($this->table, 'sub_path/2021_12_15_205804_baz');
         $this->assertDatabaseMigrationHas($this->table, 'sub_path/2022_10_27_230732_foo');
     }
+
+    public function testEmptyDirectory()
+    {
+        $this->copyEmptyDirectory();
+
+        $table = 'every_time';
+
+        $this->artisan(Names::INSTALL)->assertExitCode(0);
+
+        $this->assertDatabaseCount($table, 0);
+        $this->assertDatabaseCount($this->table, 0);
+        $this->assertDatabaseMigrationDoesntLike($this->table, $table);
+        $this->artisan(Names::MIGRATE)->assertExitCode(0);
+
+        $this->assertDatabaseCount($table, 0);
+        $this->assertDatabaseCount($this->table, 0);
+    }
 }

tests/Concerns/Files.php

@@ -22,6 +22,14 @@ protected function copyFiles(): void
         );
     }
 
+    protected function copyEmptyDirectory(): void
+    {
+        File::copyDirectory(
+            __DIR__ . '/../fixtures/app/empty',
+            $this->targetDirectory()
+        );
+    }
+
     protected function copyDI(): void
     {
         File::copyDirectory(