We need to take a look at how promotions are activated. In what case would a promotion be activated for a line item? How about for an order? Or a shipment? These are things we need to consider before we refactor anything to do with `Promotion#activate`. I don't think having one method that does three things is a good idea.