/*
 * The Alluxio Open Foundation licenses this work under the Apache License, version 2.0
 * (the "License"). You may not use this work except in compliance with the License, which is
 * available at www.apache.org/licenses/LICENSE-2.0
 *
 * This software is distributed on an "AS IS" basis, WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND,
 * either express or implied, as more fully set forth in the License.
 *
 * See the NOTICE file distributed with this work for information regarding copyright ownership.
 */

package alluxio.retry;

import org.junit.Assert;
import org.junit.Test;

/**
 * Tests for the {@link TimeoutRetry} class.
 */
public class TimeoutRetryTest {

  /**
   * Tests that the provided timeout is respected.
   */
  @Test
  public void timeout() {
    final long timeoutMs = 50;
    final int sleepMs = 10;
    int attempts = 0;
    TimeoutRetry timeoutRetry = new TimeoutRetry(timeoutMs, sleepMs);
    Assert.assertEquals(0, timeoutRetry.getRetryCount());
    long startMs = System.currentTimeMillis();
    while (timeoutRetry.attemptRetry()) {
      attempts++;
    }
    long endMs = System.currentTimeMillis();
    Assert.assertTrue(attempts > 0);
    Assert.assertTrue((endMs - startMs) >= timeoutMs);
    Assert.assertEquals(attempts, timeoutRetry.getRetryCount());
  }

  /**
   * Tests the sleep parameter between attempts.
   */
  @Test
  public void sleep() {
    final long timeoutMs = 50;
    final int sleepMs = 10;
    int attempts = 0;
    TimeoutRetry timeoutRetry = new TimeoutRetry(timeoutMs, sleepMs);
    Assert.assertEquals(0, timeoutRetry.getRetryCount());
    long startMs = System.currentTimeMillis();
    while (timeoutRetry.attemptRetry()) {
      attempts++;
    }
    long endMs = System.currentTimeMillis();
    Assert.assertTrue(attempts > 0);
    Assert.assertTrue((endMs - startMs) >= timeoutMs);
    Assert.assertEquals(attempts, timeoutRetry.getRetryCount());
    Assert.assertTrue(attempts <= timeoutMs / sleepMs);
  }
}

@@ -1293,681 +1293,65 @@
 ;%0A  
-%7D%0A%0A  /**%0A   * Tests the sleep parameter between attempts.%0A   */%0A  @Test%0A  public void sleep() %7B%0A    final long timeoutMs = 50;%0A    final int sleepMs = 10;%0A    int attempts = 0;%0A    TimeoutRetry timeoutRetry = new TimeoutRetry(timeoutMs, sleepMs);%0A    Assert.assertEquals(0, timeoutRetry.getRetryCount());%0A    long startMs = System.currentTimeMillis();%0A    while (timeoutRetry.attemptRetry()) %7B%0A      attempts++;%0A    %7D%0A    long endMs = System.currentTimeMillis();%0A    Assert.assertTrue(attempts %3E 0);%0A    Assert.assertTrue((endMs - startMs) %3E= timeoutMs);%0A    Assert.assertEquals(attempts, timeoutRetry.getRetryCount());%0A    Assert.assertTrue(attempts %3C= timeoutMs / sleepMs
+  Assert.assertTrue(attempts %3C= (timeoutMs / sleepMs) + 1
 );%0A 

Make timeout test less flaky