Add tests for TypeNames.

Original Pull Request: #3357

Author: christophstrobl

src/test/java/org/springframework/data/javapoet/JavaPoetUnitTests.java

@@ -125,22 +125,22 @@ void shouldRenderSimpleReturn() {
 	void shouldRenderConditionalLongReturn() {
 
 		CodeBlock block = LordOfTheStrings.returning(Long.class).whenLong("1L").whenBoxedLong("$T.valueOf(1)", Long.class)
-				.otherwise("😫").build();
+				.otherwise(":-[").build();
 		assertThat(block).hasToString("return java.lang.Long.valueOf(1)");
 
 		block = LordOfTheStrings.returning(Long.class).whenBoxedLong("$T.valueOf(1)", Long.class).whenLong("1L")
-				.otherwise("😫").build();
+				.otherwise(":-[").build();
 		assertThat(block).hasToString("return java.lang.Long.valueOf(1)");
 
 		block = LordOfTheStrings.returning(long.class).whenBoxedLong("$T.valueOf(1)", Long.class).whenLong("1L")
-				.otherwise("😫").build();
+				.otherwise(":-[").build();
 		assertThat(block).hasToString("return 1L");
 
-		block = LordOfTheStrings.returning(Long.class).whenBoxed(Long.class, "$T.valueOf(1)", Long.class).otherwise("😫")
+		block = LordOfTheStrings.returning(Long.class).whenBoxed(Long.class, "$T.valueOf(1)", Long.class).otherwise(":-[")
 				.build();
 		assertThat(block).hasToString("return java.lang.Long.valueOf(1)");
 
-		block = LordOfTheStrings.returning(Long.class).whenBoxed(long.class, "$T.valueOf(1)", Long.class).otherwise("😫")
+		block = LordOfTheStrings.returning(Long.class).whenBoxed(long.class, "$T.valueOf(1)", Long.class).otherwise(":-[")
 				.build();
 		assertThat(block).hasToString("return java.lang.Long.valueOf(1)");
 	}
@@ -149,44 +149,44 @@ void shouldRenderConditionalLongReturn() {
 	void shouldRenderConditionalIntReturn() {
 
 		CodeBlock block = LordOfTheStrings.returning(Integer.class).whenBoxed(long.class, "$T.valueOf(1)", Long.class)
-				.otherwise("😫").build();
-		assertThat(block).hasToString("return 😫");
+				.otherwise(":-[").build();
+		assertThat(block).hasToString("return :-[");
 
-		block = LordOfTheStrings.returning(Integer.class).whenBoxedInteger("$T.valueOf(1)", Integer.class).otherwise("😫")
+		block = LordOfTheStrings.returning(Integer.class).whenBoxedInteger("$T.valueOf(1)", Integer.class).otherwise(":-[")
 				.build();
 		assertThat(block).hasToString("return java.lang.Integer.valueOf(1)");
 
 		block = LordOfTheStrings.returning(int.class).whenBoxedInteger("$T.valueOf(1)", Integer.class).whenInt("1")
-				.otherwise("😫").build();
+				.otherwise(":-[").build();
 		assertThat(block).hasToString("return 1");
 	}
 
 	@Test // GH-3357
 	void shouldRenderConditionalBooleanReturn() {
 
-		CodeBlock block = LordOfTheStrings.returning(boolean.class).whenBoolean("$L", true).otherwise("😫").build();
+		CodeBlock block = LordOfTheStrings.returning(boolean.class).whenBoolean("$L", true).otherwise(":-[").build();
 		assertThat(block).hasToString("return true");
 
-		block = LordOfTheStrings.returning(Boolean.class).whenBoolean("$L", true).otherwise("😫").build();
+		block = LordOfTheStrings.returning(Boolean.class).whenBoolean("$L", true).otherwise(":-[").build();
 		assertThat(block).hasToString("return true");
 	}
 
 	@Test // GH-3357
 	void shouldRenderConditionalNumericReturn() {
 
-		CodeBlock block = LordOfTheStrings.returning(boolean.class).number("someNumericVariable").otherwise("😫").build();
-		assertThat(block).hasToString("return 😫");
+		CodeBlock block = LordOfTheStrings.returning(boolean.class).number("someNumericVariable").otherwise(":-[").build();
+		assertThat(block).hasToString("return :-[");
 
-		block = LordOfTheStrings.returning(long.class).number("someNumericVariable").otherwise("😫").build();
+		block = LordOfTheStrings.returning(long.class).number("someNumericVariable").otherwise(":-[").build();
 		assertThat(block).hasToString("return someNumericVariable != null ? someNumericVariable.longValue() : 0L");
 
-		block = LordOfTheStrings.returning(Long.class).number("someNumericVariable").otherwise("😫").build();
+		block = LordOfTheStrings.returning(Long.class).number("someNumericVariable").otherwise(":-[").build();
 		assertThat(block).hasToString("return someNumericVariable != null ? someNumericVariable.longValue() : null");
 
-		block = LordOfTheStrings.returning(int.class).number("someNumericVariable").otherwise("😫").build();
+		block = LordOfTheStrings.returning(int.class).number("someNumericVariable").otherwise(":-[").build();
 		assertThat(block).hasToString("return someNumericVariable != null ? someNumericVariable.intValue() : 0");
 
-		block = LordOfTheStrings.returning(Integer.class).number("someNumericVariable").otherwise("😫").build();
+		block = LordOfTheStrings.returning(Integer.class).number("someNumericVariable").otherwise(":-[").build();
 		assertThat(block).hasToString("return someNumericVariable != null ? someNumericVariable.intValue() : null");
 	}
 

src/test/java/org/springframework/data/javapoet/TypeNamesUnitTests.java

@@ -0,0 +1,62 @@
+/*
+ * Copyright 2025 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.
+ * You may obtain a copy of the License at
+ *
+ *      https://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+package org.springframework.data.javapoet;
+
+import static org.assertj.core.api.AssertionsForInterfaceTypes.assertThat;
+
+import java.util.Set;
+import java.util.stream.Stream;
+
+import org.junit.jupiter.params.ParameterizedTest;
+import org.junit.jupiter.params.provider.Arguments;
+import org.junit.jupiter.params.provider.MethodSource;
+import org.springframework.core.ResolvableType;
+import org.springframework.javapoet.ParameterizedTypeName;
+import org.springframework.javapoet.TypeName;
+
+/**
+ * @author Christoph Strobl
+ */
+class TypeNamesUnitTests {
+
+	static Stream<Arguments> typeNames() {
+		return Stream.of(Arguments.of(ResolvableType.forClass(String.class), TypeName.get(String.class)),
+				Arguments.of(ResolvableType.forClassWithGenerics(Set.class, String.class),
+						ParameterizedTypeName.get(Set.class, String.class)),
+				Arguments.of(ResolvableType.forClass(Void.class), TypeName.get(Void.class)),
+				Arguments.of(ResolvableType.forClass(Iterable.class, Object.class), TypeName.get(Iterable.class)));
+	}
+
+	static Stream<Arguments> classNames() {
+		return Stream.of(Arguments.of(ResolvableType.forClass(String.class), TypeName.get(String.class)),
+				Arguments.of(ResolvableType.forClassWithGenerics(Set.class, String.class), TypeName.get(Set.class)),
+				Arguments.of(ResolvableType.forClass(Void.class), TypeName.get(Void.class)),
+				Arguments.of(ResolvableType.forClass(Iterable.class, Object.class), TypeName.get(Iterable.class)));
+	}
+
+	@ParameterizedTest // GH-3357
+	@MethodSource
+	void typeNames(ResolvableType resolvableType, TypeName expected) {
+		assertThat(TypeNames.typeName(resolvableType)).isEqualTo(expected);
+	}
+
+	@ParameterizedTest // GH-3357
+	@MethodSource
+	void classNames(ResolvableType resolvableType, TypeName expected) {
+		assertThat(TypeNames.className(resolvableType)).isEqualTo(expected);
+	}
+
+}