cangaroo 1.0.0 → 1.1.0

data/spec/fixtures/json_payload_connection_response.json

@@ -0,0 +1,10 @@
+{
+  "request_id": "c143e548-afed-44ab-9997-43614d6c762b",
+  "summary": "Product added with Shopify ID of 5775282183 was added.",
+  "products": [
+    {
+      "id": "SKUPRODUCT",
+      "shopify_id": "5775282183"
+    }
+  ]
+}

data/spec/interactors/cangaroo/handle_request_spec.rb

@@ -8,9 +8,7 @@ describe Cangaroo::HandleRequest do
 
     let(:interactors) do
       [Cangaroo::AuthenticateConnection,
-       Cangaroo::ValidateJsonSchema,
-       Cangaroo::CountJsonObject,
-       Cangaroo::PerformJobs]
+       Cangaroo::PerformFlow]
     end
 
     it { is_expected.to eql interactors }

data/spec/interactors/cangaroo/perform_flow_spec.rb

@@ -0,0 +1,17 @@
+require 'rails_helper'
+
+describe Cangaroo::PerformFlow do
+  subject { described_class.new }
+
+  context 'organizes interactors' do
+    subject { described_class.organized }
+
+    let(:interactors) do
+      [Cangaroo::ValidateJsonSchema,
+       Cangaroo::CountJsonObject,
+       Cangaroo::PerformJobs]
+    end
+
+    it { is_expected.to eql interactors }
+  end
+end

data/spec/interactors/cangaroo/validate_json_schema_spec.rb

@@ -5,7 +5,15 @@ describe Cangaroo::ValidateJsonSchema do
 
   describe '.call' do
     context 'when json is well formatted' do
-      let(:json_body) { JSON.parse(load_fixture('json_payload_ok.json')) }
+      let(:json_body) { parse_fixture('json_payload_ok.json') }
+
+      it 'succeeds' do
+        expect(context).to be_a_success
+      end
+    end
+
+    context 'when json is well formatted and come from a connection' do
+      let(:json_body) { parse_fixture('json_payload_connection_response.json') }
 
       it 'succeeds' do
         expect(context).to be_a_success
@@ -14,9 +22,7 @@ describe Cangaroo::ValidateJsonSchema do
 
     context 'when json is not well formatted' do
       describe 'with wrong main key' do
-        let(:json_body) do
-          JSON.parse(load_fixture('json_payload_wrong_key.json'))
-        end
+        let(:json_body) { parse_fixture('json_payload_wrong_key.json') }
 
         it 'fails' do
           expect(context).to be_a_failure
@@ -32,7 +38,7 @@ describe Cangaroo::ValidateJsonSchema do
       end
 
       describe 'without an object id' do
-        let(:json_body) { JSON.parse(load_fixture('json_payload_no_id.json')) }
+        let(:json_body) { parse_fixture('json_payload_no_id.json') }
 
         it 'fails' do
           expect(context).to be_a_failure

data/spec/jobs/cangaroo/job_spec.rb

@@ -9,39 +9,50 @@ module Cangaroo
     end
 
     let(:job_class) { FakeJob }
-    let(:source_connection) { create(:cangaroo_connection) }
+    let(:destination_connection) { create(:cangaroo_connection) }
     let(:type) { 'orders' }
     let(:payload) { { id: 'O123' } }
+    let(:connection_response) { parse_fixture('json_payload_connection_response.json') }
 
     let(:options) do
-      { connection: source_connection,
+      { connection: destination_connection,
         type: type,
         payload: payload }
     end
 
     let(:client) do
-      Cangaroo::Webhook::Client.new(source_connection, '/webhook_path')
+      Cangaroo::Webhook::Client.new(destination_connection, '/webhook_path')
     end
 
     before do
-      client.stub(:post)
+      client.stub(:post).and_return(connection_response)
       allow(Cangaroo::Webhook::Client).to receive(:new).and_return(client)
+      allow(Cangaroo::PerformFlow).to receive(:call)
     end
 
     describe '#perform' do
+      let(:job) { job_class.new(options) }
+
       it 'instantiates a Cangaroo::Webhook::Client' do
         expect(Cangaroo::Webhook::Client).to receive(:new)
-          .with(source_connection, '/webhook_path')
+          .with(destination_connection, '/webhook_path')
           .and_return(client)
-        FakeJob.perform_now(options)
+        job_class.perform_now(options)
       end
 
       it 'calls post on client' do
-        job = job_class.new(options)
         job.perform
         expect(client).to have_received(:post)
           .with(job.transform, job.job_id, email: 'info@nebulab.it')
       end
+
+      it 'restart the flow' do
+        job.perform
+        expect(Cangaroo::PerformFlow).to have_received(:call)
+          .with(source_connection: destination_connection,
+                json_body: connection_response.to_json,
+                jobs: Rails.configuration.cangaroo.jobs)
+      end
     end
 
     describe '#perform?' do

data/spec/rails_helper.rb

@@ -1,7 +1,10 @@
 require 'simplecov'
+require 'codeclimate-test-reporter'
+
 SimpleCov.start 'rails' do
   add_group 'Commands', 'app/commands'
 end
+CodeClimate::TestReporter.start
 
 # This file is copied to spec/ when you run 'rails generate rspec:install'
 ENV['RAILS_ENV'] ||= 'test'
@@ -50,6 +53,11 @@ end
 RSpec.configure do |config|
   config.include FactoryGirl::Syntax::Methods
 
+  # whitelist codeclimate.com so test coverage can be reported
+  config.after(:suite) do
+    WebMock.disable_net_connect!(allow: 'codeclimate.com')
+  end
+
   config.before(:suite) do
     DatabaseCleaner.strategy = :transaction
     DatabaseCleaner.clean_with(:truncation)
@@ -60,25 +68,7 @@ RSpec.configure do |config|
       example.run
     end
   end
-  # Remove this line if you're not using ActiveRecord or ActiveRecord fixtures
-  # config.fixture_path = "#{::Rails.root}/spec/fixtures"
-
-  # If you're not using ActiveRecord, or you'd prefer not to run each of your
-  # examples within a transaction, remove the following line or assign false
-  # instead of true.
-  # config.use_transactional_fixtures = true
 
-  # RSpec Rails can automatically mix in different behaviours to your tests
-  # based on their file location, for example enabling you to call `get` and
-  # `post` in specs under `spec/controllers`.
-  #
-  # You can disable this behaviour by removing the line below, and instead
-  # explicitly tag your specs with their type, e.g.:
-  #
-  #     RSpec.describe UsersController, :type => :controller do
-  #       # ...
-  #     end
-  #
   # The different available types are documented in the features, such as in
   # https://relishapp.com/rspec/rspec-rails/docs
   config.infer_spec_type_from_file_location!

data/spec/support/spec_helpers.rb

@@ -3,6 +3,10 @@ module Cangaroo
     def load_fixture(filename)
       File.read(File.expand_path("../../fixtures/#{filename}", __FILE__))
     end
+
+    def parse_fixture(filename)
+      JSON.parse(load_fixture(filename))
+    end
   end
 end
 

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: cangaroo
 version: !ruby/object:Gem::Version
-  version: 1.0.0
+  version: 1.1.0
 platform: ruby
 authors:
 - Alessio Rocco
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2015-12-09 00:00:00.000000000 Z
+date: 2016-03-25 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails
@@ -68,7 +68,7 @@ dependencies:
       - !ruby/object:Gem::Version
         version: 0.13.7
 - !ruby/object:Gem::Dependency
-  name: pg
+  name: codeclimate-test-reporter
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
@@ -82,7 +82,7 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
-  name: rspec-rails
+  name: database_cleaner
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
@@ -110,7 +110,7 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
-  name: database_cleaner
+  name: pg
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
@@ -124,7 +124,7 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
-  name: shoulda-matchers
+  name: rspec-rails
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
@@ -138,7 +138,21 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
-  name: webmock
+  name: rubocop
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: shoulda-matchers
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
@@ -166,7 +180,7 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
-  name: rubocop
+  name: webmock
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
@@ -193,6 +207,7 @@ files:
 - app/interactors/cangaroo/authenticate_connection.rb
 - app/interactors/cangaroo/count_json_object.rb
 - app/interactors/cangaroo/handle_request.rb
+- app/interactors/cangaroo/perform_flow.rb
 - app/interactors/cangaroo/perform_jobs.rb
 - app/interactors/cangaroo/validate_json_schema.rb
 - app/jobs/cangaroo/job.rb
@@ -248,12 +263,14 @@ files:
 - spec/dummy/public/500.html
 - spec/dummy/public/favicon.ico
 - spec/factories/cangaroo_connections.rb
+- spec/fixtures/json_payload_connection_response.json
 - spec/fixtures/json_payload_no_id.json
 - spec/fixtures/json_payload_ok.json
 - spec/fixtures/json_payload_wrong_key.json
 - spec/interactors/cangaroo/authenticate_connection_spec.rb
 - spec/interactors/cangaroo/count_json_object_spec.rb
 - spec/interactors/cangaroo/handle_request_spec.rb
+- spec/interactors/cangaroo/perform_flow_spec.rb
 - spec/interactors/cangaroo/perform_jobs_spec.rb
 - spec/interactors/cangaroo/validate_json_schema_spec.rb
 - spec/jobs/cangaroo/job_spec.rb
@@ -329,12 +346,14 @@ test_files:
 - spec/dummy/Rakefile
 - spec/dummy/README.rdoc
 - spec/factories/cangaroo_connections.rb
+- spec/fixtures/json_payload_connection_response.json
 - spec/fixtures/json_payload_no_id.json
 - spec/fixtures/json_payload_ok.json
 - spec/fixtures/json_payload_wrong_key.json
 - spec/interactors/cangaroo/authenticate_connection_spec.rb
 - spec/interactors/cangaroo/count_json_object_spec.rb
 - spec/interactors/cangaroo/handle_request_spec.rb
+- spec/interactors/cangaroo/perform_flow_spec.rb
 - spec/interactors/cangaroo/perform_jobs_spec.rb
 - spec/interactors/cangaroo/validate_json_schema_spec.rb
 - spec/jobs/cangaroo/job_spec.rb