Use of the new method for all registered claims

Author: Spomky

src/Builder.php

@@ -415,11 +415,22 @@ public function withClaim($name, $value)
             trigger_error('The use of the method "withClaim" is deprecated for registered claims. Please use dedicated method instead.', E_USER_DEPRECATED);
         }
 
-        if ($name === RegisteredClaims::AUDIENCE) {
-            return $this->permittedFor($value);
+        switch ($name) {
+            case RegisteredClaims::ID:
+                return $this->identifiedBy($value);
+            case RegisteredClaims::EXPIRATION_TIME:
+                return $this->expiresAt($value);
+            case RegisteredClaims::NOT_BEFORE:
+                return $this->canOnlyBeUsedAfter($value);
+            case RegisteredClaims::ISSUED_AT:
+                return $this->issuedAt($value);
+            case RegisteredClaims::ISSUER:
+                return $this->issuedBy($value);
+            case RegisteredClaims::AUDIENCE:
+                return $this->permittedFor($value);
+            default:
+                return $this->configureClaim($name, $value);
         }
-
-        return $this->configureClaim($name, $value);
     }
 
     /**

test/unit/BuilderTest.php

@@ -567,38 +567,25 @@ public function withClaimMustKeepAFluentInterface()
      * @param string $name
      * @param mixed $value
      * @param mixed $expected
+     * @param null|string $otherMessage
      *
      * @covers ::__construct
      * @covers ::withClaim
      * @covers \Lcobucci\JWT\Token\RegisteredClaimGiven
      *
      * @dataProvider dataWithClaimDeprecationNotice
      */
-    public function withClaimShouldSendDeprecationNoticeWhenTryingToConfigureARegisteredClaim($name, $value, $expected)
+    public function withClaimShouldSendDeprecationNoticeWhenTryingToConfigureARegisteredClaim($name, $value, $expected, $otherMessage = null)
     {
-        $key = $this->createMock(Key::class);
-        $signature = $this->createMock(Signature::class);
-        $signature
-            ->expects(static::once())
-            ->method('hash')
-            ->willReturn('--hash--')
-        ;
-
-        $signer = $this->createMock(Signer::class);
-        $signer
-            ->expects(static::once())
-            ->method('sign')
-            ->willReturn($signature)
-        ;
-
-
-
         $this->expectDeprecation('The use of the method "withClaim" is deprecated for registered claims. Please use dedicated method instead.');
+        if ($otherMessage) {
+            $this->expectDeprecation($otherMessage);
+        }
 
         $token = $this
             ->createBuilder()
             ->withClaim($name, $value)
-            ->getToken($signer, $key)
+            ->getToken(new None(), Key\InMemory::plainText(''))
         ;
 
         self::assertEquals($expected, $token->claims()->get($name));
@@ -615,9 +602,9 @@ public function dataWithClaimDeprecationNotice()
             ['aud', 'him', ['him']],
             ['jti', '0123456789ABCDEF', '0123456789ABCDEF'],
             ['iss', 'you', 'you'],
-            ['exp', $nowPlus1HourAsDate->getTimestamp(), $nowPlus1HourAsDate->getTimestamp()],
-            ['iat', $now, $nowAsDate->getTimestamp()],
-            ['nbf', $now, $nowAsDate->getTimestamp()],
+            ['exp', $nowPlus1HourAsDate->getTimestamp(), $nowPlus1HourAsDate, 'Using integers for registered date claims is deprecated, please use DateTimeImmutable objects instead.'],
+            ['iat', $now, $nowAsDate, 'Using integers for registered date claims is deprecated, please use DateTimeImmutable objects instead.'],
+            ['nbf', $now, $nowAsDate, 'Using integers for registered date claims is deprecated, please use DateTimeImmutable objects instead.'],
         ];
     }