Mocking a value object like this is odd. Why not simply construct the TimeValue? That's much simpler. I don't see what creating this mocks helps (it does not help decoupling, it does not help performance, it does not help maintainability...)