Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
89 commits
Select commit Hold shift + click to select a range
33887a7
Update DirectLinkingEObjectInputStream.java
markdomeng Apr 23, 2025
4985394
Update DirectLinkingEObjectInputStream.java
markdomeng Apr 23, 2025
22c3faa
Update DirectLinkingEObjectInputStream.java
markdomeng Apr 23, 2025
0596c49
Update DirectLinkingEObjectInputStream.java
markdomeng Apr 23, 2025
ff0ce99
Update DirectLinkingEObjectInputStream.java
markdomeng Apr 23, 2025
ac78d7d
Merge pull request #1117 from markdomeng/markdomeng-patch-1
rubenporras Apr 23, 2025
913ea83
chore(deps): bump com.puppycrawl.tools:checkstyle in /ddk-parent
dependabot[bot] Apr 28, 2025
6f64e58
chore(deps): bump pmd.version from 7.12.0 to 7.13.0 in /ddk-parent
dependabot[bot] Apr 28, 2025
29be034
fix: load model when loading model associations.
andrewL-avlq Apr 28, 2025
71cc1a1
Merge pull request #1118 from dsldevkit/dependabot/maven/ddk-parent/c…
rubenporras Apr 29, 2025
735a1d2
Merge pull request #1119 from dsldevkit/dependabot/maven/ddk-parent/p…
rubenporras Apr 29, 2025
dd59172
Merge pull request #1120 from andrewL-avlq/ACF-12752
andrewL-avlq Apr 29, 2025
3eed92a
feat: allow pinning new resource desc. in ResourceDescriptionDelta
ava-fred May 9, 2025
4894ff7
Merge pull request #1121 from ava-fred/work.250509
ava-fred May 9, 2025
95468e4
Release 15.3.0.v20250519-1223-REL
May 19, 2025
1a8acdc
Next development version: 15.4.0-SNAPSHOT
May 19, 2025
1d5cd11
Merge pull request #1122 from ava-fred/release_15.3.0.v20250519-1223-REL
ava-fred May 19, 2025
d87a9bb
chore(deps): bump tycho.version from 4.0.12 to 4.0.13 in /ddk-parent
dependabot[bot] May 26, 2025
9846e52
chore(deps): bump com.puppycrawl.tools:checkstyle in /ddk-parent
dependabot[bot] May 26, 2025
3527589
Merge pull request #1124 from dsldevkit/dependabot/maven/ddk-parent/c…
rubenporras May 26, 2025
4a55863
Merge pull request #1123 from dsldevkit/dependabot/maven/ddk-parent/t…
joaodinissf May 27, 2025
2025d75
chore: improve thread safety of cat.ddk.caching.MapCache
ava-fred May 28, 2025
45cd117
Merge pull request #1125 from ava-fred/work.250516
ava-fred May 29, 2025
6ade928
chore(deps): bump org.apache.maven.plugins:maven-clean-plugin
dependabot[bot] Jun 2, 2025
139bf7f
Merge pull request #1127 from dsldevkit/dependabot/maven/ddk-parent/o…
rubenporras Jun 2, 2025
0e5d084
fix: another thread safety issue in MapCache
ava-fred Jun 5, 2025
edb7f60
Merge pull request #1130 from ava-fred/work.250516
rubenporras Jun 5, 2025
fa4d288
Release 15.4.0.v20250610-1134-REL
Jun 10, 2025
9fc3595
Next development version: 15.5.0-SNAPSHOT
Jun 10, 2025
a1f6c61
Merge pull request #1132 from ava-fred/release_15.4.0.v20250610-1134-REL
ava-fred Jun 10, 2025
351b260
chore(deps): bump org.codehaus.mojo:build-helper-maven-plugin
dependabot[bot] Jun 10, 2025
3420ffb
chore: upgrade Xtext to 2.39
ava-fred Jun 16, 2025
89a6531
Merge pull request #1134 from ava-fred/work.250616.b
ava-fred Jun 16, 2025
cfe4bec
chore(deps): bump com.puppycrawl.tools:checkstyle in /ddk-parent
dependabot[bot] Jun 23, 2025
434b5b4
Merge pull request #1135 from dsldevkit/dependabot/maven/ddk-parent/c…
rubenporras Jun 24, 2025
7a3ee34
chore(deps): bump pmd.version from 7.13.0 to 7.14.0
rubenporras Jun 24, 2025
9faf9a0
Merge pull request #1131 from dsldevkit/dependabot/maven/ddk-parent/o…
rubenporras Jun 24, 2025
102dc35
Merge pull request #1136 from rubenporras/pmd
rubenporras Jun 24, 2025
ae62ec7
chore(deps): bump com.github.spotbugs:spotbugs-maven-plugin
dependabot[bot] Jun 30, 2025
07eeb53
chore(deps): bump pmd.version from 7.14.0 to 7.15.0 in /ddk-parent
dependabot[bot] Jun 30, 2025
510abba
Merge pull request #1139 from dsldevkit/dependabot/maven/ddk-parent/p…
rubenporras Jul 1, 2025
e01c286
Merge pull request #1137 from dsldevkit/dependabot/maven/ddk-parent/c…
rubenporras Jul 1, 2025
a614879
chore(deps): bump org.apache.maven.plugins:maven-pmd-plugin
dependabot[bot] Jul 1, 2025
e17419d
Merge pull request #1138 from dsldevkit/dependabot/maven/ddk-parent/o…
rubenporras Jul 1, 2025
1532737
chore(deps): bump com.puppycrawl.tools:checkstyle in /ddk-parent
dependabot[bot] Jul 1, 2025
d0b3145
Merge pull request #1140 from dsldevkit/dependabot/maven/ddk-parent/c…
rubenporras Jul 1, 2025
6106a8b
fix: update launch, classpath and MANIFEST.MF files to specify Java 21
rubenporras Jul 1, 2025
ec04c1a
Merge pull request #1141 from rubenporras/java21
rubenporras Jul 2, 2025
da707bd
feat: Change the compilation target to Java 21
rubenporras Jul 2, 2025
faa534f
Merge pull request #1142 from rubenporras/java21
rubenporras Jul 2, 2025
cbe5934
Release 15.5.0.v20250707-1302-REL
Jul 7, 2025
edb5b26
Next development version: 16.0.0-SNAPSHOT
Jul 7, 2025
aa840c9
Merge pull request #1143 from ava-fred/release_15.5.0.v20250707-1302-REL
ava-fred Jul 7, 2025
21a88d3
chore(deps): bump org.apache.maven.plugins:maven-enforcer-plugin
dependabot[bot] Jul 7, 2025
2afa1d8
Merge pull request #1144 from dsldevkit/dependabot/maven/ddk-parent/o…
rubenporras Jul 8, 2025
205b046
chore: fix deprecated use of URL constructor
gregdyke Jul 10, 2025
546dde1
Merge pull request #1145 from gregdyke/fix_url
gregdyke Jul 10, 2025
968db28
chore(deps): bump org.apache.maven.plugins:maven-enforcer-plugin
dependabot[bot] Jul 21, 2025
f7a6fad
chore(deps): bump com.puppycrawl.tools:checkstyle in /ddk-parent
dependabot[bot] Aug 11, 2025
db7f99d
chore(deps): bump com.github.spotbugs:spotbugs in /ddk-parent
dependabot[bot] Aug 11, 2025
6e2fb3d
chore(deps): bump com.puppycrawl.tools:checkstyle in /ddk-parent
dependabot[bot] Aug 11, 2025
ce3abd6
Merge pull request #1149 from dsldevkit/dependabot/maven/ddk-parent/c…
rubenporras Aug 12, 2025
9d57556
Merge pull request #1148 from dsldevkit/dependabot/maven/ddk-parent/c…
rubenporras Aug 12, 2025
9c2a7a9
chore(deps): bump actions/checkout from 4 to 5
dependabot[bot] Aug 12, 2025
acf1068
chore(deps): bump actions/setup-java from 4 to 5
dependabot[bot] Aug 21, 2025
98fb4ac
Merge pull request #1151 from dsldevkit/dependabot/github_actions/act…
rubenporras Aug 21, 2025
7a9e403
chore(deps): bump com.github.spotbugs:spotbugs-maven-plugin
dependabot[bot] Aug 26, 2025
4019bff
Merge pull request #1152 from dsldevkit/dependabot/maven/ddk-parent/c…
rubenporras Aug 26, 2025
24b5b96
Force CR LF in text files
rubenporras Aug 27, 2025
fa2cc39
Merge pull request #225 from rubenporras/dependabot/maven/ddk-parent/…
rubenporras Aug 27, 2025
f4d23cd
Merge pull request #1153 from rubenporras/master
rubenporras Aug 27, 2025
60b11d3
Release 16.0.0.v20250902-1229-REL
Sep 2, 2025
dbb07a6
Next development version: 16.1.0-SNAPSHOT
Sep 2, 2025
8f928a4
Merge pull request #1154 from ava-fred/release_16.0.0.v20250902-1229-REL
rubenporras Sep 2, 2025
6a39b88
test: migrate JUnit 4 to JUnit 5
joaodinissf Sep 3, 2025
e51ea54
test: use Junit5 in target file
joaodinissf Sep 3, 2025
37e3a76
test: adapt MANIFEST.MF to use JUnit5
joaodinissf Sep 3, 2025
35af83f
test: Include junit Vintage to support use of @RunWith
joaodinissf Sep 3, 2025
d50af7b
test: make all JUnit 5 tests package-private
joaodinissf Sep 3, 2025
d743b63
fixup! test: make all JUnit 5 tests package-private
joaodinissf Sep 3, 2025
16d66d9
test: replace JUnit4 "Assert" imports with JUnit5 equivalents
joaodinissf Sep 4, 2025
96fe913
fixup! test: replace JUnit4 "Assert" imports with JUnit5 equivalents
joaodinissf Sep 4, 2025
da5dc67
"org.junit.Test" --> "org.junit.jupiter.api.Test"
joaodinissf Sep 4, 2025
0210b42
"org.junit.Ignore" --> "org.junit.jupiter.api.Disabled"
joaodinissf Sep 4, 2025
65b1ef4
Remove JUnit3's "extends TestCase" pattern
joaodinissf Sep 4, 2025
c283292
"Assert." --> "Assertions."
joaodinissf Sep 4, 2025
5aaf965
Roll back non-JUnit assert change
joaodinissf Sep 4, 2025
4e2b220
Add missing "Assertions." imports
joaodinissf Sep 4, 2025
13e495b
Add missing JUnit entries to MANIFEST.MF files
joaodinissf Sep 4, 2025
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
The table of contents is too big for display.
Diff view
Diff view
  •  
  •  
  •  
1 change: 1 addition & 0 deletions .gitattributes
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
text eol=crlf
8 changes: 4 additions & 4 deletions .github/workflows/verify.yml
Original file line number Diff line number Diff line change
Expand Up @@ -4,8 +4,8 @@ jobs:
pmd:
runs-on: ubuntu-22.04
steps:
- uses: actions/checkout@v4
- uses: actions/setup-java@v4
- uses: actions/checkout@v5
- uses: actions/setup-java@v5
with:
distribution: 'temurin'
java-version: '21'
Expand All @@ -26,9 +26,9 @@ jobs:
uses: stCarolas/setup-maven@v5
with:
maven-version: 3.9.9
- uses: actions/checkout@v4
- uses: actions/checkout@v5
- name: Set up JDK 21
uses: actions/setup-java@v4
uses: actions/setup-java@v5
with:
distribution: 'temurin'
java-version: '21'
Expand Down
2 changes: 1 addition & 1 deletion com.avaloq.tools.ddk.check.core.test/.classpath
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
<?xml version="1.0" encoding="UTF-8"?>
<classpath>
<classpathentry kind="con" path="org.eclipse.jdt.launching.JRE_CONTAINER/org.eclipse.jdt.internal.debug.ui.launcher.StandardVMType/JavaSE-17"/>
<classpathentry kind="con" path="org.eclipse.jdt.launching.JRE_CONTAINER/org.eclipse.jdt.internal.debug.ui.launcher.StandardVMType/JavaSE-21"/>
<classpathentry kind="con" path="org.eclipse.pde.core.requiredPlugins">
<accessrules>
<accessrule kind="accessible" pattern="org/eclipse/xtext/junit4/ui/util/**"/>
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,7 @@
<booleanAttribute key="org.eclipse.jdt.junit.KEEPRUNNING_ATTR" value="false"/>
<stringAttribute key="org.eclipse.jdt.junit.TESTNAME" value=""/>
<stringAttribute key="org.eclipse.jdt.junit.TEST_KIND" value="org.eclipse.jdt.junit.loader.junit4"/>
<stringAttribute key="org.eclipse.jdt.launching.JRE_CONTAINER" value="org.eclipse.jdt.launching.JRE_CONTAINER/org.eclipse.jdt.internal.debug.ui.launcher.StandardVMType/JavaSE-17"/>
<stringAttribute key="org.eclipse.jdt.launching.JRE_CONTAINER" value="org.eclipse.jdt.launching.JRE_CONTAINER/org.eclipse.jdt.internal.debug.ui.launcher.StandardVMType/JavaSE-21"/>
<stringAttribute key="org.eclipse.jdt.launching.MAIN_TYPE" value="com.avaloq.tools.ddk.check.test.core.CheckCoreTestSuite"/>
<stringAttribute key="org.eclipse.jdt.launching.PROGRAM_ARGUMENTS" value="-os ${target.os} -ws ${target.ws} -arch ${target.arch} -nl ${target.nl} -consoleLog"/>
<stringAttribute key="org.eclipse.jdt.launching.PROJECT_ATTR" value="com.avaloq.tools.ddk.check.core.test"/>
Expand Down
10 changes: 6 additions & 4 deletions com.avaloq.tools.ddk.check.core.test/META-INF/MANIFEST.MF
Original file line number Diff line number Diff line change
Expand Up @@ -2,9 +2,9 @@ Manifest-Version: 1.0
Bundle-ManifestVersion: 2
Bundle-Name: com.avaloq.tools.ddk.check.core.test
Bundle-SymbolicName: com.avaloq.tools.ddk.check.core.test;singleton:=true
Bundle-Version: 15.3.0.qualifier
Bundle-Version: 16.1.0.qualifier
Bundle-Vendor: Avaloq Group AG
Bundle-RequiredExecutionEnvironment: JavaSE-17
Bundle-RequiredExecutionEnvironment: JavaSE-21
Bundle-ActivationPolicy: lazy
Require-Bundle: com.avaloq.tools.ddk.check.core,
com.avaloq.tools.ddk.xtext.test.core,
Expand All @@ -20,15 +20,17 @@ Require-Bundle: com.avaloq.tools.ddk.check.core,
org.eclipse.xtend.lib,
org.eclipse.xtext.ui.testing,
org.eclipse.xtext.testing,
org.junit,
org.junit.jupiter.api,
org.mockito,
com.avaloq.tools.ddk.check.runtime.core,
org.eclipse.ui.workbench;resolution:=optional,
org.eclipse.xtext.xbase,
org.eclipse.xtext.xbase.lib,
org.apache.logging.log4j,
org.apache.commons.lang,
org.eclipse.xtext.xbase.testing
org.eclipse.xtext.xbase.testing,
org.junit,
junit-jupiter-api
Export-Package: com.avaloq.tools.ddk.check.core.test,
com.avaloq.tools.ddk.check.core.test.util,
com.avaloq.tools.ddk.check.test.core,
Expand Down
2 changes: 1 addition & 1 deletion com.avaloq.tools.ddk.check.core.test/pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@
<parent>
<artifactId>ddk-parent</artifactId>
<groupId>com.avaloq.tools.ddk</groupId>
<version>15.3.0-SNAPSHOT</version>
<version>16.1.0-SNAPSHOT</version>
<relativePath>../ddk-parent</relativePath>
</parent>

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,7 @@ import org.junit.runner.RunWith
import com.avaloq.tools.ddk.check.CheckInjectorProvider
import org.eclipse.xtext.testing.XtextRunner
import com.avaloq.tools.ddk.check.core.test.AbstractCheckGenerationTestCase
import org.junit.Test
import org.junit.jupiter.api.Test
import java.util.List
import org.eclipse.xtext.xbase.testing.JavaSource
import java.io.ByteArrayInputStream
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -11,6 +11,9 @@
package com.avaloq.tools.ddk.check.core.test;

import static com.google.common.collect.Sets.newHashSet;
import org.junit.jupiter.api.Assertions.assertEquals;
import org.junit.jupiter.api.Assertions.assertNotNull;
import org.junit.jupiter.api.Assertions.fail;

import java.io.IOException;
import java.io.InputStream;
Expand All @@ -19,8 +22,8 @@
import java.util.List;
import java.util.Set;

import org.apache.logging.log4j.Logger;
import org.apache.logging.log4j.LogManager;
import org.apache.logging.log4j.Logger;
import org.eclipse.core.resources.IFile;
import org.eclipse.core.resources.IProject;
import org.eclipse.core.resources.IWorkspaceRoot;
Expand All @@ -37,9 +40,9 @@
import org.eclipse.xtext.resource.XtextResourceSet;
import org.eclipse.xtext.ui.testing.util.IResourcesSetupUtil;
import org.eclipse.xtext.util.StringInputStream;
import org.junit.AfterClass;
import org.junit.Before;
import org.junit.BeforeClass;
import org.junit.jupiter.api.AfterAll;
import org.junit.jupiter.api.BeforeAll;
import org.junit.jupiter.api.BeforeEach;

import com.avaloq.tools.ddk.check.CheckConstants;
import com.avaloq.tools.ddk.check.ui.internal.CheckActivator;
Expand All @@ -54,14 +57,12 @@
import com.google.inject.Injector;
import com.google.inject.Provider;

import junit.framework.TestCase;


/**
* An abstract test class for tests on Check models. Allows creating a project and adding files.
*/
@SuppressWarnings({"PMD.SignatureDeclareThrowsException", "restriction"})
public abstract class AbstractCheckTestCase extends TestCase {
public abstract class AbstractCheckTestCase {
private static final int TWO_KILO_BYTES = 2048;
protected static final Logger LOGGER = LogManager.getLogger(AbstractCheckTestCase.class);
private static final PluginTestProjectManager PROJECT_MANAGER = new PluginTestProjectManager(CheckActivator.getInstance().getInjector(CheckConstants.GRAMMAR));
Expand All @@ -72,7 +73,7 @@ public abstract class AbstractCheckTestCase extends TestCase {
private Provider<XtextResourceSet> resourceSetProvider;

@Override
@Before
@BeforeEach
public void setUp() throws Exception {
getInjector().injectMembers(this);
}
Expand All @@ -83,7 +84,7 @@ public void setUp() throws Exception {
* @throws Exception
* the exception
*/
@BeforeClass
@BeforeAll
public static void prepareWorkspace() throws Exception {
PROJECT_MANAGER.setup(ImmutableList.<TestSource> of());
}
Expand Down Expand Up @@ -117,7 +118,7 @@ public <T> T get(final Class<T> clazz) {
/**
* Clean up after all tests have terminated.
*/
@AfterClass
@AfterAll
public static void cleanUp() {
PROJECT_MANAGER.teardown();
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -21,11 +21,11 @@ import org.eclipse.xtext.resource.XtextResource
import org.eclipse.xtext.testing.InjectWith
import org.eclipse.xtext.testing.XtextRunner
import org.eclipse.xtext.testing.util.ParseHelper
import org.junit.Ignore
import org.junit.Test
import org.junit.jupiter.api.Disabled
import org.junit.jupiter.api.Test
import org.junit.runner.RunWith

import static org.junit.Assert.*
import static org.junit.jupiter.api.Assertions.*

@InjectWith(typeof(CheckUiInjectorProvider))
@RunWith(typeof(XtextRunner))
Expand Down Expand Up @@ -73,8 +73,8 @@ class BasicModelTest {
}

/* Tests that Checks documented with ML_COMMENTs have an inferred description field. */
@Disabled("Fails because DocumentedImplCustom uses the null resource description provider to get the document provider")
@Test
@Ignore("Fails because DocumentedImplCustom uses the null resource description provider to get the document provider")
def void testInferingOfDescription() {
val check = util.getFirstInstanceOf(parser.parse(modelUtil.modelWithCheck), typeof(Check))
assertEquals("No documentation.", check.description)
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -10,12 +10,13 @@
*******************************************************************************/
package com.avaloq.tools.ddk.check.core.test;

import static org.junit.Assert.assertEquals;
import static org.junit.Assert.assertFalse;
import static org.junit.Assert.assertTrue;
import static org.junit.jupiter.api.Assertions.assertEquals;
import static org.junit.jupiter.api.Assertions.assertFalse;
import static org.junit.jupiter.api.Assertions.assertTrue;

import java.io.InputStream;
import java.net.MalformedURLException;
import java.net.URISyntaxException;
import java.net.URL;

import org.eclipse.emf.common.util.URI;
Expand All @@ -25,7 +26,7 @@
import org.eclipse.xtext.scoping.IScope;
import org.eclipse.xtext.testing.InjectWith;
import org.eclipse.xtext.testing.XtextRunner;
import org.junit.Test;
import org.junit.jupiter.api.Test;
import org.junit.runner.RunWith;

import com.avaloq.tools.ddk.check.CheckInjectorProvider;
Expand All @@ -40,7 +41,7 @@
*/
@RunWith(XtextRunner.class)
@InjectWith(CheckInjectorProvider.class)
public class BugAig1314 {

Check warning on line 44 in com.avaloq.tools.ddk.check.core.test/src/com/avaloq/tools/ddk/check/core/test/BugAig1314.java

View workflow job for this annotation

GitHub Actions / pmd

JUnit 5 tests should be package-private.

Reports JUnit 5 test classes and methods that are not package-private. Contrary to JUnit 4 tests, which required public visibility to be run by the engine, JUnit 5 tests can also be run if they're package-private. Marking them as such is a good practice to limit their visibility. Test methods are identified as those which use `@Test`, `@RepeatedTest`, `@TestFactory`, `@TestTemplate` or `@ParameterizedTest`. JUnit5TestShouldBePackagePrivate (Priority: 3, Ruleset: Best Practices) https://docs.pmd-code.org/pmd-doc-7.12.0/pmd_rules_java_bestpractices.html#junit5testshouldbepackageprivate

/** Constructor of super class is protected... */
private static class TestScope extends CatalogFromExtensionPointScope {
Expand All @@ -57,8 +58,8 @@
*
* @return Some syntactically correct but otherwise meaningless URL.
*/
private URL createURL() throws MalformedURLException {
return new URL("http://" + TEST_CATALOG_FILE + TEST_CATALOG_EXTENSION);
private URL createURL() throws MalformedURLException, URISyntaxException {
return new java.net.URI("http://" + TEST_CATALOG_FILE + TEST_CATALOG_EXTENSION).toURL();
}

/**
Expand Down Expand Up @@ -121,7 +122,7 @@
* Tests that querying the same scope twice doesn't make the resource set grow.
*/
@Test
public void testSameScopeUseTwice() throws MalformedURLException {
void testSameScopeUseTwice() throws MalformedURLException, URISyntaxException {
XtextResourceSet rs = new XtextResourceSet();
URL url = createURL();
ModelLocation modelLocation = createModelLocation(url);
Expand All @@ -140,7 +141,7 @@
* Tests that querying two different scopes doesn't make the resource set grow. That one was the real cause of bug AIG-1314.
*/
@Test
public void testDifferentScopeUseTwice() throws MalformedURLException {
void testDifferentScopeUseTwice() throws MalformedURLException, URISyntaxException {
XtextResourceSet rs = new XtextResourceSet();
URL url = createURL();
ModelLocation modelLocation = createModelLocation(url);
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -19,8 +19,8 @@ import org.eclipse.xtext.testing.InjectWith
import org.eclipse.xtext.testing.XtextRunner
import org.eclipse.xtext.testing.util.ParseHelper
import org.eclipse.xtext.xbase.XbasePackage
import org.junit.Assert
import org.junit.Test
import org.junit.jupiter.api.Assertions
import org.junit.jupiter.api.Test
import org.junit.runner.RunWith

@InjectWith(typeof(CheckUiInjectorProvider))
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,7 @@

import org.eclipse.xtext.testing.InjectWith;
import org.eclipse.xtext.testing.XtextRunner;
import org.junit.Test;
import org.junit.jupiter.api.Test;
import org.junit.runner.RunWith;

import com.avaloq.tools.ddk.check.CheckInjectorProvider;
Expand All @@ -26,13 +26,13 @@
*/
@InjectWith(CheckInjectorProvider.class)
@RunWith(XtextRunner.class)
public class BugDsl27 extends AbstractCheckGenerationTestCase {

Check warning on line 29 in com.avaloq.tools.ddk.check.core.test/src/com/avaloq/tools/ddk/check/core/test/BugDsl27.java

View workflow job for this annotation

GitHub Actions / pmd

JUnit 5 tests should be package-private.

Reports JUnit 5 test classes and methods that are not package-private. Contrary to JUnit 4 tests, which required public visibility to be run by the engine, JUnit 5 tests can also be run if they're package-private. Marking them as such is a good practice to limit their visibility. Test methods are identified as those which use `@Test`, `@RepeatedTest`, `@TestFactory`, `@TestTemplate` or `@ParameterizedTest`. JUnit5TestShouldBePackagePrivate (Priority: 3, Ruleset: Best Practices) https://docs.pmd-code.org/pmd-doc-7.12.0/pmd_rules_java_bestpractices.html#junit5testshouldbepackageprivate

/**
* Tests that our test source compiles fine.
*/
@Test
public void testGeneratedCodeHasNoErrors() {
@org.junit.jupiter.api.Test
void testGeneratedCodeHasNoErrors() {
try (InputStream sourceStream = BugDsl27.class.getResourceAsStream("bugdsl27/BugDsl27")) {
generateAndCompile(sourceStream);
} catch (IOException exception) {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,7 @@ import java.util.List
import org.eclipse.xtext.testing.InjectWith
import org.eclipse.xtext.testing.XtextRunner
import org.eclipse.xtext.ui.testing.util.IResourcesSetupUtil
import org.junit.Test
import org.junit.jupiter.api.Test
import org.junit.runner.RunWith

@InjectWith(typeof(CheckUiInjectorProvider))
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,7 @@ import com.avaloq.tools.ddk.check.CheckInjectorProvider
import java.io.ByteArrayInputStream
import org.eclipse.xtext.testing.InjectWith
import org.eclipse.xtext.testing.XtextRunner
import org.junit.Test
import org.junit.jupiter.api.Test
import org.junit.runner.RunWith
import java.util.List
import org.eclipse.xtext.xbase.testing.JavaSource
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,7 @@ import org.eclipse.core.resources.IProject
import org.eclipse.core.resources.IResource
import org.eclipse.xtext.testing.InjectWith
import org.eclipse.xtext.testing.XtextRunner
import org.junit.Test
import org.junit.jupiter.api.Test
import org.junit.runner.RunWith
import org.eclipse.xtext.ui.testing.util.IResourcesSetupUtil

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,7 @@ import org.eclipse.xtext.preferences.MapBasedPreferenceValues
import org.eclipse.xtext.testing.InjectWith
import org.eclipse.xtext.testing.XtextRunner
import org.eclipse.xtext.testing.formatter.FormatterTestHelper
import org.junit.Test
import org.junit.jupiter.api.Test
import org.junit.runner.RunWith

@InjectWith(typeof(CheckUiInjectorProvider))
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,7 @@ import org.eclipse.emf.common.util.URI
import org.eclipse.emf.ecore.EObject
import org.eclipse.xtext.resource.XtextResource
import org.eclipse.xtext.testing.XtextRunner
import org.junit.Test
import org.junit.jupiter.api.Test
import org.junit.runner.RunWith

import static org.mockito.Mockito.mock
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@ import org.eclipse.xtext.testing.InjectWith
import org.junit.runner.RunWith
import org.eclipse.xtext.testing.XtextRunner
import com.avaloq.tools.ddk.check.CheckUiInjectorProvider
import org.junit.Test
import org.junit.jupiter.api.Test
import com.google.inject.Inject
import org.eclipse.xtext.testing.util.ParseHelper
import com.avaloq.tools.ddk.check.check.CheckCatalog
Expand Down
Loading
Loading