Use Rollup in failing test

eclipse-buildship · Sep 25, 2023 · 35661c1 · 35661c1
1 parent 8339fa2
commit 35661c1
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 2 deletions.
diff --git a/buildSrc/src/main/groovy/eclipsebuild/testing/EclipseTestResultProcessor.java b/buildSrc/src/main/groovy/eclipsebuild/testing/EclipseTestResultProcessor.java
@@ -75,8 +75,6 @@ private void onTestRunEndedEvent(EclipseTestEvent.TestRunEnded event) {
     private static final Pattern ECLIPSE_TEST_NAME = Pattern.compile("(.*)\\((.*)\\)");
 
     private void onTestStartedEvent(EclipseTestEvent.TestStarted event) {
-        logger.info("Test started (" + event.getTestId() + "): " + event.getTestName());
-
         // TODO need idGenerator
         String testClass = event.getTestName();
         String testMethod = event.getTestName();
@@ -86,6 +84,9 @@ private void onTestStartedEvent(EclipseTestEvent.TestStarted event) {
             testMethod = matcher.group(1);
         }
 
+        logger.info("Test started (id=" + event.getTestId() + ", testName=" + event.getTestName() + ", class=" + testClass + ", method=" +  testMethod);
+
+
         String classId = getClassId(event);
         this.currentTestClass = testClass(classId, testClass, this.currentTestSuite);
         this.resultProcessor.started(this.currentTestClass, startEvent(this.currentTestSuite));

diff --git a/...roovy/org/eclipse/buildship/core/internal/configuration/WorkspaceConfigurationTest.groovy b/...roovy/org/eclipse/buildship/core/internal/configuration/WorkspaceConfigurationTest.groovy
@@ -10,6 +10,9 @@
 package org.eclipse.buildship.core.internal.configuration
 
 import org.eclipse.buildship.core.internal.test.fixtures.WorkspaceSpecification
+
+import spock.lang.Rollup
+
 import org.eclipse.buildship.core.GradleDistribution
 
 class WorkspaceConfigurationTest extends WorkspaceSpecification {
@@ -29,6 +32,7 @@ class WorkspaceConfigurationTest extends WorkspaceSpecification {
         configuration.experimentalModuleSupportEnabled == false
     }
 
+    @Rollup
     def "Can save workpsace configuration"(GradleDistribution distribution, String gradleUserHome, String javaHome, boolean offlineMode, boolean buildScansEnabled, boolean autoSync, List args, List jvmArgs, boolean showConsole, boolean showExecutions, moduleSupportEnabled) {
         setup:
         WorkspaceConfiguration orignalConfiguration = configurationManager.loadWorkspaceConfiguration()