Description
Bug report
Bug description:
In this test:
cpython/Lib/test/test_pyrepl/test_pyrepl.py
Lines 1051 to 1063 in 605022a
The second and third assert is not rerunning the ImportParser
so it's not actually testing anything right now.
We need to add this before each call to assert:
parser = ImportParser(code)
actual = parser.parse()
Please don't pick up this issue, I'd like to reserve it for someone at the PyConUS sprints :)
CPython versions tested on:
CPython main branch
Operating systems tested on:
No response
Activity
jtemporal commentedon May 19, 2025
I'm on it 👋
tomasr8 commentedon May 19, 2025
@jtemporal Let me know if you need help, I'm in the other room (316) 🙂
pythongh-134214: Fix test case in pyrepl
StanFromIreland commentedon May 19, 2025
There is a sprint label for just that:-)
gh-134214: Fix test case in pyrepl (gh-134223)
pythongh-134214: Fix test case in pyrepl (pythongh-134223)
[3.14] gh-134214: Fix test case in pyrepl (gh-134223) (gh-134229)
tomasr8 commentedon May 19, 2025
I asked about it and was told to use the
easy
label, but maybe also adding thesprint
label wouldn't hurt?tomasr8 commentedon May 19, 2025
Thanks @jtemporal !!
2 remaining items