Skip to content

Add option to run coreclr tests using interpreter #116570

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

Conversation

janvorli
Copy link
Member

This change adds support for running tests using interpreter to the generated .cmd files of the tests. It also adds a command line option to the src/tests/run.{cmd|sh} that makes the test infrastructure to run the tests using the interpreter.

This change adds support for running tests using interpreter to the
generated .cmd files of the tests. It also adds a command line option to
the src/tests/run.{cmd|sh} that makes the test infrastructure to run the
tests using the interpreter.
@janvorli janvorli added this to the 10.0.0 milestone Jun 12, 2025
@janvorli janvorli requested a review from davidwrighton June 12, 2025 00:35
@janvorli janvorli self-assigned this Jun 12, 2025
@Copilot Copilot AI review requested due to automatic review settings June 12, 2025 00:35
Copy link
Contributor

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds support for running CoreCLR tests using the interpreter by enabling a new "--interpreter" flag throughout the test infrastructure.

  • Updated run.sh to print usage info and set the RunInterpreter environment variable.
  • Adjusted run.py to add a new command-line argument and set the environment variable accordingly.
  • Revised run.cmd and target files to propagate the interpreter flag into the test execution environment.

Reviewed Changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated no comments.

Show a summary per file
File Description
src/tests/run.sh Adds interpreter option in usage and flag processing.
src/tests/run.py Adds interpreter argument and environment variable.
src/tests/run.cmd Adds interpreter flag handling in command processing.
src/tests/Common/CLRTest.Execute.Batch.targets Sets DOTNET_Interpreter when RunInterpreter is defined.
src/tests/Common/CLRTest.Execute.Bash.targets Exports DOTNET_Interpreter when RunInterpreter is set.

Copy link
Contributor

Tagging subscribers to this area: @BrzVlad, @janvorli, @kg
See info in area-owners.md if you want to be subscribed.

@janvorli janvorli requested a review from jkoritzinsky June 12, 2025 15:27
@janvorli janvorli merged commit 8c22d9e into dotnet:main Jun 16, 2025
75 checks passed
@janvorli janvorli deleted the add-option-to-run-coreclr-tests-with-interpreter branch June 16, 2025 11:05
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