Use testRuntimeOnly for JDBC/R2DBC drivers (#7295)

eddumelendez · web-flow · commit 20f6067cdbae · 2023-07-03T17:12:13.000-06:00
diff --git a/docs/examples/junit4/generic/build.gradle b/docs/examples/junit4/generic/build.gradle
@@ -6,7 +6,7 @@ dependencies {
     testImplementation project(":selenium")
     testImplementation project(":mysql")
 
-    testImplementation 'mysql:mysql-connector-java:8.0.33'
+    testRuntimeOnly 'mysql:mysql-connector-java:8.0.33'
     testImplementation "org.seleniumhq.selenium:selenium-api:4.10.0"
     testImplementation 'org.assertj:assertj-core:3.24.2'
 }
diff --git a/examples/linked-container/build.gradle b/examples/linked-container/build.gradle
@@ -9,7 +9,7 @@ dependencies {
     compileOnly 'org.slf4j:slf4j-api:1.7.36'
     implementation 'com.squareup.okhttp3:okhttp:4.11.0'
     implementation 'org.json:json:20230227'
-    testImplementation 'org.postgresql:postgresql:42.6.0'
+    testRuntimeOnly 'org.postgresql:postgresql:42.6.0'
     testImplementation 'ch.qos.logback:logback-classic:1.3.8'
     testImplementation 'org.testcontainers:postgresql'
     testImplementation 'org.assertj:assertj-core:3.24.2'
diff --git a/modules/clickhouse/build.gradle b/modules/clickhouse/build.gradle
@@ -5,6 +5,6 @@ dependencies {
     api project(':jdbc')
 
     testImplementation project(':jdbc-test')
-    testImplementation 'ru.yandex.clickhouse:clickhouse-jdbc:0.3.2'
+    testRuntimeOnly 'ru.yandex.clickhouse:clickhouse-jdbc:0.3.2'
     testImplementation 'org.assertj:assertj-core:3.24.2'
 }
diff --git a/modules/cockroachdb/build.gradle b/modules/cockroachdb/build.gradle
@@ -4,6 +4,6 @@ dependencies {
     api project(':jdbc')
 
     testImplementation project(':jdbc-test')
-    testImplementation 'org.postgresql:postgresql:42.6.0'
+    testRuntimeOnly 'org.postgresql:postgresql:42.6.0'
     testImplementation 'org.assertj:assertj-core:3.24.2'
 }
diff --git a/modules/cratedb/build.gradle b/modules/cratedb/build.gradle
@@ -4,7 +4,7 @@ dependencies {
     api project(':jdbc')
 
     testImplementation project(':jdbc-test')
-    testImplementation 'org.postgresql:postgresql:42.6.0'
+    testRuntimeOnly 'org.postgresql:postgresql:42.6.0'
 
     compileOnly 'org.jetbrains:annotations:24.0.1'
 }
diff --git a/modules/db2/build.gradle b/modules/db2/build.gradle
@@ -4,6 +4,6 @@ dependencies {
     api project(':jdbc')
 
     testImplementation project(':jdbc-test')
-    testImplementation 'com.ibm.db2:jcc:11.5.8.0'
+    testRuntimeOnly 'com.ibm.db2:jcc:11.5.8.0'
     testImplementation 'org.assertj:assertj-core:3.24.2'
 }
diff --git a/modules/mariadb/build.gradle b/modules/mariadb/build.gradle
@@ -13,5 +13,5 @@ dependencies {
     testImplementation 'org.mariadb.jdbc:mariadb-java-client:3.1.4'
 
     testImplementation testFixtures(project(':r2dbc'))
-    testImplementation 'org.mariadb:r2dbc-mariadb:1.0.3'
+    testRuntimeOnly 'org.mariadb:r2dbc-mariadb:1.0.3'
 }
diff --git a/modules/mssqlserver/build.gradle b/modules/mssqlserver/build.gradle
@@ -13,7 +13,7 @@ dependencies {
     testImplementation 'com.microsoft.sqlserver:mssql-jdbc:12.3.1.jre8-preview'
 
     testImplementation project(':r2dbc')
-    testImplementation 'io.r2dbc:r2dbc-mssql:1.0.1.RELEASE'
+    testRuntimeOnly 'io.r2dbc:r2dbc-mssql:1.0.1.RELEASE'
 
     // MSSQL's wait strategy requires the JDBC driver
     testImplementation testFixtures(project(':r2dbc'))
diff --git a/modules/mysql/build.gradle b/modules/mysql/build.gradle
@@ -10,10 +10,10 @@ dependencies {
     compileOnly 'io.asyncer:r2dbc-mysql:1.0.2'
 
     testImplementation project(':jdbc-test')
-    testImplementation 'mysql:mysql-connector-java:8.0.33'
+    testRuntimeOnly 'mysql:mysql-connector-java:8.0.33'
 
     testImplementation testFixtures(project(':r2dbc'))
-    testImplementation 'io.asyncer:r2dbc-mysql:1.0.2'
+    testRuntimeOnly 'io.asyncer:r2dbc-mysql:1.0.2'
 
     compileOnly 'org.jetbrains:annotations:24.0.1'
 }
diff --git a/modules/oracle-xe/build.gradle b/modules/oracle-xe/build.gradle
@@ -15,7 +15,7 @@ dependencies {
     compileOnly 'org.jetbrains:annotations:24.0.1'
 
     testImplementation testFixtures(project(':r2dbc'))
-    testImplementation 'com.oracle.database.r2dbc:oracle-r2dbc:1.1.1'
+    testRuntimeOnly 'com.oracle.database.r2dbc:oracle-r2dbc:1.1.1'
 }
 
 test {
diff --git a/modules/postgresql/build.gradle b/modules/postgresql/build.gradle
@@ -10,10 +10,10 @@ dependencies {
     compileOnly 'io.r2dbc:r2dbc-postgresql:0.8.13.RELEASE'
 
     testImplementation project(':jdbc-test')
-    testImplementation 'org.postgresql:postgresql:42.6.0'
+    testRuntimeOnly 'org.postgresql:postgresql:42.6.0'
 
     testImplementation testFixtures(project(':r2dbc'))
-    testImplementation 'io.r2dbc:r2dbc-postgresql:0.8.13.RELEASE'
+    testRuntimeOnly 'io.r2dbc:r2dbc-postgresql:0.8.13.RELEASE'
 
     compileOnly 'org.jetbrains:annotations:24.0.1'
 }
diff --git a/modules/presto/build.gradle b/modules/presto/build.gradle
@@ -4,6 +4,6 @@ dependencies {
     api project(':jdbc')
 
     testImplementation project(':jdbc-test')
-    testImplementation 'io.prestosql:presto-jdbc:350'
+    testRuntimeOnly 'io.prestosql:presto-jdbc:350'
     compileOnly 'org.jetbrains:annotations:24.0.1'
 }
diff --git a/modules/questdb/build.gradle b/modules/questdb/build.gradle
@@ -4,7 +4,7 @@ dependencies {
     api project(':testcontainers')
     api project(':jdbc')
 
-    testImplementation 'org.postgresql:postgresql:42.6.0'
+    testRuntimeOnly 'org.postgresql:postgresql:42.6.0'
     testImplementation project(':jdbc-test')
     testImplementation 'org.assertj:assertj-core:3.24.2'
     testImplementation 'org.questdb:questdb:7.2'
diff --git a/modules/tidb/build.gradle b/modules/tidb/build.gradle
@@ -4,7 +4,7 @@ dependencies {
     api project(':jdbc')
 
     testImplementation project(':jdbc-test')
-    testImplementation 'mysql:mysql-connector-java:8.0.33'
+    testRuntimeOnly 'mysql:mysql-connector-java:8.0.33'
 
     compileOnly 'org.jetbrains:annotations:24.0.1'
 }
diff --git a/modules/trino/build.gradle b/modules/trino/build.gradle
@@ -4,6 +4,6 @@ dependencies {
     api project(':jdbc')
 
     testImplementation project(':jdbc-test')
-    testImplementation 'io.trino:trino-jdbc:420'
+    testRuntimeOnly 'io.trino:trino-jdbc:420'
     compileOnly 'org.jetbrains:annotations:24.0.1'
 }
diff --git a/modules/yugabytedb/build.gradle b/modules/yugabytedb/build.gradle
@@ -6,5 +6,5 @@ dependencies {
     // YCQL driver
     testImplementation 'com.yugabyte:java-driver-core:4.6.0-yb-12'
     // YSQL driver
-    testImplementation 'com.yugabyte:jdbc-yugabytedb:42.3.5-yb-3'
+    testRuntimeOnly 'com.yugabyte:jdbc-yugabytedb:42.3.5-yb-3'
 }

Original file line number	Diff line number	Diff line change
`@@ -6,7 +6,7 @@ dependencies {`
`6`	`6`	`testImplementation project(":selenium")`
`7`	`7`	`testImplementation project(":mysql")`
`8`	`8`
`9`		`- testImplementation 'mysql:mysql-connector-java:8.0.33'`
	`9`	`+ testRuntimeOnly 'mysql:mysql-connector-java:8.0.33'`
`10`	`10`	`testImplementation "org.seleniumhq.selenium:selenium-api:4.10.0"`
`11`	`11`	`testImplementation 'org.assertj:assertj-core:3.24.2'`
`12`	`12`	`}`
Original file line number	Diff line number	Diff line change
`@@ -5,6 +5,6 @@ dependencies {`
`5`	`5`	`api project(':jdbc')`
`6`	`6`
`7`	`7`	`testImplementation project(':jdbc-test')`
`8`		`- testImplementation 'ru.yandex.clickhouse:clickhouse-jdbc:0.3.2'`
	`8`	`+ testRuntimeOnly 'ru.yandex.clickhouse:clickhouse-jdbc:0.3.2'`
`9`	`9`	`testImplementation 'org.assertj:assertj-core:3.24.2'`
`10`	`10`	`}`
Original file line number	Diff line number	Diff line change
`@@ -4,6 +4,6 @@ dependencies {`
`4`	`4`	`api project(':jdbc')`
`5`	`5`
`6`	`6`	`testImplementation project(':jdbc-test')`
`7`		`- testImplementation 'org.postgresql:postgresql:42.6.0'`
	`7`	`+ testRuntimeOnly 'org.postgresql:postgresql:42.6.0'`
`8`	`8`	`testImplementation 'org.assertj:assertj-core:3.24.2'`
`9`	`9`	`}`
Original file line number	Diff line number	Diff line change
`@@ -4,7 +4,7 @@ dependencies {`
`4`	`4`	`api project(':jdbc')`
`5`	`5`
`6`	`6`	`testImplementation project(':jdbc-test')`
`7`		`- testImplementation 'org.postgresql:postgresql:42.6.0'`
	`7`	`+ testRuntimeOnly 'org.postgresql:postgresql:42.6.0'`
`8`	`8`
`9`	`9`	`compileOnly 'org.jetbrains:annotations:24.0.1'`
`10`	`10`	`}`
Original file line number	Diff line number	Diff line change
`@@ -13,5 +13,5 @@ dependencies {`
`13`	`13`	`testImplementation 'org.mariadb.jdbc:mariadb-java-client:3.1.4'`
`14`	`14`
`15`	`15`	`testImplementation testFixtures(project(':r2dbc'))`
`16`		`- testImplementation 'org.mariadb:r2dbc-mariadb:1.0.3'`
	`16`	`+ testRuntimeOnly 'org.mariadb:r2dbc-mariadb:1.0.3'`
`17`	`17`	`}`
Original file line number	Diff line number	Diff line change
`@@ -15,7 +15,7 @@ dependencies {`
`15`	`15`	`compileOnly 'org.jetbrains:annotations:24.0.1'`
`16`	`16`
`17`	`17`	`testImplementation testFixtures(project(':r2dbc'))`
`18`		`- testImplementation 'com.oracle.database.r2dbc:oracle-r2dbc:1.1.1'`
	`18`	`+ testRuntimeOnly 'com.oracle.database.r2dbc:oracle-r2dbc:1.1.1'`
`19`	`19`	`}`
`20`	`20`
`21`	`21`	`test {`