fix(function): column accepts **args instead of array

Author: bivanalhar

.gitignore

@@ -5,3 +5,4 @@
 .ruby-version
 gemfiles/*.lock
 Gemfile.lock
+coverage/*

lib/active_record/userstamp/migration_additions.rb

@@ -1,14 +1,14 @@
-module ActiveRecord::Userstamp::MigrationHelper
+module ActiveRecord::Userstamp::MigrationAdditions
   extend ActiveSupport::Concern
 
-  def userstamps(*args)
+  def userstamps(**args)
     config = ActiveRecord::Userstamp.config
-    column(config.creator_attribute, :integer, *args)
-    column(config.updater_attribute, :integer, *args)
-    column(config.deleter_attribute, :integer, *args) if config.deleter_attribute.present?
+    column(config.creator_attribute, :integer, **args)
+    column(config.updater_attribute, :integer, **args)
+    column(config.deleter_attribute, :integer, **args) if config.deleter_attribute.present?
   end
 end
 
 ActiveRecord::ConnectionAdapters::TableDefinition.class_eval do
-  include ActiveRecord::Userstamp::MigrationHelper
+  include ActiveRecord::Userstamp::MigrationAdditions
 end

spec/lib/migration_spec.rb

@@ -5,10 +5,11 @@
     class self::DefaultRandom < ActiveRecord::Base
       stampable
     end
+
     subject { self.class::DefaultRandom.new }
 
     temporary_table(:default_randoms) do |t|
-      t.userstamps(null: false)
+      t.userstamps(**{null: false})
     end
 
     with_temporary_table(:default_randoms) do
@@ -30,22 +31,14 @@ class self::DefaultRandom < ActiveRecord::Base
   end
 
   context 'when overridden attribute names are used' do
-    before(:each) do
-      ActiveRecord::Userstamp.configure do |config|
-        config.creator_attribute = :created_by
-        config.updater_attribute = :updated_by
-        config.deleter_attribute = :deleted_by
-      end
-      class self.class::OverriddenRandom < ActiveRecord::Base
-        stampable
-      end
+    ActiveRecord::Userstamp.configure do |config|
+      config.creator_attribute = :created_by
+      config.updater_attribute = :updated_by
+      config.deleter_attribute = :deleted_by
     end
-    after(:each) do
-      ActiveRecord::Userstamp.configure do |config|
-        config.creator_attribute = :creator_id
-        config.updater_attribute = :updater_id
-        config.deleter_attribute = :deleter_id
-      end
+
+    class self::OverriddenRandom < ActiveRecord::Base
+      stampable
     end
 
     subject { self.class::OverriddenRandom.new }
@@ -54,7 +47,15 @@ class self.class::OverriddenRandom < ActiveRecord::Base
       t.userstamps
     end
 
-    with_temporary_table(:overridden_randoms, :each) do
+    after do
+      ActiveRecord::Userstamp.configure do |config|
+        config.creator_attribute = :creator_id
+        config.updater_attribute = :updater_id
+        config.deleter_attribute = :deleter_id
+      end
+    end
+
+    with_temporary_table(:overridden_randoms, :all) do
       it 'has a created_by attribute' do
         expect(subject.has_attribute?(:created_by)).to be true
       end