Skip to content

Commit f36d642

Browse files
feat: Migrate com.avaloq.tools.ddk.xtext.format.test to Junit5
also made a junit5 copy of the AbstractXtextTestUtil and realign the recently migrated junit5 tests to use this instead, since the old one still uses the junti4 assertions
1 parent e58520a commit f36d642

14 files changed

Lines changed: 1107 additions & 40 deletions

File tree

com.avaloq.tools.ddk.xtext.export.test/src/com/avaloq/tools/ddk/xtext/test/export/util/ExportTestUtil.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@
1212

1313
import com.avaloq.tools.ddk.xtext.export.ExportConstants;
1414
import com.avaloq.tools.ddk.xtext.export.ui.internal.ExportActivator;
15-
import com.avaloq.tools.ddk.xtext.test.AbstractXtextTestUtil;
15+
import com.avaloq.tools.ddk.xtext.test.jupiter.AbstractXtextTestUtil;
1616
import com.google.inject.Injector;
1717

1818

@@ -29,7 +29,7 @@ private static final class InstanceHolder {
2929
private static final ExportTestUtil INSTANCE = new ExportTestUtil();
3030

3131
public static ExportTestUtil get() {
32-
return INSTANCE;
32+
return INSTANCE;
3333
}
3434
}
3535

com.avaloq.tools.ddk.xtext.format.test/META-INF/MANIFEST.MF

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -10,15 +10,14 @@ Fragment-Host: com.avaloq.tools.ddk.xtext.format.ui
1010
Require-Bundle: com.avaloq.tools.ddk.xtext.format,
1111
com.google.inject,
1212
com.avaloq.tools.ddk.xtext.test.core,
13-
org.junit,
1413
org.mockito.mockito-core,
1514
org.eclipse.xtext.xbase.lib,
1615
org.eclipse.xtext.xbase.testing,
1716
org.objectweb.asm;resolution:=optional,
1817
org.eclipse.xtext.testing,
1918
junit-jupiter-api,
2019
junit-jupiter-engine,
21-
junit-vintage-engine
20+
junit-platform-suite-api
2221
Export-Package: com.avaloq.tools.ddk.xtext.test.format;x-internal=true,
2322
com.avaloq.tools.ddk.xtext.format;x-internal=true
2423
Automatic-Module-Name: com.avaloq.tools.ddk.xtext.format.test

com.avaloq.tools.ddk.xtext.format.test/src/com/avaloq/tools/ddk/xtext/format/FormatParsingTest.xtend

Lines changed: 10 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -6,13 +6,16 @@ package com.avaloq.tools.ddk.xtext.format
66
import com.avaloq.tools.ddk.xtext.format.format.FormatConfiguration
77
import com.google.inject.Inject
88
import org.eclipse.xtext.testing.InjectWith
9-
import org.eclipse.xtext.testing.XtextRunner
109
import org.eclipse.xtext.testing.util.ParseHelper
11-
import org.junit.Assert
12-
import org.junit.Test
13-
import org.junit.runner.RunWith
1410

15-
@RunWith(XtextRunner)
11+
import org.junit.jupiter.api.^extension.ExtendWith
12+
import org.eclipse.xtext.testing.extensions.InjectionExtension
13+
import org.junit.jupiter.api.Test
14+
import static org.junit.jupiter.api.Assertions.assertNotNull
15+
import static org.junit.jupiter.api.Assertions.assertTrue
16+
17+
18+
@ExtendWith(InjectionExtension)
1619
@InjectWith(FormatInjectorProvider)
1720
class FormatParsingTest {
1821
@Inject
@@ -23,8 +26,8 @@ class FormatParsingTest {
2326
val result = parseHelper.parse('''
2427
Hello Xtext!
2528
''')
26-
Assert.assertNotNull(result)
29+
assertNotNull(result)
2730
val errors = result.eResource.errors
28-
Assert.assertTrue('''Unexpected errors: «errors.join(", ")»''', errors.isEmpty)
31+
assertTrue(errors.isEmpty, '''Unexpected errors: «errors.join(", ")»''')
2932
}
3033
}

com.avaloq.tools.ddk.xtext.format.test/src/com/avaloq/tools/ddk/xtext/format/builder/FormatBuilderParticipantTest.java

Lines changed: 10 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -10,8 +10,8 @@
1010
*******************************************************************************/
1111
package com.avaloq.tools.ddk.xtext.format.builder;
1212

13-
import static org.junit.Assert.assertFalse;
14-
import static org.junit.Assert.assertTrue;
13+
import static org.junit.jupiter.api.Assertions.assertFalse;
14+
import static org.junit.jupiter.api.Assertions.assertTrue;
1515
import static org.mockito.ArgumentMatchers.argThat;
1616
import static org.mockito.Mockito.mock;
1717
import static org.mockito.Mockito.when;
@@ -21,12 +21,12 @@
2121
import org.eclipse.emf.common.util.URI;
2222
import org.eclipse.xtext.resource.IResourceDescription.Delta;
2323
import org.eclipse.xtext.resource.IResourceServiceProvider;
24-
import org.junit.Test;
24+
import org.junit.jupiter.api.Test;
2525
import org.mockito.ArgumentMatcher;
2626

2727
import com.avaloq.tools.ddk.xtext.format.ui.builder.FormatBuilderParticipant;
28-
import com.avaloq.tools.ddk.xtext.test.AbstractXtextTest;
29-
import com.avaloq.tools.ddk.xtext.test.AbstractXtextTestUtil;
28+
import com.avaloq.tools.ddk.xtext.test.jupiter.AbstractXtextTest;
29+
import com.avaloq.tools.ddk.xtext.test.jupiter.AbstractXtextTestUtil;
3030
import com.google.common.collect.ImmutableList;
3131
import com.google.inject.Guice;
3232
import com.google.inject.Injector;
@@ -71,8 +71,8 @@ protected void beforeAllTests() {
7171
public void hasCorrectExtensionTest() {
7272
IResourceServiceProvider resourceServiceProvider = mock(IResourceServiceProvider.class);
7373
when(resourceServiceProvider.canHandle(argThat(new IsUri()))).thenReturn(true, false);
74-
assertTrue("Check if the delta resource has correct extension", participant.hasCorrectExtension(delta, resourceServiceProvider));
75-
assertFalse("Check if the delta resource has incorrect extension", participant.hasCorrectExtension(delta, resourceServiceProvider));
74+
assertTrue(participant.hasCorrectExtension(delta, resourceServiceProvider), "Check if the delta resource has correct extension");
75+
assertFalse(participant.hasCorrectExtension(delta, resourceServiceProvider), "Check if the delta resource has incorrect extension");
7676
}
7777

7878
/**
@@ -81,11 +81,11 @@ public void hasCorrectExtensionTest() {
8181
@Test
8282
public void isSourceOriginatedTest() {
8383
when(uriCorrect.segments()).thenReturn(CORRECT_URI_SEGMENTS);
84-
assertTrue("Check if the delta resource has correct URI and comes from SRC directory", participant.isSourceOriginated(delta));
84+
assertTrue(participant.isSourceOriginated(delta), "Check if the delta resource has correct URI and comes from SRC directory");
8585
when(uriCorrect.segments()).thenReturn(BIN_URI_SEGMENTS);
86-
assertFalse("Check if the delta resource has correct URI and does not come from SRC directory", participant.isSourceOriginated(delta));
86+
assertFalse(participant.isSourceOriginated(delta), "Check if the delta resource has correct URI and does not come from SRC directory");
8787
when(uriCorrect.segments()).thenReturn(INCORRECT_URI_SEGMENTS);
88-
assertFalse("Check if the delta resource has incorrect URI and comes from SRC directory", participant.isSourceOriginated(delta));
88+
assertFalse(participant.isSourceOriginated(delta), "Check if the delta resource has incorrect URI and comes from SRC directory");
8989
}
9090

9191
@Override

com.avaloq.tools.ddk.xtext.format.test/src/com/avaloq/tools/ddk/xtext/format/formatting/FormatFormattingTest.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@
1212

1313
import com.avaloq.tools.ddk.xtext.test.TestSource;
1414
import com.avaloq.tools.ddk.xtext.test.format.util.FormatTestUtil;
15-
import com.avaloq.tools.ddk.xtext.test.formatting.AbstractFormattingTest;
15+
import com.avaloq.tools.ddk.xtext.test.jupiter.AbstractFormattingTest;
1616

1717

1818
public class FormatFormattingTest extends AbstractFormattingTest {

com.avaloq.tools.ddk.xtext.format.test/src/com/avaloq/tools/ddk/xtext/format/scoping/FormatScopingTest.java

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,7 @@
1010
*******************************************************************************/
1111
package com.avaloq.tools.ddk.xtext.format.scoping;
1212

13-
import static org.junit.Assert.assertFalse;
13+
import static org.junit.jupiter.api.Assertions.assertFalse;
1414

1515
import java.util.List;
1616
import java.util.Set;
@@ -27,13 +27,13 @@
2727
import org.eclipse.xtext.nodemodel.ICompositeNode;
2828
import org.eclipse.xtext.nodemodel.ILeafNode;
2929
import org.eclipse.xtext.nodemodel.util.NodeModelUtils;
30-
import org.junit.Test;
30+
import org.junit.jupiter.api.Test;
3131

3232
import com.avaloq.tools.ddk.xtext.format.format.FormatConfiguration;
3333
import com.avaloq.tools.ddk.xtext.format.format.FormatPackage;
3434
import com.avaloq.tools.ddk.xtext.format.format.GroupBlock;
3535
import com.avaloq.tools.ddk.xtext.test.format.util.FormatTestUtil;
36-
import com.avaloq.tools.ddk.xtext.test.scoping.AbstractScopingTest;
36+
import com.avaloq.tools.ddk.xtext.test.jupiter.AbstractScopingTest;
3737
import com.google.common.base.Function;
3838
import com.google.common.collect.Iterables;
3939
import com.google.common.collect.Lists;
@@ -112,7 +112,7 @@ public void allGrammarsScoped() {
112112
public void keywordScoped() {
113113
AbstractRule parserRuleA = grammarA.getRules().get(0);
114114
Set<URI> keywordURIs = Sets.newHashSet(Iterables.transform(GrammarUtil.containedKeywords(parserRuleA), TO_URI));
115-
assertFalse("No keywords found", keywordURIs.isEmpty());
115+
assertFalse(keywordURIs.isEmpty(), "No keywords found");
116116
assertScope(formatC.getRules().get(0), FormatPackage.Literals.GRAMMAR_ELEMENT_REFERENCE__KEYWORD, keywordURIs);
117117
AbstractRule parserRuleC = grammarC.getRules().get(0);
118118
assertScope(formatA.getRules().get(0), FormatPackage.Literals.GRAMMAR_ELEMENT_REFERENCE__KEYWORD, keywordURIs);
@@ -128,7 +128,7 @@ public void keywordScoped() {
128128
public void assignmentScoped() {
129129
AbstractRule parserRuleA = grammarA.getRules().get(0);
130130
Set<URI> assignmentURIs = Sets.newHashSet(Iterables.transform(GrammarUtil.containedAssignments(parserRuleA), TO_URI));
131-
assertFalse("No assignments found", assignmentURIs.isEmpty());
131+
assertFalse(assignmentURIs.isEmpty(), "No assignments found");
132132
assertScope(formatC.getRules().get(0), FormatPackage.Literals.GRAMMAR_ELEMENT_REFERENCE__ASSIGNMENT, assignmentURIs);
133133
assertScope(formatA.getRules().get(0), FormatPackage.Literals.GRAMMAR_ELEMENT_REFERENCE__ASSIGNMENT, assignmentURIs);
134134
}

com.avaloq.tools.ddk.xtext.format.test/src/com/avaloq/tools/ddk/xtext/format/validation/FormatValidationTest.java

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -10,16 +10,16 @@
1010
*******************************************************************************/
1111
package com.avaloq.tools.ddk.xtext.format.validation;
1212

13-
import static org.junit.Assert.fail;
13+
import static org.junit.jupiter.api.Assertions.fail;
1414

1515
import java.io.IOException;
1616
import java.util.List;
1717

18-
import org.junit.Test;
18+
import org.junit.jupiter.api.Test;
1919

2020
import com.avaloq.tools.ddk.xtext.format.format.FormatConfiguration;
2121
import com.avaloq.tools.ddk.xtext.test.format.util.FormatTestUtil;
22-
import com.avaloq.tools.ddk.xtext.test.validation.AbstractValidationTest;
22+
import com.avaloq.tools.ddk.xtext.test.jupiter.AbstractValidationTest;
2323
import com.google.common.collect.Lists;
2424

2525

com.avaloq.tools.ddk.xtext.format.test/src/com/avaloq/tools/ddk/xtext/test/format/FormatTestSuite.java

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -10,8 +10,8 @@
1010
*******************************************************************************/
1111
package com.avaloq.tools.ddk.xtext.test.format;
1212

13-
import org.junit.runner.RunWith;
14-
import org.junit.runners.Suite;
13+
import org.junit.platform.suite.api.SelectClasses;
14+
import org.junit.platform.suite.api.Suite;
1515

1616
import com.avaloq.tools.ddk.xtext.format.builder.FormatBuilderParticipantTest;
1717
import com.avaloq.tools.ddk.xtext.format.formatting.FormatFormattingTest;
@@ -20,10 +20,10 @@
2020

2121

2222
/**
23-
* Empty class serving only as holder for JUnit4 annotations.
23+
* Empty class serving only as holder for JUnit5 annotations.
2424
*/
25-
@RunWith(Suite.class)
26-
@Suite.SuiteClasses({FormatFormattingTest.class, FormatValidationTest.class, FormatScopingTest.class, FormatBuilderParticipantTest.class})
25+
@Suite
26+
@SelectClasses({FormatFormattingTest.class, FormatValidationTest.class, FormatScopingTest.class, FormatBuilderParticipantTest.class})
2727
public class FormatTestSuite {
2828

2929
}

com.avaloq.tools.ddk.xtext.format.test/src/com/avaloq/tools/ddk/xtext/test/format/util/FormatTestUtil.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,7 @@
1010
*******************************************************************************/
1111
package com.avaloq.tools.ddk.xtext.test.format.util;
1212

13-
import static org.junit.Assert.fail;
13+
import static org.junit.jupiter.api.Assertions.fail;
1414

1515
import java.io.IOException;
1616

@@ -20,9 +20,9 @@
2020
import com.avaloq.tools.ddk.xtext.format.FormatConstants;
2121
import com.avaloq.tools.ddk.xtext.format.format.FormatConfiguration;
2222
import com.avaloq.tools.ddk.xtext.format.ui.internal.FormatActivator;
23-
import com.avaloq.tools.ddk.xtext.test.AbstractXtextTestUtil;
2423
import com.avaloq.tools.ddk.xtext.test.ITestProjectManager;
2524
import com.avaloq.tools.ddk.xtext.test.PluginTestProjectManager;
25+
import com.avaloq.tools.ddk.xtext.test.jupiter.AbstractXtextTestUtil;
2626
import com.google.inject.Injector;
2727

2828

com.avaloq.tools.ddk.xtext.generator.test/src/com/avaloq/tools/ddk/xtext/generator/test/util/GeneratorTestUtil.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,7 @@
1111
package com.avaloq.tools.ddk.xtext.generator.test.util;
1212

1313
import com.avaloq.tools.ddk.xtext.expression.ExpressionStandaloneSetup;
14-
import com.avaloq.tools.ddk.xtext.test.AbstractXtextTestUtil;
14+
import com.avaloq.tools.ddk.xtext.test.jupiter.AbstractXtextTestUtil;
1515
import com.google.inject.Injector;
1616

1717

@@ -28,7 +28,7 @@ private static final class InstanceHolder {
2828
private static final GeneratorTestUtil INSTANCE = new GeneratorTestUtil();
2929

3030
public static GeneratorTestUtil get() {
31-
return INSTANCE;
31+
return INSTANCE;
3232
}
3333
}
3434

0 commit comments

Comments
 (0)