Add a test for passing additional parameters.

Author: Eric Koleda

README.md

@@ -340,15 +340,16 @@ method:
 
 ```js
 var authorizationUrl = getService().getAuthorizationUrl({
+  // Pass the additional parameter "lang" with the value "fr".
   lang: 'fr'
 });
 ```
 
 These values will be stored along-side Apps Script's internal information in the
-cryptographically secure `state` token, which is passed in the authorization URL
-and passed back to the redirect URI. The `state` token is automatically
-decrypted in the callback function, and you can access your parameters using the
-same `request.parameter` field used in web apps:
+encypted `state` token, which is passed in the authorization URL and passed back
+to the redirect URI. The `state` token is automatically decrypted in the
+callback function and you can access your parameters using the same
+`request.parameter` field used in web apps:
 
 ```js
 function authCallback(request) {

src/Utilities.js

@@ -42,7 +42,7 @@ function validate_(params) {
   Object.keys(params).forEach(function(name) {
     var value = params[name];
     if (!value) {
-      throw Utilities.formatString('%s is required.', name);
+      throw new Error(name + ' is required.');
     }
   });
 }

test/mocks/script.js

@@ -0,0 +1,34 @@
+var MockScriptApp = function() {};
+
+MockScriptApp.prototype.getScriptId = function() {
+  return Math.random().toString(36).substring(2);
+};
+
+MockScriptApp.prototype.newStateToken = function() {
+  return new MockStateTokenBuilder();
+};
+
+var MockStateTokenBuilder = function() {
+  this.arguments = {};
+};
+
+MockStateTokenBuilder.prototype.withMethod = function(method) {
+  this.method = method;
+  return this;
+};
+
+MockStateTokenBuilder.prototype.withArgument = function(key, value) {
+  this.arguments[key] = value;
+  return this;
+};
+
+MockStateTokenBuilder.prototype.withTimeout = function(timeout) {
+  this.timeout = timeout;
+  return this;
+};
+
+MockStateTokenBuilder.prototype.createToken = function() {
+  return JSON.stringify(this);
+};
+
+module.exports = MockScriptApp;

test/test.js

@@ -4,14 +4,11 @@ var MockUrlFetchApp = require('./mocks/urlfetchapp');
 var MockProperties = require('./mocks/properties');
 var MockCache = require('./mocks/cache');
 var MockLock = require('./mocks/lock');
+var MockScriptApp = require('./mocks/script');
 var Future = require('fibers/future');
 
 var mocks = {
-  ScriptApp: {
-    getScriptId: function() {
-      return '12345';
-    }
-  },
+  ScriptApp: new MockScriptApp(),
   UrlFetchApp: new MockUrlFetchApp(),
   Utilities: {
     base64Encode: function(data) {
@@ -405,6 +402,32 @@ describe('Service', function() {
       service.exchangeGrant_();
     });
   });
+
+  describe('#getAuthorizationUrl()', function() {
+    it('should add additional parameters to the state token', function() {
+      var service = OAuth2.createService('test')
+          .setAuthorizationBaseUrl('http://www.example.com')
+          .setClientId('abc')
+          .setClientSecret('def')
+          .setCallbackFunction('authCallback');
+      var authorizationUrl = service.getAuthorizationUrl({
+        foo: 'bar'
+      });
+
+      // Extract the state token from the URL and parse it. For example, the
+      // URL http://www.example.com?state=%7B%22a%22%3A1%7D would produce
+      // {a: 1}.
+      var querystring = authorizationUrl.split('?')[1];
+      var params = querystring.split('&').reduce(function(result, pair) {
+        var parts = pair.split('=').map(decodeURIComponent);
+        result[parts[0]] = parts[1];
+        return result;
+      }, {});
+      var state = JSON.parse(params.state);
+
+      assert.equal(state.arguments.foo, 'bar');
+    });
+  });
 });
 
 describe('Utilities', function() {