Addressing Code style: don't do star imports

spotify · Nov 14, 2016 · f49f24a · f49f24a
1 parent 3090258
commit f49f24a
Show file tree

Hide file tree

Showing 2 changed files with 17 additions and 13 deletions.
diff --git a/src/main/java/com/spotify/dns/NoCachingLookupFactory.java b/src/main/java/com/spotify/dns/NoCachingLookupFactory.java
@@ -17,23 +17,24 @@
 package com.spotify.dns;
 
 import com.google.common.base.Preconditions;
-import org.xbill.DNS.*;
+import org.xbill.DNS.Lookup;
+
 
 /**
  * A LookupFactory that delegates the creation of a lookup then forcing it to not use the cache
  */
 public class NoCachingLookupFactory implements LookupFactory {
 
-  private final LookupFactory delegate;
+    private final LookupFactory delegate;
 
-  public NoCachingLookupFactory(LookupFactory delegate) {
-      this.delegate = Preconditions.checkNotNull(delegate, "Delegate lookup factory cannot be null");
-  }
+    public NoCachingLookupFactory(LookupFactory delegate) {
+        this.delegate = Preconditions.checkNotNull(delegate, "Delegate lookup factory cannot be null");
+    }
 
-  @Override
-  public Lookup forName(String fqdn) {
-    Lookup lookup = delegate.forName(fqdn);
-    lookup.setCache(null);
-    return lookup;
-  }
+    @Override
+    public Lookup forName(String fqdn) {
+        Lookup lookup = delegate.forName(fqdn);
+        lookup.setCache(null);
+        return lookup;
+    }
 }
diff --git a/src/test/java/com/spotify/dns/NoCachingLookupFactoryTest.java b/src/test/java/com/spotify/dns/NoCachingLookupFactoryTest.java
@@ -8,9 +8,12 @@
 import org.mockito.Mock;
 import org.mockito.Mockito;
 import org.mockito.runners.MockitoJUnitRunner;
-import org.xbill.DNS.*;
+import org.xbill.DNS.DClass;
+import org.xbill.DNS.Lookup;
+import org.xbill.DNS.Type;
 
-import static org.hamcrest.CoreMatchers.*;
+import static org.hamcrest.CoreMatchers.is;
+import static org.hamcrest.CoreMatchers.notNullValue;
 import static org.junit.Assert.assertThat;
 
 @RunWith(MockitoJUnitRunner.class)