Skip to content

Commit

Permalink
Disable build-logic
Browse files Browse the repository at this point in the history
  • Loading branch information
dmikurube committed Sep 11, 2024
1 parent b9deee5 commit 92e4310
Show file tree
Hide file tree
Showing 9 changed files with 17 additions and 6 deletions.
7 changes: 7 additions & 0 deletions build-logic-obsolete/README.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,7 @@
The build-logic (equivalent to a Gradle plugin) does not seem to be required.

Only adding a `TestEngine` in the test runtime classpath would load the `TestEngine` in the test.

https://github.com/gradle/gradle/blob/v8.10.0/platforms/documentation/docs/src/docs/userguide/jvm/java_testing.adoc#filtering-test-engine

https://github.com/gradle/gradle/tree/v8.10.0/platforms/documentation/docs/src/snippets/testing/junitplatform-engine
File renamed without changes.
File renamed without changes.
14 changes: 9 additions & 5 deletions build.gradle
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
plugins {
id "java"
id "embulk-plugin-test"
// id "embulk-plugin-test"
}

repositories {
Expand All @@ -14,10 +14,11 @@ configurations {
compileClasspath.resolutionStrategy.activateDependencyLocking()
runtimeClasspath.resolutionStrategy.activateDependencyLocking()

embulkTestImplementation.extendsFrom testImplementation
embulkTestRuntime.extendsFrom testRuntime
// embulkTestImplementation.extendsFrom testImplementation
// embulkTestRuntime.extendsFrom testRuntime
}

/*
sourceSets {
embulkTest {
java {
Expand All @@ -26,15 +27,16 @@ sourceSets {
resources.srcDir file("src/embulkTest/resources")
}
}
*/

dependencies {
testImplementation "org.junit.jupiter:junit-jupiter-api:5.11.0"

testRuntimeOnly "org.junit.jupiter:junit-jupiter-engine:5.11.0"

embulkTestImplementation "org.junit.jupiter:junit-jupiter-api:5.11.0"
// embulkTestImplementation "org.junit.jupiter:junit-jupiter-api:5.11.0"

embulkTestRuntimeOnly "org.junit.jupiter:junit-jupiter-engine:5.11.0"
// embulkTestRuntimeOnly "org.junit.jupiter:junit-jupiter-engine:5.11.0"
}

// A safer and strict alternative to: "dependencies" (and "dependencies --write-locks")
Expand Down Expand Up @@ -65,6 +67,7 @@ test {
}
}

/*
task embulkTest(type: Test) {
classpath = sourceSets.embulkTest.runtimeClasspath
testClassesDirs = sourceSets.embulkTest.output.classesDirs
Expand All @@ -80,3 +83,4 @@ task embulkTest(type: Test) {
outputs.upToDateWhen { false }
}
}
*/
2 changes: 1 addition & 1 deletion settings.gradle
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
rootProject.name = "embulk"
includeBuild "build-logic"
// includeBuild "build-logic"
include "embulk-junit5-api"
include "embulk-junit5-engine"

0 comments on commit 92e4310

Please sign in to comment.