Splat options explicitly

lib/lhm/adapter.rb

@@ -37,8 +37,8 @@ def remove_column(column_name)
     # @param columns [Array<String>, Array<Symbol>, String, Symbol]
     # @param index_name [String]
     def add_index(columns, index_name = nil)
-      options = { name: index_name } if index_name
-      migration.add_index(table_name, columns, options || {})
+      options = index_name ? { name: index_name } : { }
+      migration.add_index(table_name, columns, **options)
     end
 
     # Removes the index in the given columns or by its name
@@ -51,7 +51,7 @@ def remove_index(columns, index_name = nil)
                 else
                   { column: columns }
                 end
-      migration.remove_index(table_name, options)
+      migration.remove_index(table_name, **options)
     end
 
     # Change the column to use the provided definition, through ActiveRecord
@@ -79,7 +79,7 @@ def add_unique_index(columns, index_name = nil)
       options = { unique: true }
       options.merge!(name: index_name) if index_name # rubocop:disable Performance/RedundantMerge
 
-      migration.add_index(table_name, columns, options)
+      migration.add_index(table_name, columns, **options)
     end
 
     private

spec/lhm/adapter/add_index_spec.rb

@@ -9,42 +9,45 @@
 
   before do
     allow(migration).to(
-      receive(:add_index).with(table_name, columns, options)
+      receive(:add_index).with(table_name, columns, **{})
     )
   end
 
-  let(:options) { {} }
-
   context 'when passing a single column' do
-    before { adapter.add_index(columns) }
-
     let(:columns) { :some_id_field }
 
+    before { adapter.add_index(columns) }
+
     it 'calls #add_index in the migration' do
       expect(migration).to(
-        have_received(:add_index).with(table_name, columns, options)
+        have_received(:add_index).with(table_name, columns)
       )
     end
   end
 
   context 'when passing an array of columns' do
-    before { adapter.add_index(columns) }
-
     let(:columns) { [:some_id_field, :name] }
 
+    before { adapter.add_index(columns) }
+
     it 'calls #add_index in the migration' do
       expect(migration).to(
-        have_received(:add_index).with(table_name, columns, options)
+        have_received(:add_index).with(table_name, columns)
       )
     end
   end
 
   context 'when passing also an index name' do
-    before { adapter.add_index(columns, options[:name]) }
-
     let(:columns) { [:some_id_field, :name] }
     let(:options) { { name: 'index_name' } }
 
+    before do
+      allow(migration).to(
+        receive(:add_index).with(table_name, columns, name: 'index_name')
+      )
+    end
+    before { adapter.add_index(columns, options[:name]) }
+
     it 'calls #add_index in the migration' do
       expect(migration).to(
         have_received(:add_index).with(table_name, columns, options)