Skip to content
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

Add support to specify java executable to use for test #1602

Merged
merged 3 commits into from
Sep 4, 2023

Conversation

gayanper
Copy link
Contributor

@gayanper gayanper commented Sep 2, 2023

This PR adds support for specifying a different JRE (by providing the java executable) to be used for running the test. This helps when some tests needs to be executed against a different java version to

  • Make sure certain application components are future proof.
  • Debug issues which has raised when application was running with a newer java version, Which a test is used to reproduce that error.
  • etc.

@jdneo jdneo self-requested a review September 4, 2023 02:10
@gayanper
Copy link
Contributor Author

gayanper commented Sep 4, 2023

Yes, this change was initiated for a scenario where I wanted to run some JDT debug tests with java21. This merge logic helps to resolve that. To be safe, i only merge keys which are not present in the config that passed in from PDE, but if we see a need we can extend to concat and merge existing attributes such as strings and arrays.

@jdneo jdneo added this to the 0.40.0 milestone Sep 4, 2023
@jdneo
Copy link
Member

jdneo commented Sep 4, 2023

Ok, makes sense.

Signed-off-by: Sheng Chen <sheche@microsoft.com>
Copy link
Member

@jdneo jdneo left a comment

Choose a reason for hiding this comment

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

LGTM

@jdneo jdneo merged commit ae72363 into microsoft:main Sep 4, 2023
@jdneo
Copy link
Member

jdneo commented Sep 4, 2023

Thank you @gayanper for the contribution.

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