In SERVER-51240, we removed the const qualifier from a number of function arguments in some of our unit tests - the clang-tidy rule that motivated this change is more of a check on the function declaration rather than the definitions - it is still ok, even meaningful because it communicates how the function argument is used, to retain some of the const qualifiers in the function definitions.