Option fix (#195)

* made options optional in template get

* moved optional option check before the transmission check

* added test around templates

Author: avigoldman

lib/templates.js

@@ -29,6 +29,11 @@ module.exports = function(client) {
     get: function(id, options, callback) {
       options = options || {};
 
+      if (typeof options === 'function') {
+        callback = options;
+        options = {};
+      }
+
       if (!id) {
         return Promise.reject(new Error('template id is required')).asCallback(callback);
       }

lib/transmissions.js

@@ -59,16 +59,17 @@ module.exports = function(client) {
      * @returns {Promise}
      */
     send: function(transmission, options, callback) {
-      if (!transmission || typeof transmission !== 'object') {
-        return Promise.reject(new Error('transmission object is required')).asCallback(callback);
-      }
 
       // Handle optional options argument
       if (typeof options === 'function') {
         callback = options;
         options = {};
       }
 
+      if (!transmission || typeof transmission !== 'object') {
+        return Promise.reject(new Error('transmission object is required')).asCallback(callback);
+      }
+
       const reqOpts = {
         uri: api,
         json: transmission,

test/spec/templates.spec.js

@@ -58,6 +58,15 @@ describe('Templates Library', function() {
           expect(client.get.firstCall.args[0].qs).to.deep.equal({draft: true});
         });
     });
+
+    it('should work given just an id and callback', function() {
+      var id = 'test';
+
+      return templates.get(id, function() {
+        expect(client.get.firstCall.args[0].uri).to.contain(id);
+        expect(client.get.firstCall.args[0].qs).to.deep.equal({});
+      });
+    });
   });
 
   describe('create Method', function() {