The idea is to provide its own `ResultInterface` instance, not to change the data type returned by the factory. So I think it's correct.