Remove database from test/development configuration

The installer does not create development or test databases, so don't include them in the default configuration. Fixes: #216
rails · Sep 9, 2024 · fa9b129 · fa9b129
1 parent 94ea379
commit fa9b129
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 5 deletions.
diff --git a/lib/generators/solid_cache/install/templates/config/solid_cache.yml.tt b/lib/generators/solid_cache/install/templates/config/solid_cache.yml.tt
@@ -1,5 +1,4 @@
 default: &default
-  database: <%= ENV.fetch("DATABASE", "cache") %>
   store_options:
     # Cap age of oldest cache entry to fulfill retention policies
     # max_age: <%%= 60.days.to_i %>
@@ -13,4 +12,5 @@ test:
   <<: *default
 
 production:
+  database: <%= ENV.fetch("DATABASE", "cache") %>
   <<: *default
diff --git a/test/lib/generators/solid_cache/solid_cache/install_generator_test.rb b/test/lib/generators/solid_cache/solid_cache/install_generator_test.rb
@@ -136,7 +136,6 @@ class SolidCache::InstallGeneratorTest < Rails::Generators::TestCase
       def expected_solid_cache_config
         <<~YAML
           default: &default
-            database: cache
             store_options:
               # Cap age of oldest cache entry to fulfill retention policies
               # max_age: <%= 60.days.to_i %>
@@ -150,15 +149,13 @@ def expected_solid_cache_config
             <<: *default
 
           production:
+            database: cache
             <<: *default
         YAML
       end
 
       def copy_database_config_fixture_to_destination_root(database)
         FileUtils.cp(File.expand_path("../../../../configs/#{database}-database.yml", __dir__), Pathname.new(destination_root).join("config/database.yml"))
       end
-
-      def expected_mysql_database_config
-      end
   end
 end