checkstyle

Author: richardelms

bugsnag/src/test/java/com/bugsnag/BugsnagFeatureFlagTest.java

@@ -29,10 +29,10 @@ public void tearDown() {
     @Test
     public void testAddFeatureFlag() {
         bugsnag.addFeatureFlag("flag1", "variant-a");
-        
+
         Report report = bugsnag.buildReport(new RuntimeException("Test"));
         List<FeatureFlag> flags = report.getFeatureFlags();
-        
+
         assertEquals(1, flags.size());
         assertEquals("flag1", flags.get(0).getName());
         assertEquals("variant-a", flags.get(0).getVariant());
@@ -41,10 +41,10 @@ public void testAddFeatureFlag() {
     @Test
     public void testAddFeatureFlagWithoutVariant() {
         bugsnag.addFeatureFlag("flag1");
-        
+
         Report report = bugsnag.buildReport(new RuntimeException("Test"));
         List<FeatureFlag> flags = report.getFeatureFlags();
-        
+
         assertEquals(1, flags.size());
         assertEquals("flag1", flags.get(0).getName());
         assertEquals(null, flags.get(0).getVariant());
@@ -55,12 +55,12 @@ public void testAddFeatureFlags() {
         List<FeatureFlag> flagsToAdd = new ArrayList<FeatureFlag>();
         flagsToAdd.add(new FeatureFlag("flag1", "variant-a"));
         flagsToAdd.add(new FeatureFlag("flag2", "variant-b"));
-        
+
         bugsnag.addFeatureFlags(flagsToAdd);
-        
+
         Report report = bugsnag.buildReport(new RuntimeException("Test"));
         List<FeatureFlag> flags = report.getFeatureFlags();
-        
+
         assertEquals(2, flags.size());
         assertEquals("flag1", flags.get(0).getName());
         assertEquals("flag2", flags.get(1).getName());
@@ -71,10 +71,10 @@ public void testClearFeatureFlag() {
         bugsnag.addFeatureFlag("flag1", "variant-a");
         bugsnag.addFeatureFlag("flag2", "variant-b");
         bugsnag.clearFeatureFlag("flag1");
-        
+
         Report report = bugsnag.buildReport(new RuntimeException("Test"));
         List<FeatureFlag> flags = report.getFeatureFlags();
-        
+
         assertEquals(1, flags.size());
         assertEquals("flag2", flags.get(0).getName());
     }
@@ -84,47 +84,47 @@ public void testClearFeatureFlags() {
         bugsnag.addFeatureFlag("flag1", "variant-a");
         bugsnag.addFeatureFlag("flag2", "variant-b");
         bugsnag.clearFeatureFlags();
-        
+
         Report report = bugsnag.buildReport(new RuntimeException("Test"));
         List<FeatureFlag> flags = report.getFeatureFlags();
-        
+
         assertEquals(0, flags.size());
     }
 
     @Test
     public void testClientFlagsInheritFromConfiguration() {
         Configuration config = bugsnag.getConfig();
         config.addFeatureFlag("config-flag", "config-variant");
-        
+
         Bugsnag client = new Bugsnag("api-key", false);
         client.getConfig().addFeatureFlag("config-flag", "config-variant");
-        
+
         Report report = client.buildReport(new RuntimeException("Test"));
         List<FeatureFlag> flags = report.getFeatureFlags();
-        
+
         assertEquals(1, flags.size());
         assertEquals("config-flag", flags.get(0).getName());
         assertEquals("config-variant", flags.get(0).getVariant());
-        
+
         client.close();
     }
 
     @Test
     public void testClientFlagsOverrideConfigurationFlags() {
         Configuration config = bugsnag.getConfig();
         config.addFeatureFlag("flag1", "config-variant");
-        
+
         Bugsnag client = new Bugsnag("api-key", false);
         client.getConfig().addFeatureFlag("flag1", "config-variant");
         client.addFeatureFlag("flag1", "client-variant");
-        
+
         Report report = client.buildReport(new RuntimeException("Test"));
         List<FeatureFlag> flags = report.getFeatureFlags();
-        
+
         assertEquals(1, flags.size());
         assertEquals("flag1", flags.get(0).getName());
         assertEquals("client-variant", flags.get(0).getVariant());
-        
+
         client.close();
     }
 
@@ -133,20 +133,20 @@ public void testFeatureFlagOrderPreservedAcrossScopes() {
         Configuration config = bugsnag.getConfig();
         config.addFeatureFlag("flag1", "config-variant");
         config.addFeatureFlag("flag2", "config-variant");
-        
+
         Bugsnag client = new Bugsnag("api-key", false);
         client.getConfig().addFeatureFlag("flag1", "config-variant");
         client.getConfig().addFeatureFlag("flag2", "config-variant");
         client.addFeatureFlag("flag3", "client-variant");
-        
+
         Report report = client.buildReport(new RuntimeException("Test"));
         List<FeatureFlag> flags = report.getFeatureFlags();
-        
+
         assertEquals(3, flags.size());
         assertEquals("flag1", flags.get(0).getName());
         assertEquals("flag2", flags.get(1).getName());
         assertEquals("flag3", flags.get(2).getName());
-        
+
         client.close();
     }
 }

bugsnag/src/test/java/com/bugsnag/ConfigurationFeatureFlagTest.java

@@ -1,7 +1,6 @@
 package com.bugsnag;
 
 import static org.junit.Assert.assertEquals;
-import static org.junit.Assert.assertTrue;
 
 import org.junit.Before;
 import org.junit.Test;
@@ -24,35 +23,35 @@ public void setUp() {
     @Test
     public void testAddFeatureFlag() {
         config.addFeatureFlag("flag1", "variant-a");
-        
+
         // Verify by creating a client and checking the flags are inherited
         Bugsnag bugsnag = new Bugsnag("api-key");
         bugsnag.getConfig().addFeatureFlag("flag1", "variant-a");
-        
+
         Report report = bugsnag.buildReport(new RuntimeException("Test"));
         List<FeatureFlag> flags = report.getFeatureFlags();
-        
+
         assertEquals(1, flags.size());
         assertEquals("flag1", flags.get(0).getName());
         assertEquals("variant-a", flags.get(0).getVariant());
-        
+
         bugsnag.close();
     }
 
     @Test
     public void testAddFeatureFlagWithoutVariant() {
         config.addFeatureFlag("flag1");
-        
+
         Bugsnag bugsnag = new Bugsnag("api-key");
         bugsnag.getConfig().addFeatureFlag("flag1");
-        
+
         Report report = bugsnag.buildReport(new RuntimeException("Test"));
         List<FeatureFlag> flags = report.getFeatureFlags();
-        
+
         assertEquals(1, flags.size());
         assertEquals("flag1", flags.get(0).getName());
         assertEquals(null, flags.get(0).getVariant());
-        
+
         bugsnag.close();
     }
 
@@ -61,19 +60,19 @@ public void testAddFeatureFlags() {
         List<FeatureFlag> flagsToAdd = new ArrayList<FeatureFlag>();
         flagsToAdd.add(new FeatureFlag("flag1", "variant-a"));
         flagsToAdd.add(new FeatureFlag("flag2", "variant-b"));
-        
+
         config.addFeatureFlags(flagsToAdd);
-        
+
         Bugsnag bugsnag = new Bugsnag("api-key");
         bugsnag.getConfig().addFeatureFlags(flagsToAdd);
-        
+
         Report report = bugsnag.buildReport(new RuntimeException("Test"));
         List<FeatureFlag> flags = report.getFeatureFlags();
-        
+
         assertEquals(2, flags.size());
         assertEquals("flag1", flags.get(0).getName());
         assertEquals("flag2", flags.get(1).getName());
-        
+
         bugsnag.close();
     }
 
@@ -82,18 +81,18 @@ public void testClearFeatureFlag() {
         config.addFeatureFlag("flag1", "variant-a");
         config.addFeatureFlag("flag2", "variant-b");
         config.clearFeatureFlag("flag1");
-        
+
         Bugsnag bugsnag = new Bugsnag("api-key");
         bugsnag.getConfig().addFeatureFlag("flag1", "variant-a");
         bugsnag.getConfig().addFeatureFlag("flag2", "variant-b");
         bugsnag.getConfig().clearFeatureFlag("flag1");
-        
+
         Report report = bugsnag.buildReport(new RuntimeException("Test"));
         List<FeatureFlag> flags = report.getFeatureFlags();
-        
+
         assertEquals(1, flags.size());
         assertEquals("flag2", flags.get(0).getName());
-        
+
         bugsnag.close();
     }
 
@@ -102,17 +101,17 @@ public void testClearFeatureFlags() {
         config.addFeatureFlag("flag1", "variant-a");
         config.addFeatureFlag("flag2", "variant-b");
         config.clearFeatureFlags();
-        
+
         Bugsnag bugsnag = new Bugsnag("api-key");
         bugsnag.getConfig().addFeatureFlag("flag1", "variant-a");
         bugsnag.getConfig().addFeatureFlag("flag2", "variant-b");
         bugsnag.getConfig().clearFeatureFlags();
-        
+
         Report report = bugsnag.buildReport(new RuntimeException("Test"));
         List<FeatureFlag> flags = report.getFeatureFlags();
-        
+
         assertEquals(0, flags.size());
-        
+
         bugsnag.close();
     }
 }