Author: erans Date: Thu May 1 20:43:46 2014 New Revision: 1591770 URL: http://svn.apache.org/r1591770 Log: Removed test based on the assumption that iterating over a "ConcurrentHashMap" would always provide its elements in a random order. This seemed to be the case up to Java 7 but not always in Java 8.
Modified: commons/proper/math/trunk/src/test/java/org/apache/commons/math3/ml/neuralnet/NetworkTest.java Modified: commons/proper/math/trunk/src/test/java/org/apache/commons/math3/ml/neuralnet/NetworkTest.java URL: http://svn.apache.org/viewvc/commons/proper/math/trunk/src/test/java/org/apache/commons/math3/ml/neuralnet/NetworkTest.java?rev=1591770&r1=1591769&r2=1591770&view=diff ============================================================================== --- commons/proper/math/trunk/src/test/java/org/apache/commons/math3/ml/neuralnet/NetworkTest.java (original) +++ commons/proper/math/trunk/src/test/java/org/apache/commons/math3/ml/neuralnet/NetworkTest.java Thu May 1 20:43:46 2014 @@ -113,24 +113,9 @@ public class NetworkTest { SquareNeighbourhood.VON_NEUMANN, initArray).getNetwork(); + // Check that the comparator provides a specific order. boolean isUnspecifiedOrder = false; - - // Check that the default iterator returns the neurons - // in an unspecified order. long previousId = Long.MIN_VALUE; - for (Neuron n : net) { - final long currentId = n.getIdentifier(); - if (currentId < previousId) { - isUnspecifiedOrder = true; - break; - } - previousId = currentId; - } - Assert.assertTrue(isUnspecifiedOrder); - - // Check that the comparator provides a specific order. - isUnspecifiedOrder = false; - previousId = Long.MIN_VALUE; for (Neuron n : net.getNeurons(new Network.NeuronIdentifierComparator())) { final long currentId = n.getIdentifier(); if (currentId < previousId) {