Propagate ts.home from Maven to Arquillian Tests in Glassfish Messaging Runner #1719
+3
−1
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.
Fixes Issue
When running messaging TCK in the glassfish runner, some tests reported this error:
Describe the change
The problem was in unresolved
ts.home
. This PR adds it to the test from Maven variable to the test system properties.The total amount of errors decreased from 1932 to 1494. Old summary:
vs after this change (including the previous clean up PR):
Additional context
Just run
mvn clean install
inplatform-tck/glassfish-runner/messaging-platform-tck
. I used Java 21.CC @alwin-joseph @anajosep @arjantijms @cesarhernandezgt @dblevins @m0mus @edbratt @gurunrao @jansupol @jgallimore @kazumura @kwsutter @LanceAndersen @bhatpmk @RohitKumarJain @shighbar @gthoman @brideck @OndroMih @dmatej
@starksm64 @scottmarlow