Skip to content

test: make tests fail when runtime classpath is not obtainable#5629

Merged
ndr-brt merged 1 commit into
mainfrom
meaningful-failure
Apr 3, 2026
Merged

test: make tests fail when runtime classpath is not obtainable#5629
ndr-brt merged 1 commit into
mainfrom
meaningful-failure

Conversation

@ndr-brt
Copy link
Copy Markdown
Member

@ndr-brt ndr-brt commented Apr 3, 2026

What this PR changes/adds

Add a meaningful failure in case of a failure in getting the classpath for a runtime

Why it does that

I spent some time debugging why I wasn't able to run tests from intellij fresh installation. spoiler, JAVA_HOME was not set :)

Further notes

List other areas of code that have changed but are not necessarily linked to the main feature. This could be method
signature changes, package declarations, bugs that were encountered and were fixed inline, etc.

Who will sponsor this feature?

Please @-mention the committer that will sponsor your feature.

Linked Issue(s)

Closes # <-- insert Issue number if one exists

Please be sure to take a look at the contributing guidelines and our etiquette for pull requests.

@ndr-brt ndr-brt requested a review from a team as a code owner April 3, 2026 13:06
@ndr-brt ndr-brt added the build Anything related to the CI/CD Pipeline on Github Actions label Apr 3, 2026
@ndr-brt ndr-brt merged commit d8a06ba into main Apr 3, 2026
15 of 20 checks passed
@ndr-brt ndr-brt deleted the meaningful-failure branch April 3, 2026 13:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

build Anything related to the CI/CD Pipeline on Github Actions

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants