style: improve code style for delivery api changes

bugsnag · May 4, 2018 · 72b744c · 72b744c
1 parent b15a4f4
commit 72b744c
Show file tree

Hide file tree

Showing 7 changed files with 2 additions and 22 deletions.
diff --git a/sdk/src/androidTest/java/com/bugsnag/android/BugsnagTestUtils.java b/sdk/src/androidTest/java/com/bugsnag/android/BugsnagTestUtils.java
@@ -95,15 +95,11 @@ public static Delivery generateDelivery() {
         return new Delivery() {
             @Override
             public void deliver(SessionTrackingPayload payload,
-                                Configuration config) throws DeliveryFailureException {
-
-            }
+                                Configuration config) throws DeliveryFailureException {}
 
             @Override
             public void deliver(Report report,
-                                Configuration config) throws DeliveryFailureException {
-
-            }
+                                Configuration config) throws DeliveryFailureException {}
         };
     }
 }
diff --git a/sdk/src/androidTest/java/com/bugsnag/android/ClientConfigTest.java b/sdk/src/androidTest/java/com/bugsnag/android/ClientConfigTest.java
@@ -3,9 +3,6 @@
 import static org.junit.Assert.assertArrayEquals;
 import static org.junit.Assert.assertEquals;
 import static org.junit.Assert.assertFalse;
-import static org.junit.Assert.assertNotNull;
-import static org.junit.Assert.assertNull;
-import static org.junit.Assert.assertTrue;
 
 import android.content.Context;
 import android.support.test.InstrumentationRegistry;

diff --git a/sdk/src/androidTest/java/com/bugsnag/android/DeliveryCompatTest.java b/sdk/src/androidTest/java/com/bugsnag/android/DeliveryCompatTest.java
@@ -12,7 +12,6 @@
 import java.util.Map;
 import java.util.concurrent.atomic.AtomicInteger;
 
-
 public class DeliveryCompatTest {
 
     private final Configuration config = BugsnagTestUtils.generateConfiguration();
@@ -45,7 +44,6 @@ public void deliver(Report report,
             }
         };
         deliveryCompat = new DeliveryCompat(baseDelivery);
-
     }
 
     @Test
@@ -125,5 +123,4 @@ public void testExceptionConversion() {
 
         assertNull(deliveryCompat.convertException(new RuntimeException()));
     }
-
 }
diff --git a/sdk/src/main/java/com/bugsnag/android/Client.java b/sdk/src/main/java/com/bugsnag/android/Client.java
@@ -632,7 +632,6 @@ void setErrorReportApiClient(@NonNull ErrorReportApiClient errorReportApiClient)
         if (errorReportApiClient == null) {
             throw new IllegalArgumentException("ErrorReportApiClient cannot be null.");
         }
-
         DeliveryCompat compat = (DeliveryCompat) config.getDelivery();
         compat.errorReportApiClient = errorReportApiClient;
     }
@@ -1239,9 +1238,7 @@ void deliver(@NonNull Report report, @NonNull Error error) {
             config.getDelivery().deliver(report, config);
             Logger.info("Sent 1 new error to Bugsnag");
         } catch (DeliveryFailureException exception) {
-
             switch (exception.reason) {
-
                 case CONNECTIVITY:
                     Logger.info("Could not send error(s) to Bugsnag, saving to disk to send later");
                     // Save error to disk for later sending

diff --git a/sdk/src/main/java/com/bugsnag/android/Configuration.java b/sdk/src/main/java/com/bugsnag/android/Configuration.java
@@ -453,7 +453,6 @@ public void setLaunchCrashThresholdMs(long launchCrashThresholdMs) {
 
     /**
      * Returns whether automatic breadcrumb capture or common application events is enabled.
-     *
      * @return true if automatic capture is enabled, otherwise false.
      */
     public boolean isAutomaticallyCollectingBreadcrumbs() {
@@ -474,7 +473,6 @@ public void setAutomaticallyCollectBreadcrumbs(boolean automaticallyCollectBread
 
     /**
      * Intended for internal use only - sets the type of the notifier (e.g. Android, React Native)
-     *
      * @param notifierType the notifier type
      */
     public void setNotifierType(String notifierType) {
@@ -483,7 +481,6 @@ public void setNotifierType(String notifierType) {
 
     /**
      * Intended for internal use only - sets the code bundle id for React Native
-     *
      * @param codeBundleId the code bundle id
      */
     public void setCodeBundleId(String codeBundleId) {
@@ -523,7 +520,6 @@ public void setDelivery(Delivery delivery) {
         if (delivery == null) {
             throw new IllegalArgumentException("Delivery cannot be null");
         }
-
         if (delivery instanceof DeliveryCompat) {
             this.delivery = delivery;
         } else {

diff --git a/sdk/src/main/java/com/bugsnag/android/DefaultDelivery.java b/sdk/src/main/java/com/bugsnag/android/DefaultDelivery.java
@@ -38,7 +38,6 @@ public void deliver(SessionTrackingPayload payload,
     @Override
     public void deliver(Report report,
                         Configuration config) throws DeliveryFailureException {
-
         String endpoint = config.getEndpoint();
         int status = deliver(endpoint, report, config.getErrorApiHeaders());
 
@@ -96,5 +95,4 @@ private void checkHasNetworkConnection() throws DeliveryFailureException {
             throw new DeliveryFailureException(CONNECTIVITY, "No network connection available");
         }
     }
-
 }
diff --git a/sdk/src/main/java/com/bugsnag/android/DeliveryCompat.java b/sdk/src/main/java/com/bugsnag/android/DeliveryCompat.java
@@ -52,5 +52,4 @@ DeliveryFailureException convertException(Exception exception) {
             return null;
         }
     }
-
 }