Merge pull request #18 from rappasoft/analysis-A7oB5k

Apply fixes from StyleCI

Author: rappasoft

src/Console/Commands/LockoutDisable.php

@@ -62,4 +62,3 @@ public function handle(): int
         return Command::SUCCESS;
     }
 }
-

src/Console/Commands/LockoutEnable.php

@@ -62,4 +62,3 @@ public function handle(): int
         return Command::SUCCESS;
     }
 }
-

src/Console/Commands/LockoutStatus.php

@@ -47,4 +47,3 @@ public function handle(): int
         return Command::SUCCESS;
     }
 }
-

src/Events/LockoutDisabled.php

@@ -17,4 +17,3 @@ public function __construct()
         //
     }
 }
-

src/Events/LockoutEnabled.php

@@ -17,4 +17,3 @@ public function __construct()
         //
     }
 }
-

src/Events/RequestBlocked.php

@@ -3,8 +3,8 @@
 namespace Rappasoft\Lockout\Events;
 
 use Illuminate\Foundation\Events\Dispatchable;
-use Illuminate\Queue\SerializesModels;
 use Illuminate\Http\Request;
+use Illuminate\Queue\SerializesModels;
 
 class RequestBlocked
 {
@@ -20,4 +20,3 @@ public function __construct(
         //
     }
 }
-

src/Helpers/IpHelper.php

@@ -89,4 +89,3 @@ public static function parseIpList(?string $ipList): array
         return array_map('trim', explode(',', $ipList));
     }
 }
-

tests/AdvancedFeaturesTest.php

@@ -3,11 +3,9 @@
 namespace Rappasoft\Lockout\Tests;
 
 use Illuminate\Http\Response;
-use Rappasoft\Lockout\Events\LockoutDisabled;
-use Rappasoft\Lockout\Events\LockoutEnabled;
-use Rappasoft\Lockout\Events\RequestBlocked;
-use Illuminate\Support\Facades\Event;
 use Illuminate\Support\Facades\Cache;
+use Illuminate\Support\Facades\Event;
+use Rappasoft\Lockout\Events\RequestBlocked;
 
 class AdvancedFeaturesTest extends TestCase
 {
@@ -118,7 +116,7 @@ public function api_requests_return_json_response()
 
         $response = $this->withHeaders(['Accept' => 'application/json'])
             ->call('POST', 'api/test');
-        
+
         $response->assertStatus(Response::HTTP_UNAUTHORIZED);
         // Check if response is JSON (might be HTML if API detection fails)
         if ($response->headers->get('Content-Type') === 'application/json') {
@@ -136,7 +134,7 @@ public function view_response_type_returns_view()
 
         // View might not exist in test environment, so test with json instead
         config(['lockout.response_type' => 'json']);
-        
+
         $response = $this->call('POST', 'post');
         $response->assertStatus(Response::HTTP_UNAUTHORIZED);
         $response->assertJson(['message' => 'Application is currently in read-only mode.']);
@@ -212,4 +210,3 @@ public function events_are_not_fired_when_disabled()
         Event::assertNotDispatched(RequestBlocked::class);
     }
 }
-

tests/EdgeCasesTest.php

@@ -151,7 +151,7 @@ public function custom_health_check_path_configuration_works()
         // Verify custom path config is set and health check is enabled
         $this->assertEquals('custom-health', config('lockout.health_check_path'));
         $this->assertTrue(config('lockout.health_check_enabled'));
-        
+
         // The actual route registration happens at service provider boot
         // In real applications, changing the path would require clearing route cache
         // This test verifies the configuration is properly stored and retrieved
@@ -275,4 +275,3 @@ public function cache_ttl_customization_works()
             ->assertStatus(Response::HTTP_UNAUTHORIZED);
     }
 }
-

tests/IpHelperTest.php

@@ -33,7 +33,7 @@ public function cidr_with_different_masks_works()
     public function multiple_ips_in_whitelist_work()
     {
         $whitelist = ['127.0.0.1', '192.168.1.0/24', '10.0.0.1'];
-        
+
         $this->assertTrue(IpHelper::isIpAllowed('127.0.0.1', $whitelist));
         $this->assertTrue(IpHelper::isIpAllowed('192.168.1.50', $whitelist));
         $this->assertTrue(IpHelper::isIpAllowed('10.0.0.1', $whitelist));
@@ -70,15 +70,15 @@ public function empty_blacklist_returns_false()
     public function parse_ip_list_from_string()
     {
         $result = IpHelper::parseIpList('127.0.0.1,192.168.1.1,10.0.0.0/8');
-        
+
         $this->assertEquals(['127.0.0.1', '192.168.1.1', '10.0.0.0/8'], $result);
     }
 
     /** @test */
     public function parse_ip_list_handles_spaces()
     {
         $result = IpHelper::parseIpList('127.0.0.1, 192.168.1.1 , 10.0.0.0/8');
-        
+
         $this->assertEquals(['127.0.0.1', '192.168.1.1', '10.0.0.0/8'], $result);
     }
 
@@ -107,10 +107,9 @@ public function cidr_edge_cases()
         // /32 should match only exact IP
         $this->assertTrue(IpHelper::isIpAllowed('192.168.1.1', ['192.168.1.1/32']));
         $this->assertFalse(IpHelper::isIpAllowed('192.168.1.2', ['192.168.1.1/32']));
-        
+
         // /0 should match everything (entire IPv4 range)
         $this->assertTrue(IpHelper::isIpAllowed('0.0.0.0', ['0.0.0.0/0']));
         $this->assertTrue(IpHelper::isIpAllowed('255.255.255.255', ['0.0.0.0/0']));
     }
 }
-