From 505b147bd23e26d21932af2c3336c9f4607bbcce Mon Sep 17 00:00:00 2001
From: Tavian Barnes <tavianator@tavianator.com>
Date: Wed, 7 May 2014 21:09:20 -0400
Subject: core: Make DelayedError complain if cancel() is called too late.

---
 .../src/main/java/com/tavianator/sangria/core/DelayedError.java  | 9 +++++++--
 1 file changed, 7 insertions(+), 2 deletions(-)

(limited to 'sangria-core/src/main/java/com/tavianator/sangria/core/DelayedError.java')

diff --git a/sangria-core/src/main/java/com/tavianator/sangria/core/DelayedError.java b/sangria-core/src/main/java/com/tavianator/sangria/core/DelayedError.java
index d6f86a8..2aeabca 100644
--- a/sangria-core/src/main/java/com/tavianator/sangria/core/DelayedError.java
+++ b/sangria-core/src/main/java/com/tavianator/sangria/core/DelayedError.java
@@ -25,6 +25,8 @@ import com.google.inject.CreationException;
 import com.google.inject.Injector;
 import com.google.inject.spi.Message;
 
+import static com.google.common.base.Preconditions.*;
+
 /**
  * Similar to {@link Binder#addError(String, Object...)}, but can be canceled later. Useful for enforcing correct usage
  * of fluent APIs.
@@ -35,6 +37,7 @@ import com.google.inject.spi.Message;
  */
 public class DelayedError {
     private Throwable error;
+    private boolean reported = false;
 
     /**
      * Create a {@link DelayedError}.
@@ -85,11 +88,13 @@ public class DelayedError {
      * Cancel this error.
      */
     public void cancel() {
-        this.error = null;
+        checkState(!reported, "This error has already been reported");
+        error = null;
     }
 
     @Inject
-    void inject(Injector injector) throws Throwable {
+    void reportErrors(Injector injector) throws Throwable {
+        reported = true;
         if (error != null) {
             throw error;
         }
-- 
cgit v1.2.3