/*
 * Copyright 2012-2022 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.cloud.loadbalancer.core;

import java.util.ArrayList;
import java.util.List;

import org.junit.jupiter.api.Test;
import reactor.core.publisher.Flux;

import org.springframework.cloud.client.ServiceInstance;
import org.springframework.cloud.loadbalancer.support.SimpleObjectProvider;

import static java.lang.Integer.MAX_VALUE;
import static org.assertj.core.api.Assertions.assertThat;
import static org.mockito.ArgumentMatchers.any;
import static org.mockito.Mockito.mock;
import static org.mockito.Mockito.when;

/**
 * @author Zhuozhi JI
 */
class RoundRobinLoadBalancerTest {

	@Test
	void shouldEnforceOrderWhenPositiveOverflow() {
		List<ServiceInstance> instances = new ArrayList<>();
		for (int i = 0; i < 10; i++) {
			ServiceInstance instance = mock(ServiceInstance.class);
			when(instance.getInstanceId()).thenReturn(i + "");
			instances.add(instance);
		}
		SameInstancePreferenceServiceInstanceListSupplier supplier = mock(
				SameInstancePreferenceServiceInstanceListSupplier.class);
		when(supplier.get(any())).thenReturn(Flux.just(instances));

		RoundRobinLoadBalancer loadBalancer = new RoundRobinLoadBalancer(new SimpleObjectProvider<>(supplier),
				"shouldStartFromZeroWhenPositiveOverflow", MAX_VALUE);

		for (int i = 0; i < 10; i++) {
			ServiceInstance chosen = loadBalancer.choose().block().getServer();
			assertThat(chosen.getInstanceId()).isEqualTo(i + "");
		}
	}
}

@@ -971,16 +971,59 @@
 _VALUE;%0A
+import static java.lang.Integer.MIN_VALUE;%0A
 import s
@@ -1290,41 +1290,310 @@
 ould
+Order
 Enforce
-OrderWhenPositiveOverflow(
+dWhenPositive() %7B%0A%09%09assertOrderEnforced(0);%0A%09%7D%0A%0A%09@Test%0A%09void shouldOrderEnforcedWhenNegative() %7B%0A%09%09assertOrderEnforced(MIN_VALUE);%0A%09%7D%0A%0A%09@Test%0A%09void shouldOrderEnforcedWhenPositiveOverflow() %7B%0A%09%09assertOrderEnforced(MAX_VALUE);%0A%09%7D%0A%0A%09@SuppressWarnings(%22all%22)%0A%09void assertOrderEnforced(int seed
 ) %7B%0A
@@ -2171,25 +2171,20 @@
 rflow%22, 
-MAX_VALUE
+seed
 );%0A%0A%09%09fo
@@ -2204,32 +2204,101 @@
  i %3C 10; i++) %7B%0A
+%09%09%09int instanceId = ((seed + 1 + i) & MAX_VALUE) %25 instances.size();%0A
 %09%09%09ServiceInstan
@@ -2401,16 +2401,25 @@
 qualTo(i
+nstanceId
  + %22%22);%0A

Remove unnecessary sign check, whether positive or negative (#1078)