HornsAndHooves-moribus 0.4.4 → 0.5.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
-SHA1:
-  metadata.gz: a9af55b2fc41dd5db61bc98f44e6a3f2b4de950d
-  data.tar.gz: f646aafaf86ec6389dbe8b39fe3664cb2909257e
+SHA256:
+  metadata.gz: 69f7a6978f05898888e7ef3e11471ea7606eb5211ffd7f098158bd67eec829a3
+  data.tar.gz: 6ac6db71051f8800f447e3dcd827abee37181754aa2f32e9ffddfd9e698392e4
 SHA512:
-  metadata.gz: 1ad85d605b3e35f1cf1e9c62ccb94be44d02b72af11f3b7abd5b6bfbea604ffdd1e209c8cd1a95da71e35a71dfbd02a67c27088aacb14f5a22f01e4295b4ebbd
-  data.tar.gz: a366ed36337e2516002bc426122715c49e8c4b15e58c2a8d56d479e5b1fd82de4941f23319f52bafe611f4ad07b7350684a91a1b9181dda2a140a187e19dd5d4
+  metadata.gz: eec5927604715080d2fd3ae72b71da0dfc4169de6949507904426f87e7a75768e4b953e07ce3efa46dfd0109fb66b743fa8e86abbb961d9ec2b971fbff480a16
+  data.tar.gz: 60f7975b1ad8f4818fade8a5a8f9c68d89d9244d586f6d89b6033a53fe47b7aa748378b777c9f5caa345dd90286a997f9f89d3f585901ae9f9a2c3a98427b611

data/.simplecov

@@ -12,7 +12,7 @@ SimpleCov.start do
   # Fail the build when coverage is weak:
   at_exit do
     SimpleCov.result.format!
-    threshold, actual = 96.969, SimpleCov.result.covered_percent
+    threshold, actual = 99.75, SimpleCov.result.covered_percent
     if actual < threshold
       msg = "\nLow coverage: "
       msg << red("#{actual}%")

data/README.md

@@ -6,7 +6,7 @@ Moribus is a set of tools for managing complex graphs of ActiveRecord objects
 for which there are many inbound foreign keys, attributes and associations with
 high rates of change, and business demands for well-tracked change history.
 
-##AggregatedBehavior
+## AggregatedBehavior
 
 AggregatedBehavior implements a pattern in which an object's identity
 is modeled apart from its attributes and outbound associations. This enables
@@ -26,7 +26,7 @@ to change. This is useful for objects with many inbound foreign keys and
 high-traffic attributes/associations, such as statuses. Without this pattern
 it would be difficult to avoid many StaleObjectErrors.
 
-##TrackedBehavior
+## TrackedBehavior
 
 TrackedBehavior implements history tracking on the stack of objects
 representing the identity object's attributes and outbound associations.
@@ -36,7 +36,7 @@ and will be saved as a new record with new attribute values and the
 'is_current' column as 'true'. Thus, under the hood, new attributes
 will supersede old attributes, leaving the old record as historical.
 
-##Macros, Associations and Combination
+## Macros, Associations and Combination
 
 Despite the fact that Moribus may be used by models on its own,
 its main purpose is to be used within associations, and in conjunction
@@ -105,6 +105,6 @@ rake spec
 
 ## Copyright
 
-Copyright (c) 2014 HornsAndHooves.
+Copyright (c) 2014-2018 HornsAndHooves.
 
 Copyright (c) 2013 TMX Credit.

data/lib/moribus/tracked_behavior.rb

@@ -75,8 +75,20 @@ module Moribus
           result[parent_key] = read_attribute(parent_key)
           result
         end
+        relation = klass.unscoped.where(criteria)
 
-        lock_value = klass.unscoped.where(criteria).count
+        sql = <<-SQL
+          #{relation.select("COUNT(#{lock_column_name}) AS value, 0 sort_order").to_sql}
+          UNION
+          #{relation.select("MAX(#{lock_column_name}) AS value, 1 sort_order").to_sql}
+          ORDER BY sort_order
+        SQL
+
+        result = klass.connection.execute(sql)
+        current_count = result[0]["value"]
+        current_max   = result[1]["value"]
+
+        lock_value = (current_count.to_s == "0" ? 0 : current_max.to_i + 1)
 
         write_attribute(lock_column_name, lock_value)
       end

data/lib/moribus/version.rb

@@ -1,3 +1,3 @@
 module Moribus # :nodoc:
-  VERSION = "0.4.4" # :nodoc:
+  VERSION = "0.5.1" # :nodoc:
 end

data/spec/moribus/aggregated_behavior_spec.rb

@@ -0,0 +1,131 @@
+require "spec_helper"
+
+describe Moribus::AggregatedBehavior do
+  before do
+    class SpecSuffix < MoribusSpecModel(name: :string, description: :string)
+      acts_as_enumerated
+
+      self.enumeration_model_updates_permitted = true
+      create!(name: "none", description: "")
+      create!(name: "jr"  , description: "Junior")
+    end
+
+    class SpecPersonName < MoribusSpecModel(first_name:     :string,
+                                            last_name:      :string,
+                                            spec_suffix_id: :integer
+                                          )
+      acts_as_aggregated
+      has_enumerated :spec_suffix, default: ""
+
+      validates_presence_of :first_name, :last_name
+
+      # custom writer that additionally strips first name
+      def first_name=(value)
+        self[:first_name] = value.strip
+      end
+    end
+
+    class SpecCustomerFeature < MoribusSpecModel(feature_name: :string)
+      acts_as_aggregated cache_by: :feature_name
+    end
+  end
+
+  after do
+    MoribusSpecModel.cleanup!
+  end
+
+  describe "Aggregated" do
+    context "definition" do
+      it "raises an error on an unknown option" do
+        expect{
+          Class.new(ActiveRecord::Base).class_eval do
+            acts_as_aggregated invalid_key: :error
+          end
+        }.to raise_error(ArgumentError)
+      end
+
+      it "raises an error when including AggregatedCacheBehavior without AggregatedBehavior" do
+        expect{
+          Class.new(ActiveRecord::Base).class_eval do
+            include Moribus::AggregatedCacheBehavior
+          end
+        }.to raise_error(Moribus::AggregatedCacheBehavior::NotAggregatedError)
+      end
+    end
+
+    before do
+      @existing = SpecPersonName.create! first_name: "John", last_name: "Smith"
+    end
+
+    it "doesn't duplicate records" do
+      expect {
+        SpecPersonName.create first_name: " John ", last_name: "Smith"
+      }.not_to change(SpecPersonName, :count)
+    end
+
+    it "looks up self and replaces id with existing on create" do
+      name = SpecPersonName.new first_name: "John", last_name: "Smith"
+      name.save
+      expect(name.id).to eq @existing.id
+    end
+
+    it "creates a new record if lookup fails" do
+      expect {
+        SpecPersonName.create first_name: "Alice", last_name: "Smith"
+      }.to change(SpecPersonName, :count).by(1)
+    end
+
+    it "looks up self and replaces id with existing on update" do
+      name = SpecPersonName.create first_name: "Alice", last_name: "Smith"
+      name.update_attributes first_name: "John"
+      expect(name.id).to eq @existing.id
+    end
+
+    it "calls super in save if any aggregated behaviour non content columns wasn't changed" do
+      name = SpecPersonName.create first_name: "Alice", last_name: "Smith"
+      expect {
+        name.save
+      }.to change(SpecPersonName, :count).by(0)
+    end
+
+    it "raises the expected error when 'save!' fails" do
+      name = SpecPersonName.create first_name: "Alice", last_name: "Smith"
+      name.last_name = nil
+      expect {
+        name.save!
+      }.to raise_error(ActiveRecord::RecordNotSaved)
+    end
+
+    context "with caching" do
+      before do
+        @existing = SpecCustomerFeature.create(feature_name: "Pays")
+        SpecCustomerFeature.clear_cache
+      end
+
+      it "looks up the existing value and adds it to the cache" do
+        feature = SpecCustomerFeature.new feature_name: @existing.feature_name
+
+        expect{ feature.save }.
+          to change(SpecCustomerFeature.aggregated_records_cache, :length).by(1)
+
+        expect(feature.id).to eq @existing.id
+      end
+
+      it "adds the freshly-created record to the cache" do
+        expect{ SpecCustomerFeature.create(feature_name: "Fraud") }.
+          to change(SpecCustomerFeature.aggregated_records_cache, :length).by(1)
+      end
+
+      it "freezes the cached object" do
+        feature = SpecCustomerFeature.create(feature_name: "Cancelled")
+        expect(SpecCustomerFeature.aggregated_records_cache[feature.feature_name]).to be_frozen
+      end
+
+      it "caches the clone of the record, not the record itself" do
+        feature = SpecCustomerFeature.create(feature_name: "Returned")
+        expect(SpecCustomerFeature.aggregated_records_cache[feature.feature_name].object_id).
+          not_to eq feature.object_id
+      end
+    end
+  end
+end

data/spec/moribus/alias_association_spec.rb

@@ -1,26 +1,26 @@
-require 'spec_helper'
+require "spec_helper"
 
 describe Moribus::AliasAssociation do
   before do
-    class SpecPost < MoribusSpecModel(:spec_author_id => :integer, :body => :string)
-      belongs_to :spec_author   , :alias => :creator
-      has_many   :spec_comments , :alias => :remarks
-      has_one    :spec_post_info, :alias => :information
+    class SpecPost < MoribusSpecModel(spec_author_id: :integer, body: :string)
+      belongs_to :spec_author   , alias: :creator
+      has_many   :spec_comments , alias: :remarks
+      has_one    :spec_post_info, alias: :information
 
       alias_association :author   , :spec_author
       alias_association :comments , :spec_comments
       alias_association :post_info, :spec_post_info
     end
 
-    class SpecAuthor < MoribusSpecModel(:name => :string)
+    class SpecAuthor < MoribusSpecModel(name: :string)
       has_many :spec_posts
     end
 
-    class SpecPostInfo < MoribusSpecModel(:spec_post_id => :integer, :ip => :string)
-      belongs_to :spec_post, :alias => :note
+    class SpecPostInfo < MoribusSpecModel(spec_post_id: :integer, ip: :string)
+      belongs_to :spec_post, alias: :note
     end
 
-    class SpecComment < MoribusSpecModel(:spec_post_id => :integer, :body => :string)
+    class SpecComment < MoribusSpecModel(spec_post_id: :integer, body: :string)
       belongs_to :spec_post
     end
   end
@@ -30,10 +30,10 @@ describe Moribus::AliasAssociation do
   end
 
   before do
-    author = SpecAuthor.create(:name => 'John')
-    @post = author.spec_posts.create(:body => 'Post Body')
-    @post.spec_comments.create(:body => 'Fabulous!')
-    @post.create_spec_post_info(:ip => '127.0.0.1')
+    author = SpecAuthor.create(name: "John")
+    @post  = author.spec_posts.create(body: "Post Body")
+    @post.spec_comments.create(body: "Fabulous!")
+    @post.create_spec_post_info(ip: "127.0.0.1")
   end
 
   describe "reflection aliasing" do
@@ -41,7 +41,7 @@ describe Moribus::AliasAssociation do
       expect(SpecPost.reflect_on_association(:author)).not_to be_nil
     end
 
-    it "should not raise error when using aliased name in scopes" do
+    it "does not raise error when using aliased name in scopes" do
       expect{
         SpecPost.includes(:comments).first
       }.not_to raise_error

data/spec/moribus/extensions/delegate_associated_spec.rb

@@ -0,0 +1,136 @@
+require "spec_helper"
+
+describe Moribus::Extensions::DelegateAssociated do
+  before do
+    class SpecStatus < MoribusSpecModel(name: :string, description: :string)
+      acts_as_enumerated
+
+      self.enumeration_model_updates_permitted = true
+      create!(name: "inactive", description: "Inactive")
+      create!(name: "active"  , description: "Active")
+    end
+
+    class SpecType < MoribusSpecModel(name: :string, description: :string)
+      acts_as_enumerated
+
+      self.enumeration_model_updates_permitted = true
+      create!(name: "important"  , description: "Important")
+      create!(name: "unimportant", description: "Unimportant")
+    end
+
+    class SpecSuffix < MoribusSpecModel(name: :string, description: :string)
+      acts_as_enumerated
+
+      self.enumeration_model_updates_permitted = true
+      create!(name: "none", description: "")
+      create!(name: "jr"  , description: "Junior")
+    end
+
+    class SpecPersonName < MoribusSpecModel(first_name:     :string,
+                                            middle_name:    :string,
+                                            last_name:      :string,
+                                            spec_suffix_id: :integer
+                                          )
+      acts_as_aggregated non_content_columns: :middle_name
+      has_enumerated :spec_suffix, default: ""
+
+      validates_presence_of :first_name, :last_name
+
+      # custom writer that additionally strips first name
+      def first_name=(value)
+        self[:first_name] = value.strip
+      end
+    end
+
+    class SpecCustomerInfo < MoribusSpecModel(
+                                        spec_customer_id:    :integer!,
+                                        spec_person_name_id: :integer,
+                                        spec_status_id:      :integer,
+                                        spec_type_id:        :integer,
+                                        is_current:          :boolean,
+                                        lock_version:        :integer,
+                                        created_at:          :datetime,
+                                        updated_at:          :datetime,
+                                        previous_id:         :integer
+                              )
+      attr :custom_field
+
+      belongs_to :spec_customer, inverse_of: :spec_customer_info, touch: true
+      has_aggregated :spec_person_name
+      has_enumerated :spec_status
+      has_enumerated :spec_type
+
+      acts_as_tracked by: :spec_customer, preceding_key: :previous_id
+    end
+
+    class SpecCustomer < MoribusSpecModel(spec_status_id: :integer)
+      has_one_current :spec_customer_info, inverse_of: :spec_customer
+      has_one_current :spec_customer_info_with_type, inverse_of: :spec_customer
+      has_enumerated :spec_status, default: "inactive"
+
+      delegate_associated :spec_person_name, :custom_field, :spec_type, to: :spec_customer_info
+    end
+  end
+
+  after do
+    MoribusSpecModel.cleanup!
+  end
+
+  let!(:customer) do
+    SpecCustomer.create(
+      spec_customer_info_attributes: {
+        spec_person_name_attributes: { first_name: " John ", last_name: "Smith" }
+      }
+    )
+  end
+
+  let(:info) { customer.spec_customer_info }
+
+  describe "Delegations" do
+    it "has delegated column information" do
+      expect(customer.column_for_attribute(:first_name)).not_to be_nil
+    end
+
+    it "does not delegate special methods" do
+      expect(customer).not_to respond_to(:reset_first_name)
+      expect(customer).not_to respond_to(:first_name_was)
+      expect(customer).not_to respond_to(:first_name_before_type_cast)
+      expect(customer).not_to respond_to(:first_name_will_change!)
+      expect(customer).not_to respond_to(:first_name_changed?)
+      expect(customer).not_to respond_to(:lock_version)
+    end
+
+    it "delegates methods to aggregated parts" do
+      expect(info).to respond_to(:first_name)
+      expect(info).to respond_to(:first_name=)
+      expect(info).to respond_to(:spec_suffix)
+      expect(info.last_name).to eq "Smith"
+    end
+
+    it "delegates methods to representation" do
+      expect(customer).to respond_to(:first_name)
+      expect(customer).to respond_to(:first_name=)
+      expect(customer).to respond_to(:spec_suffix)
+      expect(customer.last_name).to eq "Smith"
+      expect(customer).to respond_to(:custom_field)
+      expect(customer).to respond_to(:custom_field=)
+    end
+
+    it "properly delegates enumerated attributes" do
+      expect(customer).to respond_to(:spec_type)
+      expect(customer).to respond_to(:spec_type=)
+      customer.spec_type = :important
+      expect(customer.spec_type === :important).to eq true
+    end
+
+    it "raises NoMethodError if unknown method is received" do
+      expect{ customer.impossibru }.to raise_error(NoMethodError)
+    end
+  end
+
+  describe "#column_for_attribute" do
+    it "returns class db column if it exists" do
+      expect(customer.column_for_attribute(:id)).not_to be_nil
+    end
+  end
+end

data/spec/moribus/extensions/has_current_extension_spec.rb

@@ -0,0 +1,40 @@
+require "spec_helper"
+
+describe Moribus::Extensions::HasCurrentExtension do
+  before do
+    class SpecCustomerInfo < MoribusSpecModel(
+                                              spec_customer_id: :integer!,
+                                              is_current:       :boolean,
+                                              lock_version:     :integer,
+                                              created_at:       :datetime,
+                                              updated_at:       :datetime,
+                                              previous_id:      :integer
+                                             )
+
+      belongs_to :spec_customer, inverse_of: :spec_customer_info, touch: true
+    end
+
+    class SpecCustomer < MoribusSpecModel(spec_status_id: :integer)
+      has_one_current :spec_customer_info, inverse_of: :spec_customer, dependent: :destroy
+    end
+  end
+
+  after do
+    MoribusSpecModel.cleanup!
+  end
+
+  let(:customer) { SpecCustomer.create }
+  let!(:info)    { SpecCustomerInfo.create(spec_customer: customer, is_current: true) }
+
+  describe ".remove_target!" do
+    before do
+      allow(customer.spec_customer_info).to receive(:new_record?).and_return(true)
+    end
+
+    it "sets 'is_current' flag of overridden record to false for new record" do
+      old_info = customer.spec_customer_info
+      customer.spec_customer_info = SpecCustomerInfo.new
+      expect(old_info.is_current).to be_falsey
+    end
+  end
+end

data/spec/moribus/macros_spec.rb

@@ -1,7 +1,101 @@
-require 'spec_helper'
+require "spec_helper"
 
 # Some of the Moribus::Macros methods are tested in
 # spec/lib/moribus_spec.rb.
 describe Moribus::Macros do
+  before do
+    class SpecCustomer < MoribusSpecModel(spec_status_id: :integer)
+      has_one_current :spec_customer_info,
+        -> { where(spec_customer_id: 3) }, inverse_of: :spec_customer
+    end
 
+    class SpecCustomerInfo < MoribusSpecModel(
+                                                spec_customer_id: :integer!,
+                                                is_current:       :boolean,
+                                                lock_version:     :integer,
+                                                created_at:       :datetime,
+                                                updated_at:       :datetime,
+                                                previous_id:      :integer
+                                              )
+
+      belongs_to :spec_customer, inverse_of: :spec_customer_info, touch: true
+    end
+
+    class SpecCustomerOrder < MoribusSpecModel(:spec_status_id => :integer)
+      has_one_current :spec_customer_info_order,
+        -> { order(:created_at) }, :inverse_of => :spec_customer_order
+    end
+
+    class SpecCustomerInfoOrder < MoribusSpecModel(
+                                                spec_customer_order_id: :integer!,
+                                                is_current:             :boolean,
+                                                lock_version:           :integer,
+                                                created_at:             :datetime,
+                                                updated_at:             :datetime,
+                                                previous_id:            :integer
+                                              )
+
+      belongs_to :spec_customer_order, inverse_of: :spec_customer_info_order, touch: true
+    end
+  end
+
+  after do
+    MoribusSpecModel.cleanup!
+  end
+
+  describe ".has_one_current" do
+    let!(:customer)       { SpecCustomer.create(id: 3, spec_status_id: 2) }
+    let!(:customer_order) { SpecCustomerOrder.create(id: 9, spec_status_id: 1) }
+
+    let!(:info1) do
+      SpecCustomerInfo.create(
+        spec_customer_id: 1,
+        is_current:       true,
+        created_at:       5.days.ago,
+        updated_at:       5.days.ago
+      )
+    end
+
+    let!(:info2) do
+      SpecCustomerInfo.create(
+        spec_customer_id: 3,
+        is_current:       true,
+        created_at:       5.days.ago,
+        updated_at:       5.days.ago
+      )
+    end
+
+    let!(:info_order1) do
+      SpecCustomerInfoOrder.create(
+        spec_customer_order_id: 9,
+        is_current:             true,
+        created_at:             5.days.ago,
+        updated_at:             5.days.ago
+      )
+    end
+
+    let!(:info_order2) do
+      SpecCustomerInfoOrder.create(
+        spec_customer_order_id: 11,
+        is_current:             true,
+        created_at:             5.days.ago,
+        updated_at:             5.days.ago
+      )
+    end
+
+    it "merges has_one_current scope with has_one if given" do
+      expect(customer.spec_customer_info            ).to eq(info2)
+      expect(customer_order.spec_customer_info_order).to eq(info_order1)
+    end
+  end
+
+  describe ".normalize_reflection" do
+    subject do
+      SpecCustomer.send(:normalize_reflection, { spec_customer_info: :key_value }, :spec_customer_info)
+    end
+
+    it "returns reflection with symbol key for rails 4.1" do
+      is_expected.to eq(:key_value)
+    end
+  end
 end

data/spec/moribus_spec.rb

@@ -2,36 +2,37 @@ require "spec_helper"
 
 describe Moribus do
   before do
-    class SpecStatus < MoribusSpecModel(:name => :string, :description => :string)
+    class SpecStatus < MoribusSpecModel(name: :string, description: :string)
       acts_as_enumerated
 
       self.enumeration_model_updates_permitted = true
-      create!(:name => "inactive", :description => "Inactive")
-      create!(:name => "active"  , :description => "Active")
+      create!(name: "inactive", description: "Inactive")
+      create!(name: "active"  , description: "Active")
     end
 
-    class SpecType < MoribusSpecModel(:name => :string, :description => :string)
+    class SpecType < MoribusSpecModel(name: :string, description: :string)
       acts_as_enumerated
 
       self.enumeration_model_updates_permitted = true
-      create!(:name => "important"  , :description => "Important")
-      create!(:name => "unimportant", :description => "Unimportant")
+      create!(name: "important"  , description: "Important")
+      create!(name: "unimportant", description: "Unimportant")
     end
 
-    class SpecSuffix < MoribusSpecModel(:name => :string, :description => :string)
+    class SpecSuffix < MoribusSpecModel(name: :string, description: :string)
       acts_as_enumerated
 
       self.enumeration_model_updates_permitted = true
-      create!(:name => "none", :description => "")
-      create!(:name => "jr"  , :description => "Junior")
+      create!(name: "none", description: "")
+      create!(name: "jr"  , description: "Junior")
     end
 
-    class SpecPersonName < MoribusSpecModel(:first_name     => :string,
-                                            :last_name      => :string,
-                                            :spec_suffix_id => :integer
+    class SpecPersonName < MoribusSpecModel(first_name:     :string,
+                                            middle_name:    :string,
+                                            last_name:      :string,
+                                            spec_suffix_id: :integer
                                           )
-      acts_as_aggregated
-      has_enumerated :spec_suffix, :default => ""
+      acts_as_aggregated non_content_columns: :middle_name
+      has_enumerated :spec_suffix, default: ""
 
       validates_presence_of :first_name, :last_name
 
@@ -41,63 +42,63 @@ describe Moribus do
       end
     end
 
-    class SpecCustomerFeature < MoribusSpecModel(:feature_name => :string)
-      acts_as_aggregated :cache_by => :feature_name
+    class SpecCustomerFeature < MoribusSpecModel(feature_name: :string)
+      acts_as_aggregated cache_by: :feature_name
     end
 
     class SpecCustomerInfo < MoribusSpecModel(
-                                        :spec_customer_id    => :integer!,
-                                        :spec_person_name_id => :integer,
-                                        :spec_status_id      => :integer,
-                                        :spec_type_id        => :integer,
-                                        :is_current          => :boolean,
-                                        :lock_version        => :integer,
-                                        :created_at          => :datetime,
-                                        :updated_at          => :datetime,
-                                        :previous_id         => :integer
+                                        spec_customer_id:    :integer!,
+                                        spec_person_name_id: :integer,
+                                        spec_status_id:      :integer,
+                                        spec_type_id:        :integer,
+                                        is_current:          :boolean,
+                                        lock_version:        :integer,
+                                        created_at:          :datetime,
+                                        updated_at:          :datetime,
+                                        previous_id:         :integer
                               )
       attr :custom_field
 
-      belongs_to :spec_customer, :inverse_of => :spec_customer_info, :touch => true
+      belongs_to :spec_customer, inverse_of: :spec_customer_info, touch: true
       has_aggregated :spec_person_name
       has_enumerated :spec_status
       has_enumerated :spec_type
 
-      acts_as_tracked :by => :spec_customer, :preceding_key => :previous_id
+      acts_as_tracked by: :spec_customer, preceding_key: :previous_id
     end
 
     class SpecCustomerInfoWithType < MoribusSpecModel(
-                                        :spec_customer_id    => :integer!,
-                                        :spec_status_id      => :integer,
-                                        :spec_type_id        => :integer,
-                                        :is_current          => :boolean,
-                                        :lock_version        => :integer,
-                                        :created_at          => :datetime,
-                                        :updated_at          => :datetime
+                                        spec_customer_id: :integer!,
+                                        spec_status_id:   :integer,
+                                        spec_type_id:     :integer,
+                                        is_current:       :boolean,
+                                        lock_version:     :integer,
+                                        created_at:       :datetime,
+                                        updated_at:       :datetime
                                       )
       attr :custom_field
 
-      belongs_to :spec_customer, :inverse_of => :spec_customer_info_with_type, :touch => true
+      belongs_to :spec_customer, inverse_of: :spec_customer_info_with_type, touch: true
       has_enumerated :spec_status
       has_enumerated :spec_type
 
       acts_as_tracked :by => [:spec_customer, :spec_type]
     end
 
-    class SpecCustomer < MoribusSpecModel(:spec_status_id => :integer)
-      has_one_current :spec_customer_info, :inverse_of => :spec_customer
-      has_one_current :spec_customer_info_with_type, :inverse_of => :spec_customer
-      has_enumerated :spec_status, :default => "inactive"
+    class SpecCustomer < MoribusSpecModel(spec_status_id: :integer)
+      has_one_current :spec_customer_info, inverse_of: :spec_customer
+      has_one_current :spec_customer_info_with_type, inverse_of: :spec_customer
+      has_enumerated :spec_status, default: "inactive"
 
-      delegate_associated :spec_person_name, :custom_field, :spec_type, :to => :spec_customer_info
+      delegate_associated :spec_person_name, :custom_field, :spec_type, to: :spec_customer_info
     end
 
-    class SpecCustomerEmail < MoribusSpecModel(:spec_customer_id => :integer,
-                                               :email            => :string,
-                                               :is_current       => :boolean,
-                                               :status           => :string
+    class SpecCustomerEmail < MoribusSpecModel(spec_customer_id: :integer,
+                                               email:            :string,
+                                               is_current:       :boolean,
+                                               status:           :string
                                               )
-      connection.add_index table_name, [:email, :is_current], :unique => true
+      connection.add_index table_name, [:email, :is_current], unique: true
 
       belongs_to :spec_customer
 
@@ -112,22 +113,22 @@ describe Moribus do
   describe "common behavior" do
     before do
       @info = SpecCustomerInfo.create(
-        :spec_customer_id    => 1,
-        :spec_person_name_id => 1,
-        :is_current          => true,
-        :created_at          => 5.days.ago,
-        :updated_at          => 5.days.ago
+        spec_customer_id:    1,
+        spec_person_name_id: 1,
+        is_current:          true,
+        created_at:          5.days.ago,
+        updated_at:          5.days.ago
       )
     end
 
-    it "should revert changes if exception is raised" do
+    it "reverts changes if exception is raised" do
       old_id         = @info.id
       old_updated_at = @info.updated_at
       old_created_at = @info.created_at
 
       suppress(Exception) do
         expect {
-          @info.update_attributes :spec_customer_id => nil, :spec_person_name_id => 2
+          @info.update_attributes spec_customer_id: nil, spec_person_name_id: 2
         }.not_to change(SpecCustomerInfo, :count)
       end
 
@@ -138,130 +139,46 @@ describe Moribus do
 
       expect(@info.changed_attributes[:updated_at]).to eq(nil)
       expect(@info.changed_attributes[:created_at]).to eq(nil)
-
-    end
-  end
-
-  describe "Aggregated" do
-    context "definition" do
-      it "should raise an error on an unknown option" do
-        expect{
-          Class.new(ActiveRecord::Base).class_eval do
-            acts_as_aggregated :invalid_key => :error
-          end
-        }.to raise_error(ArgumentError)
-      end
-
-      it "should raise an error when including AggregatedCacheBehavior without AggregatedBehavior" do
-        expect{
-          Class.new(ActiveRecord::Base).class_eval do
-            include Moribus::AggregatedCacheBehavior
-          end
-        }.to raise_error(Moribus::AggregatedCacheBehavior::NotAggregatedError)
-      end
-    end
-
-    before do
-      @existing = SpecPersonName.create! :first_name => "John", :last_name => "Smith"
-    end
-
-    it "doesn't duplicate records" do
-      expect {
-        SpecPersonName.create :first_name => " John ", :last_name => "Smith"
-      }.not_to change(SpecPersonName, :count)
-    end
-
-    it "looks up self and replaces id with existing on create" do
-      name = SpecPersonName.new :first_name => "John", :last_name => "Smith"
-      name.save
-      expect(name.id).to eq @existing.id
     end
 
-    it "creates a new record if lookup fails" do
-      expect {
-        SpecPersonName.create :first_name => "Alice", :last_name => "Smith"
-      }.to change(SpecPersonName, :count).by(1)
-    end
-
-    it "looks up self and replaces id with existing on update" do
-      name = SpecPersonName.create :first_name => "Alice", :last_name => "Smith"
-      name.update_attributes :first_name => "John"
-      expect(name.id).to eq @existing.id
-    end
-
-    it "raises the expected error when 'save!' fails" do
-      name = SpecPersonName.create :first_name => "Alice", :last_name => "Smith"
-      name.last_name = nil
-      expect {
-        name.save!
-      }.to raise_error(ActiveRecord::RecordNotSaved)
-    end
-
-    context "with caching" do
-      before do
-        @existing = SpecCustomerFeature.create(:feature_name => "Pays")
-        SpecCustomerFeature.clear_cache
-      end
-
-      it "looks up the existing value and adds it to the cache" do
-        feature = SpecCustomerFeature.new :feature_name => @existing.feature_name
-
-        expect{ feature.save }.
-          to change(SpecCustomerFeature.aggregated_records_cache, :length).by(1)
-
-        expect(feature.id).to eq @existing.id
-      end
-
-      it "adds the freshly-created record to the cache" do
-        expect{ SpecCustomerFeature.create(:feature_name => "Fraud") }.
-          to change(SpecCustomerFeature.aggregated_records_cache, :length).by(1)
-      end
-
-      it "freezes the cached object" do
-        feature = SpecCustomerFeature.create(:feature_name => "Cancelled")
-        expect(SpecCustomerFeature.aggregated_records_cache[feature.feature_name]).to be_frozen
-      end
-
-      it "caches the clone of the record, not the record itself" do
-        feature = SpecCustomerFeature.create(:feature_name => "Returned")
-        expect(SpecCustomerFeature.aggregated_records_cache[feature.feature_name].object_id).
-          not_to eq feature.object_id
-      end
+    it "has module included class check methods" do
+      expect(SpecCustomerFeature.acts_as_aggregated?).to be_truthy
+      expect(SpecCustomerEmail.acts_as_tracked?).to be_truthy
     end
   end
 
   describe "Tracked" do
     before do
       @customer = SpecCustomer.create
-      @info = @customer.create_spec_customer_info :spec_person_name_id => 1
+      @info = @customer.create_spec_customer_info spec_person_name_id: 1
     end
 
     it "creates a new current record if updated" do
       expect {
-        @info.update_attributes(:spec_person_name_id => 2)
+        @info.update_attributes(spec_person_name_id: 2)
       }.to change(SpecCustomerInfo, :count).by(1)
     end
 
     it "replaces itself with new id" do
       old_id = @info.id
-      @info.update_attributes(:spec_person_name_id => 2)
+      @info.update_attributes(spec_person_name_id: 2)
       expect(@info.id).not_to eq old_id
     end
 
     it "sets is_current record to false for superseded record" do
       old_id = @info.id
-      @info.update_attributes(:spec_person_name_id => 2)
+      @info.update_attributes(spec_person_name_id: 2)
       expect(SpecCustomerInfo.find(old_id).is_current).to eq false
     end
 
     it "sets previous_id to the id of the previous record" do
       old_id = @info.id
-      @info.update_attributes(:spec_person_name_id => 2)
+      @info.update_attributes(spec_person_name_id: 2)
       expect(@info.previous_id).to eq old_id
     end
 
     it "changes is_current to false for previous one when assigning a new current record" do
-      new_info = SpecCustomerInfo.new :spec_person_name_id => 2, :is_current => true
+      new_info = SpecCustomerInfo.new spec_person_name_id: 2, is_current: true
       @customer.spec_customer_info = new_info
       expect(new_info.spec_customer_id).to eq @customer.id
       @info.reload
@@ -269,12 +186,12 @@ describe Moribus do
     end
 
     it "does not crash on superseding with 'is_current' conditional constraint" do
-      email = SpecCustomerEmail.create( :spec_customer => @customer,
-                                        :email         => "foo@bar.com",
-                                        :status        => "unverified",
-                                        :is_current    => true
+      email = SpecCustomerEmail.create( spec_customer: @customer,
+                                        email:         "foo@bar.com",
+                                        status:        "unverified",
+                                        is_current:    true
                                       )
-      expect{ email.update_attributes(:status => "verified") }.not_to raise_error
+      expect{ email.update_attributes(status: "verified") }.not_to raise_error
     end
 
     describe "updated_at and created_at" do
@@ -285,7 +202,7 @@ describe Moribus do
       after  { Timecop.return             }
 
       it "is updated on change" do
-        info = @customer.create_spec_customer_info :spec_person_name_id => 1
+        info = @customer.create_spec_customer_info spec_person_name_id: 1
         expect(info.updated_at).to eq first_time
         expect(info.created_at).to eq first_time
 
@@ -304,9 +221,9 @@ describe Moribus do
       end
 
       it "raises a stale object error" do
-        @info1.update_attributes(:spec_person_name_id => 3)
+        @info1.update_attributes(spec_person_name_id: 3)
 
-        expect{ @info2.update_attributes(:spec_person_name_id => 4) }.
+        expect{ @info2.update_attributes(spec_person_name_id: 4) }.
           to raise_error(ActiveRecord::StaleObjectError,
                          "Attempted to update_current (version #{@info2.lock_version}) a stale object: SpecCustomerInfo")
       end
@@ -315,17 +232,30 @@ describe Moribus do
         spec_customer_info = @customer.spec_customer_info
 
         expect {
-          spec_customer_info.update_attributes(:spec_person_name_id => 3)
+          spec_customer_info.update_attributes(spec_person_name_id: 3)
         }.to change { spec_customer_info.lock_version }.from(0).to(1)
 
         expect {
-          spec_customer_info.update_attributes(:spec_person_name_id => 4)
+          spec_customer_info.update_attributes(spec_person_name_id: 4)
         }.to change { spec_customer_info.lock_version }.from(1).to(2)
       end
 
+      it "does not fail to update if a lock version growth is for any reason not monotonic" do
+        spec_customer_info = @customer.spec_customer_info
+
+        spec_customer_info.update_attributes(spec_person_name_id: 3)
+        spec_customer_info.update_attributes(spec_person_name_id: 4)
+
+        SpecCustomerInfo.where(spec_customer_id: @customer.id, lock_version: 1).delete_all
+
+        expect {
+          spec_customer_info.update_attributes(spec_person_name_id: 5)
+        }.to change { spec_customer_info.lock_version }.from(2).to(3)
+      end
+
       it "does not fail if no locking_column is present" do
-        email = SpecCustomerEmail.create(:spec_customer_id => 1, :email => "foo@bar.com")
-        expect{ email.update_attributes(:email => "foo2@bar.com") }.not_to raise_error
+        email = SpecCustomerEmail.create(spec_customer_id: 1, email: "foo@bar.com")
+        expect{ email.update_attributes(email: "foo2@bar.com") }.not_to raise_error
       end
 
       it "updates lock_version column based on parent relation" do
@@ -333,29 +263,29 @@ describe Moribus do
         spec_customer_info = @customer.spec_customer_info
 
         expect {
-          spec_customer_info.update_attributes(:spec_person_name_id => 3)
+          spec_customer_info.update_attributes(spec_person_name_id: 3)
         }.to change { spec_customer_info.lock_version }.from(0).to(1)
 
         expect {
-          spec_customer_info.update_attributes(:spec_customer => @other_customer)
+          spec_customer_info.update_attributes(spec_customer: @other_customer)
         }.to change { spec_customer_info.lock_version }.from(1).to(0)
       end
 
       it "updates lock_version column base on relation list from 'by' option" do
         info_with_type =
-          @customer.reload.create_spec_customer_info_with_type(:spec_type => :important)
+          @customer.reload.create_spec_customer_info_with_type(spec_type: :important)
 
         expect( info_with_type.lock_version ).to eq 0
 
         other_info_with_type =
-          @customer.reload.create_spec_customer_info_with_type(:spec_type => :unimportant)
+          @customer.reload.create_spec_customer_info_with_type(spec_type: :unimportant)
 
         expect( other_info_with_type.lock_version ).to eq 0
 
-        info_with_type.update_attributes(:spec_status => :active)
+        info_with_type.update_attributes(spec_status: :active)
         expect( info_with_type.lock_version ).to eq 1
 
-        info_with_type.update_attributes(:spec_status => :inactive)
+        info_with_type.update_attributes(spec_status: :inactive)
         expect( info_with_type.lock_version ).to eq 2
 
         expect( other_info_with_type.lock_version ).to eq 0
@@ -365,7 +295,7 @@ describe Moribus do
     describe "with Aggregated" do
       before do
         @info.spec_person_name =
-          SpecPersonName.create(:first_name => "John", :last_name => "Smith")
+          SpecPersonName.create(first_name: "John", last_name: "Smith")
         @info.save
         @info.reload
       end
@@ -374,59 +304,12 @@ describe Moribus do
         old_id = @info.id
         @customer.spec_customer_info.spec_person_name.first_name = "Alice"
         expect{ @customer.save }.to change(@info, :spec_person_name_id)
-        expect(@info.id).not_to eq old_id
-        expect(@info.is_current).to eq true
-        expect(SpecCustomerInfo.find(old_id).is_current).to eq false
-      end
-    end
-  end
-
-  describe "Delegations" do
-    before do
-      @customer = SpecCustomer.create(
-        :spec_customer_info_attributes => {
-          :spec_person_name_attributes => {:first_name => " John ", :last_name => "Smith"} } )
-      @info = @customer.spec_customer_info
-    end
 
-    it "has delegated column information" do
-      expect(@customer.column_for_attribute(:first_name)).not_to be_nil
-    end
-
-    it "does not delegate special methods" do
-      expect(@customer).not_to respond_to(:reset_first_name)
-      expect(@customer).not_to respond_to(:first_name_was)
-      expect(@customer).not_to respond_to(:first_name_before_type_cast)
-      expect(@customer).not_to respond_to(:first_name_will_change!)
-      expect(@customer).not_to respond_to(:first_name_changed?)
-      expect(@customer).not_to respond_to(:lock_version)
-    end
+        expect(@info.id        ).not_to eq old_id
+        expect(@info.is_current).to     eq true
 
-    it "delegates methods to aggregated parts" do
-      expect(@info).to respond_to(:first_name)
-      expect(@info).to respond_to(:first_name=)
-      expect(@info).to respond_to(:spec_suffix)
-      expect(@info.last_name).to eq "Smith"
-    end
-
-    it "delegates methods to representation" do
-      expect(@customer).to respond_to(:first_name)
-      expect(@customer).to respond_to(:first_name=)
-      expect(@customer).to respond_to(:spec_suffix)
-      expect(@customer.last_name).to eq "Smith"
-      expect(@customer).to respond_to(:custom_field)
-      expect(@customer).to respond_to(:custom_field=)
-    end
-
-    it "properly delegates enumerated attributes" do
-      expect(@customer).to respond_to(:spec_type)
-      expect(@customer).to respond_to(:spec_type=)
-      @customer.spec_type = :important
-      expect(@customer.spec_type === :important).to eq true
-    end
-
-    it "raises NoMethodError if unknown method is received" do
-      expect{ @customer.impossibru }.to raise_error(NoMethodError)
+        expect(SpecCustomerInfo.find(old_id).is_current).to eq false
+      end
     end
   end
 end

data/spec/support/moribus_spec_model.rb

@@ -2,10 +2,12 @@
 module MoribusSpecModel
   def self.cleanup!
     ObjectSpace.each_object(MoribusSpecModel) do |klass|
-      if Object.const_defined? klass.name.to_sym
-        conn, table_name = klass.connection, klass.table_name
-        conn.drop_table(table_name) if conn.tables.include?(table_name)
-        Object.send :remove_const, klass.name.to_sym
+      unless klass.name.nil?
+        if Object.const_defined? klass.name.to_sym
+          conn, table_name = klass.connection, klass.table_name
+          conn.drop_table(table_name) if conn.tables.include?(table_name)
+          Object.send :remove_const, klass.name.to_sym
+        end
       end
     end
     ActiveSupport::Dependencies::Reference.clear!

data/tags

@@ -27,12 +27,16 @@ HasAggregatedExtension	lib/moribus/extensions/has_aggregated_extension.rb	5;"	m
 HasCurrentExtension	lib/moribus/extensions/has_current_extension.rb	5;"	m	line:5	class:Moribus.Extensions
 Helper	lib/moribus/extensions/has_aggregated_extension.rb	18;"	c	line:18	class:Moribus.Extensions.HasAggregatedExtension
 K.D	coverage/assets/0.10.0/application.js	17;"	f	line:17
+K.D	coverage/assets/0.10.2/application.js	17;"	f	line:17
 K.D	coverage/assets/0.8.0/application.js	17;"	f	line:17
 K.E	coverage/assets/0.10.0/application.js	17;"	f	line:17
+K.E	coverage/assets/0.10.2/application.js	17;"	f	line:17
 K.E	coverage/assets/0.8.0/application.js	17;"	f	line:17
 K.l	coverage/assets/0.10.0/application.js	17;"	f	line:17
+K.l	coverage/assets/0.10.2/application.js	17;"	f	line:17
 K.l	coverage/assets/0.8.0/application.js	17;"	f	line:17
 K.m	coverage/assets/0.10.0/application.js	17;"	f	line:17
+K.m	coverage/assets/0.10.2/application.js	17;"	f	line:17
 K.m	coverage/assets/0.8.0/application.js	17;"	f	line:17
 Macros	lib/moribus/macros.rb	4;"	m	line:4	class:Moribus
 Moribus	lib/moribus.rb	6;"	m	line:6
@@ -46,23 +50,39 @@ Moribus	lib/moribus/extensions/has_current_extension.rb	1;"	m	line:1
 Moribus	lib/moribus/macros.rb	1;"	m	line:1
 Moribus	lib/moribus/tracked_behavior.rb	1;"	m	line:1
 Moribus	lib/moribus/version.rb	1;"	m	line:1
-MoribusSpecModel	spec/support/moribus_spec_model.rb	26;"	f	line:26
+MoribusSpecModel	spec/support/moribus_spec_model.rb	28;"	f	line:28
 MoribusSpecModel	spec/support/moribus_spec_model.rb	2;"	m	line:2
 SpecAuthor	spec/moribus/alias_association_spec.rb	15;"	c	line:15
 SpecComment	spec/moribus/alias_association_spec.rb	23;"	c	line:23
-SpecCustomer	spec/moribus_spec.rb	87;"	c	line:87
-SpecCustomerEmail	spec/moribus_spec.rb	95;"	c	line:95
-SpecCustomerFeature	spec/moribus_spec.rb	44;"	c	line:44
-SpecCustomerInfo	spec/moribus_spec.rb	48;"	c	line:48
-SpecCustomerInfoWithType	spec/moribus_spec.rb	69;"	c	line:69
+SpecCustomer	spec/moribus/extensions/delegate_associated_spec.rb	66;"	c	line:66
+SpecCustomer	spec/moribus/extensions/has_current_extension_spec.rb	17;"	c	line:17
+SpecCustomer	spec/moribus/macros_spec.rb	7;"	c	line:7
+SpecCustomer	spec/moribus_spec.rb	88;"	c	line:88
+SpecCustomerEmail	spec/moribus_spec.rb	96;"	c	line:96
+SpecCustomerFeature	spec/moribus/aggregated_behavior_spec.rb	28;"	c	line:28
+SpecCustomerFeature	spec/moribus_spec.rb	45;"	c	line:45
+SpecCustomerInfo	spec/moribus/extensions/delegate_associated_spec.rb	45;"	c	line:45
+SpecCustomerInfo	spec/moribus/extensions/has_current_extension_spec.rb	5;"	c	line:5
+SpecCustomerInfo	spec/moribus/macros_spec.rb	12;"	c	line:12
+SpecCustomerInfo	spec/moribus_spec.rb	49;"	c	line:49
+SpecCustomerInfoOrder	spec/moribus/macros_spec.rb	29;"	c	line:29
+SpecCustomerInfoWithType	spec/moribus_spec.rb	70;"	c	line:70
+SpecCustomerOrder	spec/moribus/macros_spec.rb	24;"	c	line:24
+SpecPersonName	spec/moribus/aggregated_behavior_spec.rb	13;"	c	line:13
+SpecPersonName	spec/moribus/extensions/delegate_associated_spec.rb	29;"	c	line:29
 SpecPersonName	spec/moribus_spec.rb	29;"	c	line:29
 SpecPost	spec/moribus/alias_association_spec.rb	5;"	c	line:5
 SpecPostInfo	spec/moribus/alias_association_spec.rb	19;"	c	line:19
+SpecStatus	spec/moribus/extensions/delegate_associated_spec.rb	5;"	c	line:5
 SpecStatus	spec/moribus_spec.rb	5;"	c	line:5
+SpecSuffix	spec/moribus/aggregated_behavior_spec.rb	5;"	c	line:5
+SpecSuffix	spec/moribus/extensions/delegate_associated_spec.rb	21;"	c	line:21
 SpecSuffix	spec/moribus_spec.rb	21;"	c	line:21
+SpecType	spec/moribus/extensions/delegate_associated_spec.rb	13;"	c	line:13
 SpecType	spec/moribus_spec.rb	13;"	c	line:13
 TrackedBehavior	lib/moribus/tracked_behavior.rb	8;"	m	line:8	class:Moribus
 V.N	coverage/assets/0.10.0/application.js	17;"	f	line:17
+V.N	coverage/assets/0.10.2/application.js	17;"	f	line:17
 V.N	coverage/assets/0.8.0/application.js	17;"	f	line:17
 acts_as_aggregated	lib/moribus.rb	28;"	f	line:28	class:Moribus.ClassMethods
 acts_as_aggregated?	lib/moribus.rb	58;"	f	line:58	class:Moribus.ClassMethods
@@ -77,27 +97,38 @@ alias_collection_association_methods	lib/moribus/alias_association.rb	108;"	f	li
 alias_singular_association_methods	lib/moribus/alias_association.rb	98;"	f	line:98	class:Moribus.AliasAssociation.ClassMethods
 association	lib/moribus/extensions.rb	24;"	f	line:24	class:Moribus.Extensions
 b$.bA	coverage/assets/0.10.0/application.js	17;"	f	line:17
+b$.bA	coverage/assets/0.10.2/application.js	17;"	f	line:17
 b$.bA	coverage/assets/0.8.0/application.js	17;"	f	line:17
 b$.bW	coverage/assets/0.10.0/application.js	17;"	f	line:17
+b$.bW	coverage/assets/0.10.2/application.js	17;"	f	line:17
 b$.bW	coverage/assets/0.8.0/application.js	17;"	f	line:17
 b$.bX	coverage/assets/0.10.0/application.js	17;"	f	line:17
+b$.bX	coverage/assets/0.10.2/application.js	17;"	f	line:17
 b$.bX	coverage/assets/0.8.0/application.js	17;"	f	line:17
 b$.bY	coverage/assets/0.10.0/application.js	17;"	f	line:17
+b$.bY	coverage/assets/0.10.2/application.js	17;"	f	line:17
 b$.bY	coverage/assets/0.8.0/application.js	17;"	f	line:17
 b$.bZ	coverage/assets/0.10.0/application.js	17;"	f	line:17
+b$.bZ	coverage/assets/0.10.2/application.js	17;"	f	line:17
 b$.bZ	coverage/assets/0.8.0/application.js	17;"	f	line:17
 b$.bi	coverage/assets/0.10.0/application.js	17;"	f	line:17
+b$.bi	coverage/assets/0.10.2/application.js	17;"	f	line:17
 b$.bi	coverage/assets/0.8.0/application.js	17;"	f	line:17
 b$.bj	coverage/assets/0.10.0/application.js	17;"	f	line:17
+b$.bj	coverage/assets/0.10.2/application.js	17;"	f	line:17
 b$.bj	coverage/assets/0.8.0/application.js	17;"	f	line:17
 b$.bk	coverage/assets/0.10.0/application.js	17;"	f	line:17
+b$.bk	coverage/assets/0.10.2/application.js	17;"	f	line:17
 b$.bk	coverage/assets/0.8.0/application.js	17;"	f	line:17
 b$.bl	coverage/assets/0.10.0/application.js	17;"	f	line:17
+b$.bl	coverage/assets/0.10.2/application.js	17;"	f	line:17
 b$.bl	coverage/assets/0.8.0/application.js	17;"	f	line:17
 b$.bm	coverage/assets/0.10.0/application.js	17;"	f	line:17
+b$.bm	coverage/assets/0.10.2/application.js	17;"	f	line:17
 b$.bm	coverage/assets/0.8.0/application.js	17;"	f	line:17
 belongs_to	lib/moribus/alias_association.rb	39;"	f	line:39	class:Moribus.AliasAssociation.ClassMethods
 bh.bg	coverage/assets/0.10.0/application.js	17;"	f	line:17
+bh.bg	coverage/assets/0.10.2/application.js	17;"	f	line:17
 bh.bg	coverage/assets/0.8.0/application.js	17;"	f	line:17
 bold	lib/colorized_text.rb	30;"	f	line:30	class:ColorizedText
 cache_aggregated_record	lib/moribus/aggregated_cache_behavior.rb	63;"	f	line:63	class:Moribus.AggregatedCacheBehavior
@@ -108,19 +139,24 @@ colorize	lib/colorized_text.rb	10;"	f	line:10	class:ColorizedText
 column_for_attribute	lib/moribus/extensions/delegate_associated.rb	35;"	f	line:35	class:Moribus.Extensions.DelegateAssociated
 content_changed?	lib/moribus/tracked_behavior.rb	48;"	f	line:48	class:Moribus.TrackedBehavior
 cp.ce	coverage/assets/0.10.0/application.js	17;"	f	line:17
+cp.ce	coverage/assets/0.10.2/application.js	17;"	f	line:17
 cp.ce	coverage/assets/0.8.0/application.js	17;"	f	line:17
 cp.cf	coverage/assets/0.10.0/application.js	17;"	f	line:17
+cp.cf	coverage/assets/0.10.2/application.js	17;"	f	line:17
 cp.cf	coverage/assets/0.8.0/application.js	17;"	f	line:17
-current_lock_value	lib/moribus/tracked_behavior.rb	132;"	f	line:132
-current_to_false_sql_statement	lib/moribus/tracked_behavior.rb	105;"	f	line:105	class:Moribus.TrackedBehavior
+current_lock_value	lib/moribus/tracked_behavior.rb	144;"	f	line:144
+current_to_false_sql_statement	lib/moribus/tracked_behavior.rb	117;"	f	line:117	class:Moribus.TrackedBehavior
 define_delegation_module	lib/moribus/extensions/has_aggregated_extension.rb	40;"	f	line:40	class:Moribus.Extensions.HasAggregatedExtension.Helper
 define_effective_reader_for	lib/moribus/macros.rb	144;"	f	line:144	class:Moribus.Macros
 delegate_associated	lib/moribus/macros.rb	51;"	f	line:51	class:Moribus.Macros
 effective_	lib/moribus/macros.rb	146;"	f	line:146	class:Moribus.Macros.define_effective_reader_for
 extend	lib/moribus/extensions/has_aggregated_extension.rb	32;"	f	line:32	class:Moribus.Extensions.HasAggregatedExtension.Helper
 extend_has_aggregated_reflection	lib/moribus/extensions.rb	16;"	f	line:16	class:Moribus.Extensions.ClassMethods
-first_name=	spec/moribus_spec.rb	39;"	f	line:39	class:SpecPersonName
+first_name=	spec/moribus/aggregated_behavior_spec.rb	23;"	f	line:23	class:SpecPersonName
+first_name=	spec/moribus/extensions/delegate_associated_spec.rb	40;"	f	line:40	class:SpecPersonName
+first_name=	spec/moribus_spec.rb	40;"	f	line:40	class:SpecPersonName
 function.cr	coverage/assets/0.10.0/application.js	17;"	f	line:17
+function.cr	coverage/assets/0.10.2/application.js	17;"	f	line:17
 function.cr	coverage/assets/0.8.0/application.js	17;"	f	line:17
 get_parent_relation_keys	lib/moribus.rb	71;"	f	line:71	class:Moribus.ClassMethods
 green	lib/colorized_text.rb	20;"	f	line:20	class:ColorizedText
@@ -131,8 +167,10 @@ has_one_current	lib/moribus/macros.rb	78;"	f	line:78	class:Moribus.Macros
 include_delegation_module	lib/moribus/extensions/has_aggregated_extension.rb	81;"	f	line:81	class:Moribus.Extensions.HasAggregatedExtension.Helper
 initialize	lib/moribus/extensions/has_aggregated_extension.rb	26;"	f	line:26	class:Moribus.Extensions.HasAggregatedExtension.Helper
 k.function.J	coverage/assets/0.10.0/application.js	17;"	f	line:17
+k.function.J	coverage/assets/0.10.2/application.js	17;"	f	line:17
 k.function.J	coverage/assets/0.8.0/application.js	17;"	f	line:17
 k.var.e	coverage/assets/0.10.0/application.js	17;"	f	line:17
+k.var.e	coverage/assets/0.10.2/application.js	17;"	f	line:17
 k.var.e	coverage/assets/0.8.0/application.js	17;"	f	line:17
 lookup_relation	lib/moribus/aggregated_behavior.rb	72;"	f	line:72	class:Moribus.AggregatedBehavior
 lookup_self	lib/moribus/aggregated_behavior.rb	65;"	f	line:65	class:Moribus.AggregatedBehavior

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: HornsAndHooves-moribus
 version: !ruby/object:Gem::Version
-  version: 0.4.4
+  version: 0.5.1
 platform: ruby
 authors:
 - HornsAndHooves
@@ -11,7 +11,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2017-03-30 00:00:00.000000000 Z
+date: 2018-11-29 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails
@@ -184,7 +184,10 @@ files:
 - spec/dummy/public/500.html
 - spec/dummy/public/favicon.ico
 - spec/dummy/script/rails
+- spec/moribus/aggregated_behavior_spec.rb
 - spec/moribus/alias_association_spec.rb
+- spec/moribus/extensions/delegate_associated_spec.rb
+- spec/moribus/extensions/has_current_extension_spec.rb
 - spec/moribus/macros_spec.rb
 - spec/moribus_spec.rb
 - spec/spec_helper.rb
@@ -210,7 +213,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.6.10
+rubygems_version: 2.7.8
 signing_key: 
 specification_version: 4
 summary: Introduces Aggregated and Tracked behavior to ActiveRecord::Base models
@@ -248,7 +251,10 @@ test_files:
 - spec/dummy/public/500.html
 - spec/dummy/public/favicon.ico
 - spec/dummy/script/rails
+- spec/moribus/aggregated_behavior_spec.rb
 - spec/moribus/alias_association_spec.rb
+- spec/moribus/extensions/delegate_associated_spec.rb
+- spec/moribus/extensions/has_current_extension_spec.rb
 - spec/moribus/macros_spec.rb
 - spec/moribus_spec.rb
 - spec/spec_helper.rb