code cleanup: deleted dead code regarding unset functionality.

Author: praveenag

cypher-shell/src/integration-test/java/org/neo4j/shell/commands/CypherShellIntegrationTest.java

@@ -178,8 +178,5 @@ public void paramsAndListVariables() throws CommandException {
         List<String> queryResult = captor.getAllValues();
         assertThat(queryResult.get(0), is("{ bob }\n" + randomLong));
         assertEquals(randomLong, shell.getAll().get("bob"));
-
-        shell.remove("bob");
-        assertTrue(shell.getAll().isEmpty());
     }
 }

cypher-shell/src/main/java/org/neo4j/shell/CypherShell.java

@@ -173,12 +173,6 @@ public Map<String, Object> getAll() {
         return queryParams;
     }
 
-    @Override
-    @Nonnull
-    public Optional remove(@Nonnull String name) {
-        return Optional.ofNullable(queryParams.remove(name));
-    }
-
     public void setCommandHelper(@Nonnull CommandHelper commandHelper) {
         this.commandHelper = commandHelper;
     }

cypher-shell/src/main/java/org/neo4j/shell/VariableHolder.java

@@ -22,10 +22,4 @@ public interface VariableHolder {
      */
     @Nonnull
     Map<String, Object> getAll();
-
-    /**
-     *
-     * @param name of variable to delete
-     */
-    Optional remove(@Nonnull String name);
 }

cypher-shell/src/test/java/org/neo4j/shell/CypherShellTest.java

@@ -144,9 +144,6 @@ public void verifyVariableMethods() throws CommandException {
         Optional result = offlineTestShell.set("bob", "99");
         assertEquals("99", result.get());
         assertEquals("99", offlineTestShell.getAll().get("bob"));
-
-        offlineTestShell.remove("bob");
-        assertTrue(offlineTestShell.getAll().isEmpty());
     }
 
     @Test
@@ -232,13 +229,6 @@ public void specifyingACypherStringShouldGiveAStringRunner() throws IOException
         }
     }
 
-    @Test
-    public void unsetAlreadyClearedValue() throws CommandException {
-        // when
-        // then
-        assertFalse("Expected param to be unset", offlineTestShell.remove("unknown var").isPresent());
-    }
-
     @Test
     public void setWithSomeBoltError() throws CommandException {
         // then