Apply comment

Signed-off-by: yongjunhong <[email protected]>

Author: YongGoose

fixture-monkey/src/main/java/com/navercorp/fixturemonkey/annotation/Order.java

@@ -18,13 +18,18 @@
 
 package com.navercorp.fixturemonkey.annotation;
 
+import static org.apiguardian.api.API.Status.EXPERIMENTAL;
+
 import java.lang.annotation.ElementType;
 import java.lang.annotation.Retention;
 import java.lang.annotation.RetentionPolicy;
 import java.lang.annotation.Target;
 
+import org.apiguardian.api.API;
+
 @Retention(RetentionPolicy.RUNTIME)
 @Target({ElementType.TYPE})
+@API(since = "1.1.10", status = EXPERIMENTAL)
 public @interface Order {
 	int value() default 0;
 }

fixture-monkey/src/main/java/com/navercorp/fixturemonkey/customizer/MonkeyManipulatorFactory.java

@@ -18,6 +18,8 @@
 
 package com.navercorp.fixturemonkey.customizer;
 
+import static java.util.stream.Collectors.toList;
+
 import java.util.ArrayList;
 import java.util.Collections;
 import java.util.Comparator;
@@ -202,21 +204,8 @@ private DefaultArbitraryBuilder<?> findRegisteredArbitraryBuilder(
 			.sorted(Comparator.comparingInt(PriorityMatcherOperator::getPriority))
 			.collect(Collectors.toList());
 
-		return getHighestPriorityOperator(priorityOperators);
-	}
-
-	private DefaultArbitraryBuilder<?> getHighestPriorityOperator(
-		List<PriorityMatcherOperator<? extends ArbitraryBuilder<?>>> priorityOperators
-	) {
-		if (priorityOperators.isEmpty()) {
-			return null;
-		}
-
-		int highestPriority = priorityOperators.get(0).getPriority();
-		List<PriorityMatcherOperator<? extends ArbitraryBuilder<?>>> highestPriorityOperators = priorityOperators
-			.stream()
-			.filter(it -> it.getPriority() == highestPriority)
-			.collect(Collectors.toList());
+		List<PriorityMatcherOperator<? extends ArbitraryBuilder<?>>> highestPriorityOperators
+			= getHighestPriorityOperators(priorityOperators);
 
 		if (highestPriorityOperators.size() > 1) {
 			Collections.shuffle(highestPriorityOperators, Randoms.current());
@@ -229,6 +218,20 @@ private DefaultArbitraryBuilder<?> getHighestPriorityOperator(
 			.orElse(null);
 	}
 
+	private List<PriorityMatcherOperator<? extends ArbitraryBuilder<?>>> getHighestPriorityOperators(
+		List<PriorityMatcherOperator<? extends ArbitraryBuilder<?>>> priorityOperators
+	) {
+		if (priorityOperators.isEmpty()) {
+			return priorityOperators;
+		}
+
+		int highestPriority = priorityOperators.get(0).getPriority();
+
+		return priorityOperators.stream()
+			.filter(it -> it.getPriority() == highestPriority)
+			.collect(toList());
+	}
+
 	public ManipulatorSet newManipulatorSet(ManipulatorHolderSet manipulatorHolderSet) {
 		int baseSequence = sequence.getAndIncrement();