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

jaikiran pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/ant.git


The following commit(s) were added to refs/heads/master by this push:
     new 87781366f fix JavaTest.testResultPropertyZeroNoFork
87781366f is described below

commit 87781366f94a52820036091144210655beb69be1
Author: Jaikiran Pai <jaiki...@apache.org>
AuthorDate: Wed Aug 16 16:47:47 2023 +0530

    fix JavaTest.testResultPropertyZeroNoFork
---
 .../org/apache/tools/ant/taskdefs/JavaTest.java    | 22 +++++++++++-----------
 1 file changed, 11 insertions(+), 11 deletions(-)

diff --git a/src/tests/junit/org/apache/tools/ant/taskdefs/JavaTest.java 
b/src/tests/junit/org/apache/tools/ant/taskdefs/JavaTest.java
index 3faf3a6fa..a6482daa9 100644
--- a/src/tests/junit/org/apache/tools/ant/taskdefs/JavaTest.java
+++ b/src/tests/junit/org/apache/tools/ant/taskdefs/JavaTest.java
@@ -73,17 +73,16 @@ public class JavaTest {
 
     private boolean runFatalTests = false;
 
-    private static final boolean allowedToIssueSystemExit;
-    private static final String SKIP_MSG_CAUSE_SYSTEM_EXIT_USE =
+    private static final boolean allowedToUseSecurityManager;
+    private static final String SKIP_SEC_MGR_USAGE =
             "Skipping test on current Java version " + 
JavaEnvUtils.getJavaVersion()
-                    + " because test calls System.exit() in non-forked VM";
+                    + " because SecurityManager is no longer supported";
     static {
         final JavaVersion javaVersion = new JavaVersion();
         javaVersion.setAtMost("17");
-        // don't run tests which call System.exit() on a non-forked VM because
-        // Ant no longer sets a custom SecurityManager to prevent the VM exit
-        // for Java versions >= 18
-        allowedToIssueSystemExit = javaVersion.eval();
+        // don't run tests which require usage of SecurityManager.
+        // Ant no longer sets a custom SecurityManager for Java versions >= 18
+        allowedToUseSecurityManager = javaVersion.eval();
     }
 
     /**
@@ -222,14 +221,14 @@ public class JavaTest {
     @Test
     public void testRunFail() {
         assumeTrue("Fatal tests have not been set to run", runFatalTests);
-        assumeTrue(SKIP_MSG_CAUSE_SYSTEM_EXIT_USE, allowedToIssueSystemExit);
+        assumeTrue(SKIP_SEC_MGR_USAGE, allowedToUseSecurityManager);
         buildRule.executeTarget("testRunFail");
     }
 
     @Test
     public void testRunFailFoe() {
         assumeTrue("Fatal tests have not been set to run", runFatalTests);
-        assumeTrue(SKIP_MSG_CAUSE_SYSTEM_EXIT_USE, allowedToIssueSystemExit);
+        assumeTrue(SKIP_SEC_MGR_USAGE, allowedToUseSecurityManager);
         thrown.expect(BuildException.class);
         thrown.expectMessage("Java returned:");
         buildRule.executeTarget("testRunFailFoe");
@@ -282,20 +281,21 @@ public class JavaTest {
 
     @Test
     public void testResultPropertyZeroNoFork() {
+        assumeTrue(SKIP_SEC_MGR_USAGE, allowedToUseSecurityManager);
         buildRule.executeTarget("testResultPropertyZeroNoFork");
         assertEquals("0", buildRule.getProject().getProperty("exitcode"));
     }
 
     @Test
     public void testResultPropertyNonZeroNoFork() {
-        assumeTrue(SKIP_MSG_CAUSE_SYSTEM_EXIT_USE, allowedToIssueSystemExit);
+        assumeTrue(SKIP_SEC_MGR_USAGE, allowedToUseSecurityManager);
         buildRule.executeTarget("testResultPropertyNonZeroNoFork");
         assertEquals("-1", buildRule.getProject().getProperty("exitcode"));
      }
 
     @Test
     public void testRunFailWithFailOnError() {
-        assumeTrue(SKIP_MSG_CAUSE_SYSTEM_EXIT_USE, allowedToIssueSystemExit);
+        assumeTrue(SKIP_SEC_MGR_USAGE, allowedToUseSecurityManager);
         thrown.expect(BuildException.class);
         thrown.expectMessage("Java returned:");
         buildRule.executeTarget("testRunFailWithFailOnError");

Reply via email to