Make ActionMailer::Base more consistent in style

I thought it would be nice to have more consistency in style here. Linted with RuboCop using the defaults from The Ruby Style Guide. Some of these rules may be considered debatable (e.g. 'Use fail instead of raise' or 'Prefer find over detect'). I'd be happy to revert anything here if that's the case.