Separate chain calls (#51)

yiisoft · Jun 17, 2022 · 627b111 · 627b111
1 parent 4a075e8
commit 627b111
Show file tree

Hide file tree

Showing 3 changed files with 18 additions and 6 deletions.
diff --git a/src/JwtMethod.php b/src/JwtMethod.php
@@ -65,7 +65,9 @@ public function authenticate(ServerRequestInterface $request): ?IdentityInterfac
             return null;
         }
 
-        $this->getClaimCheckerManager()->check($claims);
+        $this
+            ->getClaimCheckerManager()
+            ->check($claims);
         return $this->identityRepository->findIdentity((string)$claims[$this->identifier]);
     }
 

diff --git a/src/TokenFactory.php b/src/TokenFactory.php
@@ -48,7 +48,9 @@ public function __construct(
     public function create(array $payload, string $format, ?int $signatureIndex = null): string
     {
         $jwsBuilder = new JWSBuilder($this->algorithmManager);
-        $jws = $jwsBuilder->create()->withPayload(Json::encode($payload));
+        $jws = $jwsBuilder
+            ->create()
+            ->withPayload(Json::encode($payload));
         $jwk = $this->keyFactory->create();
 
         foreach ($this->algorithmManager->list() as $algorithm) {

diff --git a/tests/JwtMethodTest.php b/tests/JwtMethodTest.php
@@ -48,7 +48,9 @@ public function testSuccessfulQueryParameterAuthentication(): void
         $payload = $this->getPayload();
         $token = $tokenFactory->create($payload, CompactSerializer::NAME);
         $result = (new JwtMethod($identityRepository, $this->getTokenRepository()))->authenticate(
-            $this->createRequest()->withQueryParams(['access-token' => $token])
+            $this
+                ->createRequest()
+                ->withQueryParams(['access-token' => $token])
         );
 
         $this->assertNotNull($result);
@@ -71,7 +73,9 @@ public function testEmptyPayload(): void
         $tokenManager = $this->getTokenFactory();
         $token = $tokenManager->create([], CompactSerializer::NAME);
         $result = (new JwtMethod($identityRepository, $this->getTokenRepository()))->authenticate(
-            $this->createRequest()->withQueryParams(['access-token' => $token])
+            $this
+                ->createRequest()
+                ->withQueryParams(['access-token' => $token])
         );
 
         $this->assertNull($result);
@@ -83,8 +87,12 @@ public function testChallengeIsCorrect(): void
         $identityRepository = new FakeIdentityRepository($this->createIdentity());
         $authenticationMethod = new JwtMethod($identityRepository, $this->getTokenRepository());
 
-        $this->assertEquals(400, $authenticationMethod->challenge($response)->getStatusCode());
-        $this->assertNotEmpty($authenticationMethod->challenge($response)->getHeaderLine(Header::WWW_AUTHENTICATE));
+        $this->assertEquals(400, $authenticationMethod
+            ->challenge($response)
+            ->getStatusCode());
+        $this->assertNotEmpty($authenticationMethod
+            ->challenge($response)
+            ->getHeaderLine(Header::WWW_AUTHENTICATE));
     }
 
     public function testCheckTokenIsExpired(): void