Merge pull request #8 from Nordstrom/pr/fix-unchecked-throw

<T> extends Throwable

Author: sbabcoc

src/main/java/com/nordstrom/common/base/UncheckedThrow.java

@@ -36,7 +36,7 @@ public static RuntimeException throwUnchecked(final Throwable thrown) {
      * @throws T dummy declaration to satisfy the compiler
      */
     @SuppressWarnings("unchecked")
-    private static <T extends Exception> void propagate(Throwable thrown) throws T {
+    private static <T extends Throwable> void propagate(Throwable thrown) throws T {
         // Due to generic type erasure, this cast only serves to satisfy the compiler
         // that the requirement to declare the thrown exception has been met.
         throw (T) thrown;

src/test/java/com/nordstrom/common/base/UncheckedThrowTest.java

@@ -0,0 +1,35 @@
+package com.nordstrom.common.base;
+
+import java.io.IOException;
+
+import org.testng.annotations.Test;
+
+public class UncheckedThrowTest {
+    
+    @Test(expectedExceptions = {IOException.class})
+    public void testCheckedException() {
+        try {
+            throwCheckedException();
+        } catch (Throwable t) {
+            throw UncheckedThrow.throwUnchecked(t);
+        }
+    }
+    
+    @Test(expectedExceptions = {AssertionError.class})
+    public void testUncheckedException() {
+        try {
+            throwUncheckedException();
+        } catch (Throwable t) {
+            throw UncheckedThrow.throwUnchecked(t);
+        }
+    }
+    
+    private void throwCheckedException() throws IOException {
+        throw new IOException("This is a checked exception");
+    }
+    
+    private void throwUncheckedException() {
+        throw new AssertionError("This is an unchecked exception");
+    }
+
+}