add test scenario to make sure deprecated warning is triggered

clio · Jun 11, 2024 · 5d2621f · 5d2621f
1 parent 24efc6a
commit 5d2621f
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 0 deletions.
diff --git a/lib/jit_preloader/preloader.rb b/lib/jit_preloader/preloader.rb
@@ -3,6 +3,10 @@ class Preloader < ActiveRecord::Associations::Preloader
 
     attr_accessor :records
 
+    def foo(**kwargs)
+      kwargs
+    end
+
     if Gem::Version.new(ActiveRecord::VERSION::STRING) >= Gem::Version.new("7.0.0")
       def self.attach(records)
         new(records: records.dup, associations: nil).tap do |loader|

diff --git a/spec/lib/jit_preloader/preloader_spec.rb b/spec/lib/jit_preloader/preloader_spec.rb
@@ -49,6 +49,11 @@
     ->(event, data){ source_map[data[:source]] << data[:association] }
   end
 
+
+  it "should warn about keyword arguments" do
+    expect(described_class.new(records: nil, associations: nil).foo({a: 1})).to eq({a: 1})
+  end
+
   context "for single table inheritance" do
     context "when preloading an aggregate for a child model" do
       let!(:contact_book) { ContactBook.create(name: "The Yellow Pages") }