capsule_crm 1.0.0 → 1.0.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 0e6a15562202d4d1a360a5beeb9d8918c7eb5e7e
-  data.tar.gz: 63a2206a1f4c450e50df285cbec99514648a9829
+  metadata.gz: 0b2ac3b80afb32c0e5e01fb90ed2677ec5bbc633
+  data.tar.gz: 0e12ab771e075cad5a39598da55d81015111620a
 SHA512:
-  metadata.gz: 68511e44926bb512b68e06419975305a3e1e940b4805b37b6cc4b3b378cee42dc44da112935e5d22bf83136055ce3fb1fa8c1a18179742c0299fa2ddf33461bb
-  data.tar.gz: c0aec7b22bc4ab680edaf83a5e52f67cbf81af988680b1e8c072012f936fc9275076eaf60fda178801f465114cc93c2e3ef658f9eb71669339c650027e8e05b9
+  metadata.gz: 377306fa489bb3aa8e2f1c19a64d233b47763b7d6ca250fa32ac30a5a536173f1c726dc17b8d402dced4578f9694797fd776095ed811c9d8e8bc6e047d9ec2bc
+  data.tar.gz: 6a34894ec8bd04703a4767dbebe8a91c48164f477a079729be646e8d59e21d4f074bcc7d11fc7c8b6b5ae60215ca1f38f7e53107cc439c03bc5dfaafc70aa767

data/CHANGELOG.md

@@ -1,5 +1,14 @@
 # Changelog
 
+## 1.0.0
+
+- Using SemVer and this gem is being used in production, so bumped to version 1
+- Handle 401 responses by raising a CapsuleCRM::Errors::Unauthorized error
+
+## 0.10.1
+
+- Automatically strformat the lastmodified param for GET requests
+
 ## 0.10.0
 
 - Added Track support

data/README.md

@@ -15,12 +15,6 @@ Ready](http://badge.waffle.io/mattbeedle/capsule_crm.png)](http://waffle.io/matt
 
 CapsuleCRM provides an ActiveModel compliant interface to the capsulecrm API
 
-## Versioning
-
-This project follows [semver](http://semver.org/spec/v2.0.0.html) so there will
-be no breaking changes in minor/patch version changes. It is pre 1.0
-though, so it's not finished and expect to find bugs.
-
 ## Installation
 
 Add this line to your application's Gemfile:
@@ -312,12 +306,18 @@ CapsuleCRM::Currency.all
 
 1.9.3, 2.0, jruby-19mode
 
+## Versioning
+
+This project follows [semver](http://semver.org/spec/v2.0.0.html) so there will
+be no breaking changes in minor/patch version changes.
+
 ## Feedback
 
 Please use github issues to give feedback. If you have a bug to report then an
 accompanying failing test would be great. Extra points for a full pull request
 with fix.
 
+
 ## Contributing
 
 1. Fork it
@@ -325,3 +325,11 @@ with fix.
 3. Commit your changes (`git commit -am 'Add some feature'`)
 4. Push to the branch (`git push origin my-new-feature`)
 5. Create new Pull Request
+
+## Contributers
+
+Many thanks to the following contributers:
+
+- @srbaker
+- @clod81
+- @danthompson

data/lib/capsule_crm.rb

@@ -25,6 +25,7 @@ require 'capsule_crm/website'
 require 'capsule_crm/hash_helper'
 require 'capsule_crm/results_proxy'
 require 'capsule_crm/errors/record_invalid'
+require 'capsule_crm/errors/record_not_saved'
 require 'capsule_crm/errors/unauthorized'
 require 'capsule_crm/contacts'
 require 'capsule_crm/contactable'

data/lib/capsule_crm/associations/has_many_proxy.rb

@@ -10,18 +10,20 @@ module CapsuleCRM
       end
 
       def method_missing(name, *args, &block)
-        @target.send(name, *args, &block)
+        target.send(name, *args, &block)
       end
 
       def build(attributes = {})
         target_klass.new(attributes).tap do |item|
           item.send("#{source}=", parent)
-          @target << item
+          target << item
         end
       end
 
       def create(attributes = {})
-        build(attributes).save
+        build(attributes).tap do |record|
+          record_not_saved(record) unless parent.persisted?
+        end.save
       end
 
       def tap
@@ -31,6 +33,14 @@ module CapsuleCRM
 
       private
 
+      def record_not_saved(record)
+        raise ::CapsuleCRM::Errors::RecordNotSaved.new(record)
+      end
+
+      def target
+        @target
+      end
+
       def parent
         @parent
       end

data/lib/capsule_crm/attachment.rb

@@ -2,8 +2,12 @@ module CapsuleCRM
   class Attachment
     include Virtus
 
+    include ActiveModel::Validations
+
     attribute :id,            Integer
     attribute :filename,      String
     attribute :content_type,  String
+
+    validates :id, numericality: { allow_blank: true }
   end
 end

data/lib/capsule_crm/case.rb

@@ -18,6 +18,7 @@ module CapsuleCRM
     attribute :close_date, Date
     attribute :owner, String
 
+    validates :id, numericality: { allow_blank: true }
     validates :name, presence: true
     validates :party, presence: true
 

data/lib/capsule_crm/connection.rb

@@ -13,7 +13,7 @@ module CapsuleCRM
     end
 
     def self.post(path, params = {})
-      process_post_response request(:post, path, params)
+      process_post_response request(:post, path, params.to_json)
     end
 
     def self.put(path, params)
@@ -56,7 +56,7 @@ module CapsuleCRM
     def self.process_post_response(response)
       if response.headers['Location'] &&
         match = response.headers['Location'].match(/\/(?<id>\d+)$/)
-        { id: match[:id] }
+        { id: match[:id].to_i }
       else
         true
       end

data/lib/capsule_crm/custom_field.rb

@@ -17,6 +17,7 @@ module CapsuleCRM
     attribute :boolean, Boolean
     attribute :text, String
 
+    validates :id, numericality: { allow_blank: true }
     validates :label, presence: true
 
     belongs_to :party, class_name: 'CapsuleCRM::Party'

data/lib/capsule_crm/errors/record_not_saved.rb

@@ -0,0 +1,18 @@
+module CapsuleCRM
+  module Errors
+    class RecordNotSaved < StandardError
+      attr_reader :record
+
+      def initialize(record)
+        @record = record
+      end
+
+      def message
+        [
+          'CapsuleCRM::Errors::RecordNotSaved',
+          'You cannot call create unless the parent is saved'
+        ].join(': ')
+      end
+    end
+  end
+end

data/lib/capsule_crm/history.rb

@@ -25,6 +25,7 @@ module CapsuleCRM
       foreign_key: :case_id
     belongs_to :opportunity,  class_name: 'CapsuleCRM::Opportunity'
 
+    validates :id, numericality: { allow_blank: true }
     validates :note, presence: true
     validates :party, :kase, :opportunity,
       presence: { if: :belongs_to_required? }

data/lib/capsule_crm/milestone.rb

@@ -1,8 +1,8 @@
 module CapsuleCRM
   class Milestone
     include Virtus
-
     include CapsuleCRM::Collection
+    include ActiveModel::Validations
 
     attribute :id, Integer
     attribute :name, String
@@ -10,6 +10,8 @@ module CapsuleCRM
     attribute :probability, Float
     attribute :complete, Boolean
 
+    validates :id, numericality: { allow_blank: true }
+
     def self.all
       init_collection(
         CapsuleCRM::Connection.

data/lib/capsule_crm/opportunity.rb

@@ -24,8 +24,9 @@ module CapsuleCRM
 
     attr_accessor :milestone, :owner
 
+    validates :id, numericality: { allow_blank: true }
     validates :name, presence: true
-    validates :party_id, presence: true
+    validates :party, presence: true
     validates :milestone, presence: true
 
     has_many :tasks, class_name: 'CapsuleCRM::Task', source: :opportunity

data/lib/capsule_crm/organization.rb

@@ -21,7 +21,7 @@ module CapsuleCRM
 
     has_many :people, class_name: 'CapsuleCRM::Person', source: :organization
     has_many :custom_fields, class_name: 'CapsuleCRM::CustomField',
-      source: :organization
+      source: :party
 
     # Public: Get all people from Capsule. The list can be restricted
     # and/or paginated with various query parameters sent through the options
@@ -134,7 +134,7 @@ module CapsuleCRM
     # Returns a CapsuleCRM::Organization
     def save!
       if valid?
-        new_record ? create_record : update_record
+        new_record? ? create_record : update_record
       else
         raise CapsuleCRM::Errors::RecordInvalid.new(self)
       end

data/lib/capsule_crm/person.rb

@@ -19,8 +19,10 @@ module CapsuleCRM
     belongs_to :organization, class_name: 'CapsuleCRM::Organization',
       foreign_key: :organisation_id
 
-    has_many :custom_fields, class_name: 'CapsuleCRM::CustomField', source: :person
+    has_many :custom_fields, class_name: 'CapsuleCRM::CustomField',
+      source: :party
 
+    validates :id, numericality: { allow_blank: true }
     validates :first_name, presence: { if: :first_name_required? }
     validates :last_name, presence: { if: :last_name_required? }
 

data/lib/capsule_crm/task.rb

@@ -22,6 +22,7 @@ module CapsuleCRM
     belongs_to :case, class_name: 'CapsuleCRM::Case'
     belongs_to :owner, class_name: 'CapsuleCRM::User'
 
+    validates :id, numericality: { allow_blank: true }
     validates :description, presence: true
     validates :due_date, presence: { unless: :due_date_time }
     validates :due_date_time, presence: { unless: :due_date }

data/lib/capsule_crm/track.rb

@@ -17,6 +17,8 @@ module CapsuleCRM
     has_many :opportunities, class_name: 'CapsuleCRM::Opportunity'
     has_many :cases, class_name: 'CapsuleCRM::Case'
 
+    validates :id, numericality: { allow_blank: true }
+
     def self.all
       init_collection(
         CapsuleCRM::Connection.get('/api/tracks')['tracks']['track']

data/lib/capsule_crm/version.rb

@@ -1,3 +1,3 @@
 module CapsuleCrm
-  VERSION = '1.0.0'
+  VERSION = '1.0.1'
 end

data/spec/fabricators/opportunity_fabricator.rb

@@ -1,5 +1,4 @@
 Fabricator(:opportunity, class_name: :'CapsuleCRM::Opportunity') do
   name          'Test'
   milestone_id  1
-  party_id      { rand(10) }
 end

data/spec/lib/capsule_crm/associations/has_many_proxy_spec.rb

@@ -7,6 +7,10 @@ class Parent
   attribute :id, Integer
 
   has_many :children, class_name: 'Child', source: :parent
+
+  def persisted?
+    !!id
+  end
 end
 
 class Child
@@ -62,10 +66,22 @@ describe CapsuleCRM::Associations::HasManyProxy do
   end
 
   describe '#create' do
-    before { parent.children.create(name: 'a test name') }
+    subject { parent.children.create(name: 'a test name') }
+    context 'when the parent is persisted' do
+      before do
+        parent.id = Random.rand(1..10)
+        subject
+      end
+
+      it 'should persist the child' do
+        expect(parent.children.last).to be_persisted
+      end
+    end
 
-    it 'should persist the child' do
-      parent.children.last.should be_persisted
+    context 'when the parent is not persisted' do
+      it 'should raise a RecordNotSaved error' do
+        expect { subject }.to raise_error(CapsuleCRM::Errors::RecordNotSaved)
+      end
     end
   end
 end

data/spec/lib/capsule_crm/attachment_spec.rb

@@ -0,0 +1,7 @@
+require 'spec_helper'
+
+describe CapsuleCRM::Attachment do
+  describe 'validations' do
+    it { should validate_numericality_of(:id) }
+  end
+end

data/spec/lib/capsule_crm/case_spec.rb

@@ -8,9 +8,11 @@ describe CapsuleCRM::Case do
       to_return(body: File.read('spec/support/all_users.json'))
   end
 
-  it { should validate_presence_of(:name) }
-
-  it { should validate_presence_of(:party) }
+  describe 'validations' do
+    it { should validate_numericality_of(:id) }
+    it { should validate_presence_of(:name) }
+    it { should validate_presence_of(:party) }
+  end
 
   describe '._for_track' do
     let(:track) { CapsuleCRM::Track.new }

data/spec/lib/capsule_crm/connection_spec.rb

@@ -23,11 +23,97 @@ describe CapsuleCRM::Connection do
         stub_request(:get, /\/api\/v1\/foo/).
           to_return(status: 401, body: '<html></html>')
       end
-      subject { CapsuleCRM::Connection.get('/api/v1/food') }
+      subject { CapsuleCRM::Connection.get('/api/v1/foo') }
 
       it 'should raise an Unauthorized error' do
-        expect { CapsuleCRM::Connection.get('/api/v1/foo') }.
-          to raise_error(CapsuleCRM::Errors::Unauthorized)
+        expect { subject }.to raise_error(CapsuleCRM::Errors::Unauthorized)
+      end
+    end
+  end
+
+  describe '.post' do
+    subject { CapsuleCRM::Connection.post('/foo', { bar: :baz }) }
+
+    context 'when it is a success' do
+      context 'when it returns a location header' do
+        before do
+          stub_request(:post, /.*/).to_return(
+            status: 200, headers: { 'Location' => "http://foo.bar/#{id}" }
+          )
+        end
+        let(:id) { Random.rand(1..100) }
+
+        it 'should return the ID' do
+          expect(subject).to eql({ id: id })
+        end
+      end
+
+      context 'when it does not return a location header' do
+        before do
+          stub_request(:post, /.*/).to_return(status: 200, body: {}.to_json)
+        end
+
+        it 'should return true' do
+          expect(subject).to eql(true)
+        end
+      end
+    end
+
+    context 'when the request is not authorized' do
+      before do
+        stub_request(:post, /.*/).to_return(status: 401, body: '<html></html>')
+      end
+
+      it 'should raise an Unauthorized error' do
+        expect { subject }.to raise_error(CapsuleCRM::Errors::Unauthorized)
+      end
+    end
+  end
+
+  describe '.put' do
+    subject { CapsuleCRM::Connection.put('/foo', { bar: :baz }) }
+
+    context 'when it is a success' do
+      before do
+        stub_request(:put, /.*/).to_return(status: 200, body: {}.to_json)
+      end
+
+      it 'should return true' do
+        expect(subject).to eql(true)
+      end
+    end
+
+    context 'when it is not authorized' do
+      before do
+        stub_request(:put, /\.*/).to_return(status: 401)
+      end
+
+      it 'should raise an Unauthorized error' do
+        expect { subject }.to raise_error(CapsuleCRM::Errors::Unauthorized)
+      end
+    end
+  end
+
+  describe '.delete' do
+    subject { CapsuleCRM::Connection.delete('/foo') }
+
+    context 'when it is a success' do
+      before do
+        stub_request(:delete, /.*/).to_return(status: 200)
+      end
+
+      it 'should return true' do
+        expect(subject).to eql(true)
+      end
+    end
+
+    context 'when it is not authorized' do
+      before do
+        stub_request(:delete, /.*/).to_return(status: 401)
+      end
+
+      it 'should raise an Unauthorized error' do
+        expect { subject }.to raise_error(CapsuleCRM::Errors::Unauthorized)
       end
     end
   end

data/spec/lib/capsule_crm/custom_field_spec.rb

@@ -10,7 +10,10 @@ describe CapsuleCRM::CustomField do
       to_return(body: File.read('spec/support/milestones.json'))
   end
 
-  it { should validate_presence_of(:label) }
+  describe 'validations' do
+    it { should validate_numericality_of(:id) }
+    it { should validate_presence_of(:label) }
+  end
 
   describe '._for_party' do
     let(:party) { Fabricate.build(:organization, id: 1) }

data/spec/lib/capsule_crm/history_spec.rb

@@ -10,7 +10,43 @@ describe CapsuleCRM::History do
       to_return(body: File.read('spec/support/milestones.json'))
   end
 
-  it { should validate_presence_of(:note) }
+  describe 'validations' do
+    subject { described_class.new }
+
+    it { should validate_numericality_of(:id) }
+    it { should validate_presence_of(:note) }
+    it { should validate_presence_of(:kase) }
+    it { should validate_presence_of(:party) }
+    it { should validate_presence_of(:opportunity) }
+
+    context 'when it belongs to a case' do
+      before do
+        subject.kase = double('CapsuleCRM::Case', id: Random.rand(1..10))
+      end
+
+      it { should_not validate_presence_of(:party) }
+      it { should_not validate_presence_of(:opportunity) }
+    end
+
+    context 'when it belongs to a party' do
+      before do
+        subject.party = double('CapsuleCRM::Party', id: Random.rand(1..10))
+      end
+
+      it { should_not validate_presence_of(:kase) }
+      it { should_not validate_presence_of(:opportunity) }
+    end
+
+    context 'when it belongs to an opportunity' do
+      before do
+        subject.opportunity =
+          double('CapsuleCRM::Opportunity', id: Random.rand(1..10))
+      end
+
+      it { should_not validate_presence_of(:party) }
+      it { should_not validate_presence_of(:kase) }
+    end
+  end
 
   describe '_.for_party' do
     let(:party) { Fabricate.build(:person, id: 1) }

data/spec/lib/capsule_crm/milestone_spec.rb

@@ -3,6 +3,10 @@ require 'spec_helper'
 describe CapsuleCRM::Milestone do
   before { configure }
 
+  describe 'validations' do
+    it { should validate_numericality_of(:id) }
+  end
+
   describe '.all' do
     before do
       stub_request(:get, /\/api\/opportunity\/milestones$/).

data/spec/lib/capsule_crm/opportunity_spec.rb

@@ -13,26 +13,13 @@ describe CapsuleCRM::Opportunity do
       to_return(body: File.read('spec/support/all_users.json'))
   end
 
-  it { should validate_presence_of(:name) }
+  describe 'validations' do
+    subject { described_class.new }
 
-  it { should validate_presence_of(:milestone) }
-
-  context 'when milestone_id is set' do
-    subject { CapsuleCRM::Opportunity.new(milestone_id: 1) }
-
-    before do
-      subject.valid?
-    end
-
-    it { subject.errors[:milestone].should be_blank }
-  end
-
-  context 'when milestone is set' do
-    let(:milestone) { CapsuleCRM::Milestone.new name: 'Bid' }
-
-    subject { CapsuleCRM::Opportunity.new(milestone: milestone) }
-
-    it { should_not validate_presence_of(:milestone_id) }
+    it { should validate_numericality_of(:id) }
+    it { should validate_presence_of(:name) }
+    it { should validate_presence_of(:milestone) }
+    it { should validate_presence_of(:party) }
   end
 
   describe '._for_track' do
@@ -215,7 +202,6 @@ describe CapsuleCRM::Opportunity do
     end
 
     it { opportunity.party_id.should eql(1) }
-
     it { opportunity.milestone_id.should eql(3) }
   end
 
@@ -226,25 +212,23 @@ describe CapsuleCRM::Opportunity do
           'Location' => 'https://sample.capsulecrm.com/api/opportunity/59'
         })
       end
-
       let(:request_path) do
         [
           'https://1234:@company.capsulecrm.com/api/party/',
-          opportunity_attributes[:party_id],
-          '/opportunity'
+          opportunity_attributes[:party].id, '/opportunity'
         ].join
       end
-
-      let(:opportunity_attributes) { Fabricate.attributes_for(:opportunity) }
+      let(:party) { Fabricate.build(:person, id: Random.rand(1..10)) }
+      let(:opportunity_attributes) do
+        Fabricate.attributes_for(:opportunity).merge(party: party)
+      end
 
       subject do
         CapsuleCRM::Opportunity.create opportunity_attributes
       end
 
       it { should be_a(CapsuleCRM::Opportunity) }
-
       it { should be_persisted }
-
       it { subject.id.should eql(59) }
 
       context 'when the opportunity has a track' do
@@ -288,15 +272,17 @@ describe CapsuleCRM::Opportunity do
           'Location' => 'https://sample.capsulecrm.com/api/opportunity/71'
         })
       end
+      let(:party) { Fabricate.build(:person, id: Random.rand(1..10)) }
+      let(:opportunity_attributes) do
+        Fabricate.attributes_for(:opportunity).merge(party: party)
+      end
 
       subject do
-        CapsuleCRM::Opportunity.create Fabricate.attributes_for(:opportunity)
+        CapsuleCRM::Opportunity.create opportunity_attributes
       end
 
       it { should be_a(CapsuleCRM::Opportunity) }
-
       it { should be_persisted }
-
       it { subject.id.should eql(71) }
     end
 
@@ -365,9 +351,9 @@ describe CapsuleCRM::Opportunity do
           'Location' => 'https://sample.capsulecrm.com/api/party/100'
         }, status: 200)
       end
-
+      let(:party) { Fabricate.build(:person) }
       let(:opportunity) do
-        Fabricate.build(:opportunity)
+        Fabricate.build(:opportunity, party: party)
       end
 
       before { opportunity.save }
@@ -388,9 +374,9 @@ describe CapsuleCRM::Opportunity do
             'Location' => 'https://sample.capsulecrm.com/api/party/100',
           }, status: 200)
         end
-
+        let(:party) { Fabricate.build(:person) }
         let(:opportunity) do
-          Fabricate.build(:opportunity)
+          Fabricate.build(:opportunity, party: party)
         end
 
         before { opportunity.save! }

data/spec/lib/capsule_crm/organization_spec.rb

@@ -11,6 +11,22 @@ describe CapsuleCRM::Organization do
       to_return(body: File.read('spec/support/all_users.json'))
   end
 
+  describe '#custom_fields' do
+    before do
+      stub_request(:get, /\/api\/party\/#{organization.id}\/customfields/).
+        to_return(body: File.read('spec/support/no_customfields.json'))
+    end
+    let(:organization) { Fabricate.build(:organization, id: 1) }
+
+    describe '#build' do
+      before { organization.custom_fields.build(label: 'test', text: 'bidule') }
+
+      it 'should add a new custom field to the organization' do
+        expect(organization.custom_fields.length).to eql(1)
+      end
+    end
+  end
+
   describe '.all' do
     context 'when some parties exist' do
       before do

data/spec/lib/capsule_crm/person_spec.rb

@@ -13,6 +13,26 @@ describe CapsuleCRM::Person do
 
   before { configure }
 
+  describe 'validations' do
+    subject { described_class.new }
+
+    it { should validate_numericality_of(:id) }
+    it { should validate_presence_of(:first_name) }
+    it { should validate_presence_of(:last_name) }
+
+    context 'when the first name is set' do
+      before { subject.first_name = Faker::Lorem.word }
+
+      it { should_not validate_presence_of(:last_name) }
+    end
+
+    context 'when the last name is set' do
+      before { subject.last_name = Faker::Lorem.word }
+
+      it { should_not validate_presence_of(:first_name) }
+    end
+  end
+
   describe '#tasks' do
     let(:person) { Fabricate.build(:person, id: 1) }
 
@@ -26,6 +46,22 @@ describe CapsuleCRM::Person do
     it { should be_a(Array) }
   end
 
+  describe '#custom_fields' do
+    before do
+      stub_request(:get, /\/api\/party\/#{person.id}\/customfields/).
+        to_return(body: File.read('spec/support/no_customfields.json'))
+    end
+    let(:person) { Fabricate.build(:person, id: 1) }
+
+    describe '#build' do
+      before { person.custom_fields.build(label: 'test', text: 'bidule') }
+
+      it 'should add a custom field' do
+        expect(person.custom_fields.length).to eql(1)
+      end
+    end
+  end
+
   describe '._for_organization' do
     context 'when there are many people for the organization' do
       pending

data/spec/lib/capsule_crm/task_spec.rb

@@ -9,11 +9,12 @@ describe CapsuleCRM::Task do
       to_return(body: File.read('spec/support/person.json'))
   end
 
-  it { should validate_presence_of(:description) }
-
-  it { should validate_presence_of(:due_date) }
-
-  it { should validate_presence_of(:due_date_time) }
+  describe 'validations' do
+    it { should validate_numericality_of(:id) }
+    it { should validate_presence_of(:description) }
+    it { should validate_presence_of(:due_date) }
+    it { should validate_presence_of(:due_date_time) }
+  end
 
   describe '.find' do
     before do
@@ -24,17 +25,11 @@ describe CapsuleCRM::Task do
     subject { CapsuleCRM::Task.find(2) }
 
     it { should be_a(CapsuleCRM::Task) }
-
     it { subject.due_date.should_not be_blank }
-
     it { subject.category.should eql('Meeting') }
-
     it { subject.description.should eql('Meet with customer') }
-
     it { subject.detail.should eql('Meeting at Coffee shop') }
-
     it { subject.owner.should be_a(CapsuleCRM::User) }
-
     it { subject.party.should be_a(CapsuleCRM::Person) }
   end
 

data/spec/lib/capsule_crm/track_spec.rb

@@ -3,6 +3,10 @@ require 'spec_helper'
 describe CapsuleCRM::Track do
   before { configure }
 
+  describe 'validations' do
+    it { should validate_numericality_of(:id) }
+  end
+
   describe '.all' do
     before do
       stub_request(:get, /\/api\/tracks$/).
@@ -12,15 +16,10 @@ describe CapsuleCRM::Track do
     subject { CapsuleCRM::Track.all }
 
     it { expect(subject).to be_a(Array) }
-
     it { expect(subject.length).to eql(2) }
-
     it { expect(subject.first).to be_a(CapsuleCRM::Track) }
-
     it { expect(subject.first.description).to eql('Sales follow up') }
-
     it { expect(subject.first.capture_rule).to eql('OPPORTUNITY') }
-
     it { expect(subject.first.id).to eql(1) }
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: capsule_crm
 version: !ruby/object:Gem::Version
-  version: 1.0.0
+  version: 1.0.1
 platform: ruby
 authors:
 - Matt Beedle
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-01-06 00:00:00.000000000 Z
+date: 2014-01-08 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activemodel
@@ -272,6 +272,7 @@ files:
 - lib/capsule_crm/custom_field.rb
 - lib/capsule_crm/email.rb
 - lib/capsule_crm/errors/record_invalid.rb
+- lib/capsule_crm/errors/record_not_saved.rb
 - lib/capsule_crm/errors/unauthorized.rb
 - lib/capsule_crm/hash_helper.rb
 - lib/capsule_crm/history.rb
@@ -299,6 +300,7 @@ files:
 - spec/lib/capsule_crm/address_spec.rb
 - spec/lib/capsule_crm/associations/has_many_proxy_spec.rb
 - spec/lib/capsule_crm/associations/has_many_spec.rb
+- spec/lib/capsule_crm/attachment_spec.rb
 - spec/lib/capsule_crm/case_spec.rb
 - spec/lib/capsule_crm/connection_spec.rb
 - spec/lib/capsule_crm/contacts_spec.rb
@@ -379,6 +381,7 @@ test_files:
 - spec/lib/capsule_crm/address_spec.rb
 - spec/lib/capsule_crm/associations/has_many_proxy_spec.rb
 - spec/lib/capsule_crm/associations/has_many_spec.rb
+- spec/lib/capsule_crm/attachment_spec.rb
 - spec/lib/capsule_crm/case_spec.rb
 - spec/lib/capsule_crm/connection_spec.rb
 - spec/lib/capsule_crm/contacts_spec.rb