bug #650 Fixed AuthenticaionFailureHandler to utilize messages from c…

…ustom exceptions (EresDev)

This PR was squashed before being merged into the 2.x-dev branch (closes #650).

Discussion
----------

Fixed AuthenticaionFailureHandler to utilize messages from custom exceptions

Fixed #588

Commits
-------

9414427 Fixed AuthenticaionFailureHandler to utilize messages from custom exceptions

Security/Http/Authentication/AuthenticationFailureHandler.php

@@ -36,7 +36,10 @@ public function __construct(EventDispatcherInterface $dispatcher)
      */
     public function onAuthenticationFailure(Request $request, AuthenticationException $exception)
     {
-        $event = new AuthenticationFailureEvent($exception, new JWTAuthenticationFailureResponse());
+        $event = new AuthenticationFailureEvent(
+            $exception,
+            new JWTAuthenticationFailureResponse($exception->getMessage())
+        );
 
         if ($this->dispatcher instanceof ContractsEventDispatcherInterface) {
             $this->dispatcher->dispatch($event, Events::AUTHENTICATION_FAILURE);

Tests/Functional/GetTokenTest.php

@@ -62,7 +62,7 @@ public function testGetTokenFromInvalidCredentials()
         $this->assertArrayHasKey('message', $body, 'The response should have a "message" key containing the failure reason.');
         $this->assertArrayHasKey('code', $body, 'The response should have a "code" key containing the response status code.');
 
-        $this->assertSame('Bad credentials', $body['message']);
+        $this->assertSame('Bad credentials.', $body['message']);
         $this->assertSame(401, $body['code']);
     }
 }

Tests/Security/Http/Authentication/AuthenticationFailureHandlerTest.php

@@ -23,14 +23,16 @@ public function testOnAuthenticationFailure()
             ->disableOriginalConstructor()
             ->getMock();
 
+        $authenticationException = $this->getAuthenticationException();
+
         $handler  = new AuthenticationFailureHandler($dispatcher);
-        $response = $handler->onAuthenticationFailure($this->getRequest(), $this->getAuthenticationException());
+        $response = $handler->onAuthenticationFailure($this->getRequest(), $authenticationException);
         $content  = json_decode($response->getContent(), true);
 
         $this->assertInstanceOf('Symfony\Component\HttpFoundation\JsonResponse', $response);
         $this->assertEquals(401, $response->getStatusCode());
         $this->assertEquals(401, $content['code']);
-        $this->assertEquals('Bad credentials', $content['message']);
+        $this->assertEquals($authenticationException->getMessage(), $content['message']);
     }
 
     /**