Merge pull request #49974 from fatkodima/fix-redis-write_multi-with-e…

…xpires_in

Fix `RedisCacheStore#write_multi` with `:expires_in`

(cherry picked from commit c36f877)

activesupport/CHANGELOG.md

@@ -1,3 +1,7 @@
+*   Fix `:expires_in` option for `RedisCacheStore#write_multi`.
+
+    *fatkodima*
+
 *   Fix deserialization of non-string "purpose" field in Message serializer
 
     *Jacopo Beschi*

activesupport/lib/active_support/cache/redis_cache_store.rb

@@ -386,7 +386,7 @@ def delete_multi_entries(entries, **_options)
         end
 
         # Nonstandard store provider API to write multiple values at once.
-        def write_multi_entries(entries, expires_in: nil, race_condition_ttl: nil, **options)
+        def write_multi_entries(entries, **options)
           return if entries.empty?
 
           failsafe :write_multi_entries do

activesupport/test/cache/behaviors/cache_store_behavior.rb

@@ -174,6 +174,15 @@ def test_write_multi_empty_hash
     assert @cache.write_multi({})
   end
 
+  def test_write_multi_expires_in
+    key = SecureRandom.uuid
+    @cache.write_multi({ key => 1 }, expires_in: 10)
+
+    travel(11.seconds) do
+      assert_nil @cache.read(key)
+    end
+  end
+
   def test_fetch_multi
     key = SecureRandom.uuid
     other_key = SecureRandom.uuid