source-academy / py-slang / 30883195291

86%

Build Type

push

github

Committed by web-flow

Commit Message

Stepper: input function now working (#389)

* fixes #191

* linting

* Fix contraction-limit mismatch and add input() test coverage (#389 review)

evaluatePython() now takes the same stepLimit as getPythonSteps() and
runChunk passes PythonStepperRunnerPlugin's actual stepLimit into both
passes, so the REPL-value pass never reaches further into a program
than the Stepper-tab pass did — otherwise InputRecorder.replaying()
could run out of recorded answers for input() calls the second pass
reaches but the first never did.

Also adds the positive-path input() tests CodeRabbit flagged as
missing: a mocked requestInput resolving successfully, the prompt
surfacing via step output, arity rejection, aliasing, and
arity()/is_function() on the input builtin itself.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>

---------

Co-authored-by: Claude Sonnet 5 <noreply@anthropic.com>

Coverage Stats

4671 of 5851 branches covered (79.83%)

Branch coverage included in aggregate %.

47 of 58 new or added lines in 5 files covered. (81.03%)

1 existing line in 1 file now uncovered.

10400 of 11668 relevant lines covered (89.13%)

166686.68 hits per line