Added support for deleting a sending domain

docs/resources/sendingDomains.md

@@ -21,6 +21,10 @@ This library provides easy access to the [Sending Domains](https://www.sparkpost
   Update an existing sending domain
   * `domainBody` - a sending domain object **required**
   * `callback` - see all function
+* **delete(domain, callback)**
+  Delete an existing sending domain
+  * `domain` - the name of the domain you want to delete **required**
+  * `callback` - see all function
 * **verify(options, callback)**
   Validate the specified verification field types for a sending domain
   * `options.domain` - the name of the domain you want to verify **required**

examples/sendingDomains/delete_sendingDomain.js

@@ -0,0 +1,14 @@
+'use strict';
+
+var key = 'YOURAPIKEY'
+  , SparkPost = require('sparkpost')
+  , client = new SparkPost(key);
+
+client.sendingDomains['delete']('example1.com', function(err, data) {
+  if (err) {
+    console.log(err);
+  } else {
+    console.log(data);
+    console.log('Congrats you can use our client library!');
+  }
+});

lib/sendingDomains.js

@@ -7,13 +7,14 @@ var api = 'sending-domains'
  * "Class" declaration, Sending Domains API exposes five functions:
  * - create: creates a new sending domain
  * - update: updates an existing sending domain
+ * - delete: deletes an existing sending domain
  * - verify: validates specified verification field types on a sending domain
  * - all: retreives a list of sending domains
  * - find: retreives info about a specific sending domain
  */
 module.exports = function (client) {
 
-  return {
+  var sendingDomains = {
     all: function (callback) { //list
       var options = {
         uri: api
@@ -100,4 +101,23 @@ module.exports = function (client) {
       client.post(reqOpts, callback);
     }
   };
+
+  sendingDomains['delete'] = function(domain, callback) {
+    if (typeof domain === 'function') {
+      callback = domain;
+      domain = null;
+    }
+
+    if (!domain) {
+      callback(new Error('domain is required'));
+      return;
+    }
+
+    var options = {
+      uri: api + '/' + domain
+    };
+    client['delete'](options, callback);
+  };
+
+  return sendingDomains;
 };

test/spec/sendingDomains.spec.js

@@ -12,7 +12,8 @@ describe('Sending Domains Library', function() {
     client = {
       get: sinon.stub().yields(),
       post: sinon.stub().yields(),
-      put: sinon.stub().yields()
+      put: sinon.stub().yields(),
+      'delete': sinon.stub().yields()
     };
 
     sendingDomains = require('../../lib/sendingDomains')(client);
@@ -126,6 +127,31 @@ describe('Sending Domains Library', function() {
     });
   });
 
+  describe('delete Method', function() {
+    it('should call client delete method with the appropriate uri', function(done) {
+      sendingDomains['delete']('test', function(err, data) {
+        expect(client['delete'].firstCall.args[0].uri).to.equal('sending-domains/test');
+        done();
+      });
+    });
+
+    it('should throw an error if domain is null', function(done) {
+      sendingDomains['delete'](null, function(err) {
+        expect(err.message).to.equal('domain is required');
+        expect(client['delete']).not.to.have.been.called;
+        done();
+      });
+    });
+
+    it('should throw an error if domain is missing', function(done) {
+      sendingDomains['delete'](function(err) {
+        expect(err.message).to.equal('domain is required');
+        expect(client['delete']).not.to.have.been.called;
+        done();
+      });
+    });
+  });
+
   describe('verify Method', function() {
     it('should call client post method with the appropriate uri', function(done) {
       var options = {