An argument for keeping things as they are is it gives you an opportunity to write both developer and user (i.e. service developer) docs. Developer docs go in the provider class definition. User docs go in the @injection decorated method.

The impression of boilerplate is amplified when you're seeing a full test like this, and I really like the separation of documentation. The service developer shouldn't see or care how the injection is implemented.