I think to expect calling `subject` to perform an action is wrong because RSpec will memoize it, we should only be relying on it's return value (i.e. the "object under test", rather than the "method call under test").

Also, won't `subject { subject.suppress email }` not recurse forever?