Bad test design

Author: bbakerman

src/test/java/org/dataloader/DataLoaderTest.java

@@ -831,19 +831,25 @@ public void should_Batch_loads_occurring_within_futures() {
 
         Supplier<Object> nullValue = () -> null;
 
+        AtomicBoolean v4Called = new AtomicBoolean();
+
         CompletableFuture.supplyAsync(nullValue).thenAccept(v1 -> {
             identityLoader.load("a");
             CompletableFuture.supplyAsync(nullValue).thenAccept(v2 -> {
                 identityLoader.load("b");
                 CompletableFuture.supplyAsync(nullValue).thenAccept(v3 -> {
                     identityLoader.load("c");
                     CompletableFuture.supplyAsync(nullValue).thenAccept(
-                            v4 ->
-                                    identityLoader.load("d"));
+                            v4 -> {
+                                identityLoader.load("d");
+                                v4Called.set(true);
+                            });
                 });
             });
         });
 
+        await().untilTrue(v4Called);
+
         identityLoader.dispatchAndJoin();
 
         assertThat(loadCalls, equalTo(