[tomee-tck] branch tomee-10-with-ee-91 updated: Not sure we need endorsed anymore - TomEE 9 doesn't work with older versions of the JDK

[jgallimore]

This is an automated email from the ASF dual-hosted git repository.

jgallimore pushed a commit to branch tomee-10-with-ee-91
in repository https://gitbox.apache.org/repos/asf/tomee-tck.git


The following commit(s) were added to refs/heads/tomee-10-with-ee-91 by this 
push:
     new 60878c7  Not sure we need endorsed anymore - TomEE 9 doesn't work with 
older versions of the JDK
60878c7 is described below

commit 60878c7b0c969446739c207839afac46843ec8e0
Author: Jonathan Gallimore <j...@jrg.me.uk>
AuthorDate: Mon Aug 7 14:52:51 2023 +0100

    Not sure we need endorsed anymore - TomEE 9 doesn't work with older 
versions of the JDK
---
 runtests                                                   |  4 ++--
 .../script/openejb/tck/commands/JavaTestCommand.groovy     | 14 +++++++-------
 2 files changed, 9 insertions(+), 9 deletions(-)

diff --git a/runtests b/runtests
index b5025b2..cd95d8c 100755
--- a/runtests
+++ b/runtests
@@ -339,7 +339,7 @@ egrep '(javaee|jakarta).(ri|cts).home' ~/.m2/settings.xml | 
perl -pe 's,^ *|<[^>
 
 # Fire up Maven to do the real work
 if [ -z $nc ]; then
-exec mvn -V --file "$DIRNAME/pom.xml" \
+echo mvn -V --file "$DIRNAME/pom.xml" \
       --batch-mode \
       --errors \
       $CONFIG \
@@ -355,7 +355,7 @@ exec mvn -V --file "$DIRNAME/pom.xml" \
                -e 
"s/\(\[ERROR\].*\)/${BOLD}${TEXT_RED}\1${RESET_FORMATTING}/g" \
                -e "s/Tests run: \([^,]*\), Failures: \([^,]*\), Errors: 
\([^,]*\), Skipped: \([^,]*\)/${BOLD}${TEXT_GREEN}Tests run: 
\1${RESET_FORMATTING}, Failures: ${BOLD}${TEXT_RED}\2${RESET_FORMATTING}, 
Errors: ${BOLD}${TEXT_RED}\3${RESET_FORMATTING}, Skipped: 
${BOLD}${TEXT_YELLOW}\4${RESET_FORMATTING}/g"
 else
-exec mvn -V --file "$DIRNAME/pom.xml" \
+echo mvn -V --file "$DIRNAME/pom.xml" \
       --batch-mode \
       --errors \
       $CONFIG \
diff --git a/src/test/script/openejb/tck/commands/JavaTestCommand.groovy 
b/src/test/script/openejb/tck/commands/JavaTestCommand.groovy
index e76e744..998f4ab 100644
--- a/src/test/script/openejb/tck/commands/JavaTestCommand.groovy
+++ b/src/test/script/openejb/tck/commands/JavaTestCommand.groovy
@@ -253,13 +253,13 @@ class JavaTestCommand
 
 
                 // not sure about this ....
-                if (tckJavaHome == null || !new File(tckJavaHome as String, 
'jmods').exists()/*j9 doesnt support it*/) {
-                    sysproperty(key: "java.endorsed.dirs", file: 
"${javaeeRiHome}/lib/endorsed")
-                    sysproperty(key: "command.testExecute.endorsed.dir", 
value: "-Djava.endorsed.dirs=${javaeeCtsHome}/endorsedlib")
-                    sysproperty(key: 
"command.testExecuteEjbEmbed.endorsed.dir", value: 
"-Djava.endorsed.dirs=${openejbHome}/endorsed")
-
-                    containerJavaOpts += " -Djava.locale.providers=COMPAT"
-                }
+//                if (tckJavaHome == null || !new File(tckJavaHome as String, 
'jmods').exists()/*j9 doesnt support it*/) {
+//                    sysproperty(key: "java.endorsed.dirs", file: 
"${javaeeRiHome}/lib/endorsed")
+//                    sysproperty(key: "command.testExecute.endorsed.dir", 
value: "-Djava.endorsed.dirs=${javaeeCtsHome}/endorsedlib")
+//                    sysproperty(key: 
"command.testExecuteEjbEmbed.endorsed.dir", value: 
"-Djava.endorsed.dirs=${openejbHome}/endorsed")
+//
+//                    containerJavaOpts += " -Djava.locale.providers=COMPAT"
+//                }
 
                 // force memory on tasks because with JDK 8 it's computed with 
a bit too much
                 // containerJavaOpts += " -Xmx512m 
-Dtest.ejb.stateful.timeout.wait.seconds=60"