Stuck on an issue?

Lightrun Answers was designed to reduce the constant googling that comes with debugging 3rd party libraries. It collects links to all the places you might be looking at while hunting down a tough bug.

And, if you’re still stuck at the end, we’re happy to hop on a call to see how we can help out.

Tackle some low-hanging fruit in "test_grammar.py"

See original GitHub issue

The latest copy of the CPython grammar tests in test_grammar.py has several @skips and FIXMEs. Some of them seem easy to fix, e.g. some parser bugs or missing warnings that would be helpful, others are entire features. We should fix the easy ones and make sure there are tickets for the rest.

Issue Analytics

State:
Created 3 years ago
Comments:6 (1 by maintainers)

Top GitHub Comments

2reactions

scodercommented, Jul 12, 2020

Sorry, the “FIXME” in use_old_parser() is actually a FIXME from CPython (hopefully the only one), not from Cython. The test file is copied from the CPython test suite. Nothing to fix here.

Note that you need to debug Cython itself here, not the test that it compiles.

For debugging, I suggest copying the failing code out of the module and running it through Cython manually, rather than always compiling the huge file completely and trying to end up in the right spot to debug it.

2reactions

da-woodscommented, Jul 12, 2020

@zdutta Thinking about it more, I think that test_grammar.py is compiled in Cython rather than run as a regular Python module. I wouldn’t expect the debugger to work on it.

I think there are some methods to debug Cython code but I’ve never really tried using them myself (and I especially don’t know how to do them with the test-suite). Sorry!