diff --git a/com.avaloq.tools.ddk.xtext.ui.test/META-INF/MANIFEST.MF b/com.avaloq.tools.ddk.xtext.ui.test/META-INF/MANIFEST.MF index 3243a56d42..498b8f61d9 100644 --- a/com.avaloq.tools.ddk.xtext.ui.test/META-INF/MANIFEST.MF +++ b/com.avaloq.tools.ddk.xtext.ui.test/META-INF/MANIFEST.MF @@ -8,7 +8,6 @@ Bundle-RequiredExecutionEnvironment: JavaSE-21 Require-Bundle: org.eclipse.xtext.ui, org.eclipse.xtext.xbase.lib, org.eclipse.xtext.testing, - org.junit, org.mockito.mockito-core, com.avaloq.tools.ddk.xtext.ui, com.avaloq.tools.ddk.test.core, @@ -16,6 +15,8 @@ Require-Bundle: org.eclipse.xtext.ui, com.avaloq.tools.ddk.xtext.test.core, junit-jupiter-api, junit-jupiter-engine, - junit-vintage-engine + net.bytebuddy.byte-buddy, + org.objenesis, + junit-platform-suite-api Export-Package: com.avaloq.tools.ddk.xtext.ui.test Automatic-Module-Name: com.avaloq.tools.ddk.xtext.ui.test 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 96536c2d70..0cab33eabd 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 @@ -43,8 +43,8 @@ import org.eclipse.xtext.util.Pair; import org.eclipse.xtext.util.Tuples; import org.eclipse.xtext.validation.Issue; -import org.junit.Before; -import org.junit.Test; +import org.junit.jupiter.api.BeforeEach; +import org.junit.jupiter.api.Test; import org.mockito.ArgumentMatchers; import com.avaloq.tools.ddk.check.runtime.ui.quickfix.IModificationContextRegistry; @@ -56,7 +56,7 @@ import com.google.inject.name.Names; -public class WorkbenchResolutionAdaptorRunTest { +class WorkbenchResolutionAdaptorRunTest { private static final String TEST_FILE_NAME = "TestFileName"; //$NON-NLS-1$ @@ -95,8 +95,8 @@ protected void configure() { private final WorkbenchMarkerResolutionGenerator wmrg = injector.getInstance(WorkbenchMarkerResolutionGenerator.class); - @Before - public void setUp() throws Exception { + @BeforeEach + void setUp() { wmrg.setIssueUtil(new IssueUtil()); when(mockRegistryProvider.get()).thenReturn(mockMarkerHelpRegistry); @@ -121,14 +121,13 @@ private void mockMarkerResource(final URI uri) throws CoreException { when(mockMarker.getAttribute(eq(Issue.URI_KEY), anyString())).thenReturn(uri.toString()); when(mockMarker.isSubtypeOf(eq(MarkerTypes.ANY_VALIDATION))).thenReturn(true); when(mockStorage2UriMapper.getUri(eq(mockFile))).thenReturn(uri); - @SuppressWarnings("unchecked") Iterable> storages = Lists.newArrayList(Tuples.create((IStorage) mockFile, mock(IProject.class))); when(mockStorage2UriMapper.getStorages(eq(uri))).thenReturn(storages); when(mockLanguageResourceHelper.isLanguageResource(eq(mockFile))).thenReturn(true); } @Test - public void testRun() throws CoreException { + void testRun() throws CoreException { WorkbenchResolutionAdapter adapter = wmrg.new WorkbenchResolutionAdapter(mockIssueResolution, mockMarker); when(mockMarkerHelpRegistry.getResolutions(mockMarker)).thenReturn(mockMarkerResolutions); @@ -142,7 +141,7 @@ public void testRun() throws CoreException { } @Test - public void testSingleRun() throws CoreException { + void testSingleRun() throws CoreException { WorkbenchResolutionAdapter adapter = wmrg.new WorkbenchResolutionAdapter(mockIssueResolution, mockMarker); when(mockMarkerHelpRegistry.getResolutions(mockMarker)).thenReturn(mockMarkerResolutions); 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 4f30b927c7..074bf7eae2 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 @@ -11,8 +11,8 @@ package com.avaloq.tools.ddk.xtext.ui.quickfix; -import static org.junit.Assert.assertArrayEquals; -import static org.junit.Assert.assertEquals; +import static org.junit.jupiter.api.Assertions.assertArrayEquals; +import static org.junit.jupiter.api.Assertions.assertEquals; import static org.mockito.Mockito.mock; import static org.mockito.Mockito.when; @@ -21,13 +21,13 @@ import org.eclipse.xtext.ui.editor.quickfix.IssueResolution; import org.eclipse.xtext.ui.util.IssueUtil; import org.eclipse.xtext.validation.Issue; -import org.junit.Before; -import org.junit.Test; +import org.junit.jupiter.api.BeforeEach; +import org.junit.jupiter.api.Test; import com.avaloq.tools.ddk.xtext.ui.quickfix.WorkbenchMarkerResolutionGenerator.WorkbenchResolutionAdapter; -public class WorkbenchResolutionAdaptorTest { +class WorkbenchResolutionAdaptorTest { private static final String TEST_LABEL = "ATestLabel"; //$NON-NLS-1$ private static final String TEST_DESCRIPTION = "ATestDescription"; //$NON-NLS-1$ @@ -37,13 +37,13 @@ public class WorkbenchResolutionAdaptorTest { private final WorkbenchMarkerResolutionGenerator mockWmrg = mock(WorkbenchMarkerResolutionGenerator.class); - @Before - public void setUp() throws Exception { + @BeforeEach + void setUp() { when(mockWmrg.getIssueUtil()).thenReturn(new IssueUtil()); } @Test - public void testGetLabel() { + void testGetLabel() { IssueResolution mockIssueResolution = mock(IssueResolution.class); IMarker mockMarker = mock(IMarker.class); @@ -51,11 +51,11 @@ public void testGetLabel() { WorkbenchResolutionAdapter adapter = mockWmrg.new WorkbenchResolutionAdapter(mockIssueResolution, mockMarker); - assertEquals("Adapter delegates get label to resolution.", TEST_LABEL, adapter.getLabel()); //$NON-NLS-1$ + assertEquals(TEST_LABEL, adapter.getLabel(), "Adapter delegates get label to resolution."); //$NON-NLS-1$ } @Test - public void testGetDescription() { + void testGetDescription() { IssueResolution mockIssueResolution = mock(IssueResolution.class); IMarker mockMarker = mock(IMarker.class); @@ -63,11 +63,11 @@ public void testGetDescription() { WorkbenchResolutionAdapter adapter = mockWmrg.new WorkbenchResolutionAdapter(mockIssueResolution, mockMarker); - assertEquals("Adapter delegates get description to resolution.", TEST_DESCRIPTION, adapter.getDescription()); //$NON-NLS-1$ + assertEquals(TEST_DESCRIPTION, adapter.getDescription(), "Adapter delegates get description to resolution."); //$NON-NLS-1$ } @Test - public void testGetImage() { + void testGetImage() { IssueResolution mockIssueResolution = mock(IssueResolution.class); IMarker mockMarker = mock(IMarker.class); @@ -75,11 +75,11 @@ public void testGetImage() { WorkbenchResolutionAdapter adapter = mockWmrg.new WorkbenchResolutionAdapter(mockIssueResolution, mockMarker); - assertEquals("Adapter delegates get Image to resolution.", TEST_IMAGE, adapter.getImage()); //$NON-NLS-1$ + assertEquals(TEST_IMAGE, adapter.getImage(), "Adapter delegates get Image to resolution."); //$NON-NLS-1$ } @Test - public void findOtherMarkers() { + void findOtherMarkers() { IssueResolution mockIssueResolution = mock(IssueResolution.class); IMarker mockResolutionMarker = mock(IMarker.class); when(mockResolutionMarker.getAttribute(Issue.CODE_KEY, null)).thenReturn(CODE1); @@ -95,14 +95,14 @@ public void findOtherMarkers() { WorkbenchResolutionAdapter adapter = mockWmrg.new WorkbenchResolutionAdapter(mockIssueResolution, mockResolutionMarker); - assertArrayEquals("Adapter findOtherMarkers matching on CODE1.", matchingMarkers, adapter.findOtherMarkers(allMarkers)); //$NON-NLS-1$ + assertArrayEquals(matchingMarkers, adapter.findOtherMarkers(allMarkers), "Adapter findOtherMarkers matching on CODE1."); //$NON-NLS-1$ when(mockResolutionMarker.getAttribute(Issue.CODE_KEY, null)).thenReturn(CODE2); matchingMarkers = new IMarker[] {mockMarker2}; adapter = mockWmrg.new WorkbenchResolutionAdapter(mockIssueResolution, mockResolutionMarker); - assertArrayEquals("Adapter findOtherMarkers matching on CODE2.", matchingMarkers, adapter.findOtherMarkers(allMarkers)); //$NON-NLS-1$ + assertArrayEquals(matchingMarkers, adapter.findOtherMarkers(allMarkers), "Adapter findOtherMarkers matching on CODE2."); //$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 ef08e0d168..fee1706fab 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 @@ -11,26 +11,30 @@ package com.avaloq.tools.ddk.xtext.ui.templates; +import static org.junit.jupiter.api.Assertions.assertArrayEquals; +import static org.junit.jupiter.api.Assertions.assertThrows; + import org.eclipse.core.resources.IFile; import org.eclipse.jface.text.templates.TemplateException; import org.eclipse.jface.text.templates.TemplateVariable; import org.eclipse.xtext.XtextRuntimeModule; +import org.eclipse.xtext.testing.extensions.InjectionExtension; import org.eclipse.xtext.ui.editor.model.IXtextDocument; import org.eclipse.xtext.ui.editor.templates.XtextTemplateContext; -import org.junit.Assert; -import org.junit.Test; -import org.junit.runner.RunWith; +import org.junit.jupiter.api.AfterAll; +import org.junit.jupiter.api.BeforeAll; +import org.junit.jupiter.api.Test; +import org.junit.jupiter.api.TestInstance; +import org.junit.jupiter.api.extension.ExtendWith; import org.mockito.Mockito; -import com.avaloq.tools.ddk.test.core.AfterAll; -import com.avaloq.tools.ddk.test.core.BeforeAll; -import com.avaloq.tools.ddk.xtext.test.junit.runners.XtextClassRunner; import com.google.common.collect.Iterables; import com.google.inject.Guice; -@RunWith(XtextClassRunner.class) -public class ResourceNameTemplateVariableResolverTest { +@ExtendWith(InjectionExtension.class) +@TestInstance(TestInstance.Lifecycle.PER_CLASS) +class ResourceNameTemplateVariableResolverTest { private static final Object[] FILE = new Object[] {"file"}; //$NON-NLS-1$ private static final String FILENAME = "filename"; //$NON-NLS-1$ @@ -43,7 +47,7 @@ public class ResourceNameTemplateVariableResolverTest { private static ResourceNameTemplateVariableResolver resolver; @BeforeAll - public void beforeAll() { + void beforeAll() { mockContext = Mockito.mock(XtextTemplateContext.class); mockFile = Mockito.mock(IFile.class); @@ -57,7 +61,7 @@ public void beforeAll() { } @AfterAll - public void afterAll() { + void afterAll() { mockContext = null; mockFile = null; @@ -66,39 +70,39 @@ public void afterAll() { resolver = null; } - @Test(expected = NullPointerException.class) - public void testResolveValuesWithNullVariable() { - resolver.resolveValues(null, mockContext); + @Test + void testResolveValuesWithNullVariable() { + assertThrows(NullPointerException.class, () -> resolver.resolveValues(null, mockContext)); } - @Test(expected = NullPointerException.class) - public void testResolveValuesWithNullContext() { - resolver.resolveValues(Mockito.mock(TemplateVariable.class), null); + @Test + void testResolveValuesWithNullContext() { + assertThrows(NullPointerException.class, () -> resolver.resolveValues(Mockito.mock(TemplateVariable.class), null)); } @Test - public void testResolveValuesWithFileWithoutExtension() throws TemplateException { + void testResolveValuesWithFileWithoutExtension() throws TemplateException { final String filename = "filenamewithnoextension"; //$NON-NLS-1$ testResolveValues(FILE, filename, filename); } @Test - public void testResolveValuesWithFileWithExtension() throws TemplateException { + void testResolveValuesWithFileWithExtension() throws TemplateException { testResolveValues(FILE, "filename.with.extension", "filename.with"); //$NON-NLS-1$//$NON-NLS-2$ } @Test - public void testResolveValuesWithExtraParams() throws TemplateException { + void testResolveValuesWithExtraParams() throws TemplateException { testResolveValues(new Object[] {FILE[0], "other", "random", "values"}, FILENAME, FILENAME); //$NON-NLS-1$ //$NON-NLS-2$ //$NON-NLS-3$ } @Test - public void testResolveValuesWithUnknownParam() throws TemplateException { + void testResolveValuesWithUnknownParam() throws TemplateException { testResolveValues(new Object[] {"This is not the parameter you are looking for"}, FILENAME); //$NON-NLS-1$ } @Test - public void testResolveValuesWithWrongTypeOfParam() throws TemplateException { + void testResolveValuesWithWrongTypeOfParam() throws TemplateException { testResolveValues(new Object[] {42}, FILENAME); } @@ -112,7 +116,7 @@ public void testResolveValuesWithWrongTypeOfParam() throws TemplateException { * @param expectedResolvedValues * expected return value */ - public void testResolveValues(final Object[] values, final String filename, final String... expectedResolvedValues) throws TemplateException { + void testResolveValues(final Object[] values, final String filename, final String... expectedResolvedValues) throws TemplateException { // ARRANGE final TemplateVariable variable = helper.createTemplateVariable(resolver, "name", values); //$NON-NLS-1$ Mockito.when(mockFile.getName()).thenReturn(filename); @@ -121,7 +125,7 @@ public void testResolveValues(final Object[] values, final String filename, fina final String[] actualResolvedValues = Iterables.toArray(resolver.resolveValues(variable, mockContext), String.class); // ASSERT - Assert.assertArrayEquals("Resolved values", expectedResolvedValues, actualResolvedValues); //$NON-NLS-1$ + assertArrayEquals(expectedResolvedValues, actualResolvedValues, "Resolved values"); //$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 0cf3acb883..c37a88e1b7 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 @@ -11,7 +11,8 @@ package com.avaloq.tools.ddk.xtext.ui.templates; -import static org.junit.Assert.assertArrayEquals; +import static org.junit.jupiter.api.Assertions.assertArrayEquals; +import static org.junit.jupiter.api.Assertions.assertThrows; import static org.mockito.Mockito.mock; import java.util.List; @@ -19,49 +20,51 @@ import org.eclipse.jface.text.templates.TemplateException; import org.eclipse.jface.text.templates.TemplateVariable; import org.eclipse.xtext.XtextRuntimeModule; +import org.eclipse.xtext.testing.extensions.InjectionExtension; import org.eclipse.xtext.ui.editor.templates.XtextTemplateContext; -import org.junit.Test; -import org.junit.runner.RunWith; +import org.junit.jupiter.api.AfterAll; +import org.junit.jupiter.api.BeforeAll; +import org.junit.jupiter.api.Test; +import org.junit.jupiter.api.TestInstance; +import org.junit.jupiter.api.extension.ExtendWith; -import com.avaloq.tools.ddk.test.core.AfterAll; -import com.avaloq.tools.ddk.test.core.BeforeAll; -import com.avaloq.tools.ddk.xtext.test.junit.runners.XtextClassRunner; import com.google.inject.Guice; -@RunWith(XtextClassRunner.class) -public class SimpleEnumTemplateVariableResolverTest { +@ExtendWith(InjectionExtension.class) +@TestInstance(TestInstance.Lifecycle.PER_CLASS) +class SimpleEnumTemplateVariableResolverTest { private static XtextTemplateContext mockContext; private static TemplateVariableResolverTestHelper helper; private static SimpleEnumTemplateVariableResolver resolver; @BeforeAll - public void beforeAll() { + void beforeAll() { mockContext = mock(XtextTemplateContext.class); helper = Guice.createInjector(new XtextRuntimeModule()).getInstance(TemplateVariableResolverTestHelper.class); resolver = new SimpleEnumTemplateVariableResolver(); } @AfterAll - public void afterAll() { + void afterAll() { mockContext = null; helper = null; resolver = null; } - @Test(expected = NullPointerException.class) - public void testResolveValuesWithNullVariable() { - resolver.resolveValues(null, mockContext); + @Test + void testResolveValuesWithNullVariable() { + assertThrows(NullPointerException.class, () -> resolver.resolveValues(null, mockContext)); } @Test - public void testResolveValuesWithOneParam() throws TemplateException { + void testResolveValuesWithOneParam() throws TemplateException { testResolveValues("Value"); //$NON-NLS-1$ } @Test - public void testResolveValuesWithMultipleParams() throws TemplateException { + void testResolveValuesWithMultipleParams() throws TemplateException { testResolveValues("Value 1", "Value 2", "Value 3"); //$NON-NLS-1$//$NON-NLS-2$//$NON-NLS-3$ } @@ -73,7 +76,7 @@ private void testResolveValues(final Object... values) throws TemplateException final List resolvedValues = resolver.resolveValues(variable, mockContext); // ASSERT - assertArrayEquals("Resolved values", values, resolvedValues.toArray(new String[resolvedValues.size()])); //$NON-NLS-1$ + assertArrayEquals(values, resolvedValues.toArray(new String[resolvedValues.size()]), "Resolved values"); //$NON-NLS-1$ } } diff --git a/com.avaloq.tools.ddk.xtext.ui.test/src/com/avaloq/tools/ddk/xtext/ui/templates/TemplateProposalProviderHelperTest.xtend b/com.avaloq.tools.ddk.xtext.ui.test/src/com/avaloq/tools/ddk/xtext/ui/templates/TemplateProposalProviderHelperTest.xtend index 6285b36de7..dd2add1d7c 100644 --- a/com.avaloq.tools.ddk.xtext.ui.test/src/com/avaloq/tools/ddk/xtext/ui/templates/TemplateProposalProviderHelperTest.xtend +++ b/com.avaloq.tools.ddk.xtext.ui.test/src/com/avaloq/tools/ddk/xtext/ui/templates/TemplateProposalProviderHelperTest.xtend @@ -11,29 +11,32 @@ package com.avaloq.tools.ddk.xtext.ui.templates -import com.avaloq.tools.ddk.test.core.AfterAll -import com.avaloq.tools.ddk.test.core.BeforeAll -import com.avaloq.tools.ddk.xtext.test.junit.runners.XtextClassRunner import com.google.inject.Guice import org.eclipse.jface.text.IDocument import org.eclipse.jface.text.IRegion import org.eclipse.jface.text.Position import org.eclipse.jface.text.templates.Template import org.eclipse.xtext.XtextRuntimeModule +import org.eclipse.xtext.testing.extensions.InjectionExtension import org.eclipse.xtext.ui.editor.templates.XtextTemplateContext import org.eclipse.xtext.ui.editor.templates.XtextTemplateContextType -import org.junit.Test -import org.junit.runner.RunWith +import org.junit.jupiter.api.Test +import org.junit.jupiter.api.^extension.ExtendWith -import static org.junit.Assert.assertArrayEquals -import static org.junit.Assert.assertEquals +import static org.junit.jupiter.api.Assertions.assertThrows +import static org.junit.jupiter.api.Assertions.assertEquals +import static org.junit.jupiter.api.Assertions.assertArrayEquals import static org.mockito.ArgumentMatchers.anyInt import static org.mockito.Mockito.mock import static org.mockito.Mockito.spy import static org.mockito.Mockito.verify import static org.mockito.Mockito.when +import org.junit.jupiter.api.TestInstance +import org.junit.jupiter.api.BeforeAll +import org.junit.jupiter.api.AfterAll -@RunWith(XtextClassRunner) +@ExtendWith(InjectionExtension) +@TestInstance(TestInstance.Lifecycle.PER_CLASS) class TemplateProposalProviderHelperTest { static val SIMPLE_ENUM_VARIABLE_TYPE = new SimpleEnumTemplateVariableResolver().type @@ -80,14 +83,14 @@ class TemplateProposalProviderHelperTest { helper = null } - @Test(expected=NullPointerException) + @Test def void testCreateLiteralValuePatternWithNullName() { - helper.createLiteralValuePattern(null, 42) + assertThrows(NullPointerException, [| helper.createLiteralValuePattern(null, 42)]) } - @Test(expected=IllegalArgumentException) + @Test def void testCreateLiteralValuePatternWithNameContainingWhitespace() { - helper.createLiteralValuePattern("Contains whitespace", 42) + assertThrows(IllegalArgumentException, [| helper.createLiteralValuePattern("Contains whitespace", 42)]) } @Test @@ -134,37 +137,36 @@ class TemplateProposalProviderHelperTest { // ASSERT verify(helperSpy).createTemplateVariablePattern(SIMPLE_ENUM_VARIABLE_TYPE, VARIABLE_NAME, defaultValue) - assertEquals("Expected result", expectedResult, actualResult) + assertEquals(expectedResult, actualResult, "Expected result") } - @Test(expected=NullPointerException) + @Test def void testCreateTemplateVariablePatternWithNullType() { - helper.createTemplateVariablePattern(null, VARIABLE_NAME) + assertThrows(NullPointerException, [| helper.createTemplateVariablePattern(null, VARIABLE_NAME)]) } - @Test(expected=IllegalArgumentException) + @Test def void testCreateTemplateVariablePatternWithTypeContainingWhitespace() { - helper.createTemplateVariablePattern("Contains whitespace", VARIABLE_NAME) + assertThrows(IllegalArgumentException, [|helper.createTemplateVariablePattern("Contains whitespace", VARIABLE_NAME)]) } - @Test(expected=NullPointerException) + @Test def void testCreateTemplateVariablePatternWithNullName() { - helper.createTemplateVariablePattern(SIMPLE_ENUM_VARIABLE_TYPE, null) + assertThrows(NullPointerException, [| helper.createTemplateVariablePattern(SIMPLE_ENUM_VARIABLE_TYPE, null)]) } - @Test(expected=IllegalArgumentException) def void testCreateTemplateVariablePatternWithNameContainingWhitespace() { - helper.createTemplateVariablePattern(SIMPLE_ENUM_VARIABLE_TYPE, "Contains whitespace") + assertThrows(IllegalArgumentException, [| helper.createTemplateVariablePattern(SIMPLE_ENUM_VARIABLE_TYPE, "Contains whitespace")]) } - @Test(expected=NullPointerException) + @Test def void testCreateTemplateVariablePatternWithNull() { - helper.createTemplateVariablePattern(SIMPLE_ENUM_VARIABLE_TYPE, VARIABLE_NAME, null) + assertThrows(NullPointerException, [| helper.createTemplateVariablePattern(SIMPLE_ENUM_VARIABLE_TYPE, VARIABLE_NAME, null)]) } - @Test(expected=IllegalArgumentException) + @Test def void testCreateTemplateVariablePatternWithNoValues() { - helper.createTemplateVariablePattern(SIMPLE_ENUM_VARIABLE_TYPE, VARIABLE_NAME) + assertThrows(IllegalArgumentException, [| helper.createTemplateVariablePattern(SIMPLE_ENUM_VARIABLE_TYPE, VARIABLE_NAME)]) } @Test @@ -256,8 +258,8 @@ class TemplateProposalProviderHelperTest { val actualValues = templateBuffer.variables.get(0).values // ASSERT - assertEquals("Expected result", expectedResult, actualResult) - assertArrayEquals("Expected values", expectedValues, actualValues) + assertEquals(expectedResult, actualResult, "Expected result") + assertArrayEquals(expectedValues, actualValues, "Expected values") } } diff --git a/com.avaloq.tools.ddk.xtext.ui.test/src/com/avaloq/tools/ddk/xtext/ui/templates/TemplateVariableResolverTestHelper.java b/com.avaloq.tools.ddk.xtext.ui.test/src/com/avaloq/tools/ddk/xtext/ui/templates/TemplateVariableResolverTestHelper.java index 35fe8a7a65..d5b5a1414f 100644 --- a/com.avaloq.tools.ddk.xtext.ui.test/src/com/avaloq/tools/ddk/xtext/ui/templates/TemplateVariableResolverTestHelper.java +++ b/com.avaloq.tools.ddk.xtext.ui.test/src/com/avaloq/tools/ddk/xtext/ui/templates/TemplateVariableResolverTestHelper.java @@ -11,7 +11,7 @@ package com.avaloq.tools.ddk.xtext.ui.templates; -import static org.junit.Assert.assertEquals; +import static org.junit.jupiter.api.Assertions.assertEquals; import org.eclipse.jface.text.templates.TemplateException; import org.eclipse.jface.text.templates.TemplateTranslator; @@ -55,7 +55,7 @@ public TemplateVariable createTemplateVariable(final TemplateVariableResolver re // and has protected constructors thus cannot be directly instantiated final String pattern = helper.createTemplateVariablePattern(resolver.getType(), name, values); final TemplateVariable[] variables = translator.translate(pattern).getVariables(); - assertEquals("Exactly one variable should be returned", 1, variables.length); //$NON-NLS-1$ + assertEquals(1, variables.length, "Exactly one variable should be returned"); //$NON-NLS-1$ return variables[0]; } 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 fca2d4860f..3295293667 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,27 +11,18 @@ package com.avaloq.tools.ddk.xtext.ui.test; -import org.junit.runner.RunWith; - -import com.avaloq.tools.ddk.test.core.junit.runners.DiscerningSuite; -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; +import org.junit.platform.suite.api.SelectPackages; +import org.junit.platform.suite.api.Suite; /** - * Empty class serving only as holder for JUnit4 annotations. + * Junit5 version of test suites. does not implement the logic in our DiscerningSuite. */ -@RunWith(DiscerningSuite.class) -@DiscerningSuite.SuiteClasses({ +@Suite +@SelectPackages({ // @Format-Off - WorkbenchResolutionAdaptorTest.class, - WorkbenchResolutionAdaptorRunTest.class, - SimpleEnumTemplateVariableResolverTest.class, - ResourceNameTemplateVariableResolverTest.class, - TemplateProposalProviderHelperTest.class + "com.avaloq.tools.ddk.xtext.ui.quickfix", + "com.avaloq.tools.ddk.xtext.ui.templates" // @Format-On }) public class XtextUiTestSuite {