Merge pull request #143 from chrisfranko/development

RefreshTokenGrant: update error messages

Author: HappyZombies

lib/grant-types/refresh-token-grant-type.js

@@ -101,7 +101,7 @@ RefreshTokenGrantType.prototype.getRefreshToken = function(request, client) {
       }
 
       if (token.client.id !== client.id) {
-        throw new InvalidGrantError('Invalid grant: refresh token is invalid');
+        throw new InvalidGrantError('Invalid grant: refresh token was issued to another client');
       }
 
       if (token.refreshTokenExpiresAt && !(token.refreshTokenExpiresAt instanceof Date)) {
@@ -130,7 +130,7 @@ RefreshTokenGrantType.prototype.revokeToken = function(token) {
   return promisify(this.model.revokeToken, 1).call(this.model, token)
     .then(function(status) {
       if (!status) {
-        throw new InvalidGrantError('Invalid grant: refresh token is invalid');
+        throw new InvalidGrantError('Invalid grant: refresh token is invalid or could not be revoked');
       }
 
       return token;

test/integration/grant-types/refresh-token-grant-type_test.js

@@ -262,7 +262,7 @@ describe('RefreshTokenGrantType integration', function() {
         .then(should.fail)
         .catch(function(e) {
           e.should.be.an.instanceOf(InvalidGrantError);
-          e.message.should.equal('Invalid grant: refresh token is invalid');
+          e.message.should.equal('Invalid grant: refresh token was issued to another client');
         });
     });
 
@@ -304,7 +304,7 @@ describe('RefreshTokenGrantType integration', function() {
         .then(should.fail)
         .catch(function(e) {
           e.should.be.an.instanceOf(InvalidGrantError);
-          e.message.should.equal('Invalid grant: refresh token is invalid');
+          e.message.should.equal('Invalid grant: refresh token was issued to another client');
         });
     });