HornsAndHooves-moribus 0.6.0 → 0.7.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 664930f1565d0d6eaa50dc8fba6c3f90763e70c709a748f2e572f493ef2f3de5
-  data.tar.gz: b9541ec330424313c0e71f9605af2f2cf615d6df92b2a29961a97675cc457683
+  metadata.gz: 0d973c4c4d01859f5d577ec5c594b78d36c5e6064885b843804a984670d6e2c4
+  data.tar.gz: b7a269f5cc0bb99e3833d219f593e726b98be3371904cfe7d1deb16a45af1ee0
 SHA512:
-  metadata.gz: 5af699607d6d67710af3e538cc23138b0a934236c5ce2b234ebc9a3349f989c6e7eaa1184a9608d5b3f77e5ff2898d2f96757d5805d39e7a1d4af349947cbef3
-  data.tar.gz: 7a8cec103bc680b45800f888adfda114b7cbe55d3ef5547694f2227cd79afff4f2e3b503782ae79b84f8beab7a80b6c0452369d464c8c8a9daf01e9c3c8335b3
+  metadata.gz: e193850f47dabd551af7e4cf31ef9b4e92653151533fc4190b8aa2fb0b2a74142c606b7de75821a273a53040402e1cfad9a1498e019cd45bf4886834baf4e3ec
+  data.tar.gz: e575f9663606d2367b594537a4cbd9222cbe44c36e0386dd6bf40378ba3ec39d66f2e626b0f1914a24850238254ded9905957775ecb1821d99b48a08d8b446e6

data/.simplecov

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

data/HornsAndHooves-moribus.gemspec

@@ -20,7 +20,7 @@ Gem::Specification.new do |s|
   s.require_paths = ["lib"]
 
   # specify any dependencies here; for example:
-  s.add_dependency "rails",      "> 4.0", "< 5"
+  s.add_dependency "rails",      "~> 5.0"
   s.add_dependency "power_enum", ">= 2.7.0"
   s.add_dependency "yard",       ">= 0"
 

data/lib/moribus/alias_association.rb

@@ -23,30 +23,25 @@ module Moribus
       # association-specific methods. See module description for example.
       def alias_association(alias_name, association_name)
         if reflection = reflect_on_association(association_name)
-          # Use Rails 4.1.x+ behavior, if available:
-          if ActiveRecord::Reflection.respond_to? :add_reflection then
-            ActiveRecord::Reflection.add_reflection self, alias_name, reflection
-          else
-            # Rails 4.0.x behavior:
-            reflections[alias_name] = reflections[association_name]
-          end
+          ActiveRecord::Reflection.add_reflection self, alias_name, reflection
+
           alias_association_methods(alias_name, reflection)
           reflection
         end
       end
 
       # Allows :alias option to alias belongs_to association
-      def belongs_to(name, scope = nil, options = {})
+      def belongs_to(name, scope = nil, **options)
         options = scope if scope.is_a?(Hash)
 
         alias_name = options.delete(:alias)
-        reflection = super(name, scope, options)
+        reflection = super(name, scope, **options)
         alias_association(alias_name, name) if alias_name
         reflection
       end
 
       # Allows :alias option to alias has_many association
-      def has_many(name, scope = nil, options = {}, &extension)
+      def has_many(name, scope = nil, **options, &extension)
         options = scope if scope.is_a?(Hash)
 
         alias_name = options.delete(:alias)
@@ -56,7 +51,7 @@ module Moribus
       end
 
       # Allows :alias option to alias has_one association
-      def has_one(name, scope = nil, options = {})
+      def has_one(name, scope = nil, **options)
         options = scope if scope.is_a?(Hash)
 
         alias_name = options.delete(:alias)

data/lib/moribus/extensions/delegate_associated.rb

@@ -33,7 +33,7 @@ module Moribus
       # presented in Customer, the code will result in exception without
       # the following hook:
       def column_for_attribute(name)
-        unless (column = super).nil?
+        unless (column = super).nil? || column.is_a?(ActiveRecord::ConnectionAdapters::NullColumn)
           return column
         end
 

data/lib/moribus/extensions/has_aggregated_extension.rb

@@ -71,7 +71,7 @@ module Moribus
             [name, "#{name}="]
           end
           klass.define_attribute_methods
-          attribute_methods = klass.generated_attribute_methods.instance_methods.select{ |m| m !~ EXCLUDE_METHODS_REGEXP }
+          attribute_methods = klass.__send__(:generated_attribute_methods).instance_methods.select{ |m| m !~ EXCLUDE_METHODS_REGEXP }
           custom_writers = klass.instance_methods(false).map(&:to_s) & klass.column_names.map{ |name| "#{name}=" }
           (attribute_methods + enum_methods.flatten + custom_writers).map(&:to_sym)
         end

data/lib/moribus/extensions/has_current_extension.rb

@@ -11,14 +11,12 @@ module Moribus
         else
           # Use custom update to avoid running ActiveRecord optimistic locking
           # and to avoid updating lock_version column:
-          klass          = target.class
-          is_current_col = klass.columns.detect { |c| c.name == "is_current" }
-          id_column      = klass.columns.detect { |c| c.name == klass.primary_key }
+          klass = target.class
 
           sql =  "UPDATE #{klass.quoted_table_name} " \
-                 "SET \"is_current\" = #{klass.quote_value(false, is_current_col)} "
+                 "SET \"is_current\" = #{klass.connection.quote(false)} "
           sql << "WHERE #{klass.quoted_primary_key} = " \
-                 "#{klass.quote_value(target.send(klass.primary_key), id_column)} "
+                 "#{klass.connection.quote(target.send(klass.primary_key))} "
 
           klass.connection.update sql
         end

data/lib/moribus/tracked_behavior.rb

@@ -116,7 +116,6 @@ module Moribus
     # TODO: need to find way to track stale objects
     def current_to_false_sql_statement
       klass              = self.class
-      is_current_col     = klass.columns.detect { |c| c.name == "is_current" }
       lock_column_name   = klass.locking_column
       lock_value         = current_lock_value
       lock_column        = if lock_value
@@ -124,17 +123,16 @@ module Moribus
                            else
                              nil
                            end
-      id_column          = klass.columns.detect { |c| c.name == klass.primary_key }
       quoted_lock_column = klass.connection.quote_column_name(lock_column_name)
 
       "UPDATE #{klass.quoted_table_name} " \
-      "SET \"is_current\" = #{klass.quote_value(false, is_current_col)} ".tap do |sql|
+      "SET \"is_current\" = #{klass.connection.quote(false)} ".tap do |sql|
         sql << "WHERE #{klass.quoted_primary_key} = " \
-               "#{klass.quote_value(@_before_to_new_record_values[:id], id_column)}"
+               "#{klass.connection.quote(@_before_to_new_record_values[:id])}"
 
         if lock_value
-          sql << " AND \"is_current\" = #{klass.quote_value(true, is_current_col)}"
-          sql << " AND #{quoted_lock_column} = #{klass.quote_value(lock_value, lock_column)}"
+          sql << " AND \"is_current\" = #{klass.connection.quote(true)}"
+          sql << " AND #{quoted_lock_column} = #{klass.connection.quote(lock_value)}"
         end
       end
     end

data/lib/moribus/version.rb

@@ -1,3 +1,3 @@
 module Moribus # :nodoc:
-  VERSION = "0.6.0" # :nodoc:
+  VERSION = "0.7.0" # :nodoc:
 end

data/spec/dummy/tmp/development_secret.txt

@@ -0,0 +1 @@
+02c924139ba4207bce788dd378b8f1f99976099325bbad4e85204e34d04970e039409db0f08ceea070238f4f5a41bdbc94fb6e51aeb0996a4ee32bcd3c31ede1

data/spec/moribus/extensions/delegate_associated_spec.rb

@@ -132,5 +132,9 @@ describe Moribus::Extensions::DelegateAssociated do
     it "returns class db column if it exists" do
       expect(customer.column_for_attribute(:id)).not_to be_nil
     end
+
+    it "returns class db column if it doesn't exist" do
+      expect(customer.column_for_attribute(:foobar)).to be_nil
+    end
   end
 end

data/spec/moribus_spec.rb

@@ -225,7 +225,7 @@ describe Moribus do
 
         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")
+                         /Attempted to update_current \(version #{@info2.lock_version}\) a stale object: SpecCustomerInfo\./)
       end
 
       it "updates lock_version incrementally for each new record" do

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: HornsAndHooves-moribus
 version: !ruby/object:Gem::Version
-  version: 0.6.0
+  version: 0.7.0
 platform: ruby
 authors:
 - HornsAndHooves
@@ -11,28 +11,22 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2019-10-14 00:00:00.000000000 Z
+date: 2019-10-18 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ">"
-      - !ruby/object:Gem::Version
-        version: '4.0'
-    - - "<"
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '5'
+        version: '5.0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ">"
-      - !ruby/object:Gem::Version
-        version: '4.0'
-    - - "<"
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '5'
+        version: '5.0'
 - !ruby/object:Gem::Dependency
   name: power_enum
   requirement: !ruby/object:Gem::Requirement
@@ -184,6 +178,7 @@ files:
 - spec/dummy/public/500.html
 - spec/dummy/public/favicon.ico
 - spec/dummy/script/rails
+- spec/dummy/tmp/development_secret.txt
 - spec/moribus/aggregated_behavior_spec.rb
 - spec/moribus/alias_association_spec.rb
 - spec/moribus/extensions/delegate_associated_spec.rb
@@ -251,6 +246,7 @@ test_files:
 - spec/dummy/public/500.html
 - spec/dummy/public/favicon.ico
 - spec/dummy/script/rails
+- spec/dummy/tmp/development_secret.txt
 - spec/moribus/aggregated_behavior_spec.rb
 - spec/moribus/alias_association_spec.rb
 - spec/moribus/extensions/delegate_associated_spec.rb