Add test engines as surefire plugin dependencies

This removes junit-vintage-engine as a test dependency
as well as kotlintest-runner-junit5.
The engines are only needed during test execution, but should
not be available for test compilation.

For this to work, the latest surefire plugin is required.

pmd-java/pom.xml

@@ -149,11 +149,6 @@
             <artifactId>junit</artifactId>
             <scope>test</scope>
         </dependency>
-        <dependency>
-            <groupId>org.junit.vintage</groupId>
-            <artifactId>junit-vintage-engine</artifactId>
-            <scope>test</scope>
-        </dependency>
 
         <dependency>
             <groupId>com.github.stefanbirkner</groupId>
@@ -176,11 +171,6 @@
             <artifactId>tree-printers</artifactId>
             <scope>test</scope>
         </dependency>
-        <dependency>
-            <groupId>io.kotlintest</groupId>
-            <artifactId>kotlintest-runner-junit5</artifactId>
-            <scope>test</scope>
-        </dependency>
         <dependency>
             <groupId>io.kotlintest</groupId>
             <artifactId>kotlintest-assertions</artifactId>

pmd-java/src/test/kotlin/net/sourceforge/pmd/lang/java/ast/ASTCatchStatementTest.kt

@@ -3,7 +3,6 @@ package net.sourceforge.pmd.lang.java.ast
 import io.kotlintest.matchers.collections.shouldContainExactly
 import io.kotlintest.should
 import io.kotlintest.shouldBe
-import io.kotlintest.specs.FunSpec
 import net.sourceforge.pmd.lang.java.ast.JavaVersion.*
 import net.sourceforge.pmd.lang.java.ast.JavaVersion.Companion.Earliest
 import net.sourceforge.pmd.lang.java.ast.JavaVersion.Companion.Latest

pmd-java/src/test/kotlin/net/sourceforge/pmd/lang/java/ast/ParserTestSpec.kt

@@ -3,7 +3,6 @@ package net.sourceforge.pmd.lang.java.ast
 import io.kotlintest.AbstractSpec
 import io.kotlintest.TestContext
 import io.kotlintest.TestType
-import io.kotlintest.specs.IntelliMarker
 import net.sourceforge.pmd.lang.ast.test.Assertions
 import io.kotlintest.should as kotlintestShould
 
@@ -15,7 +14,7 @@ import io.kotlintest.should as kotlintestShould
  *
  * @author Clément Fournier
  */
-abstract class ParserTestSpec(body: ParserTestSpec.() -> Unit) : AbstractSpec(), IntelliMarker {
+abstract class ParserTestSpec(body: ParserTestSpec.() -> Unit) : AbstractSpec() {
 
     init {
         body()