synchronisable 0.0.9 → 1.0.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 38932ee4a35a464734cae4beacbef691ffab0de5
-  data.tar.gz: 28c7843ec8b46d09ec7208bb40f161d13b240203
+  metadata.gz: 9ce97779ff8534be6e3b14f5e872e8788638badd
+  data.tar.gz: 9d5796423c0e433d71103d972a1a070d2d690bb0
 SHA512:
-  metadata.gz: 67242c4c189ae6bd9ed7023e86b07a1384162858d778dde9bcd222d4848d7bd006b9047a35dc37ba0bd3a7e91827ee0e327cf6e858b8c0dc8a370f7a1aada696
-  data.tar.gz: ec7082b706d7ab227e6f1d31839a7cd158243b00ded14de8d90c45028f434b43f561c0b98aefd0b1743a01ea8bae97c5d22403acdc6109243691dfbb360acfb5
+  metadata.gz: 3f2d9314d4e4e4fa31c4061038051531279311b0e25bd33e15293dd29304c86f86eee1f9d44223f73029ffbc864df789f0759e2ab150f0dc9361350ae898528f
+  data.tar.gz: f43e651e7528bd8e6ce9b7a09dba7ed6cb156e4f5ba4e66d5b1a8c906d877b78af7063c6fe8af44715392c27eaf63ca8199892cb44f078ba02fd24201f85cbde

data/TODO.md

@@ -5,14 +5,15 @@ Primary objectives
 - [x] sync method
 - [x] dependent syncronization & mapping
 - [x] tests for Synchronisable.sync
-- [ ] fix a mess with Context
+- [ ] fix a mess with Context: return an array of sync contexts,
+      or aggregate all syncronization contexts into just one
 - [ ] destroy_missed
-- [ ] worker.rb refactoring
+- [x] worker.rb refactoring
 - [x] integrate with travis, stillmaintained, gemnasium,
-    codeclimate, coveralls, inch-pages, codersclan
+      codeclimate, coveralls, inch-pages, codersclan
 - [ ] write a good README
 - [ ] extended interface
--   [ ] sync with include
+  - [ ] sync with include
   - [x] sync with ids array
   - [ ] handle case when association type is a :hash
   - [ ] sync method for collection proxy (Model.where(condition).sync)

data/lib/synchronisable/controller.rb

@@ -0,0 +1,105 @@
+require 'pry-byebug'
+require 'colorize'
+
+require 'synchronisable/error_handler'
+require 'synchronisable/context'
+require 'synchronisable/input_parser'
+require 'synchronisable/source'
+require 'synchronisable/models/import'
+require 'synchronisable/helper/logging'
+require 'synchronisable/worker/record'
+require 'synchronisable/worker/associations'
+
+module Synchronisable
+  # Responsible for model synchronization.
+  #
+  # @api private
+  class Controller
+    include Helper::Logging
+
+    attr_reader :logger
+
+    class << self
+      # Creates a new instance of controller and initiates model synchronization.
+      #
+      # @overload call(model, data, options)
+      #   @param model [Class] model class to be synchronized
+      #   @param options [Hash] synchronization options
+      #   @option options [Hash] :include assocations to be synchronized.
+      #     Use this option to override `has_one` & `has_many` assocations
+      #     defined in model synchronizer.
+      # @overload call(model, data)
+      # @overload call(model)
+      #
+      # @return [Synchronisable::Context] synchronization context
+      def call(model, *args)
+        options = args.extract_options!
+        data = args.first
+
+        new(model, options).call(data)
+      end
+    end
+
+    # Initiates model synchronization.
+    #
+    # @param data [Array<Hash>, Array<String>, Array<Integer>, String, Integer]
+    #   synchronization data.
+    #   If not specified, it will try to get array of hashes to sync with
+    #   using defined gateway class or `fetch` lambda/proc
+    #   defined in corresponding synchronizer
+    #
+    # @return [Synchronisable::Context] synchronization context
+    #
+    # @see Synchronisable::InputParser
+    def call(data)
+      sync do |context|
+        error_handler = ErrorHandler.new(logger, context)
+        context.before = @model.imports_count
+
+        hashes = @input.parse(data)
+        hashes.each do |attrs|
+          source = Source.new(@model, @parent, attrs)
+
+          error_handler.handle(source) do
+            source.prepare
+
+            record_worker = Worker::Record.new(@synchronizer, source)
+            associations_worker = Worker::Associations.new(@synchronizer, source)
+
+            @synchronizer.with_sync_callbacks(source) do
+              associations_worker.sync_parent_associations
+              record_worker.sync_record
+              associations_worker.sync_child_associations
+            end
+          end
+        end
+
+        context.after = @model.imports_count
+        context.deleted = 0
+      end
+    end
+
+    private
+
+    def initialize(model, options)
+      @model, @synchronizer = model, model.synchronizer
+      @logger = @synchronizer.logger
+      @parent = options[:parent]
+
+      @input = InputParser.new(@model, @synchronizer)
+    end
+
+    def sync
+      @logger.progname = "#{@model} synchronization"
+      log_info('STARTING', :yellow, true)
+
+      context = Context.new(@model, @parent.try(:model))
+      yield context
+
+      log_info('DONE', :yellow, true)
+      log_info(context.summary_message, :cyan, true)
+
+      context
+    end
+  end
+end

data/lib/synchronisable/dsl/associations/association.rb

@@ -4,6 +4,7 @@ module Synchronisable
   module DSL
     module Associations
       # Association builder.
+      # Subclasses must implement #macro method.
       class Association
         include Synchronisable::DSL::Macro
 
@@ -17,9 +18,9 @@ module Synchronisable
           end
         end
 
-        self.valid_options = %i(key class_name required type)
+        self.valid_options = %i(key class_name required)
 
-        attr_reader :name, :model, :key, :required, :type
+        attr_reader :name, :model, :key, :required
 
         def initialize(synchronizer, name)
           @synchronizer, @name = synchronizer, name.to_sym
@@ -30,7 +31,6 @@ module Synchronisable
 
           @key = options[:key]
           @required = options[:required]
-          @type = options[:type]
 
           if options[:class_name].present?
             @model = options[:class_name].constantize
@@ -41,11 +41,14 @@ module Synchronisable
           @synchronizer.associations[@key] = self
         end
 
+        def macro
+          raise NotImplementedError
+        end
+
         protected
 
         def set_defaults
           @required ||= false
-          @type ||= :ids
 
           @model ||= @name.to_s.classify.constantize
           @key = "#{@name}_#{self.class.key_suffix}" unless @key.present?

data/lib/synchronisable/dsl/associations/belongs_to.rb

@@ -0,0 +1,16 @@
+require 'synchronisable/dsl/associations/association'
+
+module Synchronisable
+  module DSL
+    module Associations
+      # `belongs_to` association builder.
+      class BelongsTo < Association
+        key_suffix 'id'
+
+        def macro
+          :belongs_to
+        end
+      end
+    end
+  end
+end

data/lib/synchronisable/dsl/associations/has_many.rb

@@ -6,6 +6,10 @@ module Synchronisable
       # `has_many` association builder.
       class HasMany < Association
         key_suffix 'ids'
+
+        def macro
+          :has_many
+        end
       end
     end
   end

data/lib/synchronisable/dsl/associations/has_one.rb

@@ -6,6 +6,10 @@ module Synchronisable
       # `has_one` association builder.
       class HasOne < Association
         key_suffix 'id'
+
+        def macro
+          :has_one
+        end
       end
     end
   end

data/lib/synchronisable/dsl/associations.rb

@@ -1,5 +1,6 @@
 require 'synchronisable/dsl/associations/has_one'
 require 'synchronisable/dsl/associations/has_many'
+require 'synchronisable/dsl/associations/belongs_to'
 
 module Synchronisable
   module DSL
@@ -17,7 +18,7 @@ module Synchronisable
           subclass.associations = {}
         end
 
-        [HasOne, HasMany].each do |klass|
+        [HasOne, HasMany, BelongsTo].each do |klass|
           macro = klass.to_s.demodulize.underscore.to_sym
           define_method(macro) do |name, options = {}|
             klass.create(self, name, options)
@@ -34,11 +35,9 @@ module Synchronisable
         #   attributes hash doesn't required associations
         def associations_for(attrs)
           ensure_required_associations(attrs)
-          intersection = self.associations.map { |key, _| key } & attrs.keys
 
-          Hash[intersection.map { |key|
-            [self.associations[key], [*attrs[key]]]
-          }]
+          intersection = self.associations.map { |key, _| key } & attrs.keys
+          Hash[intersection.map { |key| [self.associations[key], [*attrs[key]]] }]
         end
 
         private

data/lib/synchronisable/error_handler.rb

@@ -1,3 +1,5 @@
+require 'pry-byebug'
+
 module Synchronisable
   # Helper class for synchronization errors handling.
   #
@@ -40,6 +42,7 @@ module Synchronisable
       I18n.t('errors.import_error',
         :model         => @context.model.to_s,
         :error         => e.message,
+        :backtrace     => e.backtrace.join("\n"),
         :remote_attrs  => source.remote_attrs,
         :local_attrs   => source.local_attrs,
         :import_record => source.import_record.inspect,

data/lib/synchronisable/helper/logging.rb

@@ -0,0 +1,23 @@
+module Synchronisable
+  module Helper
+    # Provides logging helper methods.
+    # Class or module that includes this one
+    # should implement `logger` method.
+    #
+    # @api private
+    module Logging
+      protected
+
+      %i(verbose colorize).each do |name|
+        define_method("#{name}_logging?".to_sym) do
+          Synchronisable.logging[name]
+        end
+      end
+
+      def log_info(msg, color = :white, force = true)
+        text = msg.colorize(color) if colorize_logging?
+        logger.info(text) if force || verbose_logging?
+      end
+    end
+  end
+end

data/lib/synchronisable/input_descriptor.rb

@@ -1,5 +1,10 @@
 module Synchronisable
+  # Provides a set of helper methods
+  # to describe user input.
+  #
   # @api private
+  #
+  # @see Synchronisable::InputParser
   class InputDescriptor
     attr_reader :data
 

data/lib/synchronisable/{input_dispatcher.rb → input_parser.rb}

@@ -1,20 +1,22 @@
 require 'synchronisable/input_descriptor'
 
 module Synchronisable
+  # Responsible for guessing the user input format.
+  #
   # @api private
-  class InputDispatcher
-    class << self
-      def dispatch(model, synchronizer, data)
-        new(model, synchronizer).dispatch(data)
-      end
-    end
-
+  class InputParser
     def initialize(model, synchronizer)
       @model = model
       @synchronizer = synchronizer
     end
 
-    def dispatch(data)
+    # Parses the user input.
+    #
+    # @param data [Array<Hash>, Array<String>, Array<Integer>, String, Integer]
+    #   synchronization data to handle.
+    #
+    # @return [Array<Hash>] array of hashes with remote attributes
+    def parse(data)
       input = InputDescriptor.new(data)
 
       result = case

data/lib/synchronisable/locale/en.yml

@@ -5,6 +5,7 @@ en:
     missed_associations: Couldn't find associations %{keys} in %{attrs}
     import_error: |
       %{model} import error: %{error}.
+      backtrace: %{backtrace},
       remote attrs: %{remote_attrs},
       local attrs: %{local_attrs},
       import record: %{import_record},

data/lib/synchronisable/locale/ru.yml

@@ -5,6 +5,7 @@ ru:
     missed_associations: В хэше %{attrs} не указаны ассоциации %{keys}
     import_error: >
       Ошибка при импорте %{model}: %{error},
+      backtrace: %{backtrace},
       remote attrs: %{remote_attrs},
       local attrs: %{local_attrs},
       import record: %{import_record},

data/lib/synchronisable/model/methods.rb

@@ -1,16 +1,17 @@
-require 'synchronisable/worker'
+require 'synchronisable/controller'
 require 'synchronisable/models/import'
 
 module Synchronisable
   module Model
     # Methods that will be attached to synchronisable model class.
     module Methods
-      # Creates a new worker, that initiates synchronization
-      # for this particular model.
+      # Creates a new controller, that initiates synchronization
+      # for this particular model and its associations.
       # If you have implemented `fetch` & `find` methods
       # in your model synchronizer, than it will be used if no data supplied.
       #
       # @overload sync(data, options)
+      #   @param data [Array<Hash>, Array<String>, Array<Integer>, String, Integer] synchronization data
       #   @param options [Hash] synchronization options
       #   @option options [Hash] :include assocations to be synchronized.
       #     Use this option to override `has_one` & `has_many` assocations
@@ -19,9 +20,8 @@ module Synchronisable
       # @overload sync(data)
       # @overload sync
       #
-      # @param data [Array<Hash>] array of hashes with remote attributes.
       #
-      # @see Synchronisable::Worker
+      # @see Synchronisable::Controller
       #
       # @example Supplying array of hashes with remote attributes
       #   FooModel.sync([
@@ -43,7 +43,7 @@ module Synchronisable
       #     :match_players => :player
       #   })
       def sync(*args)
-        Worker.run(self, *args)
+        Controller.call(self, *args)
       end
 
       # Count of import records for this model.

data/lib/synchronisable/source.rb

@@ -1,10 +1,15 @@
 module Synchronisable
+  # TODO: Massive refactoring needed
+
   # Synchronization source.
   class Source
+    CHILD_ASSOCIATION_KEYS = %i(has_one has_many)
+    PARENT_ASSOCIATION_KEYS = %i(belongs_to)
+
     attr_accessor :import_record
-    attr_reader :model, :remote_attrs,
-                :remote_id, :local_attrs,
-                :associations, :import_ids
+    attr_reader :parent_associations, :child_associations
+    attr_reader :model, :remote_attrs, :remote_id,
+                :local_attrs, :import_ids
 
     def initialize(model, parent, remote_attrs)
       @model, @parent, @synchronizer = model, parent, model.synchronizer
@@ -13,6 +18,7 @@ module Synchronisable
 
     # Prepares synchronization source:
     # `remote_id`, `local_attributes`, `import_record` and `associations`.
+    #
     # Sets foreign key if current model is specified as `has_one` or `has_many`
     # association of parent model.
     #
@@ -20,14 +26,14 @@ module Synchronisable
     def prepare
       @remote_id = @synchronizer.extract_remote_id(@remote_attrs)
       @local_attrs = @synchronizer.map_attributes(@remote_attrs)
-      @associations = @synchronizer.associations_for(@local_attrs)
 
-      # TODO: implement destroy_missed, somehow pass @import_ids,
-      # get all import records if nil
+      @associations = @synchronizer.associations_for(@local_attrs)
+      @parent_associations = @associations.select do |association|
+        PARENT_ASSOCIATION_KEYS.include? association.macro
+      end
 
-      # remove associations keys from local attributes
-      @local_attrs.delete_if do |key, _|
-        @associations.keys.any? { |a| a.key == key }
+      @child_associations = @associations.select do |association|
+        CHILD_ASSOCIATION_KEYS.include? association.macro
       end
 
       @import_record = Import.find_by(
@@ -35,25 +41,15 @@ module Synchronisable
         :synchronisable_type => @model
       )
 
-      set_foreign_key
-    end
-
-    def updatable?
-      @import_record.present? && local_record.present?
-    end
+      remove_association_keys_from_local_attrs
 
-    def update_record
-      local_record.update_attributes!(@local_attrs)
+      # TODO: This should be in Synchronisable::RecordWorker
+      set_belongs_to_parent_foreign_key
     end
 
-    def create_record_pair
-      record = @model.create!(@local_attrs)
-      @import_record = Import.create!(
-        :synchronisable_id    => record.id,
-        :synchronisable_type  => @model.to_s,
-        :remote_id            => @remote_id.to_s,
-        :attrs                => @local_attrs
-      )
+    def updatable?
+      import_record.present? &&
+      local_record.present?
     end
 
     def local_record
@@ -70,22 +66,30 @@ module Synchronisable
 
     private
 
-    def set_foreign_key
-      return unless @parent
-      name = foreign_key_name
-      @local_attrs[name] = @parent.local_record.id if name
+    def remove_association_keys_from_local_attrs
+      @local_attrs.delete_if do |key, _|
+        @associations.keys.any? { |a| a.key == key }
+      end
     end
 
-    def foreign_key_name
-      return nil unless parent_has_model_as_reflection?
-      parent_name = @parent.model.table_name.singularize
+    def set_belongs_to_parent_foreign_key
+      return unless @parent && parent_has_current_model_as_reflection?
+      @local_attrs[parent_foreign_key_name] = @parent.local_record.id
+
+    end
+
+    def parent_foreign_key_name
       "#{parent_name}_id"
     end
 
-    def parent_has_model_as_reflection?
+    def parent_name
+      @parent.model.table_name.singularize
+    end
+
+    def parent_has_current_model_as_reflection?
       @parent.model.reflections.values.any? do |reflection|
         reflection.plural_name == @model.table_name &&
-        [:has_one, :has_many].include?(reflection.macro)
+        %i(has_one has_many).include?(reflection.macro)
       end
     end
   end

data/lib/synchronisable/synchronizer.rb

@@ -169,7 +169,7 @@ module Synchronisable
       end
 
       def gateway_instance
-        @gateway ||= gateway.try(:new)
+        @gateway_instance ||= gateway.try(:new)
       end
     end
   end

data/lib/synchronisable/version.rb

@@ -1,8 +1,8 @@
 module Synchronisable
   module VERSION
-    MAJOR = 0
+    MAJOR = 1
     MINOR = 0
-    PATCH = 9
+    PATCH = 0
     SUFFIX = nil
 
     STRING = [MAJOR, MINOR, PATCH, SUFFIX].compact.join('.')

data/lib/synchronisable/worker/associations.rb

@@ -0,0 +1,56 @@
+require 'synchronisable/worker/base'
+
+module Synchronisable
+  module Worker
+    # Responsible for associations synchronization.
+    #
+    # @api private
+    class Associations < Base
+      # Synchronizes associations.
+      #
+      # @see Synchronisable::Source
+      # @see Synchronisable::DSL::Associations
+      # @see Synchronisable::DSL::Associations::Association
+      %w(child parent).each do |type|
+        define_method(:"sync_#{type}_associations") do
+          associations = @source.send(:"#{type}_associations")
+          log_info("starting #{type} associations sync", :blue) if associations.present?
+
+          associations.each do |association, ids|
+            ids.each do |id|
+              send(:"sync_#{type}_association", id, association)
+            end
+          end
+
+          log_info("done #{type} associations sync", :blue) if associations.present?
+        end
+      end
+
+      private
+
+      def sync_parent_association(id, association)
+        log_info("synchronizing parent association with id: #{id}", :blue)
+
+        @synchronizer.with_association_sync_callbacks(@source, id, association) do
+          attrs = association.model.synchronizer.find(id)
+          Controller.call(association.model, [attrs])
+
+          import_record = Import.find_by(
+            :remote_id => id,
+            :synchronisable_type => association.model.to_s
+          )
+          @source.local_attrs[association.key] = import_record.synchronisable.id
+        end
+      end
+
+      def sync_child_association(id, association)
+        log_info("synchronizing child association with id: #{id}", :blue)
+
+        @synchronizer.with_association_sync_callbacks(@source, id, association) do
+          attrs = association.model.synchronizer.find(id)
+          Controller.call(association.model, [attrs], { :parent => @source })
+        end
+      end
+    end
+  end
+end

data/lib/synchronisable/worker/base.rb

@@ -0,0 +1,23 @@
+require 'synchronisable/helper/logging'
+
+module Synchronisable
+  module Worker
+    # Base class for synchronization workers.
+    #
+    # @see Synchronisable::Worker::Record
+    # @see Synchronisable::Worker::Associations
+    #
+    # @api private
+    class Base
+      include Helper::Logging
+
+      def initialize(synchronizer, source)
+        @synchronizer, @source = synchronizer, source
+      end
+
+      def logger
+        @synchronizer.logger
+      end
+    end
+  end
+end

data/lib/synchronisable/worker/record.rb

@@ -0,0 +1,44 @@
+require 'synchronisable/worker/base'
+
+module Synchronisable
+  module Worker
+    # Responsible for record synchronization.
+    #
+    # @api private
+    class Record < Base
+      # Synchronizes record.
+      def sync_record
+        @synchronizer.with_record_sync_callbacks(@source) do
+          log_info(@source.dump_message, :green)
+
+          if @source.updatable?
+            log_info("updating #{@source.model}: #{@source.local_record.id}", :blue)
+
+            update_record
+          else
+            create_record_pair
+
+            log_info("#{@source.model} (id: #{@source.local_record.id}) was created", :blue)
+            log_info("#{@source.import_record.class}: #{@source.import_record.id} was created", :blue)
+          end
+        end
+      end
+
+      private
+
+      def update_record
+        @source.local_record.update_attributes!(@source.local_attrs)
+      end
+
+      def create_record_pair
+        record = @source.model.create!(@source.local_attrs)
+        @source.import_record = Import.create!(
+          :synchronisable_id    => record.id,
+          :synchronisable_type  => @source.model.to_s,
+          :remote_id            => @source.remote_id.to_s,
+          :attrs                => @source.local_attrs
+        )
+      end
+    end
+  end
+end

data/spec/dummy/app/gateways/team_group_statistic_gateway.rb

@@ -0,0 +1,22 @@
+class TeamGroupStatisticGateway < GatewayBase
+  def id_key
+    :id
+  end
+
+  def find(id)
+    raise NotImplementedError, 'Life is a bitch'
+  end
+
+  def source
+    @source ||= [
+      FactoryGirl.build(:remote_team_group_statistic,
+        :id       => 'team_0',
+        :team_id  => 'team_0'
+      ),
+      FactoryGirl.build(:remote_team_group_statistic,
+        :id       => 'team_1',
+        :team_id  => 'team_1'
+      ),
+    ]
+  end
+end

data/spec/dummy/app/models/stadium.rb

@@ -1,6 +1,6 @@
 class Stadium < ActiveRecord::Base
   # There is no synchronizer defined for this model,
-  # so synchronization worker is gonna try
+  # so synchronization controller is gonna try
   # to use remote attributes as is to create a record
 
   synchronisable

data/spec/dummy/app/models/team.rb

@@ -1,5 +1,6 @@
 class Team < ActiveRecord::Base
   has_many :players
+  has_one :team_group_statistic
 
   synchronisable BreakConventionTeamSynchronizer
 end

data/spec/dummy/app/models/team_group_statistic.rb

@@ -0,0 +1,6 @@
+class TeamGroupStatistic < ActiveRecord::Base
+  belongs_to :team
+  validates :team, presence: true
+
+  synchronisable
+end

data/spec/dummy/app/synchronizers/team_group_statistic_synchronizer.rb

@@ -0,0 +1,5 @@
+class TeamGroupStatisticSynchronizer < Synchronisable::Synchronizer
+  belongs_to :team
+
+  gateway TeamGroupStatisticGateway
+end

data/spec/dummy/db/migrate/20140609133855_create_team_group_statistic.rb

@@ -0,0 +1,13 @@
+class CreateTeamGroupStatistic < ActiveRecord::Migration
+  def change
+    create_table :team_group_statistics do |t|
+      t.references :team, index: true
+      t.integer :games_played
+      t.integer :games_won
+      t.integer :games_lost
+      t.integer :games_draw
+
+      t.timestamps
+    end
+  end
+end

data/spec/dummy/db/schema.rb

@@ -11,7 +11,7 @@
 #
 # It's strongly recommended that you check this file into your version control system.
 
-ActiveRecord::Schema.define(version: 20140507140039) do
+ActiveRecord::Schema.define(version: 20140609133855) do
 
   create_table "imports", force: true do |t|
     t.string   "synchronisable_type", null: false
@@ -82,6 +82,18 @@ ActiveRecord::Schema.define(version: 20140507140039) do
 
   add_index "stages", ["tournament_id"], name: "index_stages_on_tournament_id"
 
+  create_table "team_group_statistics", force: true do |t|
+    t.integer  "team_id"
+    t.integer  "games_played"
+    t.integer  "games_won"
+    t.integer  "games_lost"
+    t.integer  "games_draw"
+    t.datetime "created_at"
+    t.datetime "updated_at"
+  end
+
+  add_index "team_group_statistics", ["team_id"], name: "index_team_group_statistics_on_team_id"
+
   create_table "teams", force: true do |t|
     t.string   "name"
     t.string   "country"

data/spec/factories/remote/team_group_statistic.rb

@@ -0,0 +1,13 @@
+FactoryGirl.define do
+  factory :remote_team_group_statistic, class: Hash do
+    id { generate :team_id }
+    team_id { id }
+
+    games_played { generate :integer }
+    games_won { generate :integer }
+    games_lost { generate :integer }
+    games_draw { generate :integer }
+
+    initialize_with { attributes }
+  end
+end

data/spec/models/team_group_statistic_spec.rb

@@ -0,0 +1,15 @@
+require 'spec_helper'
+
+describe TeamGroupStatistic do
+  describe 'synchronization' do
+    context 'sync with no data specified' do
+      subject { -> { TeamGroupStatistic.sync } }
+
+      it { is_expected.to change { TeamGroupStatistic.count }.by(2) }
+      it { is_expected.to change { Team.count }.by(2) }
+      it { is_expected.to change { Player.count }.by(4) }
+
+      it { is_expected.to change { Synchronisable::Import.count }.by(8) }
+    end
+  end
+end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: synchronisable
 version: !ruby/object:Gem::Version
-  version: 0.0.9
+  version: 1.0.0
 platform: ruby
 authors:
 - Vasiliy Yorkin
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-06-05 00:00:00.000000000 Z
+date: 2014-06-09 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activerecord
@@ -302,8 +302,10 @@ files:
 - lib/synchronisable.rb
 - lib/synchronisable/attribute_mapper.rb
 - lib/synchronisable/context.rb
+- lib/synchronisable/controller.rb
 - lib/synchronisable/dsl/associations.rb
 - lib/synchronisable/dsl/associations/association.rb
+- lib/synchronisable/dsl/associations/belongs_to.rb
 - lib/synchronisable/dsl/associations/has_many.rb
 - lib/synchronisable/dsl/associations/has_one.rb
 - lib/synchronisable/dsl/macro.rb
@@ -313,8 +315,9 @@ files:
 - lib/synchronisable/error_handler.rb
 - lib/synchronisable/exceptions.rb
 - lib/synchronisable/gateway.rb
+- lib/synchronisable/helper/logging.rb
 - lib/synchronisable/input_descriptor.rb
-- lib/synchronisable/input_dispatcher.rb
+- lib/synchronisable/input_parser.rb
 - lib/synchronisable/locale/en.yml
 - lib/synchronisable/locale/ru.yml
 - lib/synchronisable/model.rb
@@ -324,7 +327,9 @@ files:
 - lib/synchronisable/synchronizer.rb
 - lib/synchronisable/synchronizers/synchronizer_default.rb
 - lib/synchronisable/version.rb
-- lib/synchronisable/worker.rb
+- lib/synchronisable/worker/associations.rb
+- lib/synchronisable/worker/base.rb
+- lib/synchronisable/worker/record.rb
 - spec/dummy/.gitignore
 - spec/dummy/Rakefile
 - spec/dummy/app/assets/images/.keep
@@ -336,6 +341,7 @@ files:
 - spec/dummy/app/gateways/player_gateway.rb
 - spec/dummy/app/gateways/stage_gateway.rb
 - spec/dummy/app/gateways/team_gateway.rb
+- spec/dummy/app/gateways/team_group_statistic_gateway.rb
 - spec/dummy/app/gateways/tournament_gateway.rb
 - spec/dummy/app/helpers/application_helper.rb
 - spec/dummy/app/mailers/.keep
@@ -346,11 +352,13 @@ files:
 - spec/dummy/app/models/stadium.rb
 - spec/dummy/app/models/stage.rb
 - spec/dummy/app/models/team.rb
+- spec/dummy/app/models/team_group_statistic.rb
 - spec/dummy/app/models/tournament.rb
 - spec/dummy/app/synchronizers/break_convention_team_synchronizer.rb
 - spec/dummy/app/synchronizers/match_synchronizer.rb
 - spec/dummy/app/synchronizers/player_synchronizer.rb
 - spec/dummy/app/synchronizers/stage_synchronizer.rb
+- spec/dummy/app/synchronizers/team_group_statistic_synchronizer.rb
 - spec/dummy/app/synchronizers/tournament_synchronizer.rb
 - spec/dummy/app/views/layouts/application.html.erb
 - spec/dummy/bin/bundle
@@ -383,6 +391,7 @@ files:
 - spec/dummy/db/migrate/20140507135800_create_tournaments.rb
 - spec/dummy/db/migrate/20140507135837_create_stages.rb
 - spec/dummy/db/migrate/20140507140039_add_stage_id_to_matches.rb
+- spec/dummy/db/migrate/20140609133855_create_team_group_statistic.rb
 - spec/dummy/db/schema.rb
 - spec/dummy/db/seeds.rb
 - spec/dummy/lib/assets/.keep
@@ -403,10 +412,12 @@ files:
 - spec/factories/remote/player.rb
 - spec/factories/remote/stage.rb
 - spec/factories/remote/team.rb
+- spec/factories/remote/team_group_statistic.rb
 - spec/factories/remote/tournament.rb
 - spec/factories/stadium.rb
 - spec/factories/team.rb
 - spec/models/match_spec.rb
+- spec/models/team_group_statistic_spec.rb
 - spec/models/team_spec.rb
 - spec/spec_helper.rb
 - spec/synchronisable/dsl/macro_spec.rb
@@ -455,6 +466,7 @@ test_files:
 - spec/dummy/app/gateways/player_gateway.rb
 - spec/dummy/app/gateways/stage_gateway.rb
 - spec/dummy/app/gateways/team_gateway.rb
+- spec/dummy/app/gateways/team_group_statistic_gateway.rb
 - spec/dummy/app/gateways/tournament_gateway.rb
 - spec/dummy/app/helpers/application_helper.rb
 - spec/dummy/app/mailers/.keep
@@ -465,11 +477,13 @@ test_files:
 - spec/dummy/app/models/stadium.rb
 - spec/dummy/app/models/stage.rb
 - spec/dummy/app/models/team.rb
+- spec/dummy/app/models/team_group_statistic.rb
 - spec/dummy/app/models/tournament.rb
 - spec/dummy/app/synchronizers/break_convention_team_synchronizer.rb
 - spec/dummy/app/synchronizers/match_synchronizer.rb
 - spec/dummy/app/synchronizers/player_synchronizer.rb
 - spec/dummy/app/synchronizers/stage_synchronizer.rb
+- spec/dummy/app/synchronizers/team_group_statistic_synchronizer.rb
 - spec/dummy/app/synchronizers/tournament_synchronizer.rb
 - spec/dummy/app/views/layouts/application.html.erb
 - spec/dummy/bin/bundle
@@ -502,6 +516,7 @@ test_files:
 - spec/dummy/db/migrate/20140507135800_create_tournaments.rb
 - spec/dummy/db/migrate/20140507135837_create_stages.rb
 - spec/dummy/db/migrate/20140507140039_add_stage_id_to_matches.rb
+- spec/dummy/db/migrate/20140609133855_create_team_group_statistic.rb
 - spec/dummy/db/schema.rb
 - spec/dummy/db/seeds.rb
 - spec/dummy/lib/assets/.keep
@@ -522,10 +537,12 @@ test_files:
 - spec/factories/remote/player.rb
 - spec/factories/remote/stage.rb
 - spec/factories/remote/team.rb
+- spec/factories/remote/team_group_statistic.rb
 - spec/factories/remote/tournament.rb
 - spec/factories/stadium.rb
 - spec/factories/team.rb
 - spec/models/match_spec.rb
+- spec/models/team_group_statistic_spec.rb
 - spec/models/team_spec.rb
 - spec/spec_helper.rb
 - spec/synchronisable/dsl/macro_spec.rb

data/lib/synchronisable/worker.rb

@@ -1,171 +0,0 @@
-require 'colorize'
-
-require 'synchronisable/error_handler'
-require 'synchronisable/context'
-require 'synchronisable/input_dispatcher'
-require 'synchronisable/source'
-require 'synchronisable/models/import'
-
-module Synchronisable
-  # Responsible for model synchronization.
-  #
-  # @api private
-  class Worker
-    class << self
-      # Creates a new instance of worker and initiates model synchronization.
-      #
-      # @overload run(model, data, options)
-      #   @param model [Class] model class to be synchronized
-      #   @param options [Hash] synchronization options
-      #   @option options [Hash] :include assocations to be synchronized.
-      #     Use this option to override `has_one` & `has_many` assocations
-      #     defined in model synchronizer.
-      # @overload run(model, data)
-      # @overload run(model)
-      #
-      # @return [Synchronisable::Context] synchronization context
-      def run(model, *args)
-        options = args.extract_options!
-        data = args.first
-
-        new(model, options).run(data)
-      end
-    end
-
-    # Initiates model synchronization.
-    #
-    # @param data [Array<Hash>] array of hashes with remote attriutes.
-    #   If not specified worker will try to get the data
-    #   using defined gateway class or `fetch` lambda/proc
-    #   defined in corresponding synchronizer
-    #
-    # @return [Synchronisable::Context] synchronization context
-    #
-    # @see Synchronisable::InputDispatcher
-    def run(data)
-      sync do |context|
-        error_handler = ErrorHandler.new(@logger, context)
-        context.before = @model.imports_count
-
-        hashes = InputDispatcher.dispatch(@model, @synchronizer, data)
-        hashes.each do |attrs|
-          source = Source.new(@model, @parent, attrs)
-          error_handler.handle(source) do
-            @synchronizer.with_sync_callbacks(source) do
-              sync_record(source)
-              sync_associations(source)
-              set_record_foreign_keys(source)
-            end
-          end
-        end
-
-        context.after = @model.imports_count
-        context.deleted = 0
-      end
-    end
-
-    private
-
-    def initialize(model, options)
-      @model, @synchronizer = model, model.synchronizer
-      @logger = @synchronizer.logger
-      @parent = options[:parent]
-    end
-
-    def sync
-      @logger.progname = "#{@model} synchronization"
-      log_info('STARTING', :yellow, true)
-
-      context = Context.new(@model, @parent.try(:model))
-      yield context
-
-      log_info('DONE', :yellow, true)
-      log_info(context.summary_message, :cyan, true)
-
-      context
-    end
-
-    # Method called by {#run} for each remote model attribute hash
-    #
-    # @param source [Synchronisable::Source] synchronization source
-    #
-    # @return [Boolean] `true` if synchronization was completed
-    #   without errors, `false` otherwise
-    def sync_record(source)
-      @synchronizer.with_record_sync_callbacks(source) do
-        source.prepare
-
-        log_info(source.dump_message, :green)
-
-        if source.updatable?
-          log_info("updating #{@model}: #{source.local_record.id}", :blue)
-          source.update_record
-        else
-          source.create_record_pair
-          log_info("#{@model} (id: #{source.local_record.id}) was created", :blue)
-          log_info("#{source.import_record.class}: #{source.import_record.id} was created", :blue)
-        end
-      end
-    end
-
-    def set_record_foreign_keys(source)
-      reflection = belongs_to_parent_reflection
-      return unless reflection
-
-      belongs_to_key = "#{reflection.plural_name.singularize}_id"
-      source.local_record.update_attributes!(
-        belongs_to_key => @parent.local_record.id
-      )
-    end
-
-    # Synchronizes associations.
-    #
-    # @param source [Synchronisable::Source] synchronization source
-    #
-    # @see Synchronisable::DSL::Associations
-    # @see Synchronisable::DSL::Associations::Association
-    def sync_associations(source)
-      log_info("starting associations sync", :blue) if source.associations.present?
-
-      source.associations.each do |association, ids|
-        ids.each { |id| sync_association(source, id, association) }
-      end
-    end
-
-    def sync_association(source, id, association)
-      log_info("synchronizing association with id: #{id}", :blue)
-
-      @synchronizer.with_association_sync_callbacks(source, id, association) do
-        attrs = association.model.synchronizer.find(id)
-        Worker.run(association.model, [attrs], { :parent => source })
-      end
-    end
-
-    # Finds a `belongs_to` reflection to the parent model.
-    #
-    # @see ActiveRecord::Reflection::AssociationReflection
-    def belongs_to_parent_reflection
-      return unless @parent
-
-      model_reflections.find do |r|
-        r.macro == :belongs_to &&
-        r.plural_name == @parent.model.table_name
-      end
-    end
-
-    def model_reflections
-      @model.reflections.values
-    end
-
-    def log_info(msg, color = :white, force = true)
-      text = msg.colorize(color) if colorize_logging?
-      @logger.info(text)         if force || verbose_logging?
-    end
-
-    %i(verbose colorize).each do |name|
-      define_method("#{name}_logging?".to_sym) do
-        Synchronisable.logging[name]
-      end
-    end
-  end
-end