Move RunCompletionModelTests at the begin of the model tests

Looks like either some tests mess up with environment or CompletionModelTests don't properly set expected environment. Let see if moving them at the begin confirms my observation. Note: seems the test suite execution order in Jenkins is opposite to tycho test definition oderr?
eclipse-jdt · Jun 24, 2024 · eaea3ad · eaea3ad
1 parent 3be7fa1
commit eaea3ad
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 5 deletions.
diff --git a/org.eclipse.jdt.core.tests.model/pom.xml b/org.eclipse.jdt.core.tests.model/pom.xml
@@ -34,9 +34,9 @@
       	<configuration>
       	  <argLine>-Xmx1G -Djdt.default.test.compliance=1.8 -DDetectVMInstallationsJob.disabled=true ${tycho.surefire.argLine}</argLine>
       	  <includes>
-      	  	<include>org/eclipse/jdt/core/tests/model/AllJavaModelTestsTracing.class</include>
-      	  	<include>org/eclipse/jdt/core/tests/dom/RunAllTestsTracing.class</include>
       	  	<include>org/eclipse/jdt/core/tests/RunFormatterTests.class</include>
+      	  	<include>org/eclipse/jdt/core/tests/dom/RunAllTestsTracing.class</include>
+      	  	<include>org/eclipse/jdt/core/tests/model/AllJavaModelTestsTracing.class</include>
       	  </includes>
          </configuration>
       </plugin>

diff --git a/org.eclipse.jdt.core.tests.model/src/org/eclipse/jdt/core/tests/model/AllJavaModelTests.java b/org.eclipse.jdt.core.tests.model/src/org/eclipse/jdt/core/tests/model/AllJavaModelTests.java
@@ -40,6 +40,9 @@ private static Class[] getAllTestClasses() {
 
 		// Enter each test here, grouping the tests that are related
 
+		// Support for completion tests
+		RunCompletionModelTests.class,
+
 		// Binding key tests
 		BindingKeyTests.class,
 
@@ -119,9 +122,6 @@ private static Class[] getAllTestClasses() {
 		ResolveTests21.class,
 		SelectionJavadocModelTests.class,
 
-		// Support for completion tests
-		RunCompletionModelTests.class,
-
 		// Prefix and suffix tests
 		NamingConventionTests.class,