Test Gradle plugin against Gradle 8.0.1

Closes gh-34457
This commit is contained in:
Andy Wilkinson 2023-03-01 18:09:57 +00:00
parent d39c94f437
commit 4f7e038f14
25 changed files with 172 additions and 121 deletions

View File

@ -1,5 +1,5 @@
/*
* Copyright 2012-2021 the original author or authors.
* Copyright 2012-2023 the original author or authors.
*
* Licensed under the Apache License, Version 2.0 (the "License");
* you may not use this file except in compliance with the License.
@ -31,6 +31,7 @@ public class BootRunClasspathApplication {
}
public static void main(String[] args) {
System.out.println("Main class name = " + BootRunClasspathApplication.class.getName());
int i = 1;
for (String entry : ManagementFactory.getRuntimeMXBean().getClassPath().split(File.pathSeparator)) {
System.out.println(i++ + ". " + entry);

View File

@ -47,53 +47,46 @@ class JavaPluginActionIntegrationTests {
@TestTemplate
void noBootJarTaskWithoutJavaPluginApplied() {
assertThat(this.gradleBuild.build("taskExists", "-PtaskName=bootJar").getOutput())
.contains("bootJar exists = false");
assertThat(this.gradleBuild.build("tasks").getOutput()).doesNotContain("bootJar");
}
@TestTemplate
void applyingJavaPluginCreatesBootJarTask() {
assertThat(this.gradleBuild.build("taskExists", "-PtaskName=bootJar", "-PapplyJavaPlugin").getOutput())
.contains("bootJar exists = true");
assertThat(this.gradleBuild.build("tasks").getOutput()).contains("bootJar");
}
@TestTemplate
void noBootRunTaskWithoutJavaPluginApplied() {
assertThat(this.gradleBuild.build("taskExists", "-PtaskName=bootRun").getOutput())
.contains("bootRun exists = false");
assertThat(this.gradleBuild.build("tasks").getOutput()).doesNotContain("bootRun");
}
@TestTemplate
void applyingJavaPluginCreatesBootRunTask() {
assertThat(this.gradleBuild.build("taskExists", "-PtaskName=bootRun", "-PapplyJavaPlugin").getOutput())
.contains("bootRun exists = true");
assertThat(this.gradleBuild.build("tasks").getOutput()).contains("bootRun");
}
@TestTemplate
void javaCompileTasksUseUtf8Encoding() {
assertThat(this.gradleBuild.build("javaCompileEncoding", "-PapplyJavaPlugin").getOutput())
.contains("compileJava = UTF-8")
assertThat(this.gradleBuild.build("build").getOutput()).contains("compileJava = UTF-8")
.contains("compileTestJava = UTF-8");
}
@TestTemplate
void javaCompileTasksUseParametersCompilerFlagByDefault() {
assertThat(this.gradleBuild.build("javaCompileTasksCompilerArgs").getOutput())
.contains("compileJava compiler args: [-parameters]")
assertThat(this.gradleBuild.build("build").getOutput()).contains("compileJava compiler args: [-parameters]")
.contains("compileTestJava compiler args: [-parameters]");
}
@TestTemplate
void javaCompileTasksUseParametersAndAdditionalCompilerFlags() {
assertThat(this.gradleBuild.build("javaCompileTasksCompilerArgs").getOutput())
assertThat(this.gradleBuild.build("build").getOutput())
.contains("compileJava compiler args: [-parameters, -Xlint:all]")
.contains("compileTestJava compiler args: [-parameters, -Xlint:all]");
}
@TestTemplate
void javaCompileTasksCanOverrideDefaultParametersCompilerFlag() {
assertThat(this.gradleBuild.build("javaCompileTasksCompilerArgs").getOutput())
.contains("compileJava compiler args: [-Xlint:all]")
assertThat(this.gradleBuild.build("build").getOutput()).contains("compileJava compiler args: [-Xlint:all]")
.contains("compileTestJava compiler args: [-Xlint:all]");
}
@ -139,16 +132,12 @@ class JavaPluginActionIntegrationTests {
@TestTemplate
void applyingJavaPluginCreatesDevelopmentOnlyConfiguration() {
assertThat(this.gradleBuild
.build("configurationExists", "-PconfigurationName=developmentOnly", "-PapplyJavaPlugin")
.getOutput()).contains("developmentOnly exists = true");
assertThat(this.gradleBuild.build("build").getOutput()).contains("developmentOnly exists = true");
}
@TestTemplate
void productionRuntimeClasspathIsConfiguredWithAttributes() {
assertThat(this.gradleBuild
.build("configurationAttributes", "-PconfigurationName=productionRuntimeClasspath", "-PapplyJavaPlugin")
.getOutput()).contains("3 productionRuntimeClasspath attributes:")
assertThat(this.gradleBuild.build("build").getOutput()).contains("3 productionRuntimeClasspath attributes:")
.contains("org.gradle.usage: java-runtime")
.contains("org.gradle.libraryelements: jar")
.contains("org.gradle.dependency.bundling: external");
@ -156,18 +145,11 @@ class JavaPluginActionIntegrationTests {
@TestTemplate
void productionRuntimeClasspathIsConfiguredWithResolvabilityAndConsumabilityThatMatchesRuntimeClasspath() {
String runtime = this.gradleBuild
.build("configurationResolvabilityAndConsumability", "-PconfigurationName=runtimeClasspath",
"-PapplyJavaPlugin")
.getOutput();
assertThat(runtime).contains("canBeResolved: true");
assertThat(runtime).contains("canBeConsumed: false");
String productionRuntime = this.gradleBuild
.build("configurationResolvabilityAndConsumability", "-PconfigurationName=productionRuntimeClasspath",
"-PapplyJavaPlugin")
.getOutput();
assertThat(productionRuntime).contains("canBeResolved: true");
assertThat(productionRuntime).contains("canBeConsumed: false");
String output = this.gradleBuild.build("build").getOutput();
assertThat(output).contains("runtimeClasspath canBeResolved: true");
assertThat(output).contains("runtimeClasspath canBeConsumed: false");
assertThat(output).contains("productionRuntimeClasspath canBeResolved: true");
assertThat(output).contains("productionRuntimeClasspath canBeConsumed: false");
}
@TestTemplate

View File

@ -22,7 +22,6 @@ import java.util.Set;
import java.util.TreeSet;
import org.gradle.testkit.runner.BuildResult;
import org.gradle.testkit.runner.TaskOutcome;
import org.junit.jupiter.api.TestTemplate;
import org.springframework.boot.gradle.junit.GradleCompatibility;
@ -45,9 +44,7 @@ class BootJarIntegrationTests extends AbstractBootArchiveIntegrationTests {
@TestTemplate
void whenAResolvableCopyOfAnUnresolvableConfigurationIsResolvedThenResolutionSucceeds() {
BuildResult build = this.gradleBuild.build("resolveResolvableCopyOfUnresolvableConfiguration");
assertThat(build.task(":resolveResolvableCopyOfUnresolvableConfiguration").getOutcome())
.isEqualTo(TaskOutcome.SUCCESS);
this.gradleBuild.expectDeprecationWarningsWithAtLeastVersion("8.0").build("build");
}
@TestTemplate

View File

@ -1,5 +1,5 @@
/*
* Copyright 2012-2021 the original author or authors.
* Copyright 2012-2023 the original author or authors.
*
* Licensed under the Apache License, Version 2.0 (the "License");
* you may not use this file except in compliance with the License.
@ -16,10 +16,14 @@
package org.springframework.boot.gradle.tasks.bundling;
import java.io.IOException;
import java.util.Arrays;
import java.util.Set;
import java.util.TreeSet;
import org.assertj.core.api.Assumptions;
import org.gradle.util.GradleVersion;
import org.springframework.boot.gradle.junit.GradleCompatibility;
/**
@ -42,4 +46,25 @@ class BootWarIntegrationTests extends AbstractBootArchiveIntegrationTests {
return contents.toArray(new String[0]);
}
@Override
void multiModuleImplicitLayers() throws IOException {
whenTestingWithTheConfigurationCacheAssumeThatTheGradleVersionIsLessThan8();
super.multiModuleImplicitLayers();
}
@Override
void multiModuleCustomLayers() throws IOException {
whenTestingWithTheConfigurationCacheAssumeThatTheGradleVersionIsLessThan8();
super.multiModuleCustomLayers();
}
private void whenTestingWithTheConfigurationCacheAssumeThatTheGradleVersionIsLessThan8() {
if (this.gradleBuild.isConfigurationCache()) {
// With Gradle 8.0, a configuration cache bug prevents ResolvedDependencies
// from processing dependencies on the runtime classpath
Assumptions.assumeThat(GradleVersion.version(this.gradleBuild.getGradleVersion()))
.isLessThan(GradleVersion.version("8.0"));
}
}
}

View File

@ -24,9 +24,11 @@ import java.util.jar.Attributes;
import java.util.jar.JarOutputStream;
import java.util.jar.Manifest;
import org.assertj.core.api.Assumptions;
import org.gradle.api.JavaVersion;
import org.gradle.testkit.runner.BuildResult;
import org.gradle.testkit.runner.TaskOutcome;
import org.gradle.util.GradleVersion;
import org.junit.jupiter.api.TestTemplate;
import org.springframework.boot.gradle.junit.GradleCompatibility;
@ -114,6 +116,13 @@ class BootRunIntegrationTests {
@TestTemplate
void applicationPluginJvmArgumentsAreUsed() throws IOException {
if (this.gradleBuild.isConfigurationCache()) {
// https://github.com/gradle/gradle/pull/23924
GradleVersion gradleVersion = GradleVersion.version(this.gradleBuild.getGradleVersion());
Assumptions.assumeThat(gradleVersion)
.isLessThan(GradleVersion.version("8.0"))
.isGreaterThanOrEqualTo(GradleVersion.version("8.1-rc-1"));
}
copyJvmArgsApplication();
BuildResult result = this.gradleBuild.build("bootRun");
assertThat(result.task(":bootRun").getOutcome()).isEqualTo(TaskOutcome.SUCCESS);

View File

@ -0,0 +1,12 @@
plugins {
id 'org.springframework.boot' version '{version}'
id 'java'
}
springBoot {
mainClass = "com.example.Main"
}
gradle.taskGraph.whenReady {
println "developmentOnly exists = ${configurations.findByName('developmentOnly') != null}"
}

View File

@ -3,15 +3,18 @@ plugins {
id 'org.springframework.boot' version '{version}'
}
springBoot {
mainClass = "com.example.Main"
}
tasks.withType(JavaCompile) {
options.compilerArgs = ['-Xlint:all']
}
task('javaCompileTasksCompilerArgs') {
doFirst {
tasks.withType(JavaCompile) {
println "${name} compiler args: ${options.compilerArgs}"
gradle.taskGraph.whenReady {
gradle.taskGraph.allTasks.each {
if (it instanceof JavaCompile) {
println "${it.name} compiler args: ${it.options.compilerArgs}"
}
}
}

View File

@ -3,14 +3,18 @@ plugins {
id 'org.springframework.boot' version '{version}'
}
springBoot {
mainClass = "com.example.Main"
}
tasks.withType(JavaCompile) {
options.compilerArgs << '-Xlint:all'
}
task('javaCompileTasksCompilerArgs') {
doFirst {
tasks.withType(JavaCompile) {
println "${name} compiler args: ${options.compilerArgs}"
gradle.taskGraph.whenReady {
gradle.taskGraph.allTasks.each {
if (it instanceof JavaCompile) {
println "${it.name} compiler args: ${it.options.compilerArgs}"
}
}
}

View File

@ -3,10 +3,14 @@ plugins {
id 'org.springframework.boot' version '{version}'
}
task('javaCompileTasksCompilerArgs') {
doFirst {
tasks.withType(JavaCompile) {
println "${name} compiler args: ${options.compilerArgs}"
springBoot {
mainClass = "com.example.Main"
}
gradle.taskGraph.whenReady {
gradle.taskGraph.allTasks.each {
if (it instanceof JavaCompile) {
println "${it.name} compiler args: ${it.options.compilerArgs}"
}
}
}

View File

@ -0,0 +1,16 @@
plugins {
id 'org.springframework.boot' version '{version}'
id 'java'
}
springBoot {
mainClass = "com.example.Main"
}
gradle.taskGraph.whenReady {
gradle.taskGraph.allTasks.each {
if (it instanceof JavaCompile) {
println "${it.name} = ${it.options.encoding}"
}
}
}

View File

@ -0,0 +1,16 @@
plugins {
id 'org.springframework.boot' version '{version}'
id 'java'
}
springBoot {
mainClass = "com.example.Main"
}
gradle.taskGraph.whenReady {
def attributes = configurations.findByName('productionRuntimeClasspath').attributes
println "${attributes.keySet().size()} productionRuntimeClasspath attributes:"
attributes.keySet().each { attribute ->
println " ${attribute}: ${attributes.getAttribute(attribute)}"
}
}

View File

@ -0,0 +1,19 @@
plugins {
id 'org.springframework.boot' version '{version}'
id 'java'
}
springBoot {
mainClass = "com.example.Main"
}
gradle.taskGraph.whenReady {
analyzeConfiguration('productionRuntimeClasspath')
analyzeConfiguration('runtimeClasspath')
}
def analyzeConfiguration(String configurationName) {
Configuration configuration = configurations.findByName(configurationName)
println "$configurationName canBeResolved: ${configuration.canBeResolved}"
println "$configurationName canBeConsumed: ${configuration.canBeConsumed}"
}

View File

@ -1,49 +0,0 @@
plugins {
id 'org.springframework.boot' version '{version}'
}
if (project.hasProperty('applyJavaPlugin')) {
apply plugin: 'java'
}
task('taskExists') {
doFirst {
println "${taskName} exists = ${tasks.findByName(taskName) != null}"
}
}
task('javaCompileEncoding') {
doFirst {
tasks.withType(JavaCompile) {
println "${name} = ${options.encoding}"
}
}
}
task('configurationExists') {
doFirst {
println "${configurationName} exists = ${configurations.findByName(configurationName) != null}"
}
}
task('configurationAttributes') {
doFirst {
def attributes = configurations.findByName(configurationName).attributes
println "${attributes.keySet().size()} ${configurationName} attributes:"
attributes.keySet().each { attribute ->
println " ${attribute}: ${attributes.getAttribute(attribute)}"
}
}
}
task('configurationResolvabilityAndConsumability') {
if (project.hasProperty("configurationName")) {
Configuration configuration = configurations.findByName(configurationName)
def canBeResolved = configuration.canBeResolved
def canBeConsumed = configuration.canBeConsumed
doFirst {
println "canBeResolved: ${canBeResolved}"
println "canBeConsumed: ${canBeConsumed}"
}
}
}

View File

@ -8,5 +8,5 @@ targetCompatibility = '1.8'
bootBuildImage {
builder = "projects.registry.vmware.com/springboot/spring-boot-cnb-builder:0.0.1"
bindings = [ "${projectDir}/bindings/ca-certificates:/platform/bindings/certificates" ]
bindings = [ "${projectDir}/bindings/ca-certificates:/platform/bindings/certificates" as String ]
}

View File

@ -8,5 +8,5 @@ targetCompatibility = '1.8'
bootBuildImage {
builder = "projects.registry.vmware.com/springboot/spring-boot-cnb-builder:0.0.1"
buildpacks = [ "file://${projectDir}/buildpack/hello-world" ]
buildpacks = [ "file://${projectDir}/buildpack/hello-world" as String ]
}

View File

@ -8,5 +8,5 @@ targetCompatibility = '1.8'
bootBuildImage {
builder = "projects.registry.vmware.com/springboot/spring-boot-cnb-builder:0.0.1"
buildpacks = [ "file://${projectDir}/hello-world.tgz" ]
buildpacks = [ "file://${projectDir}/hello-world.tgz" as String ]
}

View File

@ -7,10 +7,8 @@ bootJar {
mainClass = 'com.example.Application'
}
task resolveResolvableCopyOfUnresolvableConfiguration {
doFirst {
def copy = configurations.implementation.copyRecursive()
copy.canBeResolved = true
copy.resolve()
}
gradle.taskGraph.whenReady {
def copy = configurations.implementation.copyRecursive()
copy.canBeResolved = true
copy.resolve()
}

View File

@ -3,4 +3,6 @@ plugins {
id 'org.springframework.boot' version '{version}'
}
applicationDefaultJvmArgs = ['-Dcom.foo=bar', '-Dcom.bar=baz']
application {
applicationDefaultJvmArgs = ['-Dcom.foo=bar', '-Dcom.bar=baz']
}

View File

@ -2,9 +2,3 @@ plugins {
id 'application'
id 'org.springframework.boot' version '{version}'
}
bootRun {
doFirst {
println "Main class name = ${bootRun.mainClass.get()}"
}
}

View File

@ -34,15 +34,15 @@ public final class GradleVersions {
public static List<String> allCompatible() {
if (isJava18()) {
return Arrays.asList("7.3.3", GradleVersion.current().getVersion());
return Arrays.asList("7.3.3", GradleVersion.current().getVersion(), "8.0.1");
}
if (isJava17()) {
return Arrays.asList("7.2", GradleVersion.current().getVersion());
return Arrays.asList("7.2", GradleVersion.current().getVersion(), "8.0.1");
}
if (isJava16()) {
return Arrays.asList("7.0.2", GradleVersion.current().getVersion());
return Arrays.asList("7.0.2", GradleVersion.current().getVersion(), "8.0.1");
}
return Arrays.asList("6.8.3", "6.9.4", "7.0.2", GradleVersion.current().getVersion());
return Arrays.asList("6.8.3", "6.9.4", "7.0.2", GradleVersion.current().getVersion(), "8.0.1");
}
public static String minimumCompatible() {