Skip to content

Commit 67d963d

Browse files
author
John Burns
committed
baseline java 17 and Gradle 9
mark non-testkit base test classes for removal mark spock-based testkit base classes as deprecated remove automatic terms of use agreement
1 parent fbfdda0 commit 67d963d

File tree

13 files changed

+28
-38
lines changed

13 files changed

+28
-38
lines changed

.github/workflows/build.yml

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,6 @@ jobs:
2424
with:
2525
distribution: 'zulu'
2626
java-version: |
27-
11
2827
17
2928
21
3029
${{ matrix.java }}
@@ -33,6 +32,9 @@ jobs:
3332
uses: gradle/actions/setup-gradle@v5
3433
with:
3534
cache-overwrite-existing: true
35+
build-scan-publish: true
36+
build-scan-terms-of-use-url: 'https://gradle.com/terms-of-service'
37+
build-scan-terms-of-use-agree: 'yes'
3638
- name: Gradle build
3739
run: ./gradlew --stacktrace build
3840
env:

.github/workflows/publish.yml

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -38,6 +38,9 @@ jobs:
3838
uses: gradle/actions/setup-gradle@v5
3939
with:
4040
cache-overwrite-existing: true
41+
build-scan-publish: true
42+
build-scan-terms-of-use-url: 'https://gradle.com/terms-of-service'
43+
build-scan-terms-of-use-agree: 'yes'
4144
- name: Gradle build
4245
run: ./gradlew --stacktrace build
4346
- name: Publish candidate

build.gradle.kts

Lines changed: 1 addition & 21 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,5 @@
11
import nebula.plugin.contacts.Contact
2-
import org.jetbrains.kotlin.gradle.dsl.JvmTarget
32
import org.jetbrains.kotlin.gradle.dsl.KotlinVersion
4-
import org.jetbrains.kotlin.gradle.tasks.KotlinCompile
53

64
/*
75
* Copyright 2014-2019 Netflix, Inc.
@@ -60,23 +58,6 @@ tasks.withType<Test> {
6058
useJUnitPlatform()
6159
maxParallelForks = 2
6260
finalizedBy(tasks.named("jacocoTestReport"))
63-
javaLauncher = javaToolchains.launcherFor {
64-
languageVersion = JavaLanguageVersion.of(17)
65-
}
66-
}
67-
tasks.named<JavaCompile>("compileTestJava") {
68-
javaCompiler.set(
69-
javaToolchains.compilerFor {
70-
languageVersion = JavaLanguageVersion.of(17)
71-
}
72-
)
73-
}
74-
tasks.named<KotlinCompile>("compileTestKotlin") {
75-
kotlinJavaToolchain.toolchain.use(
76-
javaToolchains.launcherFor {
77-
languageVersion = JavaLanguageVersion.of(17)
78-
}
79-
)
8061
}
8162

8263
tasks.named("build") {
@@ -85,13 +66,12 @@ tasks.named("build") {
8566

8667
java {
8768
toolchain {
88-
languageVersion = JavaLanguageVersion.of(8)
69+
languageVersion = JavaLanguageVersion.of(17)
8970
}
9071
}
9172

9273
kotlin {
9374
compilerOptions {
94-
jvmTarget.set(JvmTarget.JVM_1_8)
9575
languageVersion.set(KotlinVersion.KOTLIN_2_0)
9676
apiVersion.set(KotlinVersion.KOTLIN_2_0)
9777
}

gradle.lockfile

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@
33
# This file is expected to be part of source control.
44
cglib:cglib-nodep:3.2.2=archRulesTestRuntimeClasspath,runtimeClasspath,testRuntimeClasspath
55
com.netflix.nebula:archrules-common:0.9.0=archRulesCompileClasspath,archRulesRuntimeClasspath,archRulesTestCompileClasspath,archRulesTestRuntimeClasspath
6-
com.netflix.nebula:nebula-archrules-core:0.14.1=archRulesCompileClasspath,archRulesRuntimeClasspath,archRulesTestCompileClasspath,archRulesTestRuntimeClasspath
6+
com.netflix.nebula:nebula-archrules-core:0.15.1=archRulesCompileClasspath,archRulesRuntimeClasspath,archRulesTestCompileClasspath,archRulesTestRuntimeClasspath
77
com.tngtech.archunit:archunit:1.4.1=archRulesCompileClasspath,archRulesRuntimeClasspath,archRulesTestCompileClasspath,archRulesTestRuntimeClasspath
88
junit:junit:4.13.2=archRulesTestCompileClasspath,archRulesTestRuntimeClasspath,compileClasspath,testCompileClasspath,testRuntimeClasspath
99
net.bytebuddy:byte-buddy-agent:1.11.13=testCompileClasspath,testRuntimeClasspath

settings.gradle

Lines changed: 0 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -8,13 +8,6 @@ plugins {
88
id 'com.gradle.develocity' version '4.2'
99
}
1010

11-
develocity {
12-
buildScan {
13-
termsOfUseUrl = 'https://gradle.com/terms-of-service'
14-
termsOfUseAgree = 'yes'
15-
}
16-
}
17-
1811
rootProject.name = 'nebula-test'
1912

2013
includeBuild(".") // allows included build to depend on root project

src/main/groovy/nebula/test/BaseIntegrationSpec.groovy

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -30,7 +30,7 @@ import spock.lang.Specification
3030
*
3131
* This will be removed in the next nebula-test major version
3232
*/
33-
@Deprecated
33+
@Deprecated(forRemoval = true)
3434
abstract class BaseIntegrationSpec extends Specification implements IntegrationBase {
3535
@Rule
3636
TestName testName = new TestName()

src/main/groovy/nebula/test/Integration.groovy

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -33,7 +33,7 @@ import java.util.function.Predicate
3333
* @deprecated in favor of Gradle TestKit-based tests
3434
*/
3535
@CompileStatic
36-
@Deprecated
36+
@Deprecated(forRemoval = true)
3737
abstract trait Integration extends IntegrationBase {
3838
private static final String DEFAULT_REMOTE_DEBUG_JVM_ARGUMENTS = "-agentlib:jdwp=transport=dt_socket,server=y,suspend=y,address=5005"
3939
private static final Integer DEFAULT_DAEMON_MAX_IDLE_TIME_IN_SECONDS_IN_MEMORY_SAFE_MODE = 15;

src/main/groovy/nebula/test/IntegrationBase.groovy

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,9 @@ import org.gradle.api.logging.LogLevel
2525
*
2626
* <p>This is testing framework agnostic and can be either extended (see {@link BaseIntegrationSpec}) or composed, by
2727
* including it inside a test class as field.
28+
* @deprecated Use <a href="https://github.com/nebula-plugins/nebula-test/wiki/TestKit-DSL">TestKit DSL</a> instead
2829
*/
30+
@Deprecated
2931
@CompileStatic
3032
abstract trait IntegrationBase {
3133
File projectDir

src/main/groovy/nebula/test/IntegrationSpec.groovy

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -29,7 +29,7 @@ import groovy.transform.CompileStatic
2929
*
3030
* This will be removed in the next nebula-test major version
3131
*/
32-
@Deprecated
32+
@Deprecated(forRemoval = true)
3333
abstract class IntegrationSpec extends BaseIntegrationSpec implements Integration {
3434
def setup() {
3535
Integration.super.initialize(getClass(), testName.methodName)

src/main/groovy/nebula/test/IntegrationTestKitBase.groovy

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -28,8 +28,10 @@ import static org.gradle.testkit.runner.TaskOutcome.UP_TO_DATE
2828

2929
/**
3030
* Base trait for implementing gradle integration tests using the {@code gradle-test-kit} runner.
31+
* @deprecated Use <a href="https://github.com/nebula-plugins/nebula-test/wiki/TestKit-DSL">TestKit DSL</a> instead
3132
*/
3233
@CompileStatic
34+
@Deprecated
3335
abstract trait IntegrationTestKitBase extends IntegrationBase {
3436
static final String LINE_END = System.getProperty('line.separator')
3537
boolean keepFiles = false

0 commit comments

Comments
 (0)