Make Spree::MigrationHelpers Ruby 3.0 compatible

core/lib/spree/migration_helpers.rb

@@ -6,9 +6,9 @@ def safe_remove_index(table, column)
       remove_index(table, column) if index_exists?(table, column)
     end
 
-    def safe_add_index(table, column, options = {})
-      if columns_exist?(table, column) && !index_exists?(table, column, options)
-        add_index(table, column, options)
+    def safe_add_index(table, column, **options)
+      if columns_exist?(table, column) && !index_exists?(table, column, **options)
+        add_index(table, column, **options)
       end
     end
 

core/spec/lib/spree/migration_helpers_spec.rb

@@ -0,0 +1,33 @@
+require "rails_helper"
+
+RSpec.describe Spree::MigrationHelpers do
+  let(:helper) do
+    double.as_null_object.tap do |object|
+      object.extend(described_class)
+      object.extend(ActiveRecord::ConnectionAdapters::SchemaStatements)
+    end
+  end
+
+  subject { helper.safe_add_index(double, double) }
+
+  # regression test
+  describe "#safe_add_index" do
+    context "when the column exists" do
+      context "and the index does" do
+        it "passes compatible arguments to index_exists?" do
+          expect { subject }.to_not raise_error(ArgumentError)
+        end
+      end
+
+      context "and the index does not" do
+        before do
+          allow(helper).to receive(:index_exists?).with(any_args).and_return(false)
+        end
+
+        it "passes compatible arguments to add_index" do
+          expect { subject }.to_not raise_error(ArgumentError)
+        end
+      end
+    end
+  end
+end