incidental style cleanup of 3 test classes

sul-dlss · Jun 1, 2017 · b5600aa · b5600aa
1 parent 0653894
commit b5600aa
Show file tree

Hide file tree

Showing 3 changed files with 1 addition and 15 deletions.
diff --git a/test/edu/stanford/dlss/was/TestDownloadResponseHandler.java b/test/edu/stanford/dlss/was/TestDownloadResponseHandler.java
@@ -15,37 +15,26 @@
 import org.junit.*;
 import static org.junit.Assert.*;
 import org.mockito.*;
-import org.mockito.Mockito.*;
 import org.mockito.invocation.*;
 import org.mockito.stubbing.*;
 
-
-
 public class TestDownloadResponseHandler {
   private static final char SEP = File.separatorChar;
   private static final String OUTPUT_DIRECTORY = new String("test" + SEP + "tmp");
   private static final String OUTPUT_FILE_PATH = new String(OUTPUT_DIRECTORY + SEP + "testDownloadResponseHandler.output");
   private static final StatusLine VALID_STATUS_LINE = new BasicStatusLine(new ProtocolVersion("HTTP 1/1", 1, 1), 200, "OK");
 
-  private void writeToFile() throws IOException {
-    new File(OUTPUT_FILE_PATH).createNewFile();
-    return;
-  }
-
-
   @Before
   public void setUp() {
     new File(OUTPUT_DIRECTORY).mkdir();
   }
 
-
   @After
   public void tearDown() {
     new File(OUTPUT_FILE_PATH).delete();
     new File(OUTPUT_DIRECTORY).delete();
   }
 
-
   @Test(expected = ClientProtocolException.class)
   public void nullEntityThrowsException() throws ClientProtocolException, HttpResponseException, IOException {
     DownloadResponseHandler handler = new DownloadResponseHandler("/dev/null");
@@ -56,7 +45,6 @@ public void nullEntityThrowsException() throws ClientProtocolException, HttpResp
     handler.handleResponse(mockResponse);
   }
 
-
   @Test
   public void validResponseDownloadsFile() throws ClientProtocolException, HttpResponseException, IOException {
     DownloadResponseHandler handler = new DownloadResponseHandler(OUTPUT_FILE_PATH);
@@ -66,6 +54,7 @@ public void validResponseDownloadsFile() throws ClientProtocolException, HttpRes
     Mockito.when(mockResponse.getStatusLine()).thenReturn(VALID_STATUS_LINE);
 
     Mockito.doAnswer(new Answer<Void>() {
+      @Override
       public Void answer(InvocationOnMock invocation) throws IOException {
         new File(OUTPUT_FILE_PATH).createNewFile();
         return null;

diff --git a/test/edu/stanford/dlss/was/TestJsonResponseHandler.java b/test/edu/stanford/dlss/was/TestJsonResponseHandler.java
@@ -15,7 +15,6 @@
 import org.junit.*;
 import static org.junit.Assert.*;
 import org.mockito.*;
-import org.mockito.Mockito.*;
 
 public class TestJsonResponseHandler {
 
@@ -33,7 +32,6 @@ public void nullEntityThrowsException() throws ClientProtocolException, HttpResp
     handler.handleResponse(mockResponse);
   }
 
-
   @Test
   public void validResponseParsesCorrectly() throws ClientProtocolException, HttpResponseException, IOException {
     JsonResponseHandler handler = new JsonResponseHandler();

diff --git a/test/edu/stanford/dlss/was/TestWasapiClient.java b/test/edu/stanford/dlss/was/TestWasapiClient.java
@@ -5,7 +5,6 @@
 import org.apache.http.client.methods.HttpGet;
 import org.apache.http.impl.client.CloseableHttpClient;
 
-import static org.hamcrest.CoreMatchers.*;
 import static org.junit.Assert.*;
 import org.junit.*;
 import static org.mockito.Mockito.*;