refactor(Event): rename Exception -> Error

Author: lemnik

…src/main/java/com/bugsnag/Exception.java …nag/src/main/java/com/bugsnag/Error.javabugsnag/src/main/java/com/bugsnag/Exception.java renamed to bugsnag/src/main/java/com/bugsnag/Error.java bugsnag/src/main/java/com/bugsnag/Exception.java renamed to bugsnag/src/main/java/com/bugsnag/Error.java

@@ -4,12 +4,12 @@
 
 import java.util.List;
 
-class Exception {
+public class Error {
     private Configuration config;
     private Throwable throwable;
     private String errorClass;
 
-    Exception(Configuration config, Throwable throwable) {
+    Error(Configuration config, Throwable throwable) {
         this.config = config;
         this.throwable = throwable;
         this.errorClass = throwable.getClass().getName();

bugsnag/src/main/java/com/bugsnag/Event.java

@@ -18,7 +18,7 @@ public class Event {
     private Configuration config;
 
     private String apiKey;
-    private final Exception exception;
+    private final Error error;
     private HandledState handledState;
     private Severity severity;
     private String groupingHash;
@@ -47,7 +47,7 @@ protected Event(Configuration config, Throwable throwable) {
     Event(Configuration config, Throwable throwable,
           HandledState handledState, Thread currentThread, FeatureFlagStore clientFeatureFlagStore) {
         this.config = config;
-        this.exception = new Exception(config, throwable);
+        this.error = new Error(config, throwable);
         this.handledState = handledState;
         this.severity = handledState.getOriginalSeverity();
         diagnostics = new Diagnostics(this.config);
@@ -78,17 +78,17 @@ protected String getPayloadVersion() {
      * @return the exceptions that make up the error.
      */
     @Expose
-    protected List<Exception> getExceptions() {
-        List<Exception> exceptions = new ArrayList<Exception>();
-        exceptions.add(exception);
+    protected List<Error> getErrors() {
+        List<Error> errors = new ArrayList<Error>();
+        errors.add(error);
 
-        Throwable currentThrowable = exception.getThrowable().getCause();
+        Throwable currentThrowable = error.getThrowable().getCause();
         while (currentThrowable != null) {
-            exceptions.add(new Exception(config, currentThrowable));
+            errors.add(new Error(config, currentThrowable));
             currentThrowable = currentThrowable.getCause();
         }
 
-        return exceptions;
+        return errors;
     }
 
     @Expose
@@ -168,14 +168,14 @@ void setSession(Session session) {
      *         report.
      */
     public Throwable getException() {
-        return exception.getThrowable();
+        return error.getThrowable();
     }
 
     /**
      * @return the class name from the exception contained in this error report.
      */
     public String getExceptionName() {
-        return exception.getErrorClass();
+        return error.getErrorClass();
     }
 
     /**
@@ -184,14 +184,14 @@ public String getExceptionName() {
      * @param exceptionName the error name
      */
     public void setExceptionName(String exceptionName) {
-        exception.setErrorClass(exceptionName);
+        error.setErrorClass(exceptionName);
     }
 
     /**
      * @return The message from the exception contained in this error report.
      */
     public String getExceptionMessage() {
-        return exception.getThrowable().getLocalizedMessage();
+        return error.getThrowable().getLocalizedMessage();
     }
 
     /**

bugsnag/src/main/java/com/bugsnag/Stackframe.java

@@ -5,7 +5,7 @@
 import java.util.ArrayList;
 import java.util.List;
 
-class Stackframe {
+public class Stackframe {
     private Configuration config;
     private StackTraceElement el;
 

bugsnag/src/test/java/com/bugsnag/AppenderTest.java

@@ -249,7 +249,7 @@ public void testProjectPackages() {
         Notification notification = delivery.getNotifications().get(0);
         Event event = notification.getEvents().get(0);
 
-        List<Stackframe> frames = event.getExceptions().get(0).getStacktrace();
+        List<Stackframe> frames = event.getErrors().get(0).getStacktrace();
         assertTrue(frames.get(0).isInProject());
         assertTrue(frames.get(1).isInProject());
         for (int i = 2; i < frames.size(); i++) {

bugsnag/src/test/java/com/bugsnag/BugsnagTest.java

@@ -105,7 +105,7 @@ public void testProjectPackages() {
             @Override
             public void deliver(Serializer serializer, Object object, Map<String, String> headers) {
                 Event event = ((Notification) object).getEvents().get(0);
-                assertTrue(event.getExceptions().get(0).getStacktrace().get(0).isInProject());
+                assertTrue(event.getErrors().get(0).getStacktrace().get(0).isInProject());
             }
 
             @Override

bugsnag/src/test/java/com/bugsnag/ExceptionTest.java

@@ -16,7 +16,7 @@
 
 public class ExceptionTest {
 
-    private Exception exception;
+    private Error error;
     private RuntimeException ogThrowable;
 
     /**
@@ -28,22 +28,22 @@ public class ExceptionTest {
     public void setUp() {
         Configuration config = new Configuration("api-key");
         ogThrowable = new RuntimeException("Test");
-        exception = new Exception(config, ogThrowable);
+        error = new Error(config, ogThrowable);
     }
 
     @Test
     public void testDefaults() {
-        assertEquals("java.lang.RuntimeException", exception.getErrorClass());
-        assertEquals("Test", exception.getMessage());
-        assertEquals(ogThrowable, exception.getThrowable());
-        assertFalse(exception.getStacktrace().isEmpty());
+        assertEquals("java.lang.RuntimeException", error.getErrorClass());
+        assertEquals("Test", error.getMessage());
+        assertEquals(ogThrowable, error.getThrowable());
+        assertFalse(error.getStacktrace().isEmpty());
     }
 
     @Test
     public void testClassOverride() {
-        exception.setErrorClass("Hello");
-        assertEquals("Hello", exception.getErrorClass());
-        assertEquals("Test", exception.getMessage());
+        error.setErrorClass("Hello");
+        assertEquals("Hello", error.getErrorClass());
+        assertEquals("Test", error.getMessage());
     }
 
     @Test
@@ -68,13 +68,13 @@ public void close() {
                 report.setExceptionName("Foo");
                 assertEquals("Foo", report.getExceptionName());
 
-                List<Exception> exceptions = report.getExceptions();
-                assertEquals(1, exceptions.size());
+                List<Error> errors = report.getErrors();
+                assertEquals(1, errors.size());
 
-                Exception exception = exceptions.get(0);
-                assertNotNull(exception);
-                assertEquals("Foo", exception.getErrorClass());
-                assertEquals("Test", exception.getMessage());
+                Error error = errors.get(0);
+                assertNotNull(error);
+                assertEquals("Foo", error.getErrorClass());
+                assertEquals("Test", error.getMessage());
             } catch (Throwable throwable) {
                 report.cancel();
             }

bugsnag/src/test/java/com/bugsnag/ThreadStateTest.java

@@ -187,7 +187,7 @@ public void testHandledStacktrace() throws IOException {
     }
 
     /**
-     * Verifies that an unhandled error uses {@link com.bugsnag.Exception#getStacktrace()}
+     * Verifies that an unhandled error uses {@link Error#getStacktrace()}
      * for the reporting thread stacktrace
      */
     @Test