Update revokeToken to allow data and options override

Author: ruipenso

src/providers/oauth-provider.js

@@ -178,24 +178,31 @@ function OAuthProvider() {
        * Revokes the `token` and removes the stored `token` from cookies
        * using the `OAuthToken`.
        *
+       * @param {object} data - Request content.
+       * @param {object} options - Optional configuration.
        * @return {promise} A response promise.
        */
 
-      revokeToken() {
-        var data = {
+      revokeToken(data, options) {
+        var refreshToken = OAuthToken.getRefreshToken();
+
+        data = angular.extend({
           client_id: config.clientId,
-          token: OAuthToken.getRefreshToken() ? OAuthToken.getRefreshToken() : OAuthToken.getAccessToken()
-        };
+          token: refreshToken ? refreshToken : OAuthToken.getAccessToken(),
+          token_type_hint: refreshToken ? 'refresh_token' : 'access_token'
+        }, data);
 
         if (null !== config.clientSecret) {
           data.client_secret = config.clientSecret;
         }
 
         data = queryString.stringify(data);
 
-        var options = {
-          headers: { 'Content-Type': 'application/x-www-form-urlencoded' }
-        };
+        options = angular.extend({
+          headers: {
+            'Content-Type': 'application/x-www-form-urlencoded'
+          }
+        }, options);
 
         return $http.post(`${config.baseUrl}${config.revokePath}`, data, options).then((response) => {
           OAuthToken.removeToken();

test/unit/providers/oauth-provider.spec.js

@@ -316,8 +316,9 @@ describe('OAuthProvider', function() {
         queryString.stringify.firstCall.args.should.have.lengthOf(1);
         queryString.stringify.firstCall.args[0].should.eql({
           client_id: defaults.clientId,
-          client_secret: defaults.clientSecret,
-          token: 'bar'
+          token: 'bar',
+          token_type_hint: 'refresh_token',
+          client_secret: defaults.clientSecret
         });
         queryString.stringify.restore();
       }));
@@ -334,6 +335,7 @@ describe('OAuthProvider', function() {
         queryString.stringify.firstCall.args[0].should.eql({
           client_id: defaults.clientId,
           token: 'foo',
+          token_type_hint: 'access_token',
           client_secret: defaults.clientSecret
         });
         queryString.stringify.restore();
@@ -343,6 +345,7 @@ describe('OAuthProvider', function() {
         var data = queryString.stringify({
           client_id: defaults.clientId,
           token: undefined,
+          token_type_hint: 'access_token',
           client_secret: defaults.clientSecret
         });
 
@@ -367,6 +370,7 @@ describe('OAuthProvider', function() {
         var data = queryString.stringify({
           client_id: defaults.clientId,
           token: 'bar',
+          token_type_hint: 'refresh_token',
           client_secret: defaults.clientSecret
         });