amorail 0.3.4 → 0.6.0

data/spec/fixtures/webhooks/list.json

@@ -0,0 +1,24 @@
+{
+  "response": {
+    "webhooks": [
+      {
+        "id": "1",
+        "url": "http://example.org",
+        "events": [
+          "add_contact"
+        ],
+        "disabled": false
+      },
+      {
+        "id": "2",
+        "url": "http://example.com",
+        "events": [
+          "add_contact",
+					"add_company"
+        ],
+        "disabled": true
+      }
+    ],
+    "server_time": 1539938502
+  }
+}

data/spec/fixtures/webhooks/subscribe.json

@@ -0,0 +1,17 @@
+{
+  "response": {
+    "webhooks": {
+      "subscribe": [
+        {
+          "url": "http://example.org",
+          "result": true
+        },
+        {
+          "url": "http://example.com",
+          "result": true
+        }
+      ]
+    },
+    "server_time": 1539941636
+  }
+}

data/spec/fixtures/webhooks/unsubscribe.json

@@ -0,0 +1,17 @@
+{
+  "response": {
+    "webhooks": {
+      "unsubscribe": [
+        {
+          "url": "http://example.org",
+          "result": true
+        },
+        {
+          "url": "http://example.com",
+          "result": true
+        }
+      ]
+    },
+    "server_time": 1539941911
+  }
+}

data/spec/helpers/webmock_helpers.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 # rubocop: disable Metrics/ModuleLength
 module AmoWebMock
   def mock_api
@@ -9,7 +11,7 @@ module AmoWebMock
     account_info_stub(Amorail.config.api_endpoint)
   end
 
-  def mock_custom_api(endpoint, usermail, api_key, properties = 'account2_response.json')
+  def mock_custom_api(endpoint, usermail, api_key, properties = 'response_2.json')
     authorize_stub(
       endpoint,
       usermail,
@@ -33,10 +35,10 @@ module AmoWebMock
         })
   end
 
-  def account_info_stub(endpoint, properties = 'account_response.json')
+  def account_info_stub(endpoint, properties = 'response_1.json')
     stub_request(:get, endpoint + '/private/api/v2/json/accounts/current')
       .to_return(
-        body: File.read("./spec/fixtures/#{properties}"),
+        body: File.read("./spec/fixtures/accounts/#{properties}"),
         headers: { 'Content-Type' => 'application/json' },
         status: 200
       )
@@ -61,7 +63,7 @@ module AmoWebMock
   def contact_create_stub(endpoint)
     stub_request(:post, endpoint + '/private/api/v2/json/contacts/set')
       .to_return(
-        body: File.read('./spec/fixtures/contact_create.json'),
+        body: File.read('./spec/fixtures/contacts/create.json'),
         headers: { 'Content-Type' => 'application/json' },
         status: 200
       )
@@ -70,7 +72,7 @@ module AmoWebMock
   def contact_update_stub(endpoint)
     stub_request(:post, endpoint + '/private/api/v2/json/contacts/set')
       .to_return(
-        body: File.read('./spec/fixtures/contact_update.json'),
+        body: File.read('./spec/fixtures/contacts/update.json'),
         headers: {
           'Content-Type' => 'application/json'
         },
@@ -84,7 +86,7 @@ module AmoWebMock
         :get,
         "#{endpoint}/private/api/v2/json/contacts/list?id=#{id}")
         .to_return(
-          body: File.read('./spec/fixtures/contact_find.json'),
+          body: File.read('./spec/fixtures/contacts/find_one.json'),
           headers: { 'Content-Type' => 'application/json' },
           status: 200
         )
@@ -102,7 +104,7 @@ module AmoWebMock
         :get,
         "#{endpoint}/private/api/v2/json/contacts/list?id=#{id}")
         .to_return(
-          body: File.read('./spec/fixtures/my_contact_find.json'),
+          body: File.read('./spec/fixtures/contacts/my_contact_find.json'),
           headers: { 'Content-Type' => 'application/json' },
           status: 200
         )
@@ -120,7 +122,7 @@ module AmoWebMock
         :get,
         "#{endpoint}/private/api/v2/json/contacts/list?query=#{query}")
         .to_return(
-          body: File.read('./spec/fixtures/contact_find_query.json'),
+          body: File.read('./spec/fixtures/contacts/find_many.json'),
           headers: { 'Content-Type' => 'application/json' },
           status: 200
         )
@@ -138,7 +140,7 @@ module AmoWebMock
         :get,
         "#{endpoint}/private/api/v2/json/contacts/list?#{ids.to_query('id')}")
         .to_return(
-          body: File.read('./spec/fixtures/contact_find_query.json'),
+          body: File.read('./spec/fixtures/contacts/find_many.json'),
           headers: { 'Content-Type' => 'application/json' },
           status: 200
         )
@@ -150,10 +152,30 @@ module AmoWebMock
     end
   end
 
+  def contacts_where_stub(endpoint, success = true, **params)
+    if success
+      stub_request(
+        :get,
+        "#{endpoint}/private/api/v2/json/contacts/list"
+      ).with(
+        query: params
+      ).to_return(
+        body: File.read('./spec/fixtures/contacts/find_many.json'),
+        headers: { 'Content-Type' => 'application/json' },
+        status: 200
+      )
+    else
+      stub_request(
+        :get,
+        "#{endpoint}/private/api/v2/json/contacts/list?query=#{query}")
+        .to_return(status: 204)
+    end
+  end
+
   def company_create_stub(endpoint)
     stub_request(:post, endpoint + '/private/api/v2/json/company/set')
       .to_return(
-        body: File.read('./spec/fixtures/contact_create.json'),
+        body: File.read('./spec/fixtures/contacts/create.json'),
         headers: { 'Content-Type' => 'application/json' },
         status: 200
       )
@@ -165,7 +187,7 @@ module AmoWebMock
         :get,
         "#{endpoint}/private/api/v2/json/leads/list?#{ids.to_query('id')}")
         .to_return(
-          body: File.read('./spec/fixtures/leads.json'),
+          body: File.read('./spec/fixtures/leads/find_many.json'),
           headers: { 'Content-Type' => 'application/json' },
           status: 200
         )
@@ -177,10 +199,37 @@ module AmoWebMock
     end
   end
 
+  def lead_create_stub(endpoint)
+    stub_request(:post, endpoint + '/private/api/v2/json/leads/set')
+      .to_return(
+        body: File.read('./spec/fixtures/leads/create.json'),
+        headers: { 'Content-Type' => 'application/json' },
+        status: 200
+      )
+  end
+
+  def lead_update_stub(endpoint, success = true)
+    fixture_file =
+      if success
+        './spec/fixtures/leads/update.json'
+      else
+        './spec/fixtures/leads/update_errors.json'
+      end
+
+    stub_request(:post, endpoint + '/private/api/v2/json/leads/set')
+      .to_return(
+        body: File.read(fixture_file),
+        headers: {
+          'Content-Type' => 'application/json'
+        },
+        status: 200
+      )
+  end
+
   def contacts_links_stub(endpoint, ids)
     stub_request(:get, endpoint + "/private/api/v2/json/contacts/links?#{ids.to_query('contacts_link')}")
       .to_return(
-        body: File.read('./spec/fixtures/contacts_links.json'),
+        body: File.read('./spec/fixtures/contacts/links.json'),
         headers: { 'Content-Type' => 'application/json' },
         status: 200
       )
@@ -190,7 +239,7 @@ module AmoWebMock
     if success
       stub_request(:get, endpoint + "/private/api/v2/json/contacts/links?#{ids.to_query('deals_link')}")
         .to_return(
-          body: File.read('./spec/fixtures/leads_links.json'),
+          body: File.read('./spec/fixtures/leads/links.json'),
           headers: { 'Content-Type' => 'application/json' },
           status: 200
         )
@@ -199,4 +248,34 @@ module AmoWebMock
         .to_return(status: 204)
     end
   end
+
+  def webhooks_list_stub(endpoint, empty: false)
+    body = empty ? '' : File.read('./spec/fixtures/webhooks/list.json')
+    stub_request(:get, "#{endpoint}/private/api/v2/json/webhooks/list")
+      .to_return(
+        body: body,
+        headers: { 'Content-Type' => 'application/json' },
+        status: 200
+      )
+  end
+
+  def webhooks_subscribe_stub(endpoint, webhooks)
+    stub_request(:post, "#{endpoint}/private/api/v2/json/webhooks/subscribe")
+      .with(body: { request: { webhooks: { subscribe: webhooks } } }.to_json)
+      .to_return(
+        body: File.read('./spec/fixtures/webhooks/subscribe.json'),
+        headers: { 'Content-Type' => 'application/json' },
+        status: 200
+      )
+  end
+
+  def webhooks_unsubscribe_stub(endpoint, webhooks)
+    stub_request(:post, "#{endpoint}/private/api/v2/json/webhooks/unsubscribe")
+      .with(body: { request: { webhooks: { unsubscribe: webhooks } } }.to_json)
+      .to_return(
+        body: File.read('./spec/fixtures/webhooks/unsubscribe.json'),
+        headers: { 'Content-Type' => 'application/json' },
+        status: 200
+      )
+  end
 end

data/spec/lead_spec.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "spec_helper"
 
 describe Amorail::Lead do
@@ -18,6 +20,7 @@ describe Amorail::Lead do
         :name,
         :price,
         :status_id,
+        :pipeline_id,
         :tags
       )
     end
@@ -29,6 +32,7 @@ describe Amorail::Lead do
         name: 'Test',
         price: 100,
         status_id: 2,
+        pipeline_id: 17,
         tags: 'test lead'
       )
     end
@@ -39,6 +43,7 @@ describe Amorail::Lead do
     specify { is_expected.to include(name: 'Test') }
     specify { is_expected.to include(price: 100) }
     specify { is_expected.to include(status_id: 2) }
+    specify { is_expected.to include(pipeline_id: 17) }
     specify { is_expected.to include(tags: 'test lead') }
   end
 
@@ -70,4 +75,29 @@ describe Amorail::Lead do
       end
     end
   end
+
+  describe "#update" do
+    subject { lead.update }
+
+    let(:lead) { described_class.new(name: 'RSpec lead', status_id: 142) }
+
+    before do
+      lead_create_stub(Amorail.config.api_endpoint)
+      lead.save!
+    end
+
+    context 'with errors in response' do
+      before do
+        lead_update_stub(Amorail.config.api_endpoint, false)
+        lead.name = 'Updated name'
+      end
+
+      it { is_expected.to be_falsey }
+
+      specify do
+        subject
+        expect(lead.errors[:base]).to include('Last modified date is older than in database')
+      end
+    end
+  end
 end

data/spec/my_contact_spec.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "spec_helper"
 
 describe MyContact do

data/spec/note_spec.rb

@@ -0,0 +1,28 @@
+# frozen_string_literal: true
+
+require 'spec_helper'
+
+describe Amorail::Note do
+  before { mock_api }
+
+  it_behaves_like 'elementable'
+
+  describe 'validations' do
+    it { is_expected.to validate_presence_of(:text) }
+    it { is_expected.to validate_presence_of(:note_type) }
+    it { is_expected.to validate_inclusion_of(:element_type).in_range(1..4) }
+  end
+
+  describe '.attributes' do
+    subject { described_class.attributes }
+
+    it_behaves_like 'entity_class'
+
+    specify do
+      is_expected.to include(
+        :text,
+        :note_type
+      )
+    end
+  end
+end

data/spec/property_spec.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "spec_helper"
 require "webmock/rspec"
 

data/spec/spec_helper.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 $LOAD_PATH.unshift(File.join(File.dirname(__FILE__), '..', 'lib'))
 $LOAD_PATH.unshift(File.dirname(__FILE__))
 
@@ -10,9 +12,9 @@ require 'helpers/webmock_helpers'
 
 # Cleanup Amorail env
 ENV.delete_if { |k, _| k =~ /amorail/i }
-ENV["AMORAIL_CONF"] = File.expand_path("../fixtures/amorail_test.yml", __FILE__)
+ENV["AMORAIL_CONF"] = File.expand_path("fixtures/amorail_test.yml", __dir__)
 
-Dir[File.expand_path("../support/**/*.rb", __FILE__)].each { |f| require f }
+Dir[File.expand_path("support/**/*.rb", __dir__)].each { |f| require f }
 
 RSpec.configure do |config|
   config.mock_with :rspec

data/spec/support/elementable_example.rb

@@ -0,0 +1,54 @@
+# frozen_string_literal: true
+
+shared_examples 'elementable' do
+  describe 'validations' do
+    it { is_expected.to validate_presence_of(:element_id) }
+    it { is_expected.to validate_presence_of(:element_type) }
+  end
+
+  describe '.attributes' do
+    subject { described_class.attributes }
+
+    specify do
+      is_expected.to include(
+        :element_type,
+        :element_id
+      )
+    end
+  end
+
+  describe '#params' do
+    subject { elementable.params }
+
+    let(:elementable) do
+      described_class.new(
+        element_id: 1,
+        element_type: 2
+      )
+    end
+
+    it { is_expected.to include(element_id: 1) }
+    it { is_expected.to include(element_type: 2) }
+  end
+
+  describe 'element type behaviour' do
+    let(:elementable) { described_class.new }
+
+    it 'set element_type on initialize' do
+      expect(described_class.new(lead: true).element_type).to eq 2
+      expect(described_class.new(lead: false).element_type).to be_nil
+      expect(described_class.new(contact: true).contact?).to be_truthy
+    end
+
+    it 'set element_type with bang method' do
+      elementable.contact!
+      expect(elementable.element_type).to eq 1
+
+      elementable.lead!
+      expect(elementable.element_type).to eq 2
+
+      elementable.company!
+      expect(elementable.element_type).to eq 3
+    end
+  end
+end

data/spec/support/entity_class_example.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'spec_helper'
 
 shared_examples 'entity_class' do

data/spec/support/leadable_example.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'spec_helper'
 
 shared_examples 'leadable' do