Browse Source

Polish "Only set imagePlatform if it has text"

See gh-43424
pull/43429/head
Moritz Halbritter 1 year ago
parent
commit
4dca6ee5d3
  1. 2
      spring-boot-project/spring-boot-tools/spring-boot-maven-plugin/src/main/java/org/springframework/boot/maven/Image.java
  2. 9
      spring-boot-project/spring-boot-tools/spring-boot-maven-plugin/src/test/java/org/springframework/boot/maven/ImageTests.java

2
spring-boot-project/spring-boot-tools/spring-boot-maven-plugin/src/main/java/org/springframework/boot/maven/Image.java

@ -295,7 +295,7 @@ public class Image { @@ -295,7 +295,7 @@ public class Image {
if (StringUtils.hasText(this.applicationDirectory)) {
request = request.withApplicationDirectory(this.applicationDirectory);
}
if (!CollectionUtils.isEmpty(this.securityOptions)) {
if (this.securityOptions != null) {
request = request.withSecurityOptions(this.securityOptions);
}
if (StringUtils.hasText(this.imagePlatform)) {

9
spring-boot-project/spring-boot-tools/spring-boot-maven-plugin/src/test/java/org/springframework/boot/maven/ImageTests.java

@ -49,6 +49,7 @@ import static org.assertj.core.api.Assertions.entry; @@ -49,6 +49,7 @@ import static org.assertj.core.api.Assertions.entry;
* @author Scott Frederick
* @author Jeroen Meijer
* @author Rafael Ceccone
* @author Moritz Halbritter
*/
class ImageTests {
@ -290,6 +291,14 @@ class ImageTests { @@ -290,6 +291,14 @@ class ImageTests {
assertThat(request.getImagePlatform()).isEqualTo(ImagePlatform.of("linux/arm64"));
}
@Test
void getBuildRequestWhenImagePlatformIsEmptyDoesntSetImagePlatform() {
Image image = new Image();
image.imagePlatform = "";
BuildRequest request = image.getBuildRequest(createArtifact(), mockApplicationContent());
assertThat(request.getImagePlatform()).isNull();
}
private Artifact createArtifact() {
return new DefaultArtifact("com.example", "my-app", VersionRange.createFromVersion("0.0.1-SNAPSHOT"), "compile",
"jar", null, new DefaultArtifactHandler());

Loading…
Cancel
Save