format & lint – cg/tests/*.py

[jGaboardi]

xref #589

This PR formats & lints cg/tests/*.py, wrapping up the libpysa/cg module.

[jGaboardi]

@martinfleis How do we want to handle all these cases of declared variables not being used in tests? Shall I update with your _ = UNUSED suggestion?

[martinfleis]

Depends. If they're not doing anything, I'd remove the whole line. If they do but we are not interested in what they return, then assigning to _ is a common pattern. The issue is that for most of these I don't know if they modify some variable used later in some way or not.

[jGaboardi]

Depends. If they're not doing anything, I'd remove the whole line. If they do but we are not interested in what they return, then assigning to _ is a common pattern. The issue is that for most of these I don't know if they modify some variable used later in some way or not.

Yeah, seems like most are simply doing checks, so the _ would be appropriate. I'll see about all of them though.

[martinfleis]

Yeah, it could just be a check to ensure there's no error. Then assigning to _ should be fine for ruff?

[jGaboardi]

Yeah, it could just be a check to ensure there's no error. Then assigning to _ should be fine for ruff?

Yep, that's what I did for those and ruff liked it.

[jGaboardi]

This is good for review, by the way.