It would also be great to have the original rename test from the linked issue since something might potentially go wrong in-between findExplicitReferences and rename action (and also to expand the testset because the existing one is not really extensive :().
and rename action (and also to expand the testset because the existing one is not really extensive :().
yeap, there is some room to improve the test code here (e.g. using an idea similar to CodeContext in TweakTesting), previously we put the test code to a wrapper foo function, it is sufficient for most cases, but not for this patch as defining a template class in a function body is forbidden in C++. I think we can adress this afterwards.
Unit tests: fail. 62042 tests passed, 1 failed and 783 were skipped.