Add more logging to tests, fix single test execution

See: #782

Author: trancexpress

org.eclipse.jdt.debug.tests/tests/org/eclipse/jdt/debug/tests/core/ModuleOptionsTests.java

@@ -18,13 +18,16 @@
 import java.util.List;
 
 import org.eclipse.core.runtime.CoreException;
+import org.eclipse.core.runtime.Status;
 import org.eclipse.debug.core.ILaunchConfiguration;
 import org.eclipse.jdt.core.IClasspathAttribute;
 import org.eclipse.jdt.core.IClasspathEntry;
 import org.eclipse.jdt.core.IJavaProject;
 import org.eclipse.jdt.core.JavaCore;
 import org.eclipse.jdt.core.JavaModelException;
 import org.eclipse.jdt.debug.tests.AbstractDebugTest;
+import org.eclipse.jdt.internal.ui.JavaPlugin;
+import org.eclipse.jdt.launching.IVMInstall;
 import org.eclipse.jdt.launching.JavaRuntime;
 
 public class ModuleOptionsTests extends AbstractDebugTest {
@@ -112,8 +115,9 @@ private int indexOfJREContainer(IClasspathEntry[] rawClasspath) {
 		return -1;
 	}
 
-	public void testAddModules1() throws JavaModelException {
+	public void testAddModules1() throws Exception {
 		IJavaProject javaProject = getProjectContext();
+		logVMInstall(javaProject);
 		List<String> defaultModules = getDefaultModules(javaProject);
 		defaultModules.add("jdk.crypto.cryptoki"); // requires jdk.crypto.ec up to Java 21
 		try {
@@ -135,8 +139,9 @@ public void testAddModules1() throws JavaModelException {
 		}
 	}
 
-	public void testLimitModules_release9() throws CoreException {
+	public void testLimitModules_release9() throws Exception {
 		IJavaProject javaProject = getProjectContext();
+		logVMInstall(javaProject);
 		try {
 			javaProject.setOption(JavaCore.COMPILER_RELEASE, JavaCore.ENABLED);
 			List<String> defaultModules = getDefaultModules(javaProject);
@@ -153,7 +158,7 @@ public void testLimitModules_release9() throws CoreException {
 							+ "jdk.net," //
 							+ "jdk.nio.mapmode," //
 							// + "jdk.packager,jdk.packager.services,jdk.plugin.dom,"
-							// + "jdk.scripting.nashorn," 
+									// + "jdk.scripting.nashorn,"
 							+ "jdk.sctp,"
 							+ "jdk.security.auth,jdk.security.jgss,jdk.unsupported," //
 							+ "jdk.unsupported.desktop,jdk.xml.dom";
@@ -178,8 +183,10 @@ public void testLimitModules_release9() throws CoreException {
 		}
 	}
 
-	public void testLimitModules1() throws JavaModelException {
+	public void testLimitModules1() throws Exception {
 		IJavaProject javaProject = getProjectContext();
+		javaProject.setOption(JavaCore.COMPILER_RELEASE, JavaCore.DISABLED);
+		logVMInstall(javaProject);
 		List<String> defaultModules = getDefaultModules(javaProject);
 		String expectedModules;
 		String moduleList = String.join(",", defaultModules);
@@ -218,4 +225,10 @@ public void testLimitModules1() throws JavaModelException {
 			removeClasspathAttributesFromSystemLibrary(javaProject);
 		}
 	}
+
+	private void logVMInstall(IJavaProject javaProject) throws CoreException {
+		IVMInstall vmInstall = JavaRuntime.getVMInstall(javaProject);
+		String msg = getName() + " project VM name: " + vmInstall.getName() + ", location: " + vmInstall.getInstallLocation();
+		JavaPlugin.log(Status.info(msg));
+	}
 }

org.eclipse.jdt.launching/launching/org/eclipse/jdt/internal/launching/DetectVMInstallationsJob.java

@@ -33,6 +33,7 @@
 import org.eclipse.core.runtime.preferences.DefaultScope;
 import org.eclipse.core.runtime.preferences.IEclipsePreferences;
 import org.eclipse.core.runtime.preferences.InstanceScope;
+import org.eclipse.jdt.internal.debug.core.JDIDebugPlugin;
 import org.eclipse.jdt.launching.IVMInstall;
 import org.eclipse.jdt.launching.IVMInstall2;
 import org.eclipse.jdt.launching.IVMInstallType;
@@ -320,16 +321,24 @@ public static void initialize() {
 			// early exit no need to read preferences or check env variable!
 			return;
 		}
-		if (System.getProperty(PROPERTY_DETECT_VM_INSTALLATIONS_JOB_DISABLED) == null && Boolean.parseBoolean(System.getenv(ENV_CI))) {
-			// exit because no explicit value for the property was given and we are running in a CI environment
-			return;
+		boolean ci = false;
+		if (System.getProperty(PROPERTY_DETECT_VM_INSTALLATIONS_JOB_DISABLED) == null) {
+			ci = Boolean.parseBoolean(System.getenv(ENV_CI));
+			if (ci) {
+				// exit because no explicit value for the property was given and we are running in a CI environment
+				return;
+			}
 		}
 		// finally look what is defined in the preferences
 		IEclipsePreferences instanceNode = InstanceScope.INSTANCE.getNode(LaunchingPlugin.getDefault().getBundle().getSymbolicName());
 		IEclipsePreferences defaultNode = DefaultScope.INSTANCE.getNode(LaunchingPlugin.getDefault().getBundle().getSymbolicName());
 		boolean defaultValue = defaultNode.getBoolean(LaunchingPlugin.PREF_DETECT_VMS_AT_STARTUP, true);
 		if (instanceNode.getBoolean(LaunchingPlugin.PREF_DETECT_VMS_AT_STARTUP, defaultValue)) {
 			new DetectVMInstallationsJob().schedule();
+			if (ci) {
+				// log that this job will run in a CI environment
+				JDIDebugPlugin.log(Status.info("DetectVMInstallationsJob scheduled")); //$NON-NLS-1$
+			}
 		}
 	}