Python: Parse class bases like function arguments

[maxfischer2781]

This PR adjusts how class bases are parsed to match the parsing of function arguments.

Previously, any node with a value field was excluded to reject dict-splats as bases; this would also reject many other valid nodes, critically subscripts as used for typing (e.g. list[int]). Now, the undesired node types are explicitly rejected.

Closes #22298.

[bettykadondi12]

Good

[hvitved]

Hi. Thanks for your contribution; could you perhaps add some test cases as well?

[maxfischer2781]

Hi. Thanks for your contribution; could you perhaps add some test cases as well?

I can try. It's not really clear to me how the parser tests work. Do I see it correctly that in python/extractor/tests/parser/ I need to add a Python <test name>_new.py alongisde its parser output in <test name>_new.expected?

[hvitved]

Do I see it correctly that in python/extractor/tests/parser/ I need to add a Python <test name>_new.py alongisde its parser output in <test name>_new.expected?

Yes, that looks like the right place to add the tests (example: #17822).

[maxfischer2781]

@hvitved thanks for the pointers! Created, verified and added a test with expected output. This should cover just about all cases of expressions regularly used as class bases.

[hvitved]

Code change looks plausible to me, but I would like someone from @github/codeql-python to confirm. Thanks again for the contribution.