capsule_crm 1.2.0 → 1.3.0

data/lib/capsule_crm/track.rb

@@ -7,22 +7,17 @@ module CapsuleCRM
     include ActiveModel::Validations
 
     include CapsuleCRM::Associations
-    include CapsuleCRM::Attributes
-    include CapsuleCRM::Collection
+    include CapsuleCRM::Querying::Configuration
+    include CapsuleCRM::Querying::FindAll
+    include CapsuleCRM::Serializable
 
     attribute :id,            Integer
     attribute :description,   String
     attribute :capture_rule,  String
 
-    has_many :opportunities, class_name: 'CapsuleCRM::Opportunity'
-    has_many :cases, class_name: 'CapsuleCRM::Case'
+    has_many :opportunities
+    has_many :cases
 
     validates :id, numericality: { allow_blank: true }
-
-    def self.all
-      init_collection(
-        CapsuleCRM::Connection.get('/api/tracks')['tracks']['track']
-      )
-    end
   end
 end

data/lib/capsule_crm/user.rb

@@ -7,6 +7,9 @@ module CapsuleCRM
     include ActiveModel::Validations
 
     include CapsuleCRM::Associations
+    include CapsuleCRM::Querying::Configuration
+    include CapsuleCRM::Querying::FindAll
+    include CapsuleCRM::Serializable
 
     attribute :username, String
     attribute :name, String
@@ -16,21 +19,6 @@ module CapsuleCRM
 
     belongs_to :party, class_name: 'CapsuleCRM::Party'
 
-    # Public: Retrieve all users from CapsuleCRM
-    #
-    # Examples:
-    #
-    # CapsuleCRM::User.all
-    #
-    # Returns a CapsuleCRM::ResultsProxy of CapsuleCRM::User objects
-    def self.all
-      CapsuleCRM::ResultsProxy.new(
-        CapsuleCRM::Connection.get('/api/users')['users']['user'].map do |item|
-          new item
-        end
-      )
-    end
-
     def id
       username
     end

data/lib/capsule_crm/version.rb

@@ -1,3 +1,3 @@
 module CapsuleCrm
-  VERSION = '1.2.0'
+  VERSION = '1.3.0'
 end

data/lib/capsule_crm/website.rb

@@ -1,12 +1,14 @@
 module CapsuleCRM
   class Website
     include Virtus
+    include CapsuleCRM::Serializable
 
     extend  ActiveModel::Naming
     include ActiveModel::Validations
-    include ActiveModel::Serializers::JSON
 
-    include CapsuleCRM::CapsuleJsonable
+    serializable_config do |config|
+      config.include_root = false
+    end
 
     attribute :type
     attribute :web_service

data/lib/capsule_crm.rb

@@ -1,18 +1,21 @@
 require 'active_model'
+require 'active_support/core_ext/string/inflections'
 require 'faraday'
 require 'faraday_middleware'
 require 'virtus'
-require 'capsule_crm/attributes'
-require 'capsule_crm/capsule_jsonable'
+require 'capsule_crm/gettable'
 require 'capsule_crm/taggable'
-require 'capsule_crm/collection'
 require 'capsule_crm/associations'
-require 'capsule_crm/address'
-require 'capsule_crm/case'
 require 'capsule_crm/connection'
+require 'capsule_crm/normalizer'
+require 'capsule_crm/persistence'
+require 'capsule_crm/querying'
 require 'capsule_crm/serializer'
+require 'capsule_crm/serializable'
 
+require 'capsule_crm/address'
 require 'capsule_crm/attachment'
+require 'capsule_crm/case'
 require 'capsule_crm/country'
 require 'capsule_crm/currency'
 require 'capsule_crm/email'
@@ -33,6 +36,7 @@ require 'capsule_crm/contacts'
 require 'capsule_crm/contactable'
 require 'capsule_crm/configuration'
 require 'capsule_crm/custom_field'
+require 'capsule_crm/custom_field_definition'
 require 'capsule_crm/opportunity'
 require 'capsule_crm/organization'
 require 'capsule_crm/participant'
@@ -53,4 +57,10 @@ module CapsuleCRM
     yield(self.configuration)
     self.configuration
   end
+
+  def self.log(message, level = :debug)
+    if self.configuration.perform_logging
+      self.configuration.logger.send(level, message)
+    end
+  end
 end

data/spec/fabricators/case_fabricator.rb

@@ -1,3 +1,4 @@
 Fabricator(:case, class_name: :'CapsuleCRM::Case') do
   name { Faker::Lorem.sentence }
+  party { Fabricate.build(:person, id: Random.rand(1..10)) }
 end

data/spec/fabricators/history_fabricator.rb

@@ -1,3 +1,4 @@
 Fabricator(:history, class_name: :'CapsuleCRM::History') do
   note { Faker::Lorem.paragraph }
+  party { Fabricate.build(:person, id: Random.rand(1..10)) }
 end

data/spec/fabricators/opportunity_fabricator.rb

@@ -1,4 +1,5 @@
 Fabricator(:opportunity, class_name: :'CapsuleCRM::Opportunity') do
   name          'Test'
   milestone_id  1
+  party         { Fabricate.build(:person, id: Random.rand(1..10)) }
 end

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

@@ -0,0 +1,48 @@
+require 'spec_helper'
+
+describe CapsuleCRM::Associations::BelongsToFinder do
+  before { configure }
+
+  describe '#call' do
+    let(:association) do
+      double(
+        'CapsuleCRM::Associations::BelongsToAssociation', inverse: inverse
+      )
+    end
+    let(:inverse) do
+      double('CapsuleCRM::Associations::HasManyAssociation')
+    end
+    let(:normalizer) do
+      double('CapsuleCRM::Normalizer', normalize_collection: [])
+    end
+    let(:finder) { described_class.new(association) }
+    before do
+      finder.singular = 'person'
+      finder.plural = 'customfields'
+      finder.normalizer = normalizer
+    end
+
+    context 'when an ID is supplied' do
+      before do
+        stub_request(:get, /.*/).
+          to_return(body: File.read('spec/support/all_customfields.json'))
+      end
+      subject { finder.call(1) }
+
+      it 'should send a request' do
+        subject
+        expect(WebMock).to have_requested(
+          :get, "https://1234:@company.capsulecrm.com/api/person/1/customfields"
+        )
+      end
+    end
+
+    context 'when a nil ID is supplied' do
+      subject { finder.call(nil) }
+
+      it 'should return an array' do
+        expect(subject).to be_a(Array)
+      end
+    end
+  end
+end

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

@@ -0,0 +1,34 @@
+require 'spec_helper'
+
+class BelongsToTest
+  include Virtus
+  include CapsuleCRM::Associations
+
+  belongs_to :person
+end
+
+describe CapsuleCRM::Associations::BelongsTo do
+  describe '.belongs_to' do
+    subject { BelongsToTest.new }
+
+    it 'should create a foreign key getter' do
+      expect(subject).to respond_to(:person_id)
+    end
+
+    it 'should create a foreign key setter' do
+      expect(subject).to respond_to(:person_id=)
+    end
+
+    it 'should create a class level finder method' do
+      expect(subject.class).to respond_to(:_for_person)
+    end
+
+    it 'should create an association setter method' do
+      expect(subject).to respond_to(:person=)
+    end
+
+    it 'should create an association getter method' do
+      expect(subject).to respond_to(:person)
+    end
+  end
+end

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

@@ -3,10 +3,18 @@ require 'spec_helper'
 class Parent
   include Virtus
   include CapsuleCRM::Associations
+  include CapsuleCRM::Querying::Configuration
+
+  queryable_config do |config|
+    config.plural = :people
+    config.singular = :person
+  end
 
   attribute :id, Integer
 
   has_many :children, class_name: 'Child', source: :parent
+  has_many :embedded_children, class_name: 'Child', source: :parent_as_embedded,
+    embedded: true
 
   def persisted?
     !!id
@@ -16,10 +24,21 @@ end
 class Child
   include Virtus
   include CapsuleCRM::Associations
+  include CapsuleCRM::Querying::Configuration
+  include CapsuleCRM::Serializable
+
+  queryable_config do |config|
+    config.plural = :children
+  end
+
+  serializable_config do |config|
+    config.collection_root = :childs
+  end
 
   attribute :name
 
-  belongs_to :parent
+  belongs_to :parent, class_name: 'Parent'
+  belongs_to :parent_as_embedded, class_name: 'Parent'
 
   def self._for_parent(parent_id)
     []
@@ -40,10 +59,9 @@ describe CapsuleCRM::Associations::HasManyProxy do
       parent, Child, target, :parent
     )
   end
-
   let(:parent) { Parent.new }
-
   let(:target) { [] }
+  before { configure }
 
   describe '#build' do
     before { parent.children.build(name: 'a test name') }
@@ -66,21 +84,43 @@ describe CapsuleCRM::Associations::HasManyProxy do
   end
 
   describe '#create' do
-    subject { parent.children.create(name: 'a test name') }
-    context 'when the parent is persisted' do
-      before do
-        parent.id = Random.rand(1..10)
-        subject
+    context 'when the child is embedded' do
+      let(:name) { Faker::Lorem.word }
+      subject { parent.embedded_children.create(name: name) }
+
+      context 'when the parent is persisted' do
+        before do
+          stub_request(:put, /.*/).to_return(status: 200)
+          parent.id = Random.rand(1..10)
+          subject
+        end
+
+        it 'should send a put request to capsule' do
+          expect(WebMock).to have_requested(
+            :put, "https://1234:@company.capsulecrm.com/api/person/#{parent.id}/children"
+          ).with(body: { childs: { child: [{ name: name }] } })
+        end
       end
+    end
 
-      it 'should persist the child' do
-        expect(parent.children.last).to be_persisted
+    context 'when the child is not embedded' do
+      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
-    end
 
-    context 'when the parent is not persisted' do
-      it 'should raise a RecordNotSaved error' do
-        expect { subject }.to raise_error(CapsuleCRM::Errors::RecordNotSaved)
+      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/associations/has_many_spec.rb

@@ -1,8 +1,21 @@
 require 'spec_helper'
 
-describe CapsuleCRM::Associations::HasMany do
+class HasManyTest
+  include CapsuleCRM::Associations
+
+  has_many :people
+end
 
+describe CapsuleCRM::Associations::HasMany do
   describe '#has_many' do
-    pending
+    subject { HasManyTest.new }
+
+    it 'should create an accessor method' do
+      expect(subject).to respond_to(:people)
+    end
+
+    it 'should create a setter method' do
+      expect(subject).to respond_to(:people=)
+    end
   end
 end