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

markt pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/tomcat.git


The following commit(s) were added to refs/heads/main by this push:
     new 5d49fc9  Remove use of JreCompat as it is not required with Java 11+
5d49fc9 is described below

commit 5d49fc9ca4e9ce757687051e92c2a6d053744dc7
Author: Mark Thomas <ma...@apache.org>
AuthorDate: Wed Jul 28 13:59:33 2021 +0100

    Remove use of JreCompat as it is not required with Java 11+
---
 test/jakarta/el/TestImportHandlerStandardPackages.java | 14 +++-----------
 1 file changed, 3 insertions(+), 11 deletions(-)

diff --git a/test/jakarta/el/TestImportHandlerStandardPackages.java 
b/test/jakarta/el/TestImportHandlerStandardPackages.java
index a895030..24d90e8 100644
--- a/test/jakarta/el/TestImportHandlerStandardPackages.java
+++ b/test/jakarta/el/TestImportHandlerStandardPackages.java
@@ -30,8 +30,6 @@ import java.util.stream.Stream;
 import org.junit.Assert;
 import org.junit.Test;
 
-import org.apache.tomcat.util.compat.JreCompat;
-
 public class TestImportHandlerStandardPackages {
 
     @Test
@@ -54,16 +52,10 @@ public class TestImportHandlerStandardPackages {
     private void checkPackageClassList(String packageName, Set<String> 
classNames) throws Exception {
 
         if ("java.lang".equals(packageName)) {
-            // The code below is designed to run on Java 9 so skip this check
-            // if running on Java 8. The test has previously been run with Java
-            // 9 (and later) so it is not necessary that this is executed on
-            // every test run. The intention is that it will catch new classes
-            // when the tests are run on a newer JRE.
+            // The intention is that this test will catch new classes when the
+            // tests are run on a newer JRE.
             // The latest version of the JRE where this test is known to pass 
is
-            // - OpenJDK 14 EA 27
-            if (!JreCompat.isJre9Available()) {
-                return;
-            }
+            // - OpenJDK 17 EA 28
             getJavaBaseClasses().filter(c -> (c.startsWith("java/lang/")))
                     .filter(c -> c.lastIndexOf('/') == 9)             // 
Exclude sub-packages
                     .filter(c -> c.endsWith(".class"))                // 
Exclude non-class resources

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org

Reply via email to