merge

.travis.yml

@@ -17,7 +17,4 @@ deploy:
   on:
     tags: true
     all_branches: true
-    repo: bendrucker/clearbit-node
-env:
-  global:
-    secure: CHoeYn6R9p4vQ96KkysrCSIiN7J0EU1x1jNuevgYVgVol+b5ukvDVUBTaSEzeSznuCotrZdhPp6/CQNELMDJowTXUgbwdp7FUPss5Bp5WjMJHt+YIUw3vjaeucPawX2uzVyNjFY2pCG2IDl13LAvDEWuqMvB6NboX9uBFpW8ah0=
+    repo: clearbit/clearbit-node

README.md

-clearbit-node [![Build Status](https://travis-ci.org/bendrucker/clearbit-node.svg?branch=master)](https://travis-ci.org/bendrucker/clearbit-node) [![Code Climate](https://codeclimate.com/github/bendrucker/clearbit-node/badges/gpa.svg)](https://codeclimate.com/github/bendrucker/clearbit-node) [![Test Coverage](https://codeclimate.com/github/bendrucker/clearbit-node/badges/coverage.svg)](https://codeclimate.com/github/bendrucker/clearbit-node)
+clearbit-node [![Build Status](https://travis-ci.org/clearbit/clearbit-node.svg?branch=master)](https://travis-ci.org/clearbit/clearbit-node) [![Code Climate](https://codeclimate.com/github/clearbit/clearbit-node/badges/gpa.svg)](https://codeclimate.com/github/clearbit/clearbit-node) [![Test Coverage](https://codeclimate.com/github/clearbit/clearbit-node/badges/coverage.svg)](https://codeclimate.com/github/clearbit/clearbit-node)
 =============
 
 Node library for querying the [Clearbit](https://clearbit.com) business intelligence APIs. Currently supports:

package.json

@@ -8,7 +8,7 @@
   },
   "repository": {
     "type": "git",
-    "url": "https://github.com/bendrucker/clearbit-node.git"
+    "url": "https://github.com/clearbit/clearbit-node.git"
   },
   "keywords": [
     "clearbit",
@@ -18,9 +18,9 @@
   "author": "Ben Drucker <bvdrucker@gmail.com> (http://www.bendrucker.me)",
   "license": "MIT",
   "bugs": {
-    "url": "https://github.com/bendrucker/clearbit-node/issues"
+    "url": "https://github.com/clearbit/clearbit-node/issues"
   },
-  "homepage": "https://github.com/bendrucker/clearbit-node",
+  "homepage": "https://github.com/clearbit/clearbit-node",
   "dependencies": {
     "bluebird": "2",
     "create-error": "0.3",

src/client.js

@@ -15,9 +15,9 @@ function ClearbitClient (config) {
   assert(!!config.key, 'An API key must be provided');
   this.key = config.key;
 
-  this.Person = require('./person')(this);
+  this.Person = require('./person').Person(this);
+  this.PersonCompany = require('./person').PersonCompany(this);
   this.Company = require('./company')(this);
-  this.PersonCompany = require('./person_company')(this);
 }
 
 var base = 'https://%s%s.clearbit.com/v%s';
@@ -48,7 +48,7 @@ function generateQuery () {
 }
 
 ClearbitClient.prototype.request = function (options) {
-  options = _.defaults(options || /* istanbul ignore next */ {}, {
+  options = _.defaults(options, {
     method: 'get',
     query: {}
   });

src/company.js

 'use strict';
 
 var assert   = require('assert');
-var _       = require('lodash');
-var Promise = require('bluebird');
-var utils   = require('./utils');
+var resource = require('./resource');
 
-module.exports = function (client) {
-  function Company (data) {
-    _.extend(this, data);
-  }
-
-  Company.find = Promise.method(function (options) {
-    assert(options && options.domain, 'A domain must be provided');
-    return this.client.request(_.extend({
+module.exports = resource.create('Company', {
   api: 'company',
-      path: '/companies/domain/' + options.domain
-    }, options))
-    .bind(this)
-    .then(utils.cast)
-    .catch(utils.isQueued, function () {
-      throw new this.QueuedError('Company lookup queued');
-    })
-    .catch(utils.isUnknownRecord, function () {
-      throw new this.NotFoundError('Company not found');
-    });
-  });
-
-  Company.prototype.client = Company.client = client;
-  Company.NotFoundError = utils.NotFoundError;
-  Company.QueuedError = utils.QueuedError;
-
-  return Company;
-};
+  path: '/companies/domain/<%= domain %>'
+})
+.on('preFind', function (options) {
+  assert(options.domain, 'A domain must be provided');
+});

src/person.js

 'use strict';
 
 var assert   = require('assert');
-var _       = require('lodash');
-var Promise = require('bluebird');
-var utils   = require('./utils');
+var resource = require('./resource');
 
-module.exports = function (client) {
-  function Person (data) {
-    _.extend(this, data);
-  }
+function requireEmail (options) {
+  assert(options.email, 'An email must be provided');
+}
 
-  Person.find = Promise.method(function (options) {
-    assert(options && options.email, 'An email must be provided');
-    return this.client.request(_.extend({
+exports.Person = resource.create('Person', {
   api: 'person',
-      path: '/people/email/' + options.email,
-      query: _.pick(options, 'subscribe', 'company')
-    }, options))
-    .bind(this)
-    .then(utils.cast)
-    .catch(utils.isQueued, function () {
-      throw new this.QueuedError('Person lookup queued');
-    })
-    .catch(utils.isUnknownRecord, function () {
-      throw new this.NotFoundError('Person not found');
-    });
-  });
+  path: '/people/email/<%= email %>',
+  queryKeys: 'subscribe'
+})
+.on('preFind', requireEmail);
 
-  Person.prototype.client = Person.client = client;
-  Person.NotFoundError = utils.NotFoundError;
-  Person.QueuedError = utils.QueuedError;
-
-  return Person;
-};
+exports.PersonCompany = resource.create('PersonCompany', {
+  api: 'person',
+  path: '/combined/email/<%= email %>',
+  queryKeys: 'subscribe'
+})
+.on('preFind', requireEmail);

src/person_company.js

-'use strict';
-
-var assert  = require('assert');
-var _       = require('lodash');
-var Promise = require('bluebird');
-var utils   = require('./utils');
-
-module.exports = function (client) {
-  function PersonCompany (data) {
-    _.extend(this, data);
-  }
-
-  PersonCompany.find = Promise.method(function (options) {
-    assert(options && options.email, 'An email must be provided');
-    return this.client.request(_.extend({
-      api: 'person',
-      path: '/combined/email/' + options.email,
-      query: _.pick(options, 'subscribe', 'webhook_id')
-    }, options))
-    .bind(this)
-    .then(utils.cast)
-    .catch(utils.isQueued, function () {
-      throw new this.QueuedError('PersonCompany lookup queued');
-    })
-    .catch(utils.isUnknownRecord, function () {
-      throw new this.NotFoundError('PersonCompany not found');
-    });
-  });
-
-  PersonCompany.prototype.client = PersonCompany.client = client;
-  PersonCompany.NotFoundError = utils.NotFoundError;
-  PersonCompany.QueuedError = utils.QueuedError;
-
-  return PersonCompany;
-};

src/resource.js

+'use strict';
+
+var createError  = require('create-error');
+var EventEmitter = require('events').EventEmitter;
+var _            = require('lodash');
+var Promise      = require('bluebird');
+
+function isQueued (err) {
+  return err.type === 'queued';
+}
+
+function isUnknownRecord (err) {
+  return err.type === 'unknown_record';
+}
+
+function ClearbitResource (data) {
+  _.extend(this, data);
+}
+
+ClearbitResource.find = Promise.method(function (options) {
+  options = options || /* istanbul ignore next */ {};
+  this.emit('preFind', options);
+  return this.client.request(_.extend({
+    api: this._options.api,
+    path: this._options.template(options),
+    query: _.pick(options, this._options.queryKeys)
+  }, options))
+  .bind(this)
+  .then(function (data) {
+    return new this(data);
+  })
+  .catch(isQueued, function () {
+    throw new this.QueuedError(this._name + ' lookup queued');
+  })
+  .catch(isUnknownRecord, function () {
+    throw new this.NotFoundError(this._name + ' not found');
+  });
+});
+
+function createErrors (name) {
+  return {
+    NotFoundError: createError(name + 'NotFoundError'),
+    QueuedError: createError(name + 'QueuedError')
+  };
+}
+
+exports.create = function (name, options) {
+  var Resource = function () {
+    ClearbitResource.apply(this, arguments);
+  };
+      
+  _.extend(Resource, new EventEmitter(), EventEmitter.prototype, ClearbitResource, createErrors(name), {
+    _name: name,
+    _options: _.extend({}, options, {
+      template: _.template(options.path)
+    })
+  });
+
+  return _.extend(function (client) {
+    return _.extend(Resource, {
+      client: client
+    });
+  },
+  {
+    on: function () {
+      Resource.on.apply(Resource, arguments);
+      return this;
+    }
+  });
+};

src/utils.js

-'use strict';
-
-var createError = require('create-error');
-
-exports.cast = function (data) {
-  return new this(data);
-};
-
-exports.NotFoundError = createError('NotFoundError');
-exports.QueuedError = createError('QueuedError');
-
-exports.isUnknownRecord = function (err) {
-  return err.type === 'unknown_record';
-};
-
-exports.isQueued = function (err) {
-  return err.type === 'queued';
-};
\ No newline at end of file

test/person.js

@@ -3,6 +3,7 @@
 var expect        = require('chai').use(require('chai-as-promised')).expect;
 var nock          = require('nock');
 var Person        = require('../')('k').Person;
+var PersonCompany = require('../')('k').PersonCompany;
 
 describe('Person', function () {
 
@@ -15,9 +16,10 @@ describe('Person', function () {
     mock.done();
   });
 
-  describe('Person#find', function () {
-
   var alex = require('./fixtures/person');
+  var company = require('./fixtures/company');
+
+  describe('Person#find', function () {
 
     it('can find a person by email', function () {
       mock
@@ -38,13 +40,6 @@ describe('Person', function () {
       return Person.find({email: 'alex@alexmaccaw.com', subscribe: true});
     });
 
-    it('can override the company setting', function () {
-      mock
-        .get('/v1/people/email/alex@alexmaccaw.com?company=false')
-        .reply(200, alex);
-      return Person.find({email: 'alex@alexmaccaw.com', company: false});
-    });
-
     it('can handle queued requests', function () {
       mock
         .get('/v1/people/email/alex@alexmaccaw.com')
@@ -71,4 +66,37 @@ describe('Person', function () {
 
   });
 
+  describe('PersonCompany#find', function () {
+
+    it('can find a person by email', function () {
+      mock
+        .get('/v1/combined/email/alex@alexmaccaw.com')
+        .reply(200, {
+          person: alex,
+          company: company
+        });
+      return PersonCompany.find({email: 'alex@alexmaccaw.com'})
+        .then(function (personCompany) {
+          expect(personCompany)
+            .to.be.an.instanceOf(PersonCompany)
+            .and.have.have.keys('person', 'company')
+            .and.have.deep.property('person.id', alex.id);
+        });
+    });
+
+    it('can handle queued requests', function () {
+      mock
+        .get('/v1/combined/email/alex@alexmaccaw.com')
+        .reply(202, {
+          error: {
+            type: 'queued'
+          }
+        });
+      return expect(PersonCompany.find({email: 'alex@alexmaccaw.com'}))
+        .to.be.rejectedWith(PersonCompany.QueuedError);
+    });
+
+  });
+
 });
+

test/person_company.js

-'use strict';
-
-var expect = require('chai').use(require('chai-as-promised')).expect;
-var nock   = require('nock');
-var PersonCompany = require('../')('k').PersonCompany;
-
-describe('PersonCompany', function () {
-
-  var mock;
-  before(function () {
-    mock = nock('https://person.clearbit.com');
-  });
-  after(nock.cleanAll);
-  afterEach(function () {
-    mock.done();
-  });
-
-  describe('PersonCompany#find', function () {
-
-    var alex = require('./fixtures/person');
-
-    it('can find a person by email', function () {
-      mock
-        .get('/v1/combined/email/alex@alexmaccaw.com')
-        .reply(200, alex);
-      return PersonCompany.find({email: 'alex@alexmaccaw.com'})
-        .then(function (person) {
-          expect(person)
-            .to.be.an.instanceOf(PersonCompany)
-            .and.have.property('id', alex.id);
-        });
-    });
-
-    it('can handle queued requests', function () {
-      mock
-        .get('/v1/combined/email/alex@alexmaccaw.com')
-        .reply(202, {
-          error: {
-            type: 'queued'
-          }
-        });
-      return expect(PersonCompany.find({email: 'alex@alexmaccaw.com'}))
-        .to.be.rejectedWith(PersonCompany.QueuedError);
-    });
-
-  });
-
-});