It seems that you changed more than just this function. You might want to uncomment all the functions that you modified to be sure that there is not other problem

@glemaitre I thought that when I changed this file, it was only related to one function. But, now I see it's all those functions in one file. But, I didn't run the pytest on those other functions, I only fixed putting backticks on the parameters.
Also, sometimes I don't know where the error is (what line number), so I believe what I do is make the change in a function until the error goes away.

I can work on the other 6 functions within validation, but in separate PRs, because I just ran pytest against the others and there are still errors which need to be fixed separate for each of them.

OK fine with me. The thing is that you could have delayed the change with the backtick for the other PRs as well. Like this, the scope of the changes is really limited to a single place.

I will keep that in mind for the future. :)

To avoid introducing confusing unnecessary merge conflicts with other concurrent PRs by first time contributors also working on #21350, I would rather have PRs only focused on a specific function at a time.

@reshamas I guess what @ogrisel is asking, is to revert the changes which are not related to the function you're directly addressing in this PR, and to open a separate PR for them, which I agree with, especially if it helps with fewer merge conflicts with other PRs.

Could you elaborate what the output you see is, so that we could maybe help you find the relevant places to fix?