Comment out tests rather than use @ignore

Avoids triggering 'skipped tests' messages in test output

Author: markt-asf

test/org/apache/el/parser/TestELParserPerformance.java

@@ -29,7 +29,6 @@
 import jakarta.el.ValueExpression;
 
 import org.junit.Assert;
-import org.junit.Ignore;
 import org.junit.Test;
 
 import org.apache.tomcat.util.collections.SynchronizedStack;
@@ -99,9 +98,12 @@ public void testParserInstanceReuse() throws ParseException {
     /*
      * Ignored by default since this is an absolute test primarily for
      * https://bz.apache.org/bugzilla/show_bug.cgi?id=69338
+     *
+     * Comment out @Test rather than use @Ignore since @Ignore triggers a "skipped test" message in the test output
+     * and those messages should be reserved for tests skipped due to environmental (no APR, wrong Java version, etc)
+     * issues.
      */
-    @Ignore
-    @Test
+    //@Test
     public void testAstAnd() {
 
         ELManager manager = new ELManager();
@@ -145,9 +147,12 @@ public void testAstAnd() {
     /*
      * Ignored by default since this is an absolute test primarily for
      * https://bz.apache.org/bugzilla/show_bug.cgi?id=69338
+     *
+     * Comment out @Test rather than use @Ignore since @Ignore triggers a "skipped test" message in the test output
+     * and those messages should be reserved for tests skipped due to environmental (no APR, wrong Java version, etc)
+     * issues.
      */
-    @Ignore
-    @Test
+    //@Test
     public void testAstNotEmpty() {
 
         ELManager manager = new ELManager();
@@ -214,9 +219,12 @@ private Object runTest(Callable<Object> c, String description) throws Exception
      *
      * Ignored by default since this is an absolute test primarily for
      * https://bz.apache.org/bugzilla/show_bug.cgi?id=69381
+     *
+     * Comment out @Test rather than use @Ignore since @Ignore triggers a "skipped test" message in the test output
+     * and those messages should be reserved for tests skipped due to environmental (no APR, wrong Java version, etc)
+     * issues.
      */
-    @Ignore
-    @Test
+    //@Test
     public void testExpressions() throws Exception {
 
         ELManager manager = new ELManager();