Only convert annotation processors when creating Java projects

See gh-1781

Author: mhalbritter

initializr-generator-spring/src/main/java/io/spring/initializr/generator/spring/build/maven/ConvertAnnotationProcessorsToPluginConfigBuildCustomizer.java

@@ -24,6 +24,9 @@
 import io.spring.initializr.generator.buildsystem.Dependency;
 import io.spring.initializr.generator.buildsystem.maven.MavenBuild;
 import io.spring.initializr.generator.buildsystem.maven.MavenPluginContainer;
+import io.spring.initializr.generator.language.Language;
+import io.spring.initializr.generator.language.java.JavaLanguage;
+import io.spring.initializr.generator.project.ProjectDescription;
 import io.spring.initializr.generator.spring.build.BuildCustomizer;
 import io.spring.initializr.generator.version.VersionProperty;
 import io.spring.initializr.generator.version.VersionReference;
@@ -40,8 +43,17 @@
  */
 class ConvertAnnotationProcessorsToPluginConfigBuildCustomizer implements BuildCustomizer<MavenBuild> {
 
+	private final ProjectDescription projectDescription;
+
+	ConvertAnnotationProcessorsToPluginConfigBuildCustomizer(ProjectDescription projectDescription) {
+		this.projectDescription = projectDescription;
+	}
+
 	@Override
 	public void customize(MavenBuild build) {
+		if (!isJava()) {
+			return;
+		}
 		Set<String> ids = build.dependencies().ids().collect(Collectors.toSet());
 		List<Dependency> annotationProcessors = new ArrayList<>();
 		List<Dependency> testAnnotationProcessors = new ArrayList<>();
@@ -120,6 +132,14 @@ private void configureCompilerPlugin(MavenPluginContainer plugins, List<Dependen
 		return versionReference.getValue();
 	}
 
+	private boolean isJava() {
+		Language language = this.projectDescription.getLanguage();
+		if (language == null) {
+			return false;
+		}
+		return language.id().equals(JavaLanguage.ID);
+	}
+
 	@Override
 	public int getOrder() {
 		return Ordered.LOWEST_PRECEDENCE;

initializr-generator-spring/src/main/java/io/spring/initializr/generator/spring/build/maven/MavenProjectGenerationConfiguration.java

@@ -90,8 +90,9 @@ public BuildCustomizer<MavenBuild> mavenWarPackagingConfigurer() {
 	}
 
 	@Bean
-	ConvertAnnotationProcessorsToPluginConfigBuildCustomizer convertAnnotationProcessorsToPluginConfigBuildCustomizer() {
-		return new ConvertAnnotationProcessorsToPluginConfigBuildCustomizer();
+	ConvertAnnotationProcessorsToPluginConfigBuildCustomizer convertAnnotationProcessorsToPluginConfigBuildCustomizer(
+			ProjectDescription projectDescription) {
+		return new ConvertAnnotationProcessorsToPluginConfigBuildCustomizer(projectDescription);
 	}
 
 }

initializr-generator-spring/src/test/java/io/spring/initializr/generator/spring/build/maven/ConvertAnnotationProcessorsToPluginConfigBuildCustomizerTests.java

@@ -23,8 +23,11 @@
 import io.spring.initializr.generator.buildsystem.DependencyScope;
 import io.spring.initializr.generator.buildsystem.maven.MavenBuild;
 import io.spring.initializr.generator.io.IndentingWriterFactory;
+import io.spring.initializr.generator.language.Language;
+import io.spring.initializr.generator.project.MutableProjectDescription;
 import io.spring.initializr.generator.version.VersionProperty;
 import io.spring.initializr.generator.version.VersionReference;
+import org.junit.jupiter.api.BeforeEach;
 import org.junit.jupiter.api.Test;
 
 import static org.assertj.core.api.Assertions.assertThat;
@@ -36,7 +39,14 @@
  */
 class ConvertAnnotationProcessorsToPluginConfigBuildCustomizerTests {
 
-	private final ConvertAnnotationProcessorsToPluginConfigBuildCustomizer customizer = new ConvertAnnotationProcessorsToPluginConfigBuildCustomizer();
+	private ConvertAnnotationProcessorsToPluginConfigBuildCustomizer customizer;
+
+	@BeforeEach
+	void setUp() {
+		MutableProjectDescription projectDescription = new MutableProjectDescription();
+		projectDescription.setLanguage(Language.forId("java", "17"));
+		this.customizer = new ConvertAnnotationProcessorsToPluginConfigBuildCustomizer(projectDescription);
+	}
 
 	@Test
 	void annotationProcessorIsConvertedToDefaultCompileExecution() throws IOException {
@@ -358,6 +368,41 @@ void annotationProcessorWithExclusionsHasExclusionsInPath() throws IOException {
 				""");
 	}
 
+	@Test
+	void annotationProcessorIsNotConvertedForNonJavaLanguage() throws IOException {
+		MutableProjectDescription projectDescription = new MutableProjectDescription();
+		projectDescription.setLanguage(Language.forId("kotlin", "17"));
+		ConvertAnnotationProcessorsToPluginConfigBuildCustomizer kotlinCustomizer = new ConvertAnnotationProcessorsToPluginConfigBuildCustomizer(
+				projectDescription);
+		MavenBuild build = new MavenBuild();
+		build.dependencies()
+			.add("configuration-processor",
+					Dependency.withCoordinates("org.springframework.boot", "spring-boot-configuration-processor")
+						.scope(DependencyScope.ANNOTATION_PROCESSOR)
+						.build());
+		kotlinCustomizer.customize(build);
+		String pom = generatePom(build);
+		assertThat(pom).contains("<dependency>");
+		assertThat(pom).doesNotContain("<annotationProcessorPaths>");
+	}
+
+	@Test
+	void annotationProcessorIsNotConvertedWhenLanguageIsNull() throws IOException {
+		MutableProjectDescription projectDescription = new MutableProjectDescription();
+		ConvertAnnotationProcessorsToPluginConfigBuildCustomizer nullLanguageCustomizer = new ConvertAnnotationProcessorsToPluginConfigBuildCustomizer(
+				projectDescription);
+		MavenBuild build = new MavenBuild();
+		build.dependencies()
+			.add("configuration-processor",
+					Dependency.withCoordinates("org.springframework.boot", "spring-boot-configuration-processor")
+						.scope(DependencyScope.ANNOTATION_PROCESSOR)
+						.build());
+		nullLanguageCustomizer.customize(build);
+		String pom = generatePom(build);
+		assertThat(pom).contains("<dependency>");
+		assertThat(pom).doesNotContain("<annotationProcessorPaths>");
+	}
+
 	private String generatePom(MavenBuild build) throws IOException {
 		StringWriter writer = new StringWriter();
 		new MavenBuildProjectContributor(build, IndentingWriterFactory.withDefaultSettings()).writeBuild(writer);