amoeba 3.2.0 → 3.3.0

data/spec/lib/amoeba_spec.rb

@@ -2,7 +2,7 @@ require 'spec_helper'
 
 describe 'amoeba' do
   context 'dup' do
-    before :each do
+    before do
       require ::File.dirname(__FILE__) + '/../support/data.rb'
     end
 
@@ -73,12 +73,17 @@ describe 'amoeba' do
       expect(new_post.supercats.map(&:ramblings)).to include('Copy of zomg')
       expect(new_post.supercats.map(&:other_ramblings).uniq.length).to eq(1)
       expect(new_post.supercats.map(&:other_ramblings).uniq).to include('La la la')
-      expect(new_post.contents).to eq("Here's a copy: #{old_post.contents.gsub(/dog/, 'cat')} (copied version)")
+      expect(new_post.contents).to eq("Here's a copy: #{old_post.contents.gsub(/dog/,
+                                                                               'cat')} (copied version)")
       expect(new_post.comments.length).to eq(5)
-      expect(new_post.comments.select { |c| c.nerf == 'ratatat' && c.contents.nil? }.length).to eq(1)
+      expect(new_post.comments.select do |c|
+               c.nerf == 'ratatat' && c.contents.nil?
+             end.length).to eq(1)
       expect(new_post.comments.select { |c| c.nerf == 'ratatat' }.length).to eq(2)
       expect(new_post.comments.select { |c| c.nerf == 'bonk' }.length).to eq(1)
-      expect(new_post.comments.select { |c| c.nerf == 'bonkers' && c.contents.nil? }.length).to eq(1)
+      expect(new_post.comments.select do |c|
+               c.nerf == 'bonkers' && c.contents.nil?
+             end.length).to eq(1)
 
       new_post.widgets.map(&:id).each do |id|
         expect(old_post.widgets.map(&:id)).not_to include(id)
@@ -96,7 +101,9 @@ describe 'amoeba' do
       expect(new_author.errors.messages).to be_empty
       expect(new_author.posts.first.custom_things.length).to eq(3)
       expect(new_author.posts.first.custom_things.select { |ct| ct.value == [] }.length).to eq(1)
-      expect(new_author.posts.first.custom_things.select { |ct| ct.value == [1, 2] }.length).to eq(1)
+      expect(new_author.posts.first.custom_things.select do |ct|
+               ct.value == [1, 2]
+             end.length).to eq(1)
       expect(new_author.posts.first.custom_things.select { |ct| ct.value == [78] }.length).to eq(1)
       # }}}
       # Products {{{
@@ -177,87 +184,89 @@ describe 'amoeba' do
   end
 
   context 'Using a if condition' do
+    subject { post.amoeba_dup.save! }
+
     before(:all) do
       require ::File.dirname(__FILE__) + '/../support/data.rb'
     end
+
     before { ::Post.fresh_amoeba }
 
-    subject { post.amoeba_dup.save! }
     let(:post) { Post.first }
 
     it 'includes an association with truthy condition' do
       ::Post.amoeba do
         include_association :comments, if: :truthy?
       end
-      expect { subject }.to change { Comment.count }.by(3)
+      expect { subject }.to change(Comment, :count).by(3)
     end
 
     it 'does not include an association with a falsey condition' do
       ::Post.amoeba do
         include_association :comments, if: :falsey?
       end
-      expect { subject }.not_to change { Comment.count }
+      expect { subject }.not_to change(Comment, :count)
     end
 
     it 'excludes an association with a truthy condition' do
       ::Post.amoeba do
         exclude_association :comments, if: :truthy?
       end
-      expect { subject }.not_to change { Comment.count }
+      expect { subject }.not_to change(Comment, :count)
     end
 
     it 'does not exclude an association with a falsey condition' do
       ::Post.amoeba do
         exclude_association :comments, if: :falsey?
       end
-      expect { subject }.to change { Comment.count }.by(3)
+      expect { subject }.to change(Comment, :count).by(3)
     end
 
     it 'includes associations from a given array with a truthy condition' do
       ::Post.amoeba do
         include_association [:comments], if: :truthy?
       end
-      expect { subject }.to change { Comment.count }.by(3)
+      expect { subject }.to change(Comment, :count).by(3)
     end
 
     it 'does not include associations from a given array with a falsey condition' do
       ::Post.amoeba do
         include_association [:comments], if: :falsey?
       end
-      expect { subject }.not_to change { Comment.count }
+      expect { subject }.not_to change(Comment, :count)
     end
 
     it 'does exclude associations from a given array with a truthy condition' do
       ::Post.amoeba do
         exclude_association [:comments], if: :truthy?
       end
-      expect { subject }.not_to change { Comment.count }
+      expect { subject }.not_to change(Comment, :count)
     end
 
     it 'does not exclude associations from a given array with a falsey condition' do
       ::Post.amoeba do
         exclude_association [:comments], if: :falsey?
       end
-      expect { subject }.to change { Comment.count }.by(3)
+      expect { subject }.to change(Comment, :count).by(3)
     end
   end
 
   context 'override' do
-    before :each do
+    before do
       ::Image.fresh_amoeba
       ::Image.amoeba do
         override ->(old, new) { new.product_id = 13 if old.filename == 'test.jpg' }
       end
     end
 
-    it 'should override fields' do
+    it 'overrides fields' do
       image = ::Image.create(filename: 'test.jpg', product_id: 12)
       image_dup = image.amoeba_dup
       expect(image_dup.save).to be_truthy
       expect(image_dup.product_id).to eq(13)
     end
 
-    it 'should not override fields' do
+    it 'does not override fields' do
       image = ::Image.create(filename: 'test2.jpg', product_id: 12)
       image_dup = image.amoeba_dup
       expect(image_dup.save).to be_truthy
@@ -266,7 +275,7 @@ describe 'amoeba' do
   end
 
   context 'nullify' do
-    before :each do
+    before do
       ::Image.fresh_amoeba
       ::Image.amoeba do
         nullify :product_id
@@ -276,14 +285,14 @@ describe 'amoeba' do
     let(:image) { ::Image.create(filename: 'test.jpg', product_id: 12) }
     let(:image_dup) { image.amoeba_dup }
 
-    it 'should nullify fields' do
+    it 'nullifies fields' do
       expect(image_dup.save).to be_truthy
       expect(image_dup.product_id).to be_nil
     end
   end
 
   context 'strict propagate' do
-    it 'should call #reset_amoeba' do
+    it 'calls #reset_amoeba' do
       expect(::SuperBlackBox).to receive(:reset_amoeba).and_call_original
       box = ::SuperBlackBox.create(title: 'Super Black Box', price: 9.99, length: 1, metal: '1')
       new_box = box.amoeba_dup
@@ -312,13 +321,16 @@ describe 'amoeba' do
 
   context 'preprocessing fields' do
     subject { super_admin.amoeba_dup }
-    let(:super_admin) { ::SuperAdmin.create!(email: 'user@example.com', active: true, password: 'password') }
 
-    it 'should accept "set" to set false to attribute' do
+    let(:super_admin) do
+      ::SuperAdmin.create!(email: 'user@example.com', active: true, password: 'password')
+    end
+
+    it 'accepts "set" to set false to attribute' do
       expect(subject.active).to be false
     end
 
-    it 'should skip "prepend" if it equal to false' do
+    it 'skips "prepend" if it equal to false' do
       expect(subject.password).to eq('password')
     end
   end
@@ -336,6 +348,8 @@ describe 'amoeba' do
   end
 
   context 'inheritance extended' do
+    subject { stage.amoeba_dup }
+
     let(:stage) do
       stage = CustomStage.new(title: 'My Stage', external_id: 213)
       stage.listeners.build(name: 'John')
@@ -346,19 +360,19 @@ describe 'amoeba' do
       stage
     end
 
-    subject { stage.amoeba_dup }
-
     it 'contains parent association and own associations', :aggregate_failures do
       subject
-      expect { subject.save! }.to change(Listener, :count).by(2).
-                                  and change(Specialist, :count).by(1).
-                                  and change(CustomRule, :count).by(1)
+      expect { subject.save! }.to change(Listener, :count).by(2)
+                                                          .and change(Specialist, :count).by(1)
+                                                                                         .and change(
+                                                                                           CustomRule, :count
+                                                                                         ).by(1)
 
       expect(subject.title).to eq 'My Stage'
       expect(subject.external_id).to be_nil
-      expect(subject.listeners.find_by(name: 'John')).to_not be_nil
-      expect(subject.listeners.find_by(name: 'Helen')).to_not be_nil
-      expect(subject.specialists.find_by(name: 'Jack')).to_not be_nil
+      expect(subject.listeners.find_by(name: 'John')).not_to be_nil
+      expect(subject.listeners.find_by(name: 'Helen')).not_to be_nil
+      expect(subject.specialists.find_by(name: 'Jack')).not_to be_nil
       expect(subject.custom_rules.first.description).to eq 'Kill all humans'
     end
   end

data/spec/spec_helper.rb

@@ -1,13 +1,21 @@
 require 'simplecov'
-require 'coveralls'
-SimpleCov.formatter = SimpleCov::Formatter::MultiFormatter.new([
-  SimpleCov::Formatter::HTMLFormatter,
-  Coveralls::SimpleCov::Formatter
-])
+
 SimpleCov.start do
   add_filter 'spec'
   minimum_coverage(76)
+
+  if ENV['CI']
+    require 'simplecov-lcov'
+
+    SimpleCov::Formatter::LcovFormatter.config do |c|
+      c.report_with_single_file = true
+      c.single_report_path = 'coverage/lcov.info'
+    end
+
+    formatter SimpleCov::Formatter::LcovFormatter
+  end
 end
+
 require 'active_record'
 require 'amoeba'
 

data/spec/support/data.rb

@@ -1,4 +1,3 @@
-
 u1 = User.create(name: 'Robert Johnson', email: 'bob@crossroads.com')
 u2 = User.create(name: 'Miles Davis', email: 'miles@kindofblue.com')
 
@@ -8,7 +7,8 @@ a2 = Author.create(full_name: 'Arthur Sees Clarck', nickname: 'strangewater')
 t = Topic.create(title: 'Ponies', description: 'Lets talk about my ponies.')
 
 # First Post {{{
-p1 = t.posts.create(owner: u1, author: a1, title: 'My little pony', contents: 'Lorum ipsum dolor rainbow bright. I like dogs, dogs are awesome.')
+p1 = t.posts.create(owner: u1, author: a1, title: 'My little pony',
+                    contents: 'Lorum ipsum dolor rainbow bright. I like dogs, dogs are awesome.')
 f1 = p1.create_post_config(is_visible: true, is_open: false, password: 'abcdefg123')
 a1 = p1.create_account(title: 'Foo')
 h1 = p1.account.create_history(some_stuff: 'Bar')
@@ -21,11 +21,15 @@ c2 = p1.comments.create(contents: 'I hate it!', nerf: 'whapaow')
 c3 = p1.comments.create(contents: 'kthxbbq!!11!!!1!eleven!!', nerf: 'bonk')
 [0, 0, 1, 2, 1, 0].each { |stars| c3.ratings.create(num_stars: stars) }
 
-%w(funny wtf cats).each { |value| p1.tags << Tag.create(value: value) }
+%w[funny wtf cats].each { |value| p1.tags << Tag.create(value: value) }
 
-['My Sidebar', 'Photo Gallery', 'Share & Like'].each { |value| p1.widgets << Widget.create(value: value) }
+['My Sidebar', 'Photo Gallery', 'Share & Like'].each do |value|
+  p1.widgets << Widget.create(value: value)
+end
 
-['This is important', "You've been warned", "Don't forget"].each { |value| p1.notes << Note.create(value: value) }
+['This is important', "You've been warned", "Don't forget"].each do |value|
+  p1.notes << Note.create(value: value)
+end
 
 p1.save
 
@@ -85,15 +89,18 @@ necklace1.sections << jewelry
 necklace1.sections << accessories
 necklace1.save
 
-
-company = Company.create(name:'ABC Industries')
-employee = company.employees.create(name:'Joe',ssn:'1111111111',salary:10000.0)
-employee_address = employee.addresses.create(street: '123 My Street',unit:'103',city:'Hollywood',state:'CA',zip:'90210')
-employee_address_2 = employee.addresses.create(street: '124 My Street',unit:'103',city:'Follywood',state:'CA',zip:'90210')
-employee_photo = employee.photos.create(name: 'Portrait', size: 12345)
-customer = company.customers.create(email:'my@email.address',password:'password')
-customer_address = customer.addresses.create(street: '321 My Street',unit:'301',city:'Bollywood',state:'IN',zip:'11111')
-customer_address_2 = customer.addresses.create(street: '321 My Drive',unit:'311',city:'Mollywood',state:'IN',zip:'21111')
-customer_photo = customer.photos.create(name: 'Mug Shot', size: 54321)
+company = Company.create(name: 'ABC Industries')
+employee = company.employees.create(name: 'Joe', ssn: '1111111111', salary: 10_000.0)
+employee_address = employee.addresses.create(street: '123 My Street', unit: '103', city: 'Hollywood',
+                                             state: 'CA', zip: '90210')
+employee_address_2 = employee.addresses.create(street: '124 My Street', unit: '103', city: 'Follywood',
+                                               state: 'CA', zip: '90210')
+employee_photo = employee.photos.create(name: 'Portrait', size: 12_345)
+customer = company.customers.create(email: 'my@email.address', password: 'password')
+customer_address = customer.addresses.create(street: '321 My Street', unit: '301', city: 'Bollywood',
+                                             state: 'IN', zip: '11111')
+customer_address_2 = customer.addresses.create(street: '321 My Drive', unit: '311', city: 'Mollywood',
+                                               state: 'IN', zip: '21111')
+customer_photo = customer.photos.create(name: 'Mug Shot', size: 54_321)
 
 # }}}

data/spec/support/models.rb

@@ -34,41 +34,41 @@ class Post < ActiveRecord::Base
 
   amoeba do
     enable
-    clone [:widgets, :notes]
+    clone %i[widgets notes]
     prepend title: 'Copy of '
     append contents: ' (copied version)'
     regex contents: { replace: /dog/, with: 'cat' }
     customize([
-      lambda do |orig_obj, copy_of_obj|
-        orig_obj.comments.each do |oc|
-          if oc.nerf == 'ratatat'
-            hash = oc.attributes
-            hash[:id]       = nil
-            hash[:post_id]  = nil
-            hash[:contents] = nil
-
-            cc = Comment.new(hash)
-
-            copy_of_obj.comments << cc
-          end
-        end
-      end,
-      lambda do |orig_obj, copy_of_obj|
-        orig_obj.comments.each do |oc|
-          if oc.nerf == 'bonk'
-            hash = oc.attributes
-            hash[:id]       = nil
-            hash[:post_id]  = nil
-            hash[:contents] = nil
-            hash[:nerf]     = 'bonkers'
-
-            cc = Comment.new(hash)
-
-            copy_of_obj.comments << cc
-          end
-        end
-      end
-    ])
+                lambda do |orig_obj, copy_of_obj|
+                  orig_obj.comments.each do |oc|
+                    next unless oc.nerf == 'ratatat'
+
+                    hash = oc.attributes
+                    hash[:id]       = nil
+                    hash[:post_id]  = nil
+                    hash[:contents] = nil
+
+                    cc = Comment.new(hash)
+
+                    copy_of_obj.comments << cc
+                  end
+                end,
+                lambda do |orig_obj, copy_of_obj|
+                  orig_obj.comments.each do |oc|
+                    next unless oc.nerf == 'bonk'
+
+                    hash = oc.attributes
+                    hash[:id]       = nil
+                    hash[:post_id]  = nil
+                    hash[:contents] = nil
+                    hash[:nerf]     = 'bonkers'
+
+                    cc = Comment.new(hash)
+
+                    copy_of_obj.comments << cc
+                  end
+                end
+              ])
   end
 
   def truthy?
@@ -97,11 +97,13 @@ class CustomThing < ActiveRecord::Base
     def self.load(str)
       return [] unless str.present?
       return str if str.is_a?(Array)
+
       str.split(',').map(&:to_i)
     end
 
     def self.dump(int_array)
       return '' unless int_array.present?
+
       int_array.join(',')
     end
   end
@@ -261,7 +263,7 @@ class Photo < ActiveRecord::Base
   belongs_to :imageable, polymorphic: true
 
   amoeba do
-    customize(lambda { |original_photo,new_photo|
+    customize(lambda { |original_photo, new_photo|
       new_photo.name = original_photo.name.to_s + ' Copy'
     })
   end
@@ -273,7 +275,7 @@ class Company < ActiveRecord::Base
 
   amoeba do
     include_associations :employees,
-                        :customers
+                         :customers
   end
 end
 
@@ -283,9 +285,8 @@ class Employee < ActiveRecord::Base
   belongs_to :company
 
   amoeba do
-    include_associations [:addresses, :photos]
+    include_associations %i[addresses photos]
   end
-
 end
 
 class Customer < ActiveRecord::Base
@@ -313,7 +314,7 @@ class ObjectPrototype < MetalObject
   end
 
   def become_real
-    self.dup.becomes RealObject
+    dup.becomes RealObject
   end
 
   def remap_subobjects(relation_name)
@@ -332,7 +333,7 @@ class SubobjectPrototype < MetalObject
   end
 
   def become_subobject
-    self.dup.becomes Subobject
+    dup.becomes Subobject
   end
 end
 
@@ -360,7 +361,7 @@ class Box < ActiveRecord::Base
 end
 
 class BoxProduct < ActiveRecord::Base
-  belongs_to      :box, class_name: 'Box'
+  belongs_to :box, class_name: 'Box'
 
   amoeba do
     enable

metadata

@@ -1,44 +1,72 @@
 --- !ruby/object:Gem::Specification
 name: amoeba
 version: !ruby/object:Gem::Version
-  version: 3.2.0
+  version: 3.3.0
 platform: ruby
 authors:
 - Vaughn Draughon
 - Oleksandr Simonov
-autorequire:
+autorequire: 
 bindir: bin
 cert_chain: []
-date: 2021-06-30 00:00:00.000000000 Z
+date: 2023-09-30 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
-  name: bundler
+  name: rspec
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 1.6.0
+        version: 3.0.0
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 1.6.0
+        version: 3.0.0
 - !ruby/object:Gem::Dependency
-  name: rspec
+  name: rubocop
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: 3.0.0
+        version: '1.16'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: 3.0.0
+        version: '1.16'
+- !ruby/object:Gem::Dependency
+  name: rubocop-rake
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '0.6'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '0.6'
+- !ruby/object:Gem::Dependency
+  name: rubocop-rspec
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '2.4'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '2.4'
 - !ruby/object:Gem::Dependency
   name: sqlite3
   requirement: !ruby/object:Gem::Requirement
@@ -59,14 +87,14 @@ dependencies:
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 4.2.0
+        version: 5.2.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 4.2.0
+        version: 5.2.0
 description: |
   An extension to ActiveRecord to allow the duplication method to also copy associated children, with recursive support for nested of grandchildren. The behavior is controllable with a simple DSL both on your rails models and on the fly, i.e. per instance. Numerous configuration styles and preprocessing directives are included for power and flexibility. Supports preprocessing of field values to prepend strings such as "Copy of ", to nullify or process field values with regular expressions. Supports most association types including has_one :through and has_many :through.
 
@@ -77,24 +105,32 @@ extensions: []
 extra_rdoc_files: []
 files:
 - ".cane"
+- ".github/workflows/jruby.yml"
+- ".github/workflows/ruby_25.yml"
+- ".github/workflows/ruby_26.yml"
+- ".github/workflows/ruby_27.yml"
+- ".github/workflows/ruby_30.yml"
+- ".github/workflows/ruby_31.yml"
+- ".github/workflows/ruby_32.yml"
+- ".github/workflows/ruby_head.yml"
 - ".gitignore"
 - ".rspec"
 - ".rubocop.yml"
-- ".travis.yml"
+- ".rubocop_todo.yml"
 - Appraisals
+- CHANGELOG.md
 - Gemfile
+- LICENSE.md
 - README.md
 - Rakefile
 - amoeba.gemspec
 - defaults.reek
-- gemfiles/activerecord_4.2.gemfile
-- gemfiles/activerecord_5.0.gemfile
-- gemfiles/activerecord_5.1.gemfile
 - gemfiles/activerecord_5.2.gemfile
 - gemfiles/activerecord_6.0.gemfile
 - gemfiles/activerecord_6.1.gemfile
+- gemfiles/activerecord_7.0.gemfile
 - gemfiles/activerecord_head.gemfile
-- gemfiles/jruby_activerecord_6.1.gemfile
+- gemfiles/jruby_activerecord_7.0.gemfile
 - gemfiles/jruby_activerecord_head.gemfile
 - lib/amoeba.rb
 - lib/amoeba/class_methods.rb
@@ -114,9 +150,9 @@ files:
 - spec/support/schema.rb
 homepage: http://github.com/amoeba-rb/amoeba
 licenses:
-- BSD
+- BSD-2-Clause
 metadata: {}
-post_install_message:
+post_install_message: 
 rdoc_options: []
 require_paths:
 - lib
@@ -124,20 +160,15 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: '0'
+      version: '2.5'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.2.17
-signing_key:
+rubygems_version: 3.4.10
+signing_key: 
 specification_version: 4
 summary: Easy copying of rails models and their child associations.
-test_files:
-- spec/lib/amoeba_spec.rb
-- spec/spec_helper.rb
-- spec/support/data.rb
-- spec/support/models.rb
-- spec/support/schema.rb
+test_files: []