Skip to content

Set PYTHON_BASIC_REPL="1" for two new test#424

Merged
blink1073 merged 1 commit intoCalysto:mainfrom
ellert:basic-repl
Mar 27, 2026
Merged

Set PYTHON_BASIC_REPL="1" for two new test#424
blink1073 merged 1 commit intoCalysto:mainfrom
ellert:basic-repl

Conversation

@ellert
Copy link
Copy Markdown
Contributor

@ellert ellert commented Mar 27, 2026

Description

This PR fixes two test failures when python uses pyrepl (the default in Python 3.13 and later if the terminal has sufficient capabilites for it to be used.

ValueError: Stdin Requested but not stdin handler available

.../metakernel/replwrap.py:151: ValueError

Changes

Set PYTHON_BASIC_REPL="1" for two new test

Backwards-incompatible changes

None

Testing

Tests succeed with the proposed change.

AI usage

AI tools and models used: None

ValueError: Stdin Requested but not stdin handler available

.../metakernel/replwrap.py:151: ValueError
@codecov
Copy link
Copy Markdown

codecov bot commented Mar 27, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 90.30%. Comparing base (f3dc49f) to head (79f146d).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##             main     #424   +/-   ##
=======================================
  Coverage   90.30%   90.30%           
=======================================
  Files          51       51           
  Lines        2950     2950           
  Branches      414      414           
=======================================
  Hits         2664     2664           
  Misses        209      209           
  Partials       77       77           
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Copy link
Copy Markdown
Contributor

@blink1073 blink1073 left a comment

Choose a reason for hiding this comment

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

Thanks!

@blink1073 blink1073 merged commit 2da17fa into Calysto:main Mar 27, 2026
23 of 24 checks passed
@ellert ellert deleted the basic-repl branch March 27, 2026 14:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants