Don't check ActionMailer::Base.deliveries.empty?

Since tests are ran in semi-random order (good), it's impossible to assert
Accessing collaborators_controller requires login

The lack of the check wasn't a security hole since it would blow up on accessing
current_account.id.  We shouldn't rely on that though
Organizations controller requires login for access
Switch LifecycleMailer to use new style "deliver"