From 631dba452d538572f4475812c93f8b7b7b8a9bcc Mon Sep 17 00:00:00 2001 From: Jacopo Date: Wed, 15 Nov 2023 18:05:55 +0100 Subject: [PATCH] Stub also select_all In Rails 7.1 :select no longer depends on :exect_query but relies on :internal_exec_query instead. So let's stub directly :select_all to fix the tests. --- test/unit/solid_cache_test.rb | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/test/unit/solid_cache_test.rb b/test/unit/solid_cache_test.rb index bf7346b..818c4a8 100644 --- a/test/unit/solid_cache_test.rb +++ b/test/unit/solid_cache_test.rb @@ -57,10 +57,12 @@ def emulating_unavailability stub_matcher = ActiveRecord::Base.connection.class.any_instance stub_matcher.stubs(:exec_query).raises(ActiveRecord::StatementInvalid) stub_matcher.stubs(:exec_delete).raises(ActiveRecord::StatementInvalid) + stub_matcher.stubs(:select_all).raises(ActiveRecord::StatementInvalid) yield ActiveSupport::Cache::SolidCacheStore.new(namespace: @namespace) ensure stub_matcher.unstub(:exec_query) stub_matcher.unstub(:exec_delete) + stub_matcher.unstub(:select_all) end end @@ -82,10 +84,12 @@ def emulating_unavailability stub_matcher = ActiveRecord::Base.connection.class.any_instance stub_matcher.stubs(:exec_query).raises(ActiveRecord::StatementInvalid) stub_matcher.stubs(:exec_delete).raises(ActiveRecord::StatementInvalid) + stub_matcher.stubs(:select_all).raises(ActiveRecord::StatementInvalid) yield ActiveSupport::Cache::SolidCacheStore.new(namespace: @namespace, error_handler: ->(method:, returning:, exception:) { raise exception }) ensure stub_matcher.unstub(:exec_query) stub_matcher.unstub(:exec_delete) + stub_matcher.unstub(:select_all) end end