50% of tests in `test__opcode` are not run

[AlexWaygood]

Bug report

Bug description:

Following 40f3f11, test__opcode.py has two subclasses of unittest.TestCase in the global namespace that are both called OpListTests. This means that all tests in the first class are silently skipped in CI, as the whole class definition is overridden by the second class at import time, before unittest even tries to load the tests inside the class.

If the TestCase subclasses are given different names, the tests in the first class fail, as they rely on a helper method that was removed in 40f3f11.

Cc. @iritkatriel as author of 40f3f11

CPython versions tested on:

CPython main branch

Operating systems tested on:

Windows

Linked PRs

• gh-110923: Don't silently skip tests in test__opcode.py #110926

[iritkatriel]

Yes. You assigned to yourself because you are working on a fix?

I'd rename the second class to StackEffectTests. Then fix the first one.

[AlexWaygood]

Yes. You assigned to yourself because you are working on a fix?

I already filed one, four minutes ago ;)

• gh-110923: Don't silently skip tests in test__opcode.py #110926