-
-
Notifications
You must be signed in to change notification settings - Fork 34.5k
gh-76785: Add More Tests to test_interpreters.test_api #117662
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. Weβll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
ericsnowcurrently
merged 28 commits into
python:main
from
ericsnowcurrently:more-interpreters-api-tests
Apr 11, 2024
Merged
Changes from 1 commit
Commits
Show all changes
28 commits
Select commit
Hold shift + click to select a range
bdd8d70
InterpreterError inherits from Exception.
ericsnowcurrently 940f24d
Add _PyInterpreterState_GetIDObject().
ericsnowcurrently 528c6e3
Add _PyXI_NewInterpreter() and _PyXI_EndInterpreter().
ericsnowcurrently 0bd2e6b
Add _testinternalcapi.next_interpreter_id().
ericsnowcurrently 76e32cd
Add _testinternalcapi.exec_interpreter().
ericsnowcurrently 9b7bdc4
Sketch out tests.
ericsnowcurrently fa28f9b
Flesh out the tests.
ericsnowcurrently 5e31f6e
Add PipeEnd.
ericsnowcurrently 9111a83
Refactor _captured_script().
ericsnowcurrently 0c24e5c
Finish the tests.
ericsnowcurrently 611fa31
Add missing tests.
ericsnowcurrently bdc09f9
Add more capture/exec helpers.
ericsnowcurrently be85ff5
Fill out the tests.
ericsnowcurrently 51f18f8
Adjust the tests.
ericsnowcurrently b1f96d2
Fix clean_up_interpreters().
ericsnowcurrently cd61643
Fix test_capi.test_misc.
ericsnowcurrently 7de523d
external/unmanaged -> from_capi
ericsnowcurrently 11d38ab
Add a missing decorator.
ericsnowcurrently 1cf31b6
Fix other tests.
ericsnowcurrently 4421169
Fix test_capi.
ericsnowcurrently c75a115
Add _interpreters.capture_exception().
ericsnowcurrently b00476d
Raise ExecutionFailed when possible.
ericsnowcurrently 6a82a33
Handle OSError in the _running() script.
ericsnowcurrently 11dae3d
Add PyInterpreterState._whence.
ericsnowcurrently 7c9a2b9
Fix up _testinternalcapi.
ericsnowcurrently e68654c
Add PyInterpreterState.initialized.
ericsnowcurrently 175080c
Add tests for whence.
ericsnowcurrently 9db5a2b
Set interp-_ready on the main interpreter.
ericsnowcurrently File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Flesh out the tests.
- Loading branch information
commit fa28f9b4be380c2208e214039a7017d3348270f0
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
FYI, this is a duplicate of the above subtest
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks like this is addressed in gh-117651. I'm fine with fixing it there since it's already done. π