Merge branch 'main' of github.com:stefanzweifel/laravel-backup-restore

stefanzweifel · Aug 7, 2024 · 5d671a5 · 5d671a5
2 parents 1b53f88 + 259fcb4
commit 5d671a5
Show file tree

Hide file tree

Showing 7 changed files with 12 additions and 18 deletions.
diff --git a/src/DbImporterFactory.php b/src/DbImporterFactory.php
@@ -42,13 +42,13 @@ protected static function forDriver(string $driver): DbImporter
         $driver = Str::lower($driver);
 
         if (isset(static::$custom[$driver])) {
-            return new static::$custom[$driver]();
+            return new static::$custom[$driver];
         }
 
         return match ($driver) {
-            'mysql' => new MySql(),
-            'pgsql' => new PostgreSql(),
-            'sqlite' => new Sqlite(),
+            'mysql' => new MySql,
+            'pgsql' => new PostgreSql,
+            'sqlite' => new Sqlite,
             default => throw CannotCreateDbImporter::unsupportedDriver($driver),
         };
     }

diff --git a/src/Events/DatabaseDumpImportWasSuccessful.php b/src/Events/DatabaseDumpImportWasSuccessful.php
@@ -6,7 +6,5 @@
 
 class DatabaseDumpImportWasSuccessful
 {
-    public function __construct(readonly public string $absolutePathToDump)
-    {
-    }
+    public function __construct(readonly public string $absolutePathToDump) {}
 }
diff --git a/src/Events/DatabaseReset.php b/src/Events/DatabaseReset.php
@@ -8,7 +8,5 @@
 
 class DatabaseReset
 {
-    public function __construct(readonly public PendingRestore $pendingRestore)
-    {
-    }
+    public function __construct(readonly public PendingRestore $pendingRestore) {}
 }
diff --git a/src/Events/DatabaseRestored.php b/src/Events/DatabaseRestored.php
@@ -8,7 +8,5 @@
 
 class DatabaseRestored
 {
-    public function __construct(readonly public PendingRestore $pendingRestore)
-    {
-    }
+    public function __construct(readonly public PendingRestore $pendingRestore) {}
 }
diff --git a/tests/Actions/CheckDependenciesActionTest.php b/tests/Actions/CheckDependenciesActionTest.php
@@ -13,7 +13,7 @@
 })->with(['mysql', 'pgsql', 'sqlite']);
 
 it('throws exception if CLI dependency for given connection can not be found', function () {
-    DbImporterFactory::extend('sqlsrv', new class() extends DbImporter
+    DbImporterFactory::extend('sqlsrv', new class extends DbImporter
     {
         public function getImportCommand(string $dumpFile, string $connection): string
         {

diff --git a/tests/DbImporterFactoryTest.php b/tests/DbImporterFactoryTest.php
@@ -36,7 +36,7 @@
 ]);
 
 it('returns custom db importer instance for the given database driver', function () {
-    DbImporterFactory::extend('sqlsrv', new class() extends DbImporter
+    DbImporterFactory::extend('sqlsrv', new class extends DbImporter
     {
         public function getImportCommand(string $dumpFile, string $connection): string
         {

diff --git a/tests/HealthChecks/ResultTest.php b/tests/HealthChecks/ResultTest.php
@@ -7,7 +7,7 @@
 use Wnx\LaravelBackupRestore\HealthChecks\Result;
 
 it('returns ok result by default', function () {
-    $check = new DatabaseHasTables();
+    $check = new DatabaseHasTables;
     $result = Result::make($check);
 
     expect($result)
@@ -18,7 +18,7 @@
 });
 
 it('returns failed result when using failed method', function () {
-    $check = new DatabaseHasTables();
+    $check = new DatabaseHasTables;
     $result = Result::make($check);
 
     $result->failed('::message::');
@@ -31,7 +31,7 @@
 });
 
 test('returns ok result when using ok method', function () {
-    $check = new DatabaseHasTables();
+    $check = new DatabaseHasTables;
     $result = Result::make($check);
 
     $result->failed('::message::');