Do you need to overload this with a version that doesn't take Map, to be consistent with what was there before? If all tests + integration tests pass, then I guess it wasn't being called from any groovy so doesn't matter.