From e3fcd690a5029cb43e21f9427cd6ba97fbb07fd5 Mon Sep 17 00:00:00 2001
From: Andreas Dangel <andreas.dangel@pmd-code.org>
Date: Fri, 21 Oct 2022 10:39:13 +0200
Subject: [PATCH 1/7] Bump kotlin from 1.7.0 to 1.7.20

---
 pom.xml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/pom.xml b/pom.xml
index a6d461d9bb..a8f37f1e63 100644
--- a/pom.xml
+++ b/pom.xml
@@ -85,7 +85,7 @@
 
 
         <kotlin.compiler.jvmTarget>${maven.compiler.test.target}</kotlin.compiler.jvmTarget>
-        <kotlin.version>1.7.0</kotlin.version>
+        <kotlin.version>1.7.20</kotlin.version>
         <kotest.version>4.4.3</kotest.version>
         <dokka.version>1.6.21</dokka.version>
 

From 3dc9d92833be0b2cda4dc1a4fe701657da50da64 Mon Sep 17 00:00:00 2001
From: Andreas Dangel <andreas.dangel@pmd-code.org>
Date: Fri, 21 Oct 2022 10:39:49 +0200
Subject: [PATCH 2/7] Bump dokka from 1.6.21 to 1.7.20

---
 pom.xml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/pom.xml b/pom.xml
index a8f37f1e63..d1fd6361d8 100644
--- a/pom.xml
+++ b/pom.xml
@@ -87,7 +87,7 @@
         <kotlin.compiler.jvmTarget>${maven.compiler.test.target}</kotlin.compiler.jvmTarget>
         <kotlin.version>1.7.20</kotlin.version>
         <kotest.version>4.4.3</kotest.version>
-        <dokka.version>1.6.21</dokka.version>
+        <dokka.version>1.7.20</dokka.version>
 
 
         <javacc.version>5.0</javacc.version>

From 0240c51ff090d22c52c0419e34bffe45e789e48d Mon Sep 17 00:00:00 2001
From: Andreas Dangel <andreas.dangel@pmd-code.org>
Date: Fri, 21 Oct 2022 12:14:32 +0200
Subject: [PATCH 3/7] Bump kotest from 4.4.3 to 5.5.1

Also update JUnit5 to 5.8.2
---
 pmd-lang-test/pom.xml |  5 +++++
 pom.xml               | 48 +++++++++++++++++++++++--------------------
 2 files changed, 31 insertions(+), 22 deletions(-)

diff --git a/pmd-lang-test/pom.xml b/pmd-lang-test/pom.xml
index 52520ed9c9..150388601a 100644
--- a/pmd-lang-test/pom.xml
+++ b/pmd-lang-test/pom.xml
@@ -99,6 +99,11 @@
             <artifactId>kotest-runner-junit5-jvm</artifactId>
             <scope>compile</scope>
         </dependency>
+        <dependency>
+            <groupId>org.junit.jupiter</groupId>
+            <artifactId>junit-jupiter-api</artifactId>
+            <scope>compile</scope>
+        </dependency>
 
         <dependency>
             <groupId>org.jetbrains</groupId>
diff --git a/pom.xml b/pom.xml
index d1fd6361d8..ff9804e2c9 100644
--- a/pom.xml
+++ b/pom.xml
@@ -86,12 +86,13 @@
 
         <kotlin.compiler.jvmTarget>${maven.compiler.test.target}</kotlin.compiler.jvmTarget>
         <kotlin.version>1.7.20</kotlin.version>
-        <kotest.version>4.4.3</kotest.version>
+        <kotest.version>5.5.1</kotest.version>
+        <junit5.version>5.8.2</junit5.version> <!-- needed by kotest -->
         <dokka.version>1.7.20</dokka.version>
 
 
         <javacc.version>5.0</javacc.version>
-        <surefire.version>3.0.0-M5</surefire.version>
+        <surefire.version>3.0.0-M7</surefire.version>
         <checkstyle.version>10.3.3</checkstyle.version>
         <checkstyle.plugin.version>3.2.0</checkstyle.plugin.version>
         <pmd.plugin.version>3.19.0</pmd.plugin.version>
@@ -275,13 +276,26 @@
                         <dependency>
                             <groupId>org.junit.vintage</groupId>
                             <artifactId>junit-vintage-engine</artifactId>
-                            <version>5.7.0</version>
+                            <version>${junit5.version}</version>
                         </dependency>
                         <!-- Junit5 Platform Engine for Kotlin Tests -->
+                        <dependency>
+                            <groupId>org.junit.jupiter</groupId>
+                            <artifactId>junit-jupiter-api</artifactId>
+                            <version>${junit5.version}</version>
+                            <!-- https://github.com/kotest/kotest/issues/3255 -->
+                        </dependency>
                         <dependency>
                             <groupId>io.kotest</groupId>
                             <artifactId>kotest-runner-junit5-jvm</artifactId>
                             <version>${kotest.version}</version>
+                            <!-- https://github.com/kotest/kotest/issues/3255 -->
+                            <exclusions>
+                                <exclusion>
+                                    <groupId>org.junit.jupiter</groupId>
+                                    <artifactId>junit-jupiter-api</artifactId>
+                                </exclusion>
+                            </exclusions>
                         </dependency>
                     </dependencies>
                 </plugin>
@@ -836,6 +850,15 @@
                 <scope>test</scope>
             </dependency>
 
+            <!-- needed by kotest -->
+            <dependency>
+                <groupId>org.junit.jupiter</groupId>
+                <artifactId>junit-jupiter-api</artifactId>
+                <version>${junit5.version}</version>
+                <scope>test</scope>
+                <!-- https://github.com/kotest/kotest/issues/3255 -->
+            </dependency>
+
             <dependency>
                 <groupId>org.mockito</groupId>
                 <artifactId>mockito-core</artifactId>
@@ -899,15 +922,6 @@
                 <artifactId>kotest-runner-junit5-jvm</artifactId>
                 <version>${kotest.version}</version>
                 <scope>test</scope>
-                <exclusions>
-                    <!-- exclude transitive dependency to fix CVE-2021-29425
-                         kotest 4.4.3 depends on commons-io 2.6
-                      -->
-                    <exclusion>
-                        <groupId>commons-io</groupId>
-                        <artifactId>commons-io</artifactId>
-                    </exclusion>
-                </exclusions>
             </dependency>
             <dependency>
                 <groupId>io.kotest</groupId>
@@ -929,16 +943,6 @@
                 <scope>test</scope>
             </dependency>
 
-            <!-- transitive dependency through io.kotest/kotest-runner-junit5-jvm@4.4.3
-                 upgrade to 4.8.112 to fix [sonatype-2021-1074] CWE-611: Improper Restriction of XML External Entity Reference ('XXE')
-                 https://github.com/classgraph/classgraph/releases/tag/classgraph-4.8.112
-              -->
-            <dependency>
-                <groupId>io.github.classgraph</groupId>
-                <artifactId>classgraph</artifactId>
-                <version>4.8.112</version>
-            </dependency>
-
             <!-- transitive dependency through org.scalameta:trees_2.13
                  upgrade to 3.16.1 to fix CVE-2021-22569 A potential Denial of Service issue in protobuf-java
                  https://github.com/protocolbuffers/protobuf/security/advisories/GHSA-wrvw-hg22-4m67

From ea3b860d6149604e874a6833c7dd79ac3ec6e8a5 Mon Sep 17 00:00:00 2001
From: Andreas Dangel <andreas.dangel@pmd-code.org>
Date: Fri, 21 Oct 2022 15:23:15 +0200
Subject: [PATCH 4/7] [java] Upgrade testing dsl for kotest 5.5.1

---
 .../lang/java/ast/ASTCatchStatementTest.kt    |  2 +-
 .../pmd/lang/java/ast/KotlinTestingDsl.kt     |  2 +-
 .../pmd/lang/java/ast/ParserTestSpec.kt       | 75 ++++++++++---------
 3 files changed, 40 insertions(+), 39 deletions(-)

diff --git a/pmd-java/src/test/kotlin/net/sourceforge/pmd/lang/java/ast/ASTCatchStatementTest.kt b/pmd-java/src/test/kotlin/net/sourceforge/pmd/lang/java/ast/ASTCatchStatementTest.kt
index 73a2c2d9b7..a87fdfaea4 100644
--- a/pmd-java/src/test/kotlin/net/sourceforge/pmd/lang/java/ast/ASTCatchStatementTest.kt
+++ b/pmd-java/src/test/kotlin/net/sourceforge/pmd/lang/java/ast/ASTCatchStatementTest.kt
@@ -57,7 +57,7 @@ class ASTCatchStatementTest : ParserTestSpec({
                     }
 
                     child<ASTVariableDeclaratorId> {
-                        it.image shouldBe "e"
+                        it.name shouldBe "e"
                     }
 
                     listOf(ioe, aerr)
diff --git a/pmd-java/src/test/kotlin/net/sourceforge/pmd/lang/java/ast/KotlinTestingDsl.kt b/pmd-java/src/test/kotlin/net/sourceforge/pmd/lang/java/ast/KotlinTestingDsl.kt
index f759b45db9..d70f4b5d39 100644
--- a/pmd-java/src/test/kotlin/net/sourceforge/pmd/lang/java/ast/KotlinTestingDsl.kt
+++ b/pmd-java/src/test/kotlin/net/sourceforge/pmd/lang/java/ast/KotlinTestingDsl.kt
@@ -28,7 +28,7 @@ enum class JavaVersion : Comparable<JavaVersion> {
     J19, J19__PREVIEW;
 
     /** Name suitable for use with e.g. [JavaParsingHelper.parse] */
-    val pmdName: String = name.removePrefix("J").replaceFirst("__", "-").replace('_', '.').toLowerCase()
+    val pmdName: String = name.removePrefix("J").replaceFirst("__", "-").replace('_', '.').lowercase()
 
     val parser: JavaParsingHelper = JavaParsingHelper.WITH_PROCESSING.withDefaultVersion(pmdName)
 
diff --git a/pmd-java/src/test/kotlin/net/sourceforge/pmd/lang/java/ast/ParserTestSpec.kt b/pmd-java/src/test/kotlin/net/sourceforge/pmd/lang/java/ast/ParserTestSpec.kt
index 5239e33f65..c5e409a51f 100644
--- a/pmd-java/src/test/kotlin/net/sourceforge/pmd/lang/java/ast/ParserTestSpec.kt
+++ b/pmd-java/src/test/kotlin/net/sourceforge/pmd/lang/java/ast/ParserTestSpec.kt
@@ -4,15 +4,15 @@
 
 package net.sourceforge.pmd.lang.java.ast
 
-import io.kotest.core.config.configuration
+import io.kotest.core.names.TestName
+import io.kotest.core.source.sourceRef
 import io.kotest.core.spec.DslDrivenSpec
-import io.kotest.core.spec.style.scopes.Lifecycle
 import io.kotest.core.spec.style.scopes.RootScope
-import io.kotest.core.spec.style.scopes.RootTestRegistration
-import io.kotest.core.test.TestCaseConfig
-import io.kotest.core.test.TestContext
+import io.kotest.core.spec.style.scopes.addContainer
+import io.kotest.core.spec.style.scopes.addTest
+import io.kotest.core.test.NestedTest
+import io.kotest.core.test.TestScope
 import io.kotest.core.test.TestType
-import io.kotest.core.test.createTestName
 import net.sourceforge.pmd.lang.ast.test.Assertions
 import net.sourceforge.pmd.lang.ast.test.IntelliMarker
 import io.kotest.matchers.should as kotlintestShould
@@ -31,22 +31,16 @@ abstract class ParserTestSpec(body: ParserTestSpec.() -> Unit) : DslDrivenSpec()
         body()
     }
 
-    override fun lifecycle(): Lifecycle = Lifecycle.from(this)
-    override fun defaultConfig(): TestCaseConfig = actualDefaultConfig()
-    override fun defaultTestCaseConfig(): TestCaseConfig? = defaultTestConfig
-    override fun registration(): RootTestRegistration = RootTestRegistration.from(this)
-
-    private fun actualDefaultConfig() =
-            defaultTestConfig ?: defaultTestCaseConfig() ?: configuration.defaultTestConfig
-
-    fun test(name: String, disabled: Boolean = false, test: suspend TestContext.() -> Unit) =
-            registration().addTest(
-                    name = createTestName(name),
-                    xdisabled = disabled,
-                    test = test,
-                    config = actualDefaultConfig()
+    fun test(name: String, disabled: Boolean = false, test: suspend TestScope.() -> Unit) =
+            addTest(
+                testName = TestName(name),
+                disabled = disabled,
+                config = null,
+                type = TestType.Test,
+                test = test
             )
 
+
     /**
      * Defines a group of tests that should be named similarly,
      * with separate tests for separate versions.
@@ -66,10 +60,11 @@ abstract class ParserTestSpec(body: ParserTestSpec.() -> Unit) : DslDrivenSpec()
     fun parserTestGroup(name: String,
                         disabled: Boolean = false,
                         spec: suspend GroupTestCtx.() -> Unit) =
-            registration().addContainerTest(
-                    name = createTestName(name),
+            addContainer(
+                    testName = TestName(name),
                     test = { GroupTestCtx(this).spec() },
-                    xdisabled = disabled
+                    disabled = disabled,
+                    config = null
             )
 
     /**
@@ -117,37 +112,43 @@ abstract class ParserTestSpec(body: ParserTestSpec.() -> Unit) : DslDrivenSpec()
             }
 
     private suspend fun containedParserTestImpl(
-            context: TestContext,
+            scope: TestScope,
             name: String,
             javaVersion: JavaVersion,
             assertions: ParserTestCtx.() -> Unit) {
 
-        context.registerTestCase(
-                name = createTestName(name),
-                test = { ParserTestCtx(javaVersion).assertions() },
-                config = actualDefaultConfig(),
-                type = TestType.Test
+        val nested = NestedTest(
+            name = TestName(name),
+            test = { ParserTestCtx(javaVersion).assertions() },
+            config = null,
+            type = TestType.Test,
+            disabled = false,
+            source = sourceRef()
         )
+        scope.registerTestCase(nested)
     }
 
-    inner class GroupTestCtx(private val context: TestContext) {
+    inner class GroupTestCtx(private val scope: TestScope) {
 
         suspend fun onVersions(javaVersions: List<JavaVersion>, spec: suspend VersionedTestCtx.() -> Unit) {
             javaVersions.forEach { javaVersion ->
 
-                context.registerTestCase(
-                        name = createTestName("Java ${javaVersion.pmdName}"),
-                        test = { VersionedTestCtx(this, javaVersion).spec() },
-                        config = actualDefaultConfig(),
-                        type = TestType.Container
+                val nested = NestedTest(
+                    name = TestName("Java ${javaVersion.pmdName}"),
+                    test = { VersionedTestCtx(this, javaVersion).spec() },
+                    config = null,
+                    type = TestType.Container,
+                    disabled = false,
+                    source = sourceRef()
                 )
+                scope.registerTestCase(nested)
             }
         }
 
-        inner class VersionedTestCtx(private val context: TestContext, javaVersion: JavaVersion) : ParserTestCtx(javaVersion) {
+        inner class VersionedTestCtx(private val scope: TestScope, javaVersion: JavaVersion) : ParserTestCtx(javaVersion) {
 
             suspend infix fun String.should(matcher: Assertions<String>) {
-                containedParserTestImpl(context, "'$this'", javaVersion = javaVersion) {
+                containedParserTestImpl(scope, "'$this'", javaVersion = javaVersion) {
                     this@should kotlintestShould matcher
                 }
             }

From a0599869456edf51c9328fa0c112e0ac2f87ab7f Mon Sep 17 00:00:00 2001
From: Andreas Dangel <andreas.dangel@pmd-code.org>
Date: Thu, 27 Oct 2022 15:47:07 +0200
Subject: [PATCH 5/7] Bump kotest from 5.5.1 to 5.5.2

---
 pom.xml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/pom.xml b/pom.xml
index ff9804e2c9..ce7e2d3e03 100644
--- a/pom.xml
+++ b/pom.xml
@@ -86,7 +86,7 @@
 
         <kotlin.compiler.jvmTarget>${maven.compiler.test.target}</kotlin.compiler.jvmTarget>
         <kotlin.version>1.7.20</kotlin.version>
-        <kotest.version>5.5.1</kotest.version>
+        <kotest.version>5.5.2</kotest.version>
         <junit5.version>5.8.2</junit5.version> <!-- needed by kotest -->
         <dokka.version>1.7.20</dokka.version>
 

From 2792577da0e890ee8f7c4f665474ab94c1b8fb30 Mon Sep 17 00:00:00 2001
From: Andreas Dangel <andreas.dangel@pmd-code.org>
Date: Fri, 28 Oct 2022 10:46:16 +0200
Subject: [PATCH 6/7] Bump kotest from 5.5.2 to 5.5.3

---
 pmd-lang-test/pom.xml |  5 -----
 pom.xml               | 24 +-----------------------
 2 files changed, 1 insertion(+), 28 deletions(-)

diff --git a/pmd-lang-test/pom.xml b/pmd-lang-test/pom.xml
index 150388601a..52520ed9c9 100644
--- a/pmd-lang-test/pom.xml
+++ b/pmd-lang-test/pom.xml
@@ -99,11 +99,6 @@
             <artifactId>kotest-runner-junit5-jvm</artifactId>
             <scope>compile</scope>
         </dependency>
-        <dependency>
-            <groupId>org.junit.jupiter</groupId>
-            <artifactId>junit-jupiter-api</artifactId>
-            <scope>compile</scope>
-        </dependency>
 
         <dependency>
             <groupId>org.jetbrains</groupId>
diff --git a/pom.xml b/pom.xml
index ce7e2d3e03..fa885b2b77 100644
--- a/pom.xml
+++ b/pom.xml
@@ -86,7 +86,7 @@
 
         <kotlin.compiler.jvmTarget>${maven.compiler.test.target}</kotlin.compiler.jvmTarget>
         <kotlin.version>1.7.20</kotlin.version>
-        <kotest.version>5.5.2</kotest.version>
+        <kotest.version>5.5.3</kotest.version>
         <junit5.version>5.8.2</junit5.version> <!-- needed by kotest -->
         <dokka.version>1.7.20</dokka.version>
 
@@ -279,23 +279,10 @@
                             <version>${junit5.version}</version>
                         </dependency>
                         <!-- Junit5 Platform Engine for Kotlin Tests -->
-                        <dependency>
-                            <groupId>org.junit.jupiter</groupId>
-                            <artifactId>junit-jupiter-api</artifactId>
-                            <version>${junit5.version}</version>
-                            <!-- https://github.com/kotest/kotest/issues/3255 -->
-                        </dependency>
                         <dependency>
                             <groupId>io.kotest</groupId>
                             <artifactId>kotest-runner-junit5-jvm</artifactId>
                             <version>${kotest.version}</version>
-                            <!-- https://github.com/kotest/kotest/issues/3255 -->
-                            <exclusions>
-                                <exclusion>
-                                    <groupId>org.junit.jupiter</groupId>
-                                    <artifactId>junit-jupiter-api</artifactId>
-                                </exclusion>
-                            </exclusions>
                         </dependency>
                     </dependencies>
                 </plugin>
@@ -850,15 +837,6 @@
                 <scope>test</scope>
             </dependency>
 
-            <!-- needed by kotest -->
-            <dependency>
-                <groupId>org.junit.jupiter</groupId>
-                <artifactId>junit-jupiter-api</artifactId>
-                <version>${junit5.version}</version>
-                <scope>test</scope>
-                <!-- https://github.com/kotest/kotest/issues/3255 -->
-            </dependency>
-
             <dependency>
                 <groupId>org.mockito</groupId>
                 <artifactId>mockito-core</artifactId>

From a5b6264ad7c40411bb72afec7a29eefc186531a5 Mon Sep 17 00:00:00 2001
From: Andreas Dangel <andreas.dangel@pmd-code.org>
Date: Fri, 4 Nov 2022 11:05:01 +0100
Subject: [PATCH 7/7] Bump kotest from 5.5.3 to 5.5.4

---
 pom.xml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/pom.xml b/pom.xml
index 719b1c735c..615102c9d7 100644
--- a/pom.xml
+++ b/pom.xml
@@ -86,7 +86,7 @@
 
         <kotlin.compiler.jvmTarget>${maven.compiler.test.target}</kotlin.compiler.jvmTarget>
         <kotlin.version>1.7.20</kotlin.version>
-        <kotest.version>5.5.3</kotest.version>
+        <kotest.version>5.5.4</kotest.version>
         <junit5.version>5.8.2</junit5.version> <!-- needed by kotest -->
         <dokka.version>1.7.20</dokka.version>