Merge pull request #54 from Nordstrom/pr/fix-before-class-npe

Resolve #53: If thread runner lacks atomic test, use failure description

Author: sbabcoc

src/main/java/com/nordstrom/automation/junit/RunAnnouncer.java

@@ -145,6 +145,9 @@ static <T> AtomicTest<T> getAtomicTestOf(Object testKey) {
      */
     private static <T> AtomicTest<T> setTestFailure(Failure failure) {
         AtomicTest<T> atomicTest = getAtomicTestOf(Run.getThreadRunner());
+        if (atomicTest == null) {
+            atomicTest = getAtomicTestOf(failure.getDescription());
+        }
         atomicTest.setThrowable(failure.getException());
         return atomicTest;
     }

src/test/java/com/nordstrom/automation/junit/BeforeClassFailureTest.java

@@ -0,0 +1,24 @@
+package com.nordstrom.automation.junit;
+
+import static org.junit.Assert.assertEquals;
+import static org.junit.Assert.assertFalse;
+
+import org.junit.runner.JUnitCore;
+import org.junit.runner.Result;
+import org.junit.runner.notification.Failure;
+import org.testng.annotations.Test;
+
+public class BeforeClassFailureTest {
+
+    @Test
+    public void verifyBeforeClassExceptionHandling() {
+        JUnitCore runner = new JUnitCore();
+        Result result = runner.run(BeforeClassThrowsException.class);
+        assertFalse(result.wasSuccessful());
+        assertEquals(1, result.getFailureCount());
+        Failure failure = result.getFailures().get(0);
+        assertEquals(RuntimeException.class, failure.getException().getClass());
+        assertEquals("Must be failed", failure.getMessage());
+    }
+    
+}

src/test/java/com/nordstrom/automation/junit/BeforeClassThrowsException.java

@@ -0,0 +1,25 @@
+package com.nordstrom.automation.junit;
+
+import static org.junit.Assert.assertTrue;
+
+import org.junit.BeforeClass;
+import org.junit.Test;
+
+public class BeforeClassThrowsException extends TestBase {
+    
+    @BeforeClass
+    public static void failing() {
+        throw new RuntimeException("Must be failed");
+    }
+    
+    @Test
+    public void happy() {
+        assertTrue(true);
+    }
+
+    @Test
+    public void sad() {
+        assertTrue(false);
+    }
+
+}