From 59f5b2eabc2c7f27b650b9cbaf23dd7f02400cc6 Mon Sep 17 00:00:00 2001 From: U004458 Date: Wed, 5 Nov 2025 22:48:27 +0800 Subject: [PATCH] fix: Fix junit5 test suites not running they are not executed because the master suite AllTest is still in junit4 suite. the fix includes: - changing the AllTest.java to use the junit5 suite api - use SelectClasses instead of SelectPackages, to prevent the platform engine from discovering unrelated packages which causes errors in the build - suppress the warning PMD.JUnit5TestShouldBePackagePrivate because our tests are executed in test suites and must be public - fixed discovered failing tests --- .../tools/ddk/check/core/test/BugAig1314.java | 2 +- .../tools/ddk/check/core/test/BugDsl27.java | 2 +- .../check/core/test/CheckScopingTest.xtend | 2 +- .../check/test/core/CheckCoreTestSuite.java | 27 ++++++++++++++----- .../runtime/context/CheckContextTest.java | 2 +- .../registry/CheckExtensionPointTests.java | 2 +- .../core/validation/CheckValidatorTest.java | 2 +- .../label/CheckRuleLabelProviderTest.java | 16 +++++------ .../test/core/CheckRuntimeTestSuite.java | 17 +++++++----- .../CheckExecutionEnvironmentTestSuite.java | 10 ++++--- .../tests/CheckLibraryChecksTestSuite.java | 8 +++--- .../test/HelloWorldSampleTestSuite.java | 13 ++++++--- .../typesystem/AbstractTypeProviderTest.java | 2 +- .../BuiltInTypeModelAccessTest.java | 2 +- .../typesystem/ParameterListMatcherTest.java | 2 +- .../typesystem/test/TypeSystemTestSuite.java | 14 ++++++---- .../META-INF/MANIFEST.MF | 7 ++--- .../com/avaloq/tools/ddk/xtext/AllTests.java | 8 +++--- .../WorkbenchResolutionAdaptorRunTest.java | 2 +- .../WorkbenchResolutionAdaptorTest.java | 2 +- ...ourceNameTemplateVariableResolverTest.java | 2 +- ...impleEnumTemplateVariableResolverTest.java | 2 +- .../ddk/xtext/ui/test/XtextUiTestSuite.java | 17 +++++++++--- ddk-configuration/pmd/ruleset.xml | 1 + 24 files changed, 105 insertions(+), 59 deletions(-) diff --git a/com.avaloq.tools.ddk.check.core.test/src/com/avaloq/tools/ddk/check/core/test/BugAig1314.java b/com.avaloq.tools.ddk.check.core.test/src/com/avaloq/tools/ddk/check/core/test/BugAig1314.java index 31394387ee..be1c683a8c 100644 --- a/com.avaloq.tools.ddk.check.core.test/src/com/avaloq/tools/ddk/check/core/test/BugAig1314.java +++ b/com.avaloq.tools.ddk.check.core.test/src/com/avaloq/tools/ddk/check/core/test/BugAig1314.java @@ -42,7 +42,7 @@ @ExtendWith(InjectionExtension.class) @InjectWith(CheckInjectorProvider.class) @SuppressWarnings("nls") -class BugAig1314 { +public class BugAig1314 { /** Constructor of super class is protected... */ private static class TestScope extends CatalogFromExtensionPointScope { diff --git a/com.avaloq.tools.ddk.check.core.test/src/com/avaloq/tools/ddk/check/core/test/BugDsl27.java b/com.avaloq.tools.ddk.check.core.test/src/com/avaloq/tools/ddk/check/core/test/BugDsl27.java index 3d5e7973ce..dd1ea2081d 100644 --- a/com.avaloq.tools.ddk.check.core.test/src/com/avaloq/tools/ddk/check/core/test/BugDsl27.java +++ b/com.avaloq.tools.ddk.check.core.test/src/com/avaloq/tools/ddk/check/core/test/BugDsl27.java @@ -27,7 +27,7 @@ @InjectWith(CheckInjectorProvider.class) @ExtendWith(InjectionExtension.class) @SuppressWarnings("nls") -class BugDsl27 extends AbstractCheckGenerationTestCase { +public class BugDsl27 extends AbstractCheckGenerationTestCase { /** * Tests that our test source compiles fine. diff --git a/com.avaloq.tools.ddk.check.core.test/src/com/avaloq/tools/ddk/check/core/test/CheckScopingTest.xtend b/com.avaloq.tools.ddk.check.core.test/src/com/avaloq/tools/ddk/check/core/test/CheckScopingTest.xtend index 3c031610bb..fd8572269a 100644 --- a/com.avaloq.tools.ddk.check.core.test/src/com/avaloq/tools/ddk/check/core/test/CheckScopingTest.xtend +++ b/com.avaloq.tools.ddk.check.core.test/src/com/avaloq/tools/ddk/check/core/test/CheckScopingTest.xtend @@ -76,6 +76,6 @@ class CheckScopingTest extends AbstractCheckTestCase { def void testCheckDescriptionIsInferred() { initializeTestProject val check = util.getFirstInstanceOf(getModel("CommonChecks"), typeof(Check)) - assertEquals(check.description, "Referenced check cannot be resolved", "This check is javadoc-like commented.") + assertEquals("This check is javadoc-like commented.", check.description, "Referenced check cannot be resolved") } } diff --git a/com.avaloq.tools.ddk.check.core.test/src/com/avaloq/tools/ddk/check/test/core/CheckCoreTestSuite.java b/com.avaloq.tools.ddk.check.core.test/src/com/avaloq/tools/ddk/check/test/core/CheckCoreTestSuite.java index 79371b1229..91ee1b81ec 100644 --- a/com.avaloq.tools.ddk.check.core.test/src/com/avaloq/tools/ddk/check/test/core/CheckCoreTestSuite.java +++ b/com.avaloq.tools.ddk.check.core.test/src/com/avaloq/tools/ddk/check/test/core/CheckCoreTestSuite.java @@ -10,20 +10,35 @@ *******************************************************************************/ package com.avaloq.tools.ddk.check.test.core; -import org.junit.platform.suite.api.SelectPackages; +import org.junit.platform.suite.api.SelectClasses; import org.junit.platform.suite.api.Suite; +import com.avaloq.tools.ddk.check.core.generator.IssueCodeValueTest; +import com.avaloq.tools.ddk.check.core.test.BasicModelTest; +import com.avaloq.tools.ddk.check.core.test.BugAig1314; +import com.avaloq.tools.ddk.check.core.test.BugAig830; +import com.avaloq.tools.ddk.check.core.test.BugDsl27; +import com.avaloq.tools.ddk.check.core.test.CheckScopingTest; +import com.avaloq.tools.ddk.check.core.test.IssueCodeToLabelMapGenerationTest; +import com.avaloq.tools.ddk.check.core.test.ProjectBasedTests; +import com.avaloq.tools.ddk.check.formatting.CheckFormattingTest; + /** * Junit5 version of test suites. does not implement the logic in our DiscerningSuite. */ @Suite -@SelectPackages({ +@SelectClasses({ // @Format-Off - "com.avaloq.tools.ddk.check.core.generator", - "com.avaloq.tools.ddk.check.core.test", - "com.avaloq.tools.ddk.check.formatting", - "com.avaloq.tools.ddk.check.validation" + IssueCodeValueTest.class, + BasicModelTest.class, + BugAig830.class, + CheckScopingTest.class, + IssueCodeToLabelMapGenerationTest.class, + ProjectBasedTests.class, + BugAig1314.class, + BugDsl27.class, + CheckFormattingTest.class // @Format-On }) public class CheckCoreTestSuite { diff --git a/com.avaloq.tools.ddk.check.runtime.core.test/src/com/avaloq/tools/ddk/check/runtime/context/CheckContextTest.java b/com.avaloq.tools.ddk.check.runtime.core.test/src/com/avaloq/tools/ddk/check/runtime/context/CheckContextTest.java index 5a427fc88b..6fe8e08773 100644 --- a/com.avaloq.tools.ddk.check.runtime.core.test/src/com/avaloq/tools/ddk/check/runtime/context/CheckContextTest.java +++ b/com.avaloq.tools.ddk.check.runtime.core.test/src/com/avaloq/tools/ddk/check/runtime/context/CheckContextTest.java @@ -21,7 +21,7 @@ * Provides some tests of the reflective {@link AbstractCheckContext} framework. */ @SuppressWarnings("nls") -class CheckContextTest { +public class CheckContextTest { public static final String ENABLED_ISSUE_CODE = "Enabled.Issue.Code"; public static final String DISABLED_ISSUE_CODE = "Disabled.Issue.Code"; diff --git a/com.avaloq.tools.ddk.check.runtime.core.test/src/com/avaloq/tools/ddk/check/runtime/core/registry/CheckExtensionPointTests.java b/com.avaloq.tools.ddk.check.runtime.core.test/src/com/avaloq/tools/ddk/check/runtime/core/registry/CheckExtensionPointTests.java index 109474d0ab..be407539bf 100644 --- a/com.avaloq.tools.ddk.check.runtime.core.test/src/com/avaloq/tools/ddk/check/runtime/core/registry/CheckExtensionPointTests.java +++ b/com.avaloq.tools.ddk.check.runtime.core.test/src/com/avaloq/tools/ddk/check/runtime/core/registry/CheckExtensionPointTests.java @@ -30,7 +30,7 @@ * Provides some test cases for the Check extension point. */ @SuppressWarnings("nls") -class CheckExtensionPointTests { +public class CheckExtensionPointTests { private static final String DUMMY_EXTENSION_ID = "com.avaloq.tools.ddk.check.runtime.core.test"; private static final String CHECK_EXTENSION_ID = "com.avaloq.tools.ddk.check.runtime.core"; diff --git a/com.avaloq.tools.ddk.check.runtime.core.test/src/com/avaloq/tools/ddk/check/runtime/core/validation/CheckValidatorTest.java b/com.avaloq.tools.ddk.check.runtime.core.test/src/com/avaloq/tools/ddk/check/runtime/core/validation/CheckValidatorTest.java index a8e64e841b..bb60b5682e 100644 --- a/com.avaloq.tools.ddk.check.runtime.core.test/src/com/avaloq/tools/ddk/check/runtime/core/validation/CheckValidatorTest.java +++ b/com.avaloq.tools.ddk.check.runtime.core.test/src/com/avaloq/tools/ddk/check/runtime/core/validation/CheckValidatorTest.java @@ -24,7 +24,7 @@ * Performs some basic validation tests on the {@link AbstractCheckValidator}. */ @SuppressWarnings("nls") -class CheckValidatorTest extends AbstractCheckValidator { +public class CheckValidatorTest extends AbstractCheckValidator { /** * Represents the dummy language which is also registered in the plugin.xml. */ diff --git a/com.avaloq.tools.ddk.check.runtime.core.test/src/com/avaloq/tools/ddk/check/runtime/label/CheckRuleLabelProviderTest.java b/com.avaloq.tools.ddk.check.runtime.core.test/src/com/avaloq/tools/ddk/check/runtime/label/CheckRuleLabelProviderTest.java index ad6c313734..35c30852bd 100644 --- a/com.avaloq.tools.ddk.check.runtime.core.test/src/com/avaloq/tools/ddk/check/runtime/label/CheckRuleLabelProviderTest.java +++ b/com.avaloq.tools.ddk.check.runtime.core.test/src/com/avaloq/tools/ddk/check/runtime/label/CheckRuleLabelProviderTest.java @@ -40,7 +40,7 @@ * Unit test for {@link DefaultCheckRuleLabelProvider}. */ @SuppressWarnings("nls") -class CheckRuleLabelProviderTest { +public class CheckRuleLabelProviderTest { // Test data private static final int NUM_VALIDATORS = 3; @@ -144,7 +144,7 @@ void testSuccess() { final String label = checkRuleLabelProvider.getLabel(ISSUE_CODE); // ASSERT - assertEquals("Label should be " + LABEL + " but was " + label, LABEL, label); + assertEquals(LABEL, label, "Label should be " + LABEL + " but was " + label); } /** @@ -173,7 +173,7 @@ void testWithNullID() { final String label = checkRuleLabelProvider.getLabel(null); // ASSERT - assertNull("Label should be null but was " + label, label); + assertNull(label, "Label should be null but was " + label); } /** @@ -188,7 +188,7 @@ void testWithNoValidators() { final String label = checkRuleLabelProvider.getLabel(ISSUE_CODE); // ASSERT - assertNull("Label should be null but was " + label, label); + assertNull(label, "Label should be null but was " + label); } /** @@ -206,7 +206,7 @@ void testWithNoChecks() { final String label = checkRuleLabelProvider.getLabel(ISSUE_CODE); // ASSERT - assertNull("Label should be null but was " + label, label); + assertNull(label, "Label should be null but was " + label); } /** @@ -226,7 +226,7 @@ void testWhenIssueCodeIsNotACheckIssueCode() { final String label = checkRuleLabelProvider.getLabel(notACheckIssueCode); // ASSERT - assertNull("Label should be null but was " + label, label); + assertNull(label, "Label should be null but was " + label); } /** @@ -247,7 +247,7 @@ void testCaching() { verify(mockValidator, times(1)).getIssueCodeToLabelMap(); } - assertEquals(label1 + " not equal to " + label2 + " . Equality expected", label1, label2); + assertEquals(label1, label2, label1 + " not equal to " + label2 + " . Equality expected"); } /** @@ -284,7 +284,7 @@ void testInvalidatingCache() { verify(mockValidator, times(2)).getIssueCodeToLabelMap(); } - assertEquals(label1 + " not equal to " + label2 + " . Equality expected", label1, label2); + assertEquals(label1, label2, label1 + " not equal to " + label2 + " . Equality expected"); } /** diff --git a/com.avaloq.tools.ddk.check.runtime.core.test/src/com/avaloq/tools/ddk/check/runtime/test/core/CheckRuntimeTestSuite.java b/com.avaloq.tools.ddk.check.runtime.core.test/src/com/avaloq/tools/ddk/check/runtime/test/core/CheckRuntimeTestSuite.java index 5bfc0fe7e7..610aadb756 100644 --- a/com.avaloq.tools.ddk.check.runtime.core.test/src/com/avaloq/tools/ddk/check/runtime/test/core/CheckRuntimeTestSuite.java +++ b/com.avaloq.tools.ddk.check.runtime.core.test/src/com/avaloq/tools/ddk/check/runtime/test/core/CheckRuntimeTestSuite.java @@ -10,20 +10,25 @@ *******************************************************************************/ package com.avaloq.tools.ddk.check.runtime.test.core; -import org.junit.platform.suite.api.SelectPackages; +import org.junit.platform.suite.api.SelectClasses; import org.junit.platform.suite.api.Suite; +import com.avaloq.tools.ddk.check.runtime.context.CheckContextTest; +import com.avaloq.tools.ddk.check.runtime.core.registry.CheckExtensionPointTests; +import com.avaloq.tools.ddk.check.runtime.core.validation.CheckValidatorTest; +import com.avaloq.tools.ddk.check.runtime.label.CheckRuleLabelProviderTest; + /** * Junit5 version of test suites. does not implement the logic in our DiscerningSuite. */ @Suite -@SelectPackages({ +@SelectClasses({ // @Format-Off - "com.avaloq.tools.ddk.check.runtime.context", - "com.avaloq.tools.ddk.check.runtime.core.registry", - "com.avaloq.tools.ddk.check.runtime.core.validation", - "com.avaloq.tools.ddk.check.runtime.label" + CheckContextTest.class, + CheckExtensionPointTests.class, + CheckValidatorTest.class, + CheckRuleLabelProviderTest.class // @Format-On }) public class CheckRuntimeTestSuite { diff --git a/com.avaloq.tools.ddk.check.test.runtime.tests/src/com/avaloq/tools/ddk/check/test/runtime/tests/CheckExecutionEnvironmentTestSuite.java b/com.avaloq.tools.ddk.check.test.runtime.tests/src/com/avaloq/tools/ddk/check/test/runtime/tests/CheckExecutionEnvironmentTestSuite.java index 2d3fe6be7c..a218c4de52 100644 --- a/com.avaloq.tools.ddk.check.test.runtime.tests/src/com/avaloq/tools/ddk/check/test/runtime/tests/CheckExecutionEnvironmentTestSuite.java +++ b/com.avaloq.tools.ddk.check.test.runtime.tests/src/com/avaloq/tools/ddk/check/test/runtime/tests/CheckExecutionEnvironmentTestSuite.java @@ -10,17 +10,21 @@ *******************************************************************************/ package com.avaloq.tools.ddk.check.test.runtime.tests; -import org.junit.platform.suite.api.SelectPackages; +import org.junit.platform.suite.api.SelectClasses; import org.junit.platform.suite.api.Suite; +import com.avaloq.tools.ddk.check.test.runtime.CheckConfigurationIsAppliedTest; +import com.avaloq.tools.ddk.check.test.runtime.CheckExecutionEnvironmentProjectTest; + /** * Junit5 version of test suites. does not implement the logic in our DiscerningSuite. */ @Suite -@SelectPackages({ +@SelectClasses({ // @Format-Off - "com.avaloq.tools.ddk.check.test.runtime" + CheckConfigurationIsAppliedTest.class, + CheckExecutionEnvironmentProjectTest.class // @Format-On }) public class CheckExecutionEnvironmentTestSuite { diff --git a/com.avaloq.tools.ddk.check.test.runtime.tests/src/com/avaloq/tools/ddk/check/test/runtime/tests/CheckLibraryChecksTestSuite.java b/com.avaloq.tools.ddk.check.test.runtime.tests/src/com/avaloq/tools/ddk/check/test/runtime/tests/CheckLibraryChecksTestSuite.java index b7e0d44f71..abedf3c564 100644 --- a/com.avaloq.tools.ddk.check.test.runtime.tests/src/com/avaloq/tools/ddk/check/test/runtime/tests/CheckLibraryChecksTestSuite.java +++ b/com.avaloq.tools.ddk.check.test.runtime.tests/src/com/avaloq/tools/ddk/check/test/runtime/tests/CheckLibraryChecksTestSuite.java @@ -11,17 +11,19 @@ package com.avaloq.tools.ddk.check.test.runtime.tests; -import org.junit.platform.suite.api.SelectPackages; +import org.junit.platform.suite.api.SelectClasses; import org.junit.platform.suite.api.Suite; +import com.avaloq.tools.ddk.check.test.runtime.label.IssueLabelTest; + /** * Junit5 version of test suites. does not implement the logic in our DiscerningSuite. */ @Suite -@SelectPackages({ +@SelectClasses({ // @Format-Off - "com.avaloq.tools.ddk.check.test.runtime.label" + IssueLabelTest.class // @Format-On }) public class CheckLibraryChecksTestSuite { diff --git a/com.avaloq.tools.ddk.sample.helloworld.ui.test/src/com/avaloq/tools/ddk/sample/helloworld/test/HelloWorldSampleTestSuite.java b/com.avaloq.tools.ddk.sample.helloworld.ui.test/src/com/avaloq/tools/ddk/sample/helloworld/test/HelloWorldSampleTestSuite.java index 5a876deb9c..45a1fa6074 100644 --- a/com.avaloq.tools.ddk.sample.helloworld.ui.test/src/com/avaloq/tools/ddk/sample/helloworld/test/HelloWorldSampleTestSuite.java +++ b/com.avaloq.tools.ddk.sample.helloworld.ui.test/src/com/avaloq/tools/ddk/sample/helloworld/test/HelloWorldSampleTestSuite.java @@ -11,18 +11,23 @@ package com.avaloq.tools.ddk.sample.helloworld.test; -import org.junit.platform.suite.api.SelectPackages; +import org.junit.platform.suite.api.SelectClasses; import org.junit.platform.suite.api.Suite; +import com.avaloq.tools.ddk.sample.helloworld.check.CheckConfigurationIsAppliedTest; +import com.avaloq.tools.ddk.sample.helloworld.check.CheckExecutionEnvironmentProjectTest; +import com.avaloq.tools.ddk.sample.helloworld.label.IssueLabelTest; + /** * Junit5 version of test suites. does not implement the logic in our DiscerningSuite. */ @Suite -@SelectPackages({ +@SelectClasses({ // @Format-Off - "com.avaloq.tools.ddk.sample.helloworld.check", - "com.avaloq.tools.ddk.sample.helloworld.label" + CheckConfigurationIsAppliedTest.class, + CheckExecutionEnvironmentProjectTest.class, + IssueLabelTest.class // @Format-On }) diff --git a/com.avaloq.tools.ddk.typesystem.test/src/com/avaloq/tools/ddk/typesystem/AbstractTypeProviderTest.java b/com.avaloq.tools.ddk.typesystem.test/src/com/avaloq/tools/ddk/typesystem/AbstractTypeProviderTest.java index 02a2dc82c2..cf54677ec7 100644 --- a/com.avaloq.tools.ddk.typesystem.test/src/com/avaloq/tools/ddk/typesystem/AbstractTypeProviderTest.java +++ b/com.avaloq.tools.ddk.typesystem.test/src/com/avaloq/tools/ddk/typesystem/AbstractTypeProviderTest.java @@ -34,7 +34,7 @@ @SuppressWarnings("nls") -class AbstractTypeProviderTest { +public class AbstractTypeProviderTest { protected class TypeImpl extends EObjectImpl implements IType { } diff --git a/com.avaloq.tools.ddk.typesystem.test/src/com/avaloq/tools/ddk/typesystem/BuiltInTypeModelAccessTest.java b/com.avaloq.tools.ddk.typesystem.test/src/com/avaloq/tools/ddk/typesystem/BuiltInTypeModelAccessTest.java index d694f46be7..4653a8775b 100644 --- a/com.avaloq.tools.ddk.typesystem.test/src/com/avaloq/tools/ddk/typesystem/BuiltInTypeModelAccessTest.java +++ b/com.avaloq.tools.ddk.typesystem.test/src/com/avaloq/tools/ddk/typesystem/BuiltInTypeModelAccessTest.java @@ -28,7 +28,7 @@ * Tests that the BuiltInType model instance is correct and complete. */ @SuppressWarnings("nls") -class BuiltInTypeModelAccessTest { +public class BuiltInTypeModelAccessTest { @Test void testLoadModel() { diff --git a/com.avaloq.tools.ddk.typesystem.test/src/com/avaloq/tools/ddk/typesystem/ParameterListMatcherTest.java b/com.avaloq.tools.ddk.typesystem.test/src/com/avaloq/tools/ddk/typesystem/ParameterListMatcherTest.java index d252a16a75..b83e0509d3 100644 --- a/com.avaloq.tools.ddk.typesystem.test/src/com/avaloq/tools/ddk/typesystem/ParameterListMatcherTest.java +++ b/com.avaloq.tools.ddk.typesystem.test/src/com/avaloq/tools/ddk/typesystem/ParameterListMatcherTest.java @@ -37,7 +37,7 @@ // You can't have too many tests @SuppressWarnings({"PMD.ExcessivePublicCount", "nls"}) -class ParameterListMatcherTest { +public class ParameterListMatcherTest { private static final String WRONG_NUMBER_OF_UNNAMED_FORMALS_AFTER_NAMED_FORMALS = "wrong number of unnamed formals after named formals"; private static final String UNNAMED_FORMAL_AFTER_NAMED_NOT_LOCATED = "unnamed formal after named not located"; diff --git a/com.avaloq.tools.ddk.typesystem.test/src/com/avaloq/tools/ddk/typesystem/test/TypeSystemTestSuite.java b/com.avaloq.tools.ddk.typesystem.test/src/com/avaloq/tools/ddk/typesystem/test/TypeSystemTestSuite.java index cb9be41d96..82b3a5f262 100644 --- a/com.avaloq.tools.ddk.typesystem.test/src/com/avaloq/tools/ddk/typesystem/test/TypeSystemTestSuite.java +++ b/com.avaloq.tools.ddk.typesystem.test/src/com/avaloq/tools/ddk/typesystem/test/TypeSystemTestSuite.java @@ -10,19 +10,23 @@ *******************************************************************************/ package com.avaloq.tools.ddk.typesystem.test; -import org.junit.platform.suite.api.SelectPackages; +import org.junit.platform.suite.api.SelectClasses; import org.junit.platform.suite.api.Suite; +import com.avaloq.tools.ddk.typesystem.AbstractTypeProviderTest; +import com.avaloq.tools.ddk.typesystem.BuiltInTypeModelAccessTest; +import com.avaloq.tools.ddk.typesystem.ParameterListMatcherTest; + /** * Junit5 version of test suites. does not implement the logic in our DiscerningSuite. */ @Suite -@SelectPackages({ +@SelectClasses({ // @Format-Off - "com.avaloq.tools.ddk.typesystem.AbstractTypeProviderTest", - "com.avaloq.tools.ddk.typesystem.BuiltInTypeModelAccessTest", - "com.avaloq.tools.ddk.typesystem.ParameterListMatcherTest" + AbstractTypeProviderTest.class, + BuiltInTypeModelAccessTest.class, + ParameterListMatcherTest.class // @Format-On }) public class TypeSystemTestSuite { diff --git a/com.avaloq.tools.ddk.xtext.test/META-INF/MANIFEST.MF b/com.avaloq.tools.ddk.xtext.test/META-INF/MANIFEST.MF index 8976e67bbd..327ef000bf 100644 --- a/com.avaloq.tools.ddk.xtext.test/META-INF/MANIFEST.MF +++ b/com.avaloq.tools.ddk.xtext.test/META-INF/MANIFEST.MF @@ -6,13 +6,13 @@ Bundle-Version: 16.3.0.qualifier Bundle-Vendor: Avaloq Group AG Bundle-RequiredExecutionEnvironment: JavaSE-21 Bundle-ActivationPolicy: lazy -Require-Bundle: com.avaloq.tools.ddk.xtext, +Require-Bundle: com.avaloq.tools.ddk.xtext, com.avaloq.tools.ddk.xtext.test.core, org.eclipse.core.resources, com.avaloq.tools.ddk.xtext.generator, org.eclipse.emf.mwe2.launch;resolution:=optional, org.eclipse.xtext, - org.eclipse.xtext.xtext.generator, + org.eclipse.xtext.xtext.generator, org.eclipse.xtext.util, org.eclipse.xtext.ui, com.avaloq.tools.ddk.xtext.ui, @@ -29,7 +29,8 @@ Require-Bundle: com.avaloq.tools.ddk.xtext, org.eclipse.emf.ecore, junit-jupiter-api, junit-jupiter-engine, - junit-vintage-engine + junit-vintage-engine, + junit-platform-suite-api Import-Package: org.slf4j, org.apache.log4j, com.avaloq.tools.ddk.check.runtime.test.core, com.avaloq.tools.ddk.check.test.core, diff --git a/com.avaloq.tools.ddk.xtext.test/src/com/avaloq/tools/ddk/xtext/AllTests.java b/com.avaloq.tools.ddk.xtext.test/src/com/avaloq/tools/ddk/xtext/AllTests.java index d63fd3c8e1..4915382915 100644 --- a/com.avaloq.tools.ddk.xtext.test/src/com/avaloq/tools/ddk/xtext/AllTests.java +++ b/com.avaloq.tools.ddk.xtext.test/src/com/avaloq/tools/ddk/xtext/AllTests.java @@ -10,8 +10,8 @@ *******************************************************************************/ package com.avaloq.tools.ddk.xtext; -import org.junit.runner.RunWith; -import org.junit.runners.Suite; +import org.junit.platform.suite.api.SelectClasses; +import org.junit.platform.suite.api.Suite; import com.avaloq.tools.ddk.check.runtime.test.core.CheckRuntimeTestSuite; import com.avaloq.tools.ddk.check.test.core.CheckCoreTestSuite; @@ -34,8 +34,8 @@ // CHECKSTYLE:OFF HideUtilityClassConstructor // @Format-Off -@RunWith(Suite.class) -@Suite.SuiteClasses({ +@Suite +@SelectClasses({ XtextTestSuite.class, XtextUiTestSuite.class, GeneratorTestSuite.class, diff --git a/com.avaloq.tools.ddk.xtext.ui.test/src/com/avaloq/tools/ddk/xtext/ui/quickfix/WorkbenchResolutionAdaptorRunTest.java b/com.avaloq.tools.ddk.xtext.ui.test/src/com/avaloq/tools/ddk/xtext/ui/quickfix/WorkbenchResolutionAdaptorRunTest.java index 0cab33eabd..f49cd0e81e 100644 --- a/com.avaloq.tools.ddk.xtext.ui.test/src/com/avaloq/tools/ddk/xtext/ui/quickfix/WorkbenchResolutionAdaptorRunTest.java +++ b/com.avaloq.tools.ddk.xtext.ui.test/src/com/avaloq/tools/ddk/xtext/ui/quickfix/WorkbenchResolutionAdaptorRunTest.java @@ -56,7 +56,7 @@ import com.google.inject.name.Names; -class WorkbenchResolutionAdaptorRunTest { +public class WorkbenchResolutionAdaptorRunTest { private static final String TEST_FILE_NAME = "TestFileName"; //$NON-NLS-1$ diff --git a/com.avaloq.tools.ddk.xtext.ui.test/src/com/avaloq/tools/ddk/xtext/ui/quickfix/WorkbenchResolutionAdaptorTest.java b/com.avaloq.tools.ddk.xtext.ui.test/src/com/avaloq/tools/ddk/xtext/ui/quickfix/WorkbenchResolutionAdaptorTest.java index 074bf7eae2..eff10ff59f 100644 --- a/com.avaloq.tools.ddk.xtext.ui.test/src/com/avaloq/tools/ddk/xtext/ui/quickfix/WorkbenchResolutionAdaptorTest.java +++ b/com.avaloq.tools.ddk.xtext.ui.test/src/com/avaloq/tools/ddk/xtext/ui/quickfix/WorkbenchResolutionAdaptorTest.java @@ -27,7 +27,7 @@ import com.avaloq.tools.ddk.xtext.ui.quickfix.WorkbenchMarkerResolutionGenerator.WorkbenchResolutionAdapter; -class WorkbenchResolutionAdaptorTest { +public class WorkbenchResolutionAdaptorTest { private static final String TEST_LABEL = "ATestLabel"; //$NON-NLS-1$ private static final String TEST_DESCRIPTION = "ATestDescription"; //$NON-NLS-1$ diff --git a/com.avaloq.tools.ddk.xtext.ui.test/src/com/avaloq/tools/ddk/xtext/ui/templates/ResourceNameTemplateVariableResolverTest.java b/com.avaloq.tools.ddk.xtext.ui.test/src/com/avaloq/tools/ddk/xtext/ui/templates/ResourceNameTemplateVariableResolverTest.java index fee1706fab..22aced3f95 100644 --- a/com.avaloq.tools.ddk.xtext.ui.test/src/com/avaloq/tools/ddk/xtext/ui/templates/ResourceNameTemplateVariableResolverTest.java +++ b/com.avaloq.tools.ddk.xtext.ui.test/src/com/avaloq/tools/ddk/xtext/ui/templates/ResourceNameTemplateVariableResolverTest.java @@ -34,7 +34,7 @@ @ExtendWith(InjectionExtension.class) @TestInstance(TestInstance.Lifecycle.PER_CLASS) -class ResourceNameTemplateVariableResolverTest { +public class ResourceNameTemplateVariableResolverTest { private static final Object[] FILE = new Object[] {"file"}; //$NON-NLS-1$ private static final String FILENAME = "filename"; //$NON-NLS-1$ diff --git a/com.avaloq.tools.ddk.xtext.ui.test/src/com/avaloq/tools/ddk/xtext/ui/templates/SimpleEnumTemplateVariableResolverTest.java b/com.avaloq.tools.ddk.xtext.ui.test/src/com/avaloq/tools/ddk/xtext/ui/templates/SimpleEnumTemplateVariableResolverTest.java index c37a88e1b7..6489c544fe 100644 --- a/com.avaloq.tools.ddk.xtext.ui.test/src/com/avaloq/tools/ddk/xtext/ui/templates/SimpleEnumTemplateVariableResolverTest.java +++ b/com.avaloq.tools.ddk.xtext.ui.test/src/com/avaloq/tools/ddk/xtext/ui/templates/SimpleEnumTemplateVariableResolverTest.java @@ -33,7 +33,7 @@ @ExtendWith(InjectionExtension.class) @TestInstance(TestInstance.Lifecycle.PER_CLASS) -class SimpleEnumTemplateVariableResolverTest { +public class SimpleEnumTemplateVariableResolverTest { private static XtextTemplateContext mockContext; private static TemplateVariableResolverTestHelper helper; diff --git a/com.avaloq.tools.ddk.xtext.ui.test/src/com/avaloq/tools/ddk/xtext/ui/test/XtextUiTestSuite.java b/com.avaloq.tools.ddk.xtext.ui.test/src/com/avaloq/tools/ddk/xtext/ui/test/XtextUiTestSuite.java index 3295293667..76fcf67a18 100644 --- a/com.avaloq.tools.ddk.xtext.ui.test/src/com/avaloq/tools/ddk/xtext/ui/test/XtextUiTestSuite.java +++ b/com.avaloq.tools.ddk.xtext.ui.test/src/com/avaloq/tools/ddk/xtext/ui/test/XtextUiTestSuite.java @@ -11,18 +11,27 @@ package com.avaloq.tools.ddk.xtext.ui.test; -import org.junit.platform.suite.api.SelectPackages; +import org.junit.platform.suite.api.SelectClasses; import org.junit.platform.suite.api.Suite; +import com.avaloq.tools.ddk.xtext.ui.quickfix.WorkbenchResolutionAdaptorRunTest; +import com.avaloq.tools.ddk.xtext.ui.quickfix.WorkbenchResolutionAdaptorTest; +import com.avaloq.tools.ddk.xtext.ui.templates.ResourceNameTemplateVariableResolverTest; +import com.avaloq.tools.ddk.xtext.ui.templates.SimpleEnumTemplateVariableResolverTest; +import com.avaloq.tools.ddk.xtext.ui.templates.TemplateProposalProviderHelperTest; + /** * Junit5 version of test suites. does not implement the logic in our DiscerningSuite. */ @Suite -@SelectPackages({ +@SelectClasses({ // @Format-Off - "com.avaloq.tools.ddk.xtext.ui.quickfix", - "com.avaloq.tools.ddk.xtext.ui.templates" + WorkbenchResolutionAdaptorRunTest.class, + WorkbenchResolutionAdaptorTest.class, + TemplateProposalProviderHelperTest.class, + ResourceNameTemplateVariableResolverTest.class, + SimpleEnumTemplateVariableResolverTest.class // @Format-On }) public class XtextUiTestSuite { diff --git a/ddk-configuration/pmd/ruleset.xml b/ddk-configuration/pmd/ruleset.xml index dba9be4e2e..5ae4cbd431 100644 --- a/ddk-configuration/pmd/ruleset.xml +++ b/ddk-configuration/pmd/ruleset.xml @@ -88,6 +88,7 @@ +