added error handling tests

Author: Andrew Maillet

test/integration/grant-types/authorization-code-grant-type_test.js

@@ -91,8 +91,8 @@ describe('AuthorizationCodeGrantType integration', function() {
         e.message.should.equal('Missing parameter: `request`');
       }
     });
-
-    it('should throw an error if `client` is missing', function() {
+    
+    it('should throw an error if `client` is invalid', function() {
       var client = {};
       var model = {
         getAuthorizationCode: function() { return { authorizationCode: 12345, expiresAt: new Date(new Date() * 2), user: {} }; },
@@ -110,6 +110,25 @@ describe('AuthorizationCodeGrantType integration', function() {
         });
     });
 
+    it('should throw an error if `client` is missing', function() {
+      
+      var model = {
+        getAuthorizationCode: function() { return { authorizationCode: 12345, expiresAt: new Date(new Date() * 2), user: {} }; },
+        revokeAuthorizationCode: function() {},
+        saveToken: function() {}
+      };
+      var grantType = new AuthorizationCodeGrantType({ accessTokenLifetime: 123, model: model });
+      var request = new Request({ body: { code: 12345 }, headers: {}, method: {}, query: {} });
+
+      try {
+        grantType.handle(request, null);
+      }
+      catch (e) {
+        e.should.be.an.instanceOf(InvalidArgumentError);
+        e.message.should.equal('Missing parameter: `client`');
+      }
+    });
+
     it('should return a token', function() {
       var client = { id: 'foobar' };
       var token = {};
@@ -462,6 +481,25 @@ describe('AuthorizationCodeGrantType integration', function() {
         .catch(should.fail);
     });
 
+    it('should throw an error when the auth code is invalid', function() {
+      var authorizationCode = { authorizationCode: 12345, client: {}, expiresAt: new Date(new Date() / 2), user: {} };
+      var model = {
+        getAuthorizationCode: function() {},
+        revokeAuthorizationCode: function() { return false; },
+        saveToken: function() {}
+      };
+      var grantType = new AuthorizationCodeGrantType({ accessTokenLifetime: 123, model: model });
+
+      return grantType.revokeAuthorizationCode(authorizationCode)
+        .then(function(data) {
+          data.should.equal(authorizationCode);
+        })
+        .catch(function(e) {
+          e.should.be.an.instanceOf(InvalidGrantError);
+          e.message.should.equal('Invalid grant: authorization code is invalid');
+        });
+    });
+
     it('should support promises', function() {
       var authorizationCode = { authorizationCode: 12345, client: {}, expiresAt: new Date(new Date() / 2), user: {} };
       var model = {

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

@@ -170,7 +170,7 @@ describe('RefreshTokenGrantType integration', function() {
   });
 
   describe('getRefreshToken()', function() {
-    it('should throw an error if the requested `refreshToken` is missing', function() {
+    it('should throw an error if the `refreshToken` parameter is missing from the request body', function() {
       var client = {};
       var model = {
         getRefreshToken: function() {},
@@ -190,35 +190,15 @@ describe('RefreshTokenGrantType integration', function() {
       }
     });
 
-    it('should throw an error if the requested `refreshToken` is invalid', function() {
-      var client = {};
-      var model = {
-        getRefreshToken: function() {},
-        revokeToken: function() {},
-        saveToken: function() {}
-      };
-      var grantType = new RefreshTokenGrantType({ accessTokenLifetime: 120, model: model });
-      var request = new Request({ body: { refresh_token: [] }, headers: {}, method: {}, query: {} });
-
-      try {
-        grantType.getRefreshToken(request, client);
-
-        should.fail();
-      } catch (e) {
-        e.should.be.an.instanceOf(InvalidRequestError);
-        e.message.should.equal('Invalid parameter: `refresh_token`');
-      }
-    });
-
-    it('should throw an error if `refreshToken` is missing', function() {
-      var client = {};
+    it('should throw an error if `refreshToken` is not found', function() {
+      var client = { id: 123 };
       var model = {
-        getRefreshToken: function() { return { accessToken: 'foo', client: { id: 123 }, user: {} }; },
+        getRefreshToken: function() { return; },
         revokeToken: function() {},
         saveToken: function() {}
       };
       var grantType = new RefreshTokenGrantType({ accessTokenLifetime: 120, model: model });
-      var request = new Request({ body: { refresh_token: 12345 }, headers: {}, method: {}, query: {} });
+      var request = new Request({ body: { refresh_token: '12345' }, headers: {}, method: {}, query: {} });
 
       return grantType.getRefreshToken(request, client)
         .then(should.fail)
@@ -286,29 +266,7 @@ describe('RefreshTokenGrantType integration', function() {
         });
     });
 
-    it('should throw an error if the request body does not contain `refresh_token`', function() {
-      var client = {};
-      var model = {
-        getRefreshToken: function() {
-          return { client: { id: 456 }, user: {} };
-        },
-        revokeToken: function() {},
-        saveToken: function() {}
-      };
-      var grantType = new RefreshTokenGrantType({ accessTokenLifetime: 120, model: model });
-      var request = new Request({ body: {}, headers: {}, method: {}, query: {} });
-
-      try {
-        grantType.getRefreshToken(request, client);
-
-        should.fail();
-      } catch (e) {
-        e.should.be.an.instanceOf(InvalidRequestError);
-        e.message.should.equal('Missing parameter: `refresh_token`');
-      }
-    });
-
-    it('should throw an error if `refresh_token` is invalid', function() {
+    it('should throw an error if `refresh_token` contains invalid characters', function() {
       var client = {};
       var model = {
         getRefreshToken: function() {
@@ -371,6 +329,26 @@ describe('RefreshTokenGrantType integration', function() {
         });
     });
 
+    it('should throw an error if `refreshTokenExpiresAt` is not a date value', function() {
+      var client = { id: 123 };
+      var model = {
+        getRefreshToken: function() {
+          return { accessToken: 'foo', client: { id: 123 }, refreshTokenExpiresAt: 'stringvalue', user: {} };
+        },
+        revokeToken: function() {},
+        saveToken: function() {}
+      };
+      var grantType = new RefreshTokenGrantType({ accessTokenLifetime: 120, model: model });
+      var request = new Request({ body: { refresh_token: 12345 }, headers: {}, method: {}, query: {} });
+
+      return grantType.getRefreshToken(request, client)
+        .then(should.fail)
+        .catch(function(e) {
+          e.should.be.an.instanceOf(ServerError);
+          e.message.should.equal('Server error: `refreshTokenExpiresAt` must be a Date instance');
+        });
+    });
+
     it('should return a token', function() {
       var client = { id: 123 };
       var token = { accessToken: 'foo', client: { id: 123 }, user: {} };