maestrano-connector-rails 1.5.0 → 2.0.0.pre.RC1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: a944dc14eab6daaf5ba9596a36958f2e363aba3d
-  data.tar.gz: 800b7d60536b0c229da755641a9e9d43089894a3
+  metadata.gz: 20af245e9479b89f2432aaedc8bfdf6648314631
+  data.tar.gz: f5d2c1d4222c0a7d8cdd3736aecaf4acc44d96b7
 SHA512:
-  metadata.gz: 989bd5a085084e768190d3b8af8302de4f8a397ea6e129da2d8834d776bf12f33ec7b0362798bfa20803f7065ce28c8f09679dfac31bdf161cc3e62597bd962a
-  data.tar.gz: 54e7614a33ff5c3cf1552f77874cd9c17880f01a4e5fb820d9d6fa2acd1ef620a2a24c6df9ed96f5545f57acabbe647b6a38fad130ce970ca1e6f20036d34b8f
+  metadata.gz: c460c160a31c3372dcbfa6ea34184668a28b249ec527a62582651db6a198a57985969571b987d64ed43b7c05909a7bc6f1969ce070f8160424a1172f4fdaadfa
+  data.tar.gz: 09dc98a9f479db3080e2b9dc1dd10e6d717cbd99f121a11d983b5d3deda5983a0efcae420da051501631f5a87540232802576367a7d1cb679fd22f89bee13d5c

data/.rubocop.yml

@@ -20,9 +20,6 @@ Style/CollectionMethods:
 Metrics/LineLength:
   Max: 320
 
-Metrics/MethodLength:
-  Max: 50
-
 Style/IndentationConsistency:
   EnforcedStyle: rails
 

data/.rubocop_todo.yml

@@ -102,6 +102,14 @@ Style/EmptyLinesAroundModuleBody:
 Style/FrozenStringLiteralComment:
   Enabled: false
 
+# Offense count: 1
+# Cop supports --auto-correct.
+# Configuration parameters: EnforcedStyle, SupportedStyles.
+# SupportedStyles: predicate, comparison
+Style/NumericPredicate:
+  Exclude:
+    - 'app/models/maestrano/connector/rails/organization.rb'
+
 # Offense count: 3
 # Configuration parameters: NamePrefix, NamePrefixBlacklist, NameWhitelist.
 # NamePrefix: is_, has_, have_

data/Gemfile

@@ -1,8 +1,8 @@
 source 'http://rubygems.org'
 
 # Add dependencies required to use your gem here.
-gem 'rails', '~> 4.2'
-gem 'maestrano-rails'
+gem 'rails'
+gem 'maestrano-rails', '1.0.0.pre.RC2'
 
 gem 'hash_mapper', '>= 0.2.2'
 gem 'haml-rails'
@@ -15,7 +15,7 @@ gem 'figaro'
 gem 'sidekiq'
 # The missing unique jobs for sidekiq
 gem 'sidekiq-unique-jobs'
-gem 'sinatra', require: false
+gem 'sinatra', require: nil
 gem 'sidekiq-cron'
 gem 'slim'
 
@@ -28,7 +28,8 @@ group :development do
   gem 'simplecov', '>= 0'
   gem 'rspec-rails'
   gem 'factory_girl_rails'
-  gem 'sqlite3'
+  gem 'activerecord-jdbcsqlite3-adapter', :platforms => :jruby
+  gem 'sqlite3', :platforms => :ruby
   gem 'shoulda-matchers'
   gem 'rubocop'
   gem 'timecop'

data/VERSION

@@ -1 +1 @@
-1.5.0
+2.0.0.pre.RC1

data/app/controllers/maestrano/auth/saml_controller.rb

@@ -24,7 +24,7 @@ class Maestrano::Auth::SamlController < Maestrano::Rails::SamlBaseController
     if session[:settings]
       session.delete(:settings)
       redirect_to main_app.root_path
-    elsif current_organization&.oauth_uid && current_organization&.sync_enabled
+    elsif current_organization && current_organization.oauth_uid && current_organization.sync_enabled
       redirect_to main_app.home_redirect_to_external_path
     else
       redirect_to main_app.root_path

data/app/controllers/maestrano/connec_controller.rb

@@ -9,7 +9,7 @@ class Maestrano::ConnecController < Maestrano::Rails::WebHookController
 
         entities.each do |entity|
           organization = Maestrano::Connector::Rails::Organization.find_by_uid_and_tenant(entity[:group_id], params[:tenant])
-          next Rails.logger.warn "Received notification from Connec! for unknown group or group without oauth: #{entity['group_id']} (tenant: #{params[:tenant]})" unless organization&.oauth_uid
+          next Rails.logger.warn "Received notification from Connec! for unknown group or group without oauth: #{entity['group_id']} (tenant: #{params[:tenant]})" unless organization && organization.oauth_uid
           next unless organization.sync_enabled && organization.synchronized_entities[entity_class_hash[:name].to_sym]
 
           Maestrano::Connector::Rails::ConnectorLogger.log('info', organization, "Received entity from Connec! webhook: Entity=#{entity_name}, Data=#{entity}")

data/app/controllers/maestrano/synchronizations_controller.rb

@@ -3,11 +3,23 @@ class Maestrano::SynchronizationsController < Maestrano::Rails::WebHookControlle
     tenant = params[:tenant]
     uid = params[:id]
     organization = Maestrano::Connector::Rails::Organization.find_by_uid_and_tenant(uid, tenant)
-    return render json: {errors: [{message: 'Organization not found', code: 404}]}, status: :not_found unless organization
-
-    status = organization_status organization
+    return render json: {errors: [{message: "Organization not found", code: 404}]}, status: :not_found unless organization
+
+    h = {
+      group_id: organization.uid,
+      sync_enabled: organization.sync_enabled
+    }
+
+    last_sync = organization.synchronizations.last
+    if last_sync
+      h.merge!(
+        status: last_sync.status,
+        message: last_sync.message,
+        updated_at: last_sync.updated_at
+      )
+    end
 
-    render_organization_sync(organization, status, 200)
+    render json: h
   end
 
   def create
@@ -15,55 +27,21 @@ class Maestrano::SynchronizationsController < Maestrano::Rails::WebHookControlle
     uid = params[:group_id]
     opts = params[:opts] || {}
     organization = Maestrano::Connector::Rails::Organization.find_by_uid_and_tenant(uid, tenant)
-    return render json: {errors: [{message: 'Organization not found', code: 404}]}, status: :not_found unless organization
-
-    status = organization_status(organization)
+    return render json: {errors: [{message: "Organization not found", code: 404}]}, status: :not_found unless organization
 
-    unless status == 'RUNNING' || status == 'ENQUEUED'
-      Maestrano::Connector::Rails::SynchronizationJob.perform_later(organization.id, opts.with_indifferent_access)
-      status = 'ENQUEUED'
-    end
-
-    render_organization_sync(organization, status, 201)
+    Maestrano::Connector::Rails::SynchronizationJob.perform_later(organization, opts.with_indifferent_access)
+    head :created
   end
 
   def toggle_sync
     tenant = params[:tenant]
     uid = params[:group_id]
     organization = Maestrano::Connector::Rails::Organization.find_by_uid_and_tenant(uid, tenant)
-    return render json: {errors: [{message: 'Organization not found', code: 404}]}, status: :not_found unless organization
+    return render json: {errors: [{message: "Organization not found", code: 404}]}, status: :not_found unless organization
 
     organization.toggle(:sync_enabled)
     organization.save
-    status = organization_status organization
-    render_organization_sync(organization, status, 200)
-  end
 
-  private
-
-    def render_organization_sync(organization, status, code)
-      h = {
-        group_id: organization.uid,
-        sync_enabled: organization.sync_enabled,
-        status: status
-      }
-      last_sync = organization.synchronizations.last
-      if last_sync
-        h[:message] = last_sync.message
-        h[:updated_at] = last_sync.updated_at
-      end
-
-      render json: h, status: code
-    end
-
-    def organization_status(organization)
-      last_sync = organization.synchronizations.last
-      if Maestrano::Connector::Rails::SynchronizationJob.find_running_job(organization.id)
-        'RUNNING'
-      elsif Maestrano::Connector::Rails::SynchronizationJob.find_job(organization.id)
-        'ENQUEUED'
-      else
-        last_sync&.status || 'DISABLED'
-      end
-    end
+    render json: {sync_enabled: organization.sync_enabled}
+  end
 end

data/app/jobs/maestrano/connector/rails/all_synchronizations_job.rb

@@ -6,7 +6,7 @@ module Maestrano::Connector::Rails
     def perform(name = nil, count = nil)
       Maestrano::Connector::Rails::Organization.where.not(oauth_provider: nil, encrypted_oauth_token: nil).each do |o|
         next unless [true, 1].include?(o.sync_enabled)
-        Maestrano::Connector::Rails::SynchronizationJob.perform_later(o.id, {})
+        Maestrano::Connector::Rails::SynchronizationJob.perform_later(o, {})
       end
     end
   end

data/app/jobs/maestrano/connector/rails/synchronization_job.rb

@@ -7,9 +7,9 @@ module Maestrano::Connector::Rails
     #  * :only_entities => [person, tasks_list]
     #  * :full_sync => true  synchronization is performed without date filtering
     #  * :connec_preemption => true|false : preemption is always|never given to connec in case of conflict (if not set, the most recently updated entity is kept)
-    def perform(organization_id, opts = {})
-      organization = Maestrano::Connector::Rails::Organization.find(organization_id)
-      return unless organization&.sync_enabled
+    def perform(organization, opts = {})
+      return unless organization.sync_enabled
+
       # Check if previous synchronization is still running
       if Synchronization.where(organization_id: organization.id, status: 'RUNNING').where(created_at: (30.minutes.ago..Time.now)).exists?
         ConnectorLogger.log('info', organization, 'Synchronization skipped: Previous synchronization is still running')
@@ -105,25 +105,6 @@ module Maestrano::Connector::Rails
       end
     end
 
-    def self.enqueued?(organization_id)
-      SynchronizationJob.find_job(organization_id).present? || SynchronizationJob.find_running_job(organization_id).present?
-    end
-
-    def self.find_job(organization_id)
-      queue = Sidekiq::Queue.new(:default)
-      queue.find do |job|
-        organization_id == job.item['args'][0]['arguments'].first
-      end
-    end
-
-    def self.find_running_job(organization_id)
-      Sidekiq::Workers.new.find do |_, _, work|
-        work['queue'] == 'default' && work['payload']['args'][0]['arguments'].first == organization_id
-      end
-    rescue
-      nil
-    end
-
     private
 
       def instanciate_entity(entity_name, organization, connec_client, external_client, opts)
@@ -139,7 +120,7 @@ module Maestrano::Connector::Rails
         entity_instance.push_entities_to_external(mapped_entities[:connec_entities])
         entity_instance.push_entities_to_connec(mapped_entities[:external_entities])
         entity_instance.after_sync(last_synchronization_date)
-
+        
         entity_instance.class.count_and_first(external ? external_entities : connec_entities)
       end
   end

data/app/models/maestrano/connector/rails/concerns/entity.rb

@@ -312,7 +312,7 @@ module Maestrano::Connector::Rails::Concerns::Entity
 
     Maestrano::Connector::Rails::ConnectorLogger.log('info', @organization, "Sending Connec! #{self.class.connec_entity_name.pluralize} to #{Maestrano::Connector::Rails::External.external_name} #{external_entity_name.pluralize}")
 
-    entities_to_send_to_connec = mapped_connec_entities_with_idmaps.map { |mapped_connec_entity_with_idmap|
+    entities_to_send_to_connec = mapped_connec_entities_with_idmaps.map{ |mapped_connec_entity_with_idmap|
       push_entity_to_external(mapped_connec_entity_with_idmap, external_entity_name)
     }.compact
 
@@ -324,7 +324,7 @@ module Maestrano::Connector::Rails::Concerns::Entity
     # with either only the id, or the id + id references
     proc = lambda do |entity|
       id = {id: [Maestrano::Connector::Rails::ConnecHelper.id_hash(entity[:idmap].external_id, @organization)]}
-      body = entity[:completed_hash]&.merge(id) || id
+      body = entity[:completed_hash] ? entity[:completed_hash].merge(id) : id
       batch_op('put', body, entity[:idmap].connec_id, self.class.normalized_connec_entity_name)
     end
     batch_calls(entities_to_send_to_connec, proc, self.class.connec_entity_name, true)

data/app/models/maestrano/connector/rails/organization.rb

@@ -62,7 +62,7 @@ module Maestrano::Connector::Rails
     end
 
     def member?(user)
-      user_organization_rels.where(user_id: user.id).count.positive?
+      user_organization_rels.where(user_id: user.id).count > 0
     end
 
     def remove_member(user)
@@ -108,7 +108,7 @@ module Maestrano::Connector::Rails
     end
 
     def last_synchronization_date
-      last_successful_synchronization&.updated_at || date_filtering_limit
+      (last_successful_synchronization && last_successful_synchronization.updated_at) || date_filtering_limit
     end
   end
 end

data/lib/generators/connector/templates/home_controller.rb

@@ -23,7 +23,7 @@ class HomeController < ApplicationController
 
   def synchronize
     return redirect_to(:back) unless is_admin
-    Maestrano::Connector::Rails::SynchronizationJob.perform_later(current_organization.id, (params['opts'] || {}).merge(forced: true))
+    Maestrano::Connector::Rails::SynchronizationJob.perform_later(current_organization, (params['opts'] || {}).merge(forced: true))
     flash[:info] = 'Synchronization requested'
     redirect_to(:back)
   end
@@ -35,7 +35,7 @@ class HomeController < ApplicationController
   private
 
     def start_synchronization(old_sync_state, organization)
-      Maestrano::Connector::Rails::SynchronizationJob.perform_later(organization.id, {})
+      Maestrano::Connector::Rails::SynchronizationJob.perform_later(organization, {})
       flash[:info] = 'Congrats, you\'re all set up! Your data are now being synced'
     end
 end

data/lib/generators/connector/templates/home_controller_spec.rb

@@ -87,7 +87,7 @@ describe HomeController, type: :controller do
         subject { post :synchronize, opts: {'opts' => 'some_opts'} }
 
         it 'calls perform_later with opts' do
-          expect(Maestrano::Connector::Rails::SynchronizationJob).to receive(:perform_later).with(organization.id, 'opts' => 'some_opts', forced: true)
+          expect(Maestrano::Connector::Rails::SynchronizationJob).to receive(:perform_later).with(organization, 'opts' => 'some_opts', forced: true)
           subject
         end
       end
@@ -96,7 +96,7 @@ describe HomeController, type: :controller do
         subject { post :synchronize }
 
         it 'calls perform_later with empty opts hash' do
-          expect(Maestrano::Connector::Rails::SynchronizationJob).to receive(:perform_later).with(organization.id, forced: true)
+          expect(Maestrano::Connector::Rails::SynchronizationJob).to receive(:perform_later).with(organization, forced: true)
           subject
         end
       end

data/maestrano-connector-rails.gemspec

@@ -2,16 +2,16 @@
 # DO NOT EDIT THIS FILE DIRECTLY
 # Instead, edit Jeweler::Tasks in Rakefile, and run 'rake gemspec'
 # -*- encoding: utf-8 -*-
-# stub: maestrano-connector-rails 1.5.0 ruby lib
+# stub: maestrano-connector-rails 2.0.0.pre.RC1 ruby lib
 
 Gem::Specification.new do |s|
   s.name = "maestrano-connector-rails"
-  s.version = "1.5.0"
+  s.version = "2.0.0.pre.RC1"
 
-  s.required_rubygems_version = Gem::Requirement.new(">= 0") if s.respond_to? :required_rubygems_version=
+  s.required_rubygems_version = Gem::Requirement.new("> 1.3.1") if s.respond_to? :required_rubygems_version=
   s.require_paths = ["lib"]
   s.authors = ["Maestrano", "Pierre Berard", "Marco Bagnasco"]
-  s.date = "2016-09-30"
+  s.date = "2016-09-01"
   s.description = "Maestrano is the next generation marketplace for SME applications. See https://maestrano.com for details."
   s.email = "developers@maestrano.com"
   s.executables = ["rails"]
@@ -168,7 +168,6 @@ Gem::Specification.new do |s|
     "spec/dummy/public/422.html",
     "spec/dummy/public/500.html",
     "spec/dummy/public/favicon.ico",
-    "spec/dummy/tmp/cache/.keep",
     "spec/factories.rb",
     "spec/integration/complex_id_references_spec.rb",
     "spec/integration/complex_naming_spec.rb",
@@ -222,8 +221,8 @@ Gem::Specification.new do |s|
     s.specification_version = 4
 
     if Gem::Version.new(Gem::VERSION) >= Gem::Version.new('1.2.0') then
-      s.add_runtime_dependency(%q<rails>, ["~> 4.2"])
-      s.add_runtime_dependency(%q<maestrano-rails>, [">= 0"])
+      s.add_runtime_dependency(%q<rails>, [">= 0"])
+      s.add_runtime_dependency(%q<maestrano-rails>, ["= 1.0.0.pre.RC2"])
       s.add_runtime_dependency(%q<hash_mapper>, [">= 0.2.2"])
       s.add_runtime_dependency(%q<haml-rails>, [">= 0"])
       s.add_runtime_dependency(%q<bootstrap-sass>, [">= 0"])
@@ -243,13 +242,14 @@ Gem::Specification.new do |s|
       s.add_development_dependency(%q<simplecov>, [">= 0"])
       s.add_development_dependency(%q<rspec-rails>, [">= 0"])
       s.add_development_dependency(%q<factory_girl_rails>, [">= 0"])
+      s.add_development_dependency(%q<activerecord-jdbcsqlite3-adapter>, [">= 0"])
       s.add_development_dependency(%q<sqlite3>, [">= 0"])
       s.add_development_dependency(%q<shoulda-matchers>, [">= 0"])
       s.add_development_dependency(%q<rubocop>, [">= 0"])
       s.add_development_dependency(%q<timecop>, [">= 0"])
     else
-      s.add_dependency(%q<rails>, ["~> 4.2"])
-      s.add_dependency(%q<maestrano-rails>, [">= 0"])
+      s.add_dependency(%q<rails>, [">= 0"])
+      s.add_dependency(%q<maestrano-rails>, ["= 1.0.0.pre.RC2"])
       s.add_dependency(%q<hash_mapper>, [">= 0.2.2"])
       s.add_dependency(%q<haml-rails>, [">= 0"])
       s.add_dependency(%q<bootstrap-sass>, [">= 0"])
@@ -269,14 +269,15 @@ Gem::Specification.new do |s|
       s.add_dependency(%q<simplecov>, [">= 0"])
       s.add_dependency(%q<rspec-rails>, [">= 0"])
       s.add_dependency(%q<factory_girl_rails>, [">= 0"])
+      s.add_dependency(%q<activerecord-jdbcsqlite3-adapter>, [">= 0"])
       s.add_dependency(%q<sqlite3>, [">= 0"])
       s.add_dependency(%q<shoulda-matchers>, [">= 0"])
       s.add_dependency(%q<rubocop>, [">= 0"])
       s.add_dependency(%q<timecop>, [">= 0"])
     end
   else
-    s.add_dependency(%q<rails>, ["~> 4.2"])
-    s.add_dependency(%q<maestrano-rails>, [">= 0"])
+    s.add_dependency(%q<rails>, [">= 0"])
+    s.add_dependency(%q<maestrano-rails>, ["= 1.0.0.pre.RC2"])
     s.add_dependency(%q<hash_mapper>, [">= 0.2.2"])
     s.add_dependency(%q<haml-rails>, [">= 0"])
     s.add_dependency(%q<bootstrap-sass>, [">= 0"])
@@ -296,6 +297,7 @@ Gem::Specification.new do |s|
     s.add_dependency(%q<simplecov>, [">= 0"])
     s.add_dependency(%q<rspec-rails>, [">= 0"])
     s.add_dependency(%q<factory_girl_rails>, [">= 0"])
+    s.add_dependency(%q<activerecord-jdbcsqlite3-adapter>, [">= 0"])
     s.add_dependency(%q<sqlite3>, [">= 0"])
     s.add_dependency(%q<shoulda-matchers>, [">= 0"])
     s.add_dependency(%q<rubocop>, [">= 0"])

data/release_notes.md

@@ -1,25 +1,36 @@
+## 1.4.0
+
+### Breaking changes
+* `map_to`, `map_to_connec` and `map_to_external` method have changed and now take two arguments.
+Additionally you should be able to remove all overloading to those methods using the new argument in hash mapper. See [documentation](https://maestrano.atlassian.net/wiki/display/DEV/Mapping+and+synchronization#Mappingandsynchronization-FAQ) for more details
+
+### Features
+* Possibility to use an extended mapper for creation only
+* Possibility to pass additional arguments to hash mapper to use in the after/before hooks
+
 ## 1.3.5
 
-## Features
+
+### Features
 * Improve generated files
 * Adds a built-in way to handle error when updating a deleted record
 
-## Fixes
+### Fixes
 * Adds safety against potential infinite loop
 
 ## 1.3.4
 `maestrano.rb` file should be updated with new synchronization paths.
 
-## Features
+### Features
 * `connec_version_lt?` method
 
-## Fixes
+### Fixes
 * Fixes multi-tenancy of synchronization endpoints
 * Fixes display of singleton entity
 
 ## 1.3.3
 
-## Fixes
+### Fixes
 * Fixes `connec_version` method
 
 ## 1.3.2

data/spec/controllers/synchronizations_controller_spec.rb

@@ -61,7 +61,6 @@ describe Maestrano::SynchronizationsController, type: :controller do
               JSON.parse({
                 group_id: organization.uid,
                 sync_enabled: organization.sync_enabled,
-                status: 'DISABLED'
               }.to_json)
             )
           end
@@ -79,7 +78,7 @@ describe Maestrano::SynchronizationsController, type: :controller do
                 sync_enabled: organization.sync_enabled,
                 status: sync2.status,
                 message: sync2.message,
-                updated_at: sync2.updated_at
+                updated_at: sync2.updated_at   
               }.to_json)
             )
           end
@@ -136,7 +135,7 @@ describe Maestrano::SynchronizationsController, type: :controller do
         end
 
         it 'queues a sync' do
-          expect(Maestrano::Connector::Rails::SynchronizationJob).to receive(:perform_later).with(organization.id, opts)
+          expect(Maestrano::Connector::Rails::SynchronizationJob).to receive(:perform_later).with(organization, opts)
           subject
         end
       end

data/spec/jobs/all_synchronizations_job_spec.rb

@@ -14,9 +14,9 @@ describe Maestrano::Connector::Rails::AllSynchronizationsJob do
 
   describe 'perform' do
     it 'does not calls sync entity' do
-      expect(Maestrano::Connector::Rails::SynchronizationJob).to_not receive(:perform_later).with(organization_not_linked.id, anything)
-      expect(Maestrano::Connector::Rails::SynchronizationJob).to_not receive(:perform_later).with(organization_not_active.id, anything)
-      expect(Maestrano::Connector::Rails::SynchronizationJob).to receive(:perform_later).with(organization_to_process.id, anything)
+      expect(Maestrano::Connector::Rails::SynchronizationJob).to_not receive(:perform_later).with(organization_not_linked, anything)
+      expect(Maestrano::Connector::Rails::SynchronizationJob).to_not receive(:perform_later).with(organization_not_active, anything)
+      expect(Maestrano::Connector::Rails::SynchronizationJob).to receive(:perform_later).with(organization_to_process, anything)
 
       subject
     end

data/spec/jobs/synchronization_job_spec.rb

@@ -3,7 +3,7 @@ require 'spec_helper'
 describe Maestrano::Connector::Rails::SynchronizationJob do
   let(:organization) { create(:organization) }
   let(:opts) { {} }
-  subject { Maestrano::Connector::Rails::SynchronizationJob.perform_now(organization.id, opts) }
+  subject { Maestrano::Connector::Rails::SynchronizationJob.perform_now(organization, opts) }
 
   def does_not_perform
     expect_any_instance_of(Maestrano::Connector::Rails::SynchronizationJob).to_not receive(:sync_entity)
@@ -20,7 +20,7 @@ describe Maestrano::Connector::Rails::SynchronizationJob do
     end
 
     context 'with sync_enabled set to true' do
-      before { organization.update(sync_enabled: true)}
+      before { organization.update(sync_enabled: true)} 
 
       context 'with a sync still running for less than 30 minutes' do
         let!(:running_sync) { create(:synchronization, organization: organization, status: 'RUNNING', created_at: 29.minutes.ago) }
@@ -77,12 +77,11 @@ describe Maestrano::Connector::Rails::SynchronizationJob do
 
       context 'subsequent sync' do
         let!(:old_sync) { create(:synchronization, partial: false, status: 'SUCCESS', organization: organization) }
-
+        
         it { performes }
 
         it 'calls sync entity on all the organization synchronized entities set to true' do
           organization.synchronized_entities[organization.synchronized_entities.keys.first] = false
-          organization.save
           expect_any_instance_of(Maestrano::Connector::Rails::SynchronizationJob).to receive(:sync_entity).exactly(organization.synchronized_entities.count - 1).times
 
           subject
@@ -173,11 +172,11 @@ describe Maestrano::Connector::Rails::SynchronizationJob do
           context 'with more than 50 entities' do
             let(:external_entities1) { [*1..50] }
             let(:external_entities2) { [*51..60] }
-
+            
             it 'calls perform_sync several time' do
               expect_any_instance_of(Entities::Person).to receive(:opts_merge!).twice
               expect(subject).to receive(:perform_sync).twice.and_call_original
-
+              
               subject.first_sync_entity('person', organization, nil, nil, nil, {}, true)
             end
           end
@@ -188,7 +187,7 @@ describe Maestrano::Connector::Rails::SynchronizationJob do
             it 'calls perform_sync once' do
               expect_any_instance_of(Entities::Person).to receive(:opts_merge!).once.with({__skip: 0})
               expect(subject).to receive(:perform_sync).once.and_call_original
-
+              
               subject.first_sync_entity('person', organization, nil, nil, nil, {}, true)
             end
           end
@@ -201,7 +200,7 @@ describe Maestrano::Connector::Rails::SynchronizationJob do
             it 'calls perform_sync once' do
               expect_any_instance_of(Entities::Person).to receive(:opts_merge!).once.with({__skip: 0})
               expect(subject).to receive(:perform_sync).once.and_call_original
-
+              
               subject.first_sync_entity('person', organization, nil, nil, nil, {}, true)
             end
           end
@@ -221,4 +220,4 @@ describe Maestrano::Connector::Rails::SynchronizationJob do
       end
     end
   end
-end
+end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: maestrano-connector-rails
 version: !ruby/object:Gem::Version
-  version: 1.5.0
+  version: 2.0.0.pre.RC1
 platform: ruby
 authors:
 - Maestrano
@@ -10,36 +10,36 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2016-09-30 00:00:00.000000000 Z
+date: 2016-09-01 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: '4.2'
+        version: '0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: '4.2'
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: maestrano-rails
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - '='
       - !ruby/object:Gem::Version
-        version: '0'
+        version: 1.0.0.pre.RC2
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - '='
       - !ruby/object:Gem::Version
-        version: '0'
+        version: 1.0.0.pre.RC2
 - !ruby/object:Gem::Dependency
   name: hash_mapper
   requirement: !ruby/object:Gem::Requirement
@@ -306,6 +306,20 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
+- !ruby/object:Gem::Dependency
+  name: activerecord-jdbcsqlite3-adapter
+  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: sqlite3
   requirement: !ruby/object:Gem::Requirement
@@ -520,7 +534,6 @@ files:
 - spec/dummy/public/422.html
 - spec/dummy/public/500.html
 - spec/dummy/public/favicon.ico
-- spec/dummy/tmp/cache/.keep
 - spec/factories.rb
 - spec/integration/complex_id_references_spec.rb
 - spec/integration/complex_naming_spec.rb
@@ -579,9 +592,9 @@ required_ruby_version: !ruby/object:Gem::Requirement
       version: '0'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
-  - - ">="
+  - - ">"
     - !ruby/object:Gem::Version
-      version: '0'
+      version: 1.3.1
 requirements: []
 rubyforge_project: 
 rubygems_version: 2.5.1

data/spec/dummy/tmp/cache/.keep

@@ -1 +0,0 @@
-