From 947da619a6b52eab45cef644887704e084c9a74e Mon Sep 17 00:00:00 2001
From: Uwe Schindler <uwe@thetaphi.de>
Date: Mon, 24 Aug 2015 00:06:11 +0200
Subject: [PATCH] Prevent bug with empty options (EnumSet.copyOf needs at least
 one element)

---
 src/main/java/de/thetaphi/forbiddenapis/Checker.java |  3 +--
 .../de/thetaphi/forbiddenapis/CheckerSetupTest.java  | 12 ++++++++++++
 2 files changed, 13 insertions(+), 2 deletions(-)

diff --git a/src/main/java/de/thetaphi/forbiddenapis/Checker.java b/src/main/java/de/thetaphi/forbiddenapis/Checker.java
index 56912e32..f05f22ce 100644
--- a/src/main/java/de/thetaphi/forbiddenapis/Checker.java
+++ b/src/main/java/de/thetaphi/forbiddenapis/Checker.java
@@ -94,9 +94,8 @@ public static enum Option {
   protected abstract void logWarn(String msg);
   protected abstract void logInfo(String msg);
   
-  //public Checker(ClassLoader loader, boolean internalRuntimeForbidden, boolean failOnMissingClasses, boolean failOnViolation, boolean defaultFailOnUnresolvableSignatures) {
   public Checker(ClassLoader loader, Option... options) {
-    this(loader, EnumSet.copyOf(Arrays.asList(options)));
+    this(loader, (options.length == 0) ? EnumSet.noneOf(Option.class) : EnumSet.copyOf(Arrays.asList(options)));
   }
   
   public Checker(ClassLoader loader, EnumSet<Option> options) {
diff --git a/src/test/java/de/thetaphi/forbiddenapis/CheckerSetupTest.java b/src/test/java/de/thetaphi/forbiddenapis/CheckerSetupTest.java
index 99568671..e7f0101b 100644
--- a/src/test/java/de/thetaphi/forbiddenapis/CheckerSetupTest.java
+++ b/src/test/java/de/thetaphi/forbiddenapis/CheckerSetupTest.java
@@ -21,6 +21,7 @@
 import static org.junit.Assume.assumeTrue;
 
 import java.util.Collections;
+import java.util.EnumSet;
 
 import org.junit.Before;
 import org.junit.Test;
@@ -55,6 +56,7 @@ protected void logInfo(String msg) {
   public void setUp() {
     checker = new MyChecker();
     assumeTrue("This test only works with a supported JDK (see docs)", checker.isSupportedJDK);
+    assertEquals(EnumSet.of(INTERNAL_RUNTIME_FORBIDDEN, FAIL_ON_MISSING_CLASSES, FAIL_ON_VIOLATION, FAIL_ON_UNRESOLVABLE_SIGNATURES), checker.options);
   }
 
   @Test
@@ -101,4 +103,14 @@ public void testMethodSignature() throws Exception {
     assertEquals(Collections.singletonMap("java/lang/Object\000toString()Ljava/lang/String;", "java.lang.Object#toString() [Foobar]"), checker.forbiddenMethods);
   }
   
+  @Test
+  public void testEmptyCtor() throws Exception {
+    Checker chk = new Checker(ClassLoader.getSystemClassLoader()) {
+      @Override protected void logError(String msg) { }
+      @Override protected void logWarn(String msg) { }
+      @Override protected void logInfo(String msg) { }
+    };
+    assertEquals(EnumSet.noneOf(Checker.Option.class), chk.options);
+  }
+  
 }