refactor: rename

apiato · Feb 10, 2024 · c015068 · c015068
1 parent ddf942d
commit c015068
Show file tree

Hide file tree

Showing 8 changed files with 24 additions and 24 deletions.
diff --git a/app/Containers/AppSection/Authentication/Actions/ApiLoginProxyForWebClientAction.php b/app/Containers/AppSection/Authentication/Actions/ApiLoginProxyForWebClientAction.php
@@ -3,7 +3,7 @@
 namespace App\Containers\AppSection\Authentication\Actions;
 
 use Apiato\Core\Exceptions\IncorrectIdException;
-use App\Containers\AppSection\Authentication\Classes\LoginFieldProcessor;
+use App\Containers\AppSection\Authentication\Classes\LoginFieldParser;
 use App\Containers\AppSection\Authentication\Exceptions\LoginFailedException;
 use App\Containers\AppSection\Authentication\Tasks\CallOAuthServerTask;
 use App\Containers\AppSection\Authentication\Tasks\MakeRefreshTokenCookieTask;
@@ -34,7 +34,7 @@ public function run(LoginProxyPasswordGrantRequest $request): AuthResult
             'scope' => '',
         ]);
 
-        $loginFields = LoginFieldProcessor::extractAll($sanitizedData);
+        $loginFields = LoginFieldParser::extractAll($sanitizedData);
 
         foreach ($loginFields as $loginField) {
             $sanitizedData['username'] = $loginField->value;

diff --git a/app/Containers/AppSection/Authentication/Actions/WebLoginAction.php b/app/Containers/AppSection/Authentication/Actions/WebLoginAction.php
@@ -3,7 +3,7 @@
 namespace App\Containers\AppSection\Authentication\Actions;
 
 use Apiato\Core\Exceptions\IncorrectIdException;
-use App\Containers\AppSection\Authentication\Classes\LoginFieldProcessor;
+use App\Containers\AppSection\Authentication\Classes\LoginFieldParser;
 use App\Containers\AppSection\Authentication\UI\WEB\Requests\LoginRequest;
 use App\Containers\AppSection\Authentication\Values\IncomingLoginField;
 use App\Ship\Parents\Actions\Action as ParentAction;
@@ -24,7 +24,7 @@ public function run(LoginRequest $request): RedirectResponse
             'remember' => false,
         ]);
 
-        $loginFields = LoginFieldProcessor::extractAll($sanitizedData);
+        $loginFields = LoginFieldParser::extractAll($sanitizedData);
         $credentials = [];
         foreach ($loginFields as $loginField) {
             if (config('appSection-authentication.login.case_sensitive')) {

diff --git a/...ntication/Classes/LoginFieldProcessor.php → ...thentication/Classes/LoginFieldParser.php b/...ntication/Classes/LoginFieldProcessor.php → ...thentication/Classes/LoginFieldParser.php
@@ -6,7 +6,7 @@
 use App\Containers\AppSection\Authentication\Values\LoginField;
 use Illuminate\Support\Arr;
 
-class LoginFieldProcessor
+class LoginFieldParser
 {
     /**
      * Extract all matching login fields from the given data.

diff --git a/.../Unit/Classes/LoginFieldProcessorTest.php → ...sts/Unit/Classes/LoginFieldParserTest.php b/.../Unit/Classes/LoginFieldProcessorTest.php → ...sts/Unit/Classes/LoginFieldParserTest.php
@@ -2,7 +2,7 @@
 
 namespace App\Containers\AppSection\Authentication\Tests\Unit\Classes;
 
-use App\Containers\AppSection\Authentication\Classes\LoginFieldProcessor;
+use App\Containers\AppSection\Authentication\Classes\LoginFieldParser;
 use App\Containers\AppSection\Authentication\Tests\UnitTestCase;
 use App\Containers\AppSection\Authentication\Values\IncomingLoginField;
 use App\Containers\AppSection\Authentication\Values\LoginField;
@@ -12,8 +12,8 @@
 use PHPUnit\Framework\Attributes\Group;
 
 #[Group('authentication')]
-#[CoversClass(LoginFieldProcessor::class)]
-final class LoginFieldProcessorTest extends UnitTestCase
+#[CoversClass(LoginFieldParser::class)]
+final class LoginFieldParserTest extends UnitTestCase
 {
     public static function loginDataProvider(): array
     {
@@ -49,7 +49,7 @@ public function testCanExtractUsernameFromValidInput(array $input, array $expect
     {
         config()->set('appSection-authentication.login.fields', ['email' => [], 'name' => []]);
 
-        $result = LoginFieldProcessor::extractAll($input);
+        $result = LoginFieldParser::extractAll($input);
 
         $this->assertEquals($result, $expected);
     }
@@ -63,7 +63,7 @@ public function testShouldDiscardUnknownFields(): void
         ];
         $expected = [new IncomingLoginField('email', 'gandalf@the.grey')];
 
-        $result = LoginFieldProcessor::extractAll($credentials);
+        $result = LoginFieldParser::extractAll($credentials);
 
         $this->assertEquals($result, $expected);
     }
@@ -77,7 +77,7 @@ public function testUsesEmailFieldAsDefaultFallback(): void
         ];
         $expected = [new IncomingLoginField('email', 'gandalf@the.grey')];
 
-        $result = LoginFieldProcessor::extractAll($credentials);
+        $result = LoginFieldParser::extractAll($credentials);
 
         $this->assertEquals($result, $expected);
     }
@@ -87,7 +87,7 @@ public function testEmptyCredentialsThrowsException(): void
         $this->expectException(\RuntimeException::class);
         $this->expectExceptionMessage('No matching login field found');
 
-        LoginFieldProcessor::extractAll([]);
+        LoginFieldParser::extractAll([]);
     }
 
     public static function invalidLoginFieldsDataProvider(): array
@@ -118,7 +118,7 @@ public function testInvalidLoginFields(mixed $invalidFields, string $exceptedExc
             'password' => 'youShallNotPass',
         ];
 
-        LoginFieldProcessor::extractAll($userDetails);
+        LoginFieldParser::extractAll($userDetails);
     }
 
     public function testMergeValidationRulesWithException(): void
@@ -132,7 +132,7 @@ public function testMergeValidationRulesWithException(): void
             'remember' => 'boolean',
         ];
 
-        LoginFieldProcessor::mergeValidationRules($newRules);
+        LoginFieldParser::mergeValidationRules($newRules);
     }
 
     public static function multiLoginFieldProvider(): array
@@ -205,7 +205,7 @@ public function testCanMergeValidationRules(array $email, array $name, array $ci
             'age' => ['nullable', 'integer'],
         ];
 
-        $result = LoginFieldProcessor::mergeValidationRules($rules);
+        $result = LoginFieldParser::mergeValidationRules($rules);
 
         $this->assertSame([
             'phone' => ['required', 'numeric'],
@@ -258,7 +258,7 @@ public function testGivenOnlyOneFieldExistsShouldNotAddMultiLoginRelatedRule(Log
             'age' => ['nullable', 'integer'],
         ];
 
-        $result = LoginFieldProcessor::mergeValidationRules($rules);
+        $result = LoginFieldParser::mergeValidationRules($rules);
 
         $this->assertSame([
             'phone' => ['required', 'numeric'],

diff --git a/...pSection/Authentication/Tests/Unit/UI/API/Requests/LoginProxyPasswordGrantRequestTest.php b/...pSection/Authentication/Tests/Unit/UI/API/Requests/LoginProxyPasswordGrantRequestTest.php
@@ -2,7 +2,7 @@
 
 namespace App\Containers\AppSection\Authentication\Tests\Unit\UI\API\Requests;
 
-use App\Containers\AppSection\Authentication\Classes\LoginFieldProcessor;
+use App\Containers\AppSection\Authentication\Classes\LoginFieldParser;
 use App\Containers\AppSection\Authentication\Tests\UnitTestCase;
 use App\Containers\AppSection\Authentication\UI\API\Requests\LoginProxyPasswordGrantRequest;
 use PHPUnit\Framework\Attributes\CoversClass;
@@ -35,7 +35,7 @@ public function testUrlParametersArray(): void
     public function testValidationRules(): void
     {
         $this->assertEquals(
-            LoginFieldProcessor::mergeValidationRules(['password' => 'required']),
+            LoginFieldParser::mergeValidationRules(['password' => 'required']),
             $this->request->rules(),
         );
     }

diff --git a/app/Containers/AppSection/Authentication/Tests/Unit/UI/WEB/Requests/LoginRequestTest.php b/app/Containers/AppSection/Authentication/Tests/Unit/UI/WEB/Requests/LoginRequestTest.php
@@ -2,7 +2,7 @@
 
 namespace App\Containers\AppSection\Authentication\Tests\Unit\UI\WEB\Requests;
 
-use App\Containers\AppSection\Authentication\Classes\LoginFieldProcessor;
+use App\Containers\AppSection\Authentication\Classes\LoginFieldParser;
 use App\Containers\AppSection\Authentication\Tests\UnitTestCase;
 use App\Containers\AppSection\Authentication\UI\WEB\Requests\LoginRequest;
 use PHPUnit\Framework\Attributes\CoversClass;
@@ -37,7 +37,7 @@ public function testValidationRules(): void
         $rules = $this->request->rules();
 
         $this->assertSame(
-            LoginFieldProcessor::mergeValidationRules([
+            LoginFieldParser::mergeValidationRules([
                 'password' => 'required',
                 'remember' => 'boolean',
             ]),

diff --git a/app/Containers/AppSection/Authentication/UI/API/Requests/LoginProxyPasswordGrantRequest.php b/app/Containers/AppSection/Authentication/UI/API/Requests/LoginProxyPasswordGrantRequest.php
@@ -2,7 +2,7 @@
 
 namespace App\Containers\AppSection\Authentication\UI\API\Requests;
 
-use App\Containers\AppSection\Authentication\Classes\LoginFieldProcessor;
+use App\Containers\AppSection\Authentication\Classes\LoginFieldParser;
 use App\Ship\Parents\Requests\Request as ParentRequest;
 
 class LoginProxyPasswordGrantRequest extends ParentRequest
@@ -26,7 +26,7 @@ public function rules(): array
             'password' => 'required',
         ];
 
-        return LoginFieldProcessor::mergeValidationRules($rules);
+        return LoginFieldParser::mergeValidationRules($rules);
     }
 
     public function authorize(): bool

diff --git a/app/Containers/AppSection/Authentication/UI/WEB/Requests/LoginRequest.php b/app/Containers/AppSection/Authentication/UI/WEB/Requests/LoginRequest.php
@@ -2,7 +2,7 @@
 
 namespace App\Containers\AppSection\Authentication\UI\WEB\Requests;
 
-use App\Containers\AppSection\Authentication\Classes\LoginFieldProcessor;
+use App\Containers\AppSection\Authentication\Classes\LoginFieldParser;
 use App\Ship\Parents\Requests\Request as ParentRequest;
 
 class LoginRequest extends ParentRequest
@@ -23,7 +23,7 @@ public function rules(): array
             'remember' => 'boolean',
         ];
 
-        return LoginFieldProcessor::mergeValidationRules($rules);
+        return LoginFieldParser::mergeValidationRules($rules);
     }
 
     public function authorize(): bool