Minor java nitpicks

- Unused imports
- Redundant type declarations
- Better exception docs

Change-Id: Iebbd03cd9e1d9cf78cc4486085619bb6f17c1ec9

src/main/java/com/googlesource/gerrit/plugins/its/phabricator/conduit/Conduit.java

diff --git a/src/main/java/com/googlesource/gerrit/plugins/its/phabricator/conduit/Conduit.java b/src/main/java/com/googlesource/gerrit/plugins/its/phabricator/conduit/Conduit.java
index 27b838f..fa49281 100644
--- a/src/main/java/com/googlesource/gerrit/plugins/its/phabricator/conduit/Conduit.java
+++ b/src/main/java/com/googlesource/gerrit/plugins/its/phabricator/conduit/Conduit.java
@@ -98,7 +98,7 @@
       log.debug("Trying to start new session");
       conduitConnect();
     }
-    Map<String, Object> conduitParams = new HashMap<String, Object>();
+    Map<String, Object> conduitParams = new HashMap<>();
     conduitParams.put("sessionKey",sessionKey);
     params.put("__conduit__", conduitParams);
   }
@@ -118,7 +118,7 @@
    * Runs the API's 'conduit.connect' method
    */
   public ConduitConnect conduitConnect() throws ConduitException {
-    Map<String, Object> params = new HashMap<String, Object>();
+    Map<String, Object> params = new HashMap<>();
     params.put("client", "its-phabricator");
     params.put("clientVersion", CONDUIT_VERSION);
     params.put("user", username);
@@ -164,7 +164,7 @@
    * Runs the API's 'maniphest.Info' method
    */
   public ManiphestInfo maniphestInfo(int taskId) throws ConduitException {
-    Map<String, Object> params = new HashMap<String, Object>();
+    Map<String, Object> params = new HashMap<>();
     fillInSession(params);
     params.put("task_id", taskId);
 
@@ -191,7 +191,7 @@
    * Runs the API's 'maniphest.update' method
    */
   public ManiphestUpdate maniphestUpdate(int taskId, String comment, Iterable<String> projects) throws ConduitException {
-    Map<String, Object> params = new HashMap<String, Object>();
+    Map<String, Object> params = new HashMap<>();
     fillInSession(params);
     params.put("id", taskId);
     if (comment != null) {
@@ -210,7 +210,7 @@
    * Runs the API's 'projectQuery' method to match exactly one project name
    */
   public ProjectInfo projectQuery(String name) throws ConduitException {
-    Map<String, Object> params = new HashMap<String, Object>();
+    Map<String, Object> params = new HashMap<>();
     fillInSession(params);
     params.put("names", Arrays.asList(name));
 

src/main/java/com/googlesource/gerrit/plugins/its/phabricator/conduit/ConduitConnection.java

diff --git a/src/main/java/com/googlesource/gerrit/plugins/its/phabricator/conduit/ConduitConnection.java b/src/main/java/com/googlesource/gerrit/plugins/its/phabricator/conduit/ConduitConnection.java
index 10644d1..2233df0 100644
--- a/src/main/java/com/googlesource/gerrit/plugins/its/phabricator/conduit/ConduitConnection.java
+++ b/src/main/java/com/googlesource/gerrit/plugins/its/phabricator/conduit/ConduitConnection.java
@@ -30,7 +30,6 @@
 import org.slf4j.LoggerFactory;
 
 import java.io.IOException;
-import java.io.UnsupportedEncodingException;
 import java.nio.charset.StandardCharsets;
 import java.util.HashMap;
 import java.util.Map;
@@ -73,7 +72,7 @@
    *
    * @param method The name of the method that should get called
    * @return The call's result, if there has been no error
-   * @throws Exception
+   * @throws ConduitException
    */
   JsonElement call(String method) throws ConduitException {
     return call(method, new HashMap<String, Object>());
@@ -85,7 +84,7 @@
    * @param method The name of the method that should get called
    * @param params A map of parameters to pass to the call
    * @return The call's result, if there has been no error
-   * @throws Exception
+   * @throws ConduitException
    */
   JsonElement call(String method, Map<String, Object> params) throws ConduitException {
     String methodUrl = apiUrlBase + method;

src/test/java/com/googlesource/gerrit/plugins/its/phabricator/conduit/ConduitTest.java

diff --git a/src/test/java/com/googlesource/gerrit/plugins/its/phabricator/conduit/ConduitTest.java b/src/test/java/com/googlesource/gerrit/plugins/its/phabricator/conduit/ConduitTest.java
index f2e80e6..58ea0b8 100644
--- a/src/test/java/com/googlesource/gerrit/plugins/its/phabricator/conduit/ConduitTest.java
+++ b/src/test/java/com/googlesource/gerrit/plugins/its/phabricator/conduit/ConduitTest.java
@@ -27,10 +27,7 @@
 import java.util.Arrays;
 import java.util.List;
 import java.util.Map;
-import java.util.Set;
 
-import com.google.gson.JsonArray;
-import com.google.gson.JsonElement;
 import com.google.gson.JsonObject;
 import com.google.gson.JsonPrimitive;
 
@@ -93,7 +90,7 @@
     JsonObject ret = new JsonObject();
     ret.add("sessionKey", new JsonPrimitive("KeyFoo"));
 
-    Capture<Map<String, Object>> paramsCapture = new Capture<Map<String, Object>>();
+    Capture<Map<String, Object>> paramsCapture = new Capture<>();
 
     expect(connection.call(eq("conduit.connect"), capture(paramsCapture)))
       .andReturn(ret)
@@ -117,7 +114,7 @@
 
     ConduitException conduitException = new ConduitException();
 
-    Capture<Map<String, Object>> paramsCapture = new Capture<Map<String, Object>>();
+    Capture<Map<String, Object>> paramsCapture = new Capture<>();
 
     expect(connection.call(eq("conduit.connect"), capture(paramsCapture)))
       .andThrow(conduitException)
@@ -146,7 +143,7 @@
     JsonObject retConnect = new JsonObject();
     retConnect.add("sessionKey", new JsonPrimitive("KeyFoo"));
 
-    Capture<Map<String, Object>> paramsCaptureConnect = new Capture<Map<String, Object>>();
+    Capture<Map<String, Object>> paramsCaptureConnect = new Capture<>();
 
     expect(connection.call(eq("conduit.connect"), capture(paramsCaptureConnect)))
       .andReturn(retConnect)
@@ -155,7 +152,7 @@
     JsonObject retRelevant = new JsonObject();
     retRelevant.add("id", new JsonPrimitive(42));
 
-    Capture<Map<String, Object>> paramsCaptureRelevant = new Capture<Map<String, Object>>();
+    Capture<Map<String, Object>> paramsCaptureRelevant = new Capture<>();
 
     expect(connection.call(eq("maniphest.info"), capture(paramsCaptureRelevant)))
     .andReturn(retRelevant)
@@ -183,7 +180,7 @@
 
     ConduitException conduitException = new ConduitException();
 
-    Capture<Map<String, Object>> paramsCapture = new Capture<Map<String, Object>>();
+    Capture<Map<String, Object>> paramsCapture = new Capture<>();
 
     expect(connection.call(eq("conduit.connect"), capture(paramsCapture)))
       .andThrow(conduitException)
@@ -214,7 +211,7 @@
     JsonObject retConnect = new JsonObject();
     retConnect.add("sessionKey", new JsonPrimitive("KeyFoo"));
 
-    Capture<Map<String, Object>> paramsCaptureConnect = new Capture<Map<String, Object>>();
+    Capture<Map<String, Object>> paramsCaptureConnect = new Capture<>();
 
     expect(connection.call(eq("conduit.connect"), capture(paramsCaptureConnect)))
       .andReturn(retConnect)
@@ -222,7 +219,7 @@
 
     ConduitException conduitException = new ConduitException();
 
-    Capture<Map<String, Object>> paramsCaptureRelevant = new Capture<Map<String, Object>>();
+    Capture<Map<String, Object>> paramsCaptureRelevant = new Capture<>();
 
     expect(connection.call(eq("maniphest.info"), capture(paramsCaptureRelevant)))
       .andThrow(conduitException)
@@ -256,7 +253,7 @@
     JsonObject retConnect = new JsonObject();
     retConnect.add("sessionKey", new JsonPrimitive("KeyFoo"));
 
-    Capture<Map<String, Object>> paramsCaptureConnect = new Capture<Map<String, Object>>();
+    Capture<Map<String, Object>> paramsCaptureConnect = new Capture<>();
 
     expect(connection.call(eq("conduit.connect"), capture(paramsCaptureConnect)))
       .andReturn(retConnect)
@@ -265,7 +262,7 @@
     JsonObject retRelevant = new JsonObject();
     retRelevant.add("id", new JsonPrimitive(42));
 
-    Capture<Map<String, Object>> paramsCaptureRelevant = new Capture<Map<String, Object>>();
+    Capture<Map<String, Object>> paramsCaptureRelevant = new Capture<>();
 
     expect(connection.call(eq("maniphest.update"), capture(paramsCaptureRelevant)))
     .andReturn(retRelevant)
@@ -296,7 +293,7 @@
     JsonObject retConnect = new JsonObject();
     retConnect.add("sessionKey", new JsonPrimitive("KeyFoo"));
 
-    Capture<Map<String, Object>> paramsCaptureConnect = new Capture<Map<String, Object>>();
+    Capture<Map<String, Object>> paramsCaptureConnect = new Capture<>();
 
     expect(connection.call(eq("conduit.connect"), capture(paramsCaptureConnect)))
       .andReturn(retConnect)
@@ -305,7 +302,7 @@
     JsonObject retRelevant = new JsonObject();
     retRelevant.add("id", new JsonPrimitive(42));
 
-    Capture<Map<String, Object>> paramsCaptureRelevant = new Capture<Map<String, Object>>();
+    Capture<Map<String, Object>> paramsCaptureRelevant = new Capture<>();
 
     expect(connection.call(eq("maniphest.update"), capture(paramsCaptureRelevant)))
     .andReturn(retRelevant)
@@ -339,7 +336,7 @@
     JsonObject retConnect = new JsonObject();
     retConnect.add("sessionKey", new JsonPrimitive("KeyFoo"));
 
-    Capture<Map<String, Object>> paramsCaptureConnect = new Capture<Map<String, Object>>();
+    Capture<Map<String, Object>> paramsCaptureConnect = new Capture<>();
 
     expect(connection.call(eq("conduit.connect"), capture(paramsCaptureConnect)))
       .andReturn(retConnect)
@@ -348,7 +345,7 @@
     JsonObject retRelevant = new JsonObject();
     retRelevant.add("id", new JsonPrimitive(42));
 
-    Capture<Map<String, Object>> paramsCaptureRelevant = new Capture<Map<String, Object>>();
+    Capture<Map<String, Object>> paramsCaptureRelevant = new Capture<>();
 
     expect(connection.call(eq("maniphest.update"), capture(paramsCaptureRelevant)))
     .andReturn(retRelevant)
@@ -381,7 +378,7 @@
 
     ConduitException conduitException = new ConduitException();
 
-    Capture<Map<String, Object>> paramsCapture = new Capture<Map<String, Object>>();
+    Capture<Map<String, Object>> paramsCapture = new Capture<>();
 
     expect(connection.call(eq("conduit.connect"), capture(paramsCapture)))
       .andThrow(conduitException)
@@ -412,7 +409,7 @@
     JsonObject retConnect = new JsonObject();
     retConnect.add("sessionKey", new JsonPrimitive("KeyFoo"));
 
-    Capture<Map<String, Object>> paramsCaptureConnect = new Capture<Map<String, Object>>();
+    Capture<Map<String, Object>> paramsCaptureConnect = new Capture<>();
 
     expect(connection.call(eq("conduit.connect"), capture(paramsCaptureConnect)))
       .andReturn(retConnect)
@@ -420,7 +417,7 @@
 
     ConduitException conduitException = new ConduitException();
 
-    Capture<Map<String, Object>> paramsCaptureRelevant = new Capture<Map<String, Object>>();
+    Capture<Map<String, Object>> paramsCaptureRelevant = new Capture<>();
 
     expect(connection.call(eq("maniphest.update"), capture(paramsCaptureRelevant)))
       .andThrow(conduitException)
@@ -454,7 +451,7 @@
     JsonObject retConnect = new JsonObject();
     retConnect.add("sessionKey", new JsonPrimitive("KeyFoo"));
 
-    Capture<Map<String, Object>> paramsCaptureConnect = new Capture<Map<String, Object>>();
+    Capture<Map<String, Object>> paramsCaptureConnect = new Capture<>();
 
     expect(connection.call(eq("conduit.connect"), capture(paramsCaptureConnect)))
       .andReturn(retConnect)
@@ -463,7 +460,7 @@
     JsonObject retRelevant = new JsonObject();
     retRelevant.add("id", new JsonPrimitive(42));
 
-    Capture<Map<String, Object>> paramsCaptureRelevant = new Capture<Map<String, Object>>();
+    Capture<Map<String, Object>> paramsCaptureRelevant = new Capture<>();
 
     expect(connection.call(eq("maniphest.info"), capture(paramsCaptureRelevant)))
     .andReturn(retRelevant)
@@ -495,7 +492,7 @@
     JsonObject retConnect = new JsonObject();
     retConnect.add("sessionKey", new JsonPrimitive("KeyFoo"));
 
-    Capture<Map<String, Object>> paramsCaptureConnect = new Capture<Map<String, Object>>();
+    Capture<Map<String, Object>> paramsCaptureConnect = new Capture<>();
 
     expect(connection.call(eq("conduit.connect"), capture(paramsCaptureConnect)))
       .andReturn(retConnect)
@@ -511,7 +508,7 @@
     JsonObject retRelevant = new JsonObject();
     retRelevant.add("data", queryDataJson);
 
-    Capture<Map<String, Object>> paramsCaptureRelevant = new Capture<Map<String, Object>>();
+    Capture<Map<String, Object>> paramsCaptureRelevant = new Capture<>();
 
     expect(connection.call(eq("project.query"), capture(paramsCaptureRelevant)))
     .andReturn(retRelevant)
@@ -544,7 +541,7 @@
     JsonObject retConnect = new JsonObject();
     retConnect.add("sessionKey", new JsonPrimitive("KeyFoo"));
 
-    Capture<Map<String, Object>> paramsCaptureConnect = new Capture<Map<String, Object>>();
+    Capture<Map<String, Object>> paramsCaptureConnect = new Capture<>();
 
     expect(connection.call(eq("conduit.connect"), capture(paramsCaptureConnect)))
       .andReturn(retConnect)
@@ -570,7 +567,7 @@
     JsonObject retRelevant = new JsonObject();
     retRelevant.add("data", queryDataJson);
 
-    Capture<Map<String, Object>> paramsCaptureRelevant = new Capture<Map<String, Object>>();
+    Capture<Map<String, Object>> paramsCaptureRelevant = new Capture<>();
 
     expect(connection.call(eq("project.query"), capture(paramsCaptureRelevant)))
     .andReturn(retRelevant)
@@ -596,7 +593,7 @@
   }
 
   private void mockConnection() throws Exception {
-    connection = createMock(ConduitConnection.class);;
+    connection = createMock(ConduitConnection.class);
     expectNew(ConduitConnection.class, URL)
       .andReturn(connection)
       .once();