Remove redundant models and corresponding test

Iterator.next is already modelled

Author: owen-mc

java/ql/src/semmle/code/java/frameworks/apache/Collections.qll

@@ -88,7 +88,6 @@ private class ApacheCollectionsModel extends SummaryModelCsv {
         // In order for the models of Iterator to work we have to use Element instead of MapKey for key data.
         "org.apache.commons.collections4;MapIterator;true;getKey;;;Element of Argument[-1];ReturnValue;value",
         "org.apache.commons.collections4;MapIterator;true;getValue;;;MapValue of Argument[-1];ReturnValue;value",
-        "org.apache.commons.collections4;MapIterator;true;next;;;Element of Argument[-1];ReturnValue;value",
         "org.apache.commons.collections4;MapIterator;true;setValue;;;MapValue of Argument[-1];ReturnValue;value",
         "org.apache.commons.collections4;MapIterator;true;setValue;;;Argument[0];MapValue of Argument[-1];value",
         "org.apache.commons.collections4;MultiMap;true;get;;;Element of MapValue of Argument[-1];Element of ReturnValue;value",
@@ -177,7 +176,6 @@ private class ApacheCollectionsModel extends SummaryModelCsv {
         "org.apache.commons.collections;KeyValue;true;getValue;;;MapValue of Argument[-1];ReturnValue;value",
         "org.apache.commons.collections;MapIterator;true;getKey;;;Element of Argument[-1];ReturnValue;value",
         "org.apache.commons.collections;MapIterator;true;getValue;;;MapValue of Argument[-1];ReturnValue;value",
-        "org.apache.commons.collections;MapIterator;true;next;;;Element of Argument[-1];ReturnValue;value",
         "org.apache.commons.collections;MapIterator;true;setValue;;;MapValue of Argument[-1];ReturnValue;value",
         "org.apache.commons.collections;MapIterator;true;setValue;;;Argument[0];MapValue of Argument[-1];value",
         "org.apache.commons.collections;MultiMap;true;get;;;Element of MapValue of Argument[-1];Element of ReturnValue;value",

java/ql/test/library-tests/frameworks/apache-collections/Test.java

@@ -2675,13 +2675,6 @@ public void test() {
 			out = in.getValue();
 			sink(out); // $ hasValueFlow
 		}
-		{
-			// "org.apache.commons.collections4;MapIterator;true;next;;;Element of Argument[-1];ReturnValue;value"
-			Object out = null;
-			MapIterator in = newOMIWithElement((String)source());
-			out = in.next();
-			sink(out); // $ hasValueFlow
-		}
 		{
 			// "org.apache.commons.collections4;MapIterator;true;setValue;;;Argument[0];MapValue of Argument[-1];value"
 			MapIterator out = null;