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

Already on GitHub? Sign in to your account

Add mailer specs #336

Merged
merged 3 commits into from Nov 14, 2012

Conversation

Projects
None yet
2 participants
Contributor

haines commented Nov 13, 2012

I added those mailer specs to confirm that we have #316 covered.

I also got rid of the System class since it was just serving as a namespace for a couple of methods, and these could just as well be defined on the main module... Draper.setup_action_controller(...)

And finally I let allows and denies accept strings as well as symbols for the method names - potentially useful for metaprogramming.

@steveklabnik steveklabnik added a commit that referenced this pull request Nov 14, 2012

@steveklabnik steveklabnik Merge pull request #336 from haines/mailer_specs
Add mailer specs
f206c41

@steveklabnik steveklabnik merged commit f206c41 into drapergem:master Nov 14, 2012

1 check passed

default The Travis build passed
Details
Owner

steveklabnik commented Nov 14, 2012

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment