fix: styling

Author: joostdebruijn

.styleci.yml

@@ -0,0 +1 @@
+preset: laravel

composer.json

@@ -23,10 +23,10 @@
         "minishlink/web-push": "^9.0"
     },
     "require-dev": {
-        "laravel/pint": "^1.21",
+        "larastan/larastan": "^3.1",
+        "laravel/pint": "^1.25",
         "mockery/mockery": "^1.0",
         "orchestra/testbench": "^9.2|^10.0",
-        "larastan/larastan": "^3.1",
         "phpunit/phpunit": "^10.5|^11.5.3",
         "rector/rector": "^2.0"
     },
@@ -55,4 +55,4 @@
     "scripts": {
         "test": "vendor/bin/phpunit"
     }
-}
+}

src/WebPushServiceProvider.php

@@ -89,13 +89,12 @@ protected function definePublishing()
             __DIR__.'/../config/webpush.php' => config_path('webpush.php'),
         ], 'config');
 
-
         $this->publishes([
             __DIR__.'/../migrations/create_push_subscriptions_table.php.stub' => $this->getMigrationFileName('create_push_subscriptions_table.php'),
         ], 'migrations');
     }
 
-        /**
+    /**
      * Returns existing migration file if found, else uses the current timestamp.
      */
     protected function getMigrationFileName(string $migrationFileName): string

tests/WebPushServiceProviderTest.php

@@ -38,7 +38,7 @@ public function it_publishes_migration(): void
         $migrationsPath = $this->app->databasePath().DIRECTORY_SEPARATOR.'migrations'.DIRECTORY_SEPARATOR;
 
         // Remove any existing matching migrations in the database/migrations folder for a clean test
-        $existing = glob($migrationsPath.'*_' . 'create_push_subscriptions_table.php');
+        $existing = glob($migrationsPath.'*_'.'create_push_subscriptions_table.php');
         foreach ($existing as $file) {
             @unlink($file);
         }
@@ -50,7 +50,7 @@ public function it_publishes_migration(): void
 
         $this->assertEquals(0, $exit);
 
-        $found = glob($migrationsPath.'*_' . 'create_push_subscriptions_table.php');
+        $found = glob($migrationsPath.'*_'.'create_push_subscriptions_table.php');
 
         $this->assertNotEmpty($found, 'No migration was published to the database/migrations path');
 
@@ -74,11 +74,11 @@ public function it_does_not_generate_duplicate_migration_if_one_already_exists()
         }
 
         // Ensure no existing matching migrations are present before the test
-        $existing = glob($migrationsPath . '*_create_push_subscriptions_table.php');
+        $existing = glob($migrationsPath.'*_create_push_subscriptions_table.php');
         foreach ($existing as $file) {
             @unlink($file);
         }
-        
+
         // Create a fake existing migration file in the app migrations directory that should be detected and reused
         $existingFilename = $migrationsPath.'2020_01_01_000000_create_push_subscriptions_table.php';
         file_put_contents($existingFilename, "<?php\n// existing migration\n");
@@ -101,7 +101,7 @@ public function it_does_not_generate_duplicate_migration_if_one_already_exists()
         $this->assertEquals(0, $exit);
 
         // After publishing, ensure no additional migration with a different timestamp suffix was created
-        $found = glob($migrationsPath.'*_' . 'create_push_subscriptions_table.php');
+        $found = glob($migrationsPath.'*_'.'create_push_subscriptions_table.php');
 
         // There should be exactly one matching migration (the existing one we created)
         $this->assertCount(1, $found, 'A duplicate migration file was created instead of reusing the existing one');