mamh-java
diff --git a/‎src/main/java/org/kohsuke/github/GitHubSanityCachedValue.java‎
Lines changed: 0 additions & 1 deletion b/‎src/main/java/org/kohsuke/github/GitHubSanityCachedValue.java‎
Lines changed: 0 additions & 1 deletion
diff --git a/‎src/test/java/org/kohsuke/github/GitHubSanityCachedValueTest.java‎
Lines changed: 26 additions & 27 deletions b/‎src/test/java/org/kohsuke/github/GitHubSanityCachedValueTest.java‎
Lines changed: 26 additions & 27 deletions
@@ -40,7 +40,6 @@ <E extends Throwable> T get(Function<T, Boolean> isExpired, SupplierThrows<T, E>
         } finally {
             readLock.unlock();
         }
-
         writeLock.lock();
         try {
             boolean stillExpired = Instant.now().getEpochSecond() > lastQueriedAtEpochSeconds || isExpired.apply(lastResult);
 
@@ -15,6 +15,12 @@
 
 public class GitHubSanityCachedValueTest {
 
+    private static void alignToStartOfSecond() {
+        while (Instant.now().getNano() > 100_000_000) {
+            Thread.yield();
+        }
+    }
+
     @Test
     public void cachesWithinSameSecond() throws Exception {
         alignToStartOfSecond();
@@ -35,28 +41,6 @@ public void cachesWithinSameSecond() throws Exception {
         assertThat(calls.get(), equalTo(1));
     }
 
-    @Test
-    public void refreshesAfterOneSecond() throws Exception {
-        GitHubSanityCachedValue<String> cachedValue = new GitHubSanityCachedValue<>();
-        AtomicInteger calls = new AtomicInteger();
-
-        String first = cachedValue.get(() -> {
-            calls.incrementAndGet();
-            return "value";
-        });
-
-        Thread.sleep(1100);
-
-        String second = cachedValue.get(() -> {
-            calls.incrementAndGet();
-            return "value";
-        });
-
-        assertThat(first, equalTo("value"));
-        assertThat(second, equalTo("value"));
-        assertThat(calls.get(), equalTo(2));
-    }
-
     @Test
     public void concurrentCallersOnlyRefreshOnce() throws Exception {
         alignToStartOfSecond();
@@ -98,10 +82,25 @@ public void concurrentCallersOnlyRefreshOnce() throws Exception {
         }
     }
 
-    private static void alignToStartOfSecond() {
-        while (Instant.now().getNano() > 100_000_000) {
-            Thread.yield();
-        }
+    @Test
+    public void refreshesAfterOneSecond() throws Exception {
+        GitHubSanityCachedValue<String> cachedValue = new GitHubSanityCachedValue<>();
+        AtomicInteger calls = new AtomicInteger();
+
+        String first = cachedValue.get(() -> {
+            calls.incrementAndGet();
+            return "value";
+        });
+
+        Thread.sleep(1100);
+
+        String second = cachedValue.get(() -> {
+            calls.incrementAndGet();
+            return "value";
+        });
+
+        assertThat(first, equalTo("value"));
+        assertThat(second, equalTo("value"));
+        assertThat(calls.get(), equalTo(2));
     }
 }
-
Original file line number	Diff line number	Diff line change
`@@ -40,7 +40,6 @@ <E extends Throwable> T get(Function<T, Boolean> isExpired, SupplierThrows<T, E>`
`40`	`40`	`} finally {`
`41`	`41`	`readLock.unlock();`
`42`	`42`	`}`
`43`		`-`
`44`	`43`	`writeLock.lock();`
`45`	`44`	`try {`
`46`	`45`	`boolean stillExpired = Instant.now().getEpochSecond() > lastQueriedAtEpochSeconds \|\| isExpired.apply(lastResult);`