Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

handle asset_host #30

Open
wants to merge 6 commits into
base: master
Choose a base branch
from
Open

handle asset_host #30

Changes from 1 commit
Commits
Show all changes
6 commits
Select commit Hold shift + click to select a range
4107951
handle an asset_host on the mailer (as a string)
afeld Dec 29, 2014
2f6f230
handle asset_hosts that are Procs
afeld Dec 29, 2014
45696cc
update information around the asset_host
afeld Dec 29, 2014
dbdb132
include the asset_host logic in the Mailer
afeld Dec 29, 2014
d206c2c
extract asset_host specs
afeld Dec 29, 2014
b46da40
re-simplify mailer_spec
afeld Dec 29, 2014
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
re-simplify mailer_spec
  • Loading branch information
@afeld
afeld committed Dec 29, 2014
commit b46da40e642fb24051c75d06699cf9ba3d4e47bd
8 changes: 3 additions & 5 deletions spec/lib/roadie/rails/mailer_spec.rb
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -4,9 +4,11 @@
module Roadie
module Rails
describe Mailer do
base_mailer = Class.new do
some_mailer = Class.new do
cattr_accessor :asset_host

include Mailer

def initialize(email = nil)
@email = email
end
Expand All @@ -16,10 +18,6 @@ def mail(options = {})
end
end

some_mailer = Class.new(base_mailer) do
include Mailer
end

describe "#roadie_options" do
it "returns Rails' roadie config" do
allow(::Rails).to receive_message_chain(:application, :config, :roadie).and_return "roadie config"
Expand Down