Move test logback config to ktor-server-test-base

ktorio · Jan 17, 2025 · b2af3f8 · b2af3f8
1 parent 2a88824
commit b2af3f8
Show file tree

Hide file tree

Showing 20 changed files with 37 additions and 78 deletions.
diff --git a/build.gradle.kts b/build.gradle.kts
@@ -57,10 +57,6 @@ subprojects {
     configureTargets()
     if (CI) configureTestTasksOnCi()
 
-    configurations {
-        maybeCreate("testOutput")
-    }
-
     kotlin {
         if (!internalProjects.contains(project.name)) explicitApi()
 

diff --git a/buildSrc/src/main/kotlin/JvmConfig.kt b/buildSrc/src/main/kotlin/JvmConfig.kt
@@ -34,20 +34,6 @@ fun Project.configureJvm() {
         }
     }
 
-    tasks.register<Jar>("jarTest") {
-        dependsOn(tasks.named("jvmTestClasses"))
-        archiveClassifier = "test"
-        from(kotlin.jvm().compilations["test"].output)
-    }
-
-    configurations {
-        val testCompile = findByName("testCompile") ?: return@configurations
-
-        val testOutput by creating {
-            extendsFrom(testCompile)
-        }
-    }
-
     val testJdk = project.testJdk
     val jvmTest = tasks.named<KotlinJvmTest>("jvmTest") {
         maxHeapSize = "2g"

diff --git a/ktor-server/build.gradle.kts b/ktor-server/build.gradle.kts
@@ -1,5 +1,5 @@
 /*
- * Copyright 2014-2024 JetBrains s.r.o and contributors. Use of this source code is governed by the Apache 2.0 license.
+ * Copyright 2014-2025 JetBrains s.r.o and contributors. Use of this source code is governed by the Apache 2.0 license.
  */
 
 description = "Wrapper for ktor-server-core and base plugins"
@@ -34,7 +34,3 @@ kotlin.sourceSets {
         }
     }
 }
-
-artifacts {
-    add("testOutput", tasks.named("jarTest"))
-}
diff --git a/ktor-server/ktor-server-cio/build.gradle.kts b/ktor-server/ktor-server-cio/build.gradle.kts
@@ -20,10 +20,4 @@ kotlin.sourceSets {
             api(project(":ktor-server:ktor-server-test-base"))
         }
     }
-    jvmTest {
-        dependencies {
-            api(project(":ktor-server:ktor-server-core", configuration = "testOutput"))
-            api(libs.logback.classic)
-        }
-    }
 }
diff --git a/ktor-server/ktor-server-core/build.gradle.kts b/ktor-server/ktor-server-core/build.gradle.kts
@@ -1,5 +1,5 @@
 /*
- * Copyright 2014-2024 JetBrains s.r.o and contributors. Use of this source code is governed by the Apache 2.0 license.
+ * Copyright 2014-2025 JetBrains s.r.o and contributors. Use of this source code is governed by the Apache 2.0 license.
  */
 
 description = ""
@@ -41,14 +41,9 @@ kotlin {
                 implementation(project(":ktor-server:ktor-server-config-yaml"))
                 implementation(project(":ktor-server:ktor-server-test-base"))
                 implementation(project(":ktor-server:ktor-server-test-suites"))
-
-                api(libs.logback.classic)
+
                 implementation(libs.mockk)
             }
         }
     }
 }
-
-artifacts {
-    add("testOutput", tasks.named("jarTest"))
-}
diff --git a/ktor-server/ktor-server-jetty-jakarta/build.gradle.kts b/ktor-server/ktor-server-jetty-jakarta/build.gradle.kts
@@ -1,5 +1,5 @@
 /*
- * Copyright 2014-2024 JetBrains s.r.o and contributors. Use of this source code is governed by the Apache 2.0 license.
+ * Copyright 2014-2025 JetBrains s.r.o and contributors. Use of this source code is governed by the Apache 2.0 license.
  */
 
 description = ""
@@ -26,8 +26,6 @@ kotlin {
                 api(project(":ktor-server:ktor-server-test-suites"))
 
                 api(libs.jetty.servlet.jakarta)
-                api(project(":ktor-server:ktor-server-core", configuration = "testOutput"))
-                api(libs.logback.classic)
             }
         }
     }

diff --git a/ktor-server/ktor-server-jetty-jakarta/ktor-server-jetty-test-http2-jakarta/build.gradle.kts b/ktor-server/ktor-server-jetty-jakarta/ktor-server-jetty-test-http2-jakarta/build.gradle.kts
@@ -1,5 +1,5 @@
 /*
- * Copyright 2014-2024 JetBrains s.r.o and contributors. Use of this source code is governed by the Apache 2.0 license.
+ * Copyright 2014-2025 JetBrains s.r.o and contributors. Use of this source code is governed by the Apache 2.0 license.
  */
 
 import org.jetbrains.kotlin.gradle.targets.jvm.tasks.KotlinJvmTest
@@ -12,9 +12,6 @@ kotlin.sourceSets {
             api(libs.jetty.servlet.jakarta)
             api(project(":ktor-server:ktor-server-core"))
             api(project(":ktor-server:ktor-server-jetty-jakarta"))
-            api(project(":ktor-server:ktor-server-core", configuration = "testOutput"))
-
-            api(libs.logback.classic)
         }
     }
 }

diff --git a/ktor-server/ktor-server-jetty/build.gradle.kts b/ktor-server/ktor-server-jetty/build.gradle.kts
@@ -1,5 +1,5 @@
 /*
- * Copyright 2014-2024 JetBrains s.r.o and contributors. Use of this source code is governed by the Apache 2.0 license.
+ * Copyright 2014-2025 JetBrains s.r.o and contributors. Use of this source code is governed by the Apache 2.0 license.
  */
 
 description = ""
@@ -25,8 +25,6 @@ kotlin {
                 api(project(":ktor-server:ktor-server-test-suites"))
 
                 api(libs.jetty.servlet)
-                api(project(":ktor-server:ktor-server-core", configuration = "testOutput"))
-                api(libs.logback.classic)
             }
         }
     }

diff --git a/ktor-server/ktor-server-jetty/ktor-server-jetty-test-http2/build.gradle.kts b/ktor-server/ktor-server-jetty/ktor-server-jetty-test-http2/build.gradle.kts
@@ -1,5 +1,5 @@
 /*
- * Copyright 2014-2024 JetBrains s.r.o and contributors. Use of this source code is governed by the Apache 2.0 license.
+ * Copyright 2014-2025 JetBrains s.r.o and contributors. Use of this source code is governed by the Apache 2.0 license.
  */
 
 import org.jetbrains.kotlin.gradle.targets.jvm.tasks.KotlinJvmTest
@@ -12,9 +12,6 @@ kotlin.sourceSets {
             api(libs.jetty.servlet)
             api(project(":ktor-server:ktor-server-core"))
             api(project(":ktor-server:ktor-server-jetty"))
-            api(project(":ktor-server:ktor-server-core", configuration = "testOutput"))
-
-            api(libs.logback.classic)
         }
     }
 }

diff --git a/ktor-server/ktor-server-netty/build.gradle.kts b/ktor-server/ktor-server-netty/build.gradle.kts
@@ -1,5 +1,5 @@
 /*
- * Copyright 2014-2024 JetBrains s.r.o and contributors. Use of this source code is governed by the Apache 2.0 license.
+ * Copyright 2014-2025 JetBrains s.r.o and contributors. Use of this source code is governed by the Apache 2.0 license.
  */
 
 import org.jetbrains.kotlin.gradle.targets.jvm.tasks.KotlinJvmTest
@@ -43,9 +43,6 @@ kotlin.sourceSets {
             api(libs.netty.tcnative)
             api(libs.netty.tcnative.boringssl.static)
             api(libs.mockk)
-            api(libs.logback.classic)
-
-            api(project(":ktor-server:ktor-server-core", configuration = "testOutput"))
         }
     }
 }

diff --git a/ktor-server/ktor-server-plugins/build.gradle.kts b/ktor-server/ktor-server-plugins/build.gradle.kts
@@ -1,5 +1,5 @@
 /*
- * Copyright 2014-2024 JetBrains s.r.o and contributors. Use of this source code is governed by the Apache 2.0 license.
+ * Copyright 2014-2025 JetBrains s.r.o and contributors. Use of this source code is governed by the Apache 2.0 license.
  */
 
 subprojects {
@@ -18,11 +18,7 @@ subprojects {
 
             jvmTest {
                 dependencies {
-                    api(project(":ktor-server:ktor-server-core", configuration = "testOutput"))
-
-                    // Version catalogs aren't accessible directly inside subprojects block
-                    // https://github.com/gradle/gradle/issues/16634#issuecomment-809345790
-                    api(rootProject.libs.logback.classic)
+                    implementation(project(":ktor-server:ktor-server-test-base"))
                 }
             }
         }

diff --git a/ktor-server/ktor-server-servlet-jakarta/build.gradle.kts b/ktor-server/ktor-server-servlet-jakarta/build.gradle.kts
@@ -1,3 +1,7 @@
+/*
+ * Copyright 2014-2025 JetBrains s.r.o and contributors. Use of this source code is governed by the Apache 2.0 license.
+ */
+
 description = ""
 
 kotlin.sourceSets {
@@ -11,7 +15,6 @@ kotlin.sourceSets {
 
     jvmTest {
         dependencies {
-            api(project(":ktor-server:ktor-server-core", configuration = "testOutput"))
             api(project(":ktor-server:ktor-server-config-yaml"))
             implementation(libs.mockk)
             implementation(libs.jakarta.servlet)

diff --git a/ktor-server/ktor-server-servlet/build.gradle.kts b/ktor-server/ktor-server-servlet/build.gradle.kts
@@ -1,3 +1,7 @@
+/*
+ * Copyright 2014-2025 JetBrains s.r.o and contributors. Use of this source code is governed by the Apache 2.0 license.
+ */
+
 description = ""
 
 kotlin.sourceSets {
@@ -10,7 +14,6 @@ kotlin.sourceSets {
 
     jvmTest {
         dependencies {
-            api(project(":ktor-server:ktor-server-core", configuration = "testOutput"))
             api(project(":ktor-server:ktor-server-config-yaml"))
             implementation(libs.mockk)
             implementation(libs.javax.servlet)

diff --git a/ktor-server/ktor-server-test-base/build.gradle.kts b/ktor-server/ktor-server-test-base/build.gradle.kts
@@ -19,6 +19,8 @@ kotlin.sourceSets {
             api(project(":ktor-client:ktor-client-apache"))
             api(project(":ktor-network:ktor-network-tls:ktor-network-tls-certificates"))
             api(project(":ktor-server:ktor-server-plugins:ktor-server-call-logging"))
+
+            api(libs.logback.classic)
         }
     }
 }
diff --git a/...-core/jvm/test-resources/logback-test.xml → ...-test-base/jvm/resources/logback-test.xml b/...-core/jvm/test-resources/logback-test.xml → ...-test-base/jvm/resources/logback-test.xml
@@ -1,3 +1,7 @@
+<!--
+  ~ Copyright 2014-2025 JetBrains s.r.o and contributors. Use of this source code is governed by the Apache 2.0 license.
+  -->
+
 <configuration>
     <appender name="STDOUT" class="ch.qos.logback.core.ConsoleAppender">
         <encoder>

diff --git a/ktor-server/ktor-server-test-host/build.gradle.kts b/ktor-server/ktor-server-test-host/build.gradle.kts
@@ -1,5 +1,5 @@
 /*
- * Copyright 2014-2024 JetBrains s.r.o and contributors. Use of this source code is governed by the Apache 2.0 license.
+ * Copyright 2014-2025 JetBrains s.r.o and contributors. Use of this source code is governed by the Apache 2.0 license.
  */
 
 description = ""
@@ -34,7 +34,6 @@ kotlin.sourceSets {
 
     jvmTest {
         dependencies {
-            api(project(":ktor-server:ktor-server-core", configuration = "testOutput"))
             api(project(":ktor-server:ktor-server-config-yaml"))
             api(libs.kotlin.test)
         }

diff --git a/ktor-server/ktor-server-test-suites/build.gradle.kts b/ktor-server/ktor-server-test-suites/build.gradle.kts
@@ -25,12 +25,4 @@ kotlin.sourceSets {
             implementation(project(":ktor-server:ktor-server-plugins:ktor-server-request-validation"))
         }
     }
-
-    jvmTest {
-        dependencies {
-            api(project(":ktor-server:ktor-server-core", configuration = "testOutput"))
-
-            api(libs.logback.classic)
-        }
-    }
 }
diff --git a/ktor-server/ktor-server-tests/build.gradle.kts b/ktor-server/ktor-server-tests/build.gradle.kts
@@ -1,3 +1,7 @@
+/*
+ * Copyright 2014-2025 JetBrains s.r.o and contributors. Use of this source code is governed by the Apache 2.0 license.
+ */
+
 description = ""
 
 plugins {
@@ -16,8 +20,6 @@ kotlin.sourceSets {
         dependencies {
             implementation(libs.jansi)
             implementation(project(":ktor-client:ktor-client-plugins:ktor-client-encoding"))
-            api(project(":ktor-server:ktor-server-core", configuration = "testOutput"))
-            api(libs.logback.classic)
             api(project(":ktor-server:ktor-server-plugins:ktor-server-sse"))
         }
     }

diff --git a/ktor-server/ktor-server-tomcat-jakarta/build.gradle.kts b/ktor-server/ktor-server-tomcat-jakarta/build.gradle.kts
@@ -1,3 +1,7 @@
+/*
+ * Copyright 2014-2025 JetBrains s.r.o and contributors. Use of this source code is governed by the Apache 2.0 license.
+ */
+
 description = ""
 
 kotlin.sourceSets {
@@ -14,8 +18,6 @@ kotlin.sourceSets {
             api(project(":ktor-server:ktor-server-test-base"))
             api(project(":ktor-server:ktor-server-test-suites"))
             api(project(":ktor-server:ktor-server-core"))
-            api(project(":ktor-server:ktor-server-core", configuration = "testOutput"))
-            api(libs.logback.classic)
         }
     }
 }
diff --git a/ktor-server/ktor-server-tomcat/build.gradle.kts b/ktor-server/ktor-server-tomcat/build.gradle.kts
@@ -1,3 +1,7 @@
+/*
+ * Copyright 2014-2025 JetBrains s.r.o and contributors. Use of this source code is governed by the Apache 2.0 license.
+ */
+
 description = ""
 
 kotlin.sourceSets {
@@ -14,8 +18,6 @@ kotlin.sourceSets {
             api(project(":ktor-server:ktor-server-test-base"))
             api(project(":ktor-server:ktor-server-test-suites"))
             api(project(":ktor-server:ktor-server-core"))
-            api(project(":ktor-server:ktor-server-core", configuration = "testOutput"))
-            api(libs.logback.classic)
         }
     }
 }