Tweak check for schema cache serialization strategy

Katrina Owen · Katrina Owen · commit 4185614677a8 · 2020-02-24T14:56:07.000-07:00
This checks for the inclusion of .dump rather than ending
with dump for the schema cache filename, in order to switch
between Marshal (.dump) and YAML (default).

This is so that we can wrap gzipping around both strategies
cleanly.
diff --git a/activerecord/lib/active_record/connection_adapters/schema_cache.rb b/activerecord/lib/active_record/connection_adapters/schema_cache.rb
@@ -9,7 +9,7 @@ def self.load_from(filename)
         return unless File.file?(filename)
 
         file = File.read(filename)
-        filename.end_with?(".dump") ? Marshal.load(file) : YAML.load(file)
+        filename.include?(".dump") ? Marshal.load(file) : YAML.load(file)
       end
 
       attr_reader :version
@@ -144,7 +144,7 @@ def dump_to(filename)
         clear!
         connection.data_sources.each { |table| add(table) }
         File.atomic_write(filename) { |f|
-          if filename.end_with?(".dump")
+          if filename.include?(".dump")
             f.write(Marshal.dump(self))
           else
             f.write(YAML.dump(self))
