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.

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

Load ActionMailer only if it is also used. #66

Closed

Conversation

ream88
Copy link
Contributor

@ream88 ream88 commented Jul 26, 2012

The working version of #65.


I have Rails app which will never use ActionMailer, so I excluded ActionMailer at all:

require 'active_record/railtie'
require 'action_controller/railtie'
# require 'action_mailer/railtie'
# require 'active_resource/railtie'
# require 'rails/test_unit/railtie'
require 'sprockets/railtie'

This commit fixes specs failing with uninitialized constant ActionMailer::TestCase::Behavior::TestHelper (NameError).

@blowmage
Copy link
Collaborator

I am partial to using this commit from @terrcin's branch:

terrcin@b2d5233

@ream88
Copy link
Contributor Author

ream88 commented Jul 26, 2012

No problem, just a working solution is needed ;)

@blowmage blowmage closed this in a8ea722 Jul 27, 2012
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants