bpo-40334: Don't skip test_parser:test_trigger_memory_error by lysnikolaou · Pull Request #19744 · python/cpython

the-knights-who-say-ni

This test has been changed to always use the old parser, so no need
for it to be skipped.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

(Why doesn't pegen trigger? Because it goes up to 200?)

(Why doesn't pegen trigger? Because it goes up to 200?)

Yes, and that's because the tokenizer enforces that restriction. The old parser was throwing a MemoryError way before that.

lysnikolaou changed the title ~~bpo-40334: Don't skip test_parser:test_trigget_memory_error~~ bpo-40334: Don't skip test_parser:test_trigger_memory_error

Apr 27, 2020