rename classes, methods and test names

ministryofjustice · Oct 10, 2023 · db1f89d · db1f89d
1 parent 62db8b7
commit db1f89d
Show file tree

Hide file tree

Showing 3 changed files with 13 additions and 13 deletions.
diff --git a/service-api/app/src/App/src/Handler/AuthRedirectHandler.php b/service-api/app/src/App/src/Handler/AuthRedirectHandler.php
@@ -5,7 +5,7 @@
 namespace App\Handler;
 
 use App\Exception\BadRequestException;
-use App\Service\Authentication\AuthenticationService;
+use App\Service\Authentication\OneLoginAuthorisationRequestService;
 use Exception;
 use Laminas\Diactoros\Response\JsonResponse;
 use Psr\Http\Message\ResponseInterface;
@@ -18,7 +18,7 @@
 class AuthRedirectHandler implements RequestHandlerInterface
 {
     public function __construct(
-        private AuthenticationService $authenticationService,
+        private OneLoginAuthorisationRequestService $authorisationRequestService,
     ) {
     }
 
@@ -44,7 +44,7 @@ public function handle(ServerRequestInterface $request): ResponseInterface
             throw new BadRequestException('ui_locale is not set to en or cy');
         }
 
-        $authorisationUri = $this->authenticationService->redirect($params['ui_locale']);
+        $authorisationUri = $this->authorisationRequestService->createAuthorisationRequest($params['ui_locale']);
 
         return new JsonResponse($authorisationUri);
     }

diff --git a/.../Authentication/AuthenticationService.php → ...n/OneLoginAuthorisationRequestService.php b/.../Authentication/AuthenticationService.php → ...n/OneLoginAuthorisationRequestService.php
@@ -15,7 +15,7 @@
 
 use function Facile\OpenIDClient\base64url_encode;
 
-class AuthenticationService
+class OneLoginAuthorisationRequestService
 {
     public function __construct(
         private JWKFactory $JWKFactory,
@@ -24,7 +24,7 @@ public function __construct(
     ) {
     }
 
-    public function redirect(string $uiLocale): string
+    public function createAuthorisationRequest(string $uiLocale): string
     {
         //TODO UML-3080 Configure cache
 
@@ -51,9 +51,9 @@ public function redirect(string $uiLocale): string
 
         $authorisationService = (new AuthorizationServiceBuilder())->build();
 
-        $redirectAuthorisationUri = '';
+        $authorisationRequest = '';
         try {
-            $redirectAuthorisationUri = $authorisationService->getAuthorizationUri(
+            $authorisationRequest = $authorisationService->getAuthorizationUri(
                 $client,
                 [
                     'scope'      => 'openid email',
@@ -69,6 +69,6 @@ public function redirect(string $uiLocale): string
             throw new RuntimeException('Could not create authorisation uri');
         }
 
-        return $redirectAuthorisationUri;
+        return $authorisationRequest;
     }
 }
diff --git a/...hentication/AuthenticationServiceTest.php → ...eLoginAuthorisationRequestServiceTest.php b/...hentication/AuthenticationServiceTest.php → ...eLoginAuthorisationRequestServiceTest.php
@@ -4,7 +4,7 @@
 
 namespace AppTest\Service\Authentication;
 
-use App\Service\Authentication\AuthenticationService;
+use App\Service\Authentication\OneLoginAuthorisationRequestService;
 use App\Service\Authentication\JWKFactory;
 use Facile\OpenIDClient\Issuer\IssuerBuilder;
 use Facile\OpenIDClient\Issuer\IssuerBuilderInterface;
@@ -16,7 +16,7 @@
 use Prophecy\Prophecy\ObjectProphecy;
 use Psr\Log\LoggerInterface;
 
-class AuthenticationServiceTest extends TestCase
+class OneLoginAuthorisationRequestServiceTest extends TestCase
 {
     use ProphecyTrait;
 
@@ -42,14 +42,14 @@ public function setup(): void
     /**
      * @test
      */
-    public function get_redirect_uri(): void
+    public function create_authorisation_request(): void
     {
-        $authenticationService = new AuthenticationService(
+        $authenticationService = new OneLoginAuthorisationRequestService(
             $this->JWKFactory->reveal(),
             $this->logger->reveal(),
             $this->issuerBuilder->reveal()
         );
-        $redirectUri           = $authenticationService->redirect('en');
+        $redirectUri           = $authenticationService->createAuthorisationRequest('en');
         $this->assertStringContainsString('client_id=client-id', $redirectUri);
         $this->assertStringContainsString('scope=openid+email', $redirectUri);
         $this->assertStringContainsString('vtr=%5B%22Cl.Cm.P2%22%5D', $redirectUri);