Polish "Adapt FieldHint to recent GraalVM versions"

See gh-29130

Author: snicoll

spring-beans/src/test/java/org/springframework/beans/factory/annotation/AutowiredAnnotationBeanRegistrationAotContributionTests.java

@@ -73,7 +73,7 @@ void contributeWhenPrivateFieldInjectionInjectsUsingReflection() {
 		RegisteredBean registeredBean = getAndApplyContribution(
 				PrivateFieldInjectionSample.class);
 		assertThat(RuntimeHintsPredicates.reflection()
-				.onField(PrivateFieldInjectionSample.class, "environment").withWriteMode())
+				.onField(PrivateFieldInjectionSample.class, "environment"))
 				.accepts(this.generationContext.getRuntimeHints());
 		compile(registeredBean, (postProcessor, compiled) -> {
 			PrivateFieldInjectionSample instance = new PrivateFieldInjectionSample();
@@ -92,7 +92,7 @@ void contributeWhenPackagePrivateFieldInjectionInjectsUsingConsumer() {
 		RegisteredBean registeredBean = getAndApplyContribution(
 				PackagePrivateFieldInjectionSample.class);
 		assertThat(RuntimeHintsPredicates.reflection()
-				.onField(PackagePrivateFieldInjectionSample.class, "environment").withWriteMode())
+				.onField(PackagePrivateFieldInjectionSample.class, "environment"))
 				.accepts(this.generationContext.getRuntimeHints());
 		compile(registeredBean, (postProcessor, compiled) -> {
 			PackagePrivateFieldInjectionSample instance = new PackagePrivateFieldInjectionSample();

spring-core-test/src/main/java/org/springframework/aot/agent/InstrumentedMethod.java

@@ -290,7 +290,7 @@ enum InstrumentedMethod {
 	 * {@link Field#set(Object, Object)}.
 	 */
 	FIELD_SET(Field.class, "set", HintType.REFLECTION,
-			invocation -> reflection().onField(invocation.getInstance()).withWriteMode()),
+			invocation -> reflection().onField(invocation.getInstance())),
 
 
 	/*

spring-core/src/test/java/org/springframework/aot/hint/FieldHintTests.java renamed to spring-core/src/main/java/org/springframework/aot/hint/FieldHint.java

@@ -16,23 +16,18 @@
 
 package org.springframework.aot.hint;
 
-import org.junit.jupiter.api.Test;
-
-import static org.assertj.core.api.Assertions.assertThat;
+import java.lang.reflect.Field;
 
 /**
- * Tests for {@link FieldHint}.
+ * A hint that describes the need for reflection on a {@link Field}.
  *
- * @author Phillip Webb
+ * @author Stephane Nicoll
+ * @since 6.0
  */
-class FieldHintTests {
+public final class FieldHint extends MemberHint {
 
-	@Test
-	void builtWithAppliesMode() {
-		FieldHint.Builder builder = new FieldHint.Builder("test");
-		assertThat(builder.build().getMode()).isEqualTo(FieldMode.WRITE);
-		FieldHint.builtWith(FieldMode.READ).accept(builder);
-		assertThat(builder.build().getMode()).isEqualTo(FieldMode.READ);
+	FieldHint(String name) {
+		super(name);
 	}
 
 }

spring-core/src/main/java/org/springframework/aot/hint/FieldMode.java

@@ -147,35 +147,14 @@ public ReflectionHints registerTypes(Iterable<TypeReference> types, Consumer<Typ
 		return this;
 	}
 
-	/**
-	 * Register the need for reflection on the specified {@link Field},
-	 * enabling {@link FieldMode#WRITE}.
-	 * @param field the field that requires reflection
-	 * @return {@code this}, to facilitate method chaining
-	 */
-	public ReflectionHints registerField(Field field) {
-		return registerField(field, FieldMode.WRITE);
-	}
-
-	/**
-	 * Register the need for reflection on the specified {@link Field}
-	 * using the specified {@link FieldMode}.
-	 * @param field the field that requires reflection
-	 * @return {@code this}, to facilitate method chaining
-	 */
-	public ReflectionHints registerField(Field field, FieldMode mode) {
-		return registerField(field, FieldHint.builtWith(mode));
-	}
-
 	/**
 	 * Register the need for reflection on the specified {@link Field}.
 	 * @param field the field that requires reflection
-	 * @param fieldHint a builder to further customize the hints of this field
 	 * @return {@code this}, to facilitate method chaining
 	 */
-	public ReflectionHints registerField(Field field, Consumer<FieldHint.Builder> fieldHint) {
+	public ReflectionHints registerField(Field field) {
 		return registerType(TypeReference.of(field.getDeclaringClass()),
-				typeHint -> typeHint.withField(field.getName(), fieldHint));
+				typeHint -> typeHint.withField(field.getName()));
 	}
 
 	/**

spring-core/src/main/java/org/springframework/aot/hint/ReflectionHints.java

@@ -44,7 +44,7 @@ public final class TypeHint implements ConditionalHint {
 	@Nullable
 	private final TypeReference reachableType;
 
-	private final Set<String> fields;
+	private final Set<FieldHint> fields;
 
 	private final Set<ExecutableHint> constructors;
 
@@ -57,7 +57,7 @@ private TypeHint(Builder builder) {
 		this.type = builder.type;
 		this.reachableType = builder.reachableType;
 		this.memberCategories = Set.copyOf(builder.memberCategories);
-		this.fields = builder.fields;
+		this.fields = builder.fields.stream().map(FieldHint::new).collect(Collectors.toSet());
 		this.constructors = builder.constructors.values().stream().map(ExecutableHint.Builder::build).collect(Collectors.toSet());
 		this.methods = builder.methods.values().stream().map(ExecutableHint.Builder::build).collect(Collectors.toSet());
 	}
@@ -89,10 +89,10 @@ public TypeReference getReachableType() {
 
 	/**
 	 * Return the fields that require reflection.
-	 * @return a stream of Strings
+	 * @return a stream of {@link FieldHint}
 	 */
-	public Set<String> fields() {
-		return this.fields;
+	public Stream<FieldHint> fields() {
+		return this.fields.stream();
 	}
 
 	/**
@@ -185,16 +185,15 @@ public Builder onReachableType(Class<?> reachableType) {
 		}
 
 		/**
-		 * Register the need for reflection on the field with the specified name,
-		 * enabling write access.
+		 * Register the need for reflection on the field with the specified name.
 		 * @param name the name of the field
 		 * @return {@code this}, to facilitate method chaining
 		 */
 		public Builder withField(String name) {
-			return withField(name);
+			this.fields.add(name);
+			return this;
 		}
 
-
 		/**
 		 * Register the need for reflection on the constructor with the specified
 		 * parameter types, enabling {@link ExecutableMode#INVOKE}.

spring-core/src/main/java/org/springframework/aot/hint/TypeHint.java

@@ -28,8 +28,6 @@
 
 import org.springframework.aot.hint.ExecutableHint;
 import org.springframework.aot.hint.ExecutableMode;
-import org.springframework.aot.hint.FieldHint;
-import org.springframework.aot.hint.FieldMode;
 import org.springframework.aot.hint.MemberCategory;
 import org.springframework.aot.hint.ReflectionHints;
 import org.springframework.aot.hint.RuntimeHints;
@@ -375,57 +373,10 @@ public static class FieldHintPredicate implements Predicate<RuntimeHints> {
 
 		private final Field field;
 
-		private FieldMode mode = FieldMode.READ;
-
-		private boolean allowUnsafeAccess;
-
 		FieldHintPredicate(Field field) {
 			this.field = field;
 		}
 
-		/**
-		 * Refine the current predicate to match if write access is allowed on the field.
-		 * @return the refined {@link RuntimeHints} predicate
-		 * @see FieldHint#isAllowWrite()
-		 * @deprecated in favor of {@link #withReadMode()} or {@link #withWriteMode()}
-		 */
-		@Deprecated
-		public FieldHintPredicate allowWrite() {
-			this.mode = FieldMode.WRITE;
-			return this;
-		}
-
-		/**
-		 * Refine the current predicate to match if read access is allowed on the field.
-		 * @return the refined {@link RuntimeHints} predicate
-		 * @see FieldHint#getMode()
-		 */
-		public FieldHintPredicate withReadMode() {
-			// FieldMode.READ is already the default and should not override a writeMode() call.
-			return this;
-		}
-
-		/**
-		 * Refine the current predicate to match if write access is allowed on the field.
-		 * @return the refined {@link RuntimeHints} predicate
-		 * @see FieldHint#getMode()
-		 */
-		public FieldHintPredicate withWriteMode() {
-			this.mode = FieldMode.WRITE;
-			return this;
-		}
-
-
-		/**
-		 * Refine the current predicate to match if unsafe access is allowed on the field.
-		 * @return the refined {@link RuntimeHints} predicate
-		 * @see FieldHint#isAllowUnsafeAccess() ()
-		 */
-		public FieldHintPredicate allowUnsafeAccess() {
-			this.allowUnsafeAccess = true;
-			return this;
-		}
-
 		@Override
 		public boolean test(RuntimeHints runtimeHints) {
 			TypeHint typeHint = runtimeHints.reflection().getTypeHint(this.field.getDeclaringClass());
@@ -447,9 +398,7 @@ private boolean memberCategoryMatch(TypeHint typeHint) {
 
 		private boolean exactMatch(TypeHint typeHint) {
 			return typeHint.fields().anyMatch(fieldHint ->
-					this.field.getName().equals(fieldHint.getName())
-							&& (fieldHint.getMode().includes(this.mode))
-							&& (!this.allowUnsafeAccess || this.allowUnsafeAccess == fieldHint.isAllowUnsafeAccess()));
+					this.field.getName().equals(fieldHint.getName()));
 		}
 	}
 

spring-core/src/main/java/org/springframework/aot/hint/predicate/ReflectionHintsPredicates.java

@@ -26,7 +26,6 @@
 import org.springframework.aot.hint.ExecutableHint;
 import org.springframework.aot.hint.ExecutableMode;
 import org.springframework.aot.hint.FieldHint;
-import org.springframework.aot.hint.FieldMode;
 import org.springframework.aot.hint.MemberCategory;
 import org.springframework.aot.hint.ReflectionHints;
 import org.springframework.aot.hint.TypeHint;
@@ -78,12 +77,6 @@ private void handleFields(Map<String, Object> attributes, Stream<FieldHint> fiel
 	private Map<String, Object> toAttributes(FieldHint hint) {
 		Map<String, Object> attributes = new LinkedHashMap<>();
 		attributes.put("name", hint.getName());
-		if (hint.getMode() == FieldMode.WRITE) {
-			attributes.put("allowWrite", true);
-		}
-		if (hint.isAllowUnsafeAccess()) {
-			attributes.put("allowUnsafeAccess", hint.isAllowUnsafeAccess());
-		}
 		return attributes;
 	}