This isn't directly relevant to this PR, but the way `VumiRouterApi` deals with routers is rather wat. I feel the constructor should either take `router_type, router_key` or `router_object` and figure out the other. Passing in an optional router object to a method on an instance that already has router_key and router_type attributes is bizarre.