Why do we need to mock this out? If the implementation of `add_metrics()` changes, all of this is wrong and it's far enough away that we could miss fixing it if the tests don't fail.