Configures action mailer asset host (#853)

thoughtbot · Jul 30, 2017 · c2f02ae · c2f02ae
1 parent 3fbdc34
commit c2f02ae
Show file tree

Hide file tree

Showing 3 changed files with 18 additions and 0 deletions.
diff --git a/lib/suspenders/actions.rb b/lib/suspenders/actions.rb
@@ -14,6 +14,11 @@ def action_mailer_host(rails_env, host)
       configure_environment(rails_env, config)
     end
 
+    def action_mailer_asset_host(rails_env, host)
+      config = "config.action_mailer.asset_host = { host: #{host} }"
+      configure_environment(rails_env, config)
+    end
+
     def configure_application_file(config)
       inject_into_file(
         "config/application.rb",

diff --git a/lib/suspenders/app_builder.rb b/lib/suspenders/app_builder.rb
@@ -303,8 +303,14 @@ def configure_simple_form
 
     def configure_action_mailer
       action_mailer_host "development", %{"localhost:3000"}
+      action_mailer_asset_host "development", %{"localhost:3000"}
       action_mailer_host "test", %{"www.example.com"}
+      action_mailer_asset_host "test", %{"www.example.com"}
       action_mailer_host "production", %{ENV.fetch("APPLICATION_HOST")}
+      action_mailer_asset_host(
+        "production",
+        %{ENV.fetch("ASSET_HOST", ENV.fetch("APPLICATION_HOST"))},
+      )
     end
 
     def configure_active_job

diff --git a/spec/features/new_project_spec.rb b/spec/features/new_project_spec.rb
@@ -155,6 +155,13 @@
       to match(/^ +config.action_mailer.delivery_method = :file$/)
   end
 
+  it "sets action mailer default host and asset host" do
+    config_key = 'config\.action_mailer\.asset_host'
+    config_value =
+      '{ host: ENV\.fetch\("ASSET_HOST", ENV\.fetch\("APPLICATION_HOST"\)\) }'
+    expect(production_config).to match(/#{config_key} = #{config_value}/)
+  end
+
   it "uses APPLICATION_HOST, not HOST in the production config" do
     expect(production_config).to match(/"APPLICATION_HOST"/)
     expect(production_config).not_to match(/"HOST"/)