This is less readable than before, but maybe we have to accept these changes to have the benefits of ruff. Renaming thei would help here though. Another option would be to turn of the performance rules for the tests (as test performance is typically not critical)

It's very much out-of-scope here, but generally speaking it's preferable to use parametric tests instead of putting assertions in a for-loop 🤷🏻

But even so, I agree that it wouldn't hurt to renamei to something more descriptive.

To tell the truth, therange(len()) →enumerate() change hasnot been suggested byruff. I added it for consistency with thedesired.keys() →desired.items() change a few lines before.

I am happy to revert therange(len)) →enumerate() change and disable thedesired.keys() →desired.items(), globally or locally.

Note that PERF rules are not necessarily about "performance" as such. In most cases, these micro-optimisations will have only (very) marginal effect on actual global performance. Rather, the idea of all these rules is to use consistent Python style.