ViewVC Help

View File | Revision Log | Show Annotations | Download File | Root Listing

root/jsr166/jsr166/src/test/tck/Collection8Test.java

Comparing jsr166/src/test/tck/Collection8Test.java (file contents):
Revision 1.33 by jsr166, Mon Nov 28 15:31:40 2016 UTC vs.
Revision 1.43 by jsr166, Thu Dec 15 01:21:22 2016 UTC

#	Line 12 | Line 12 | import java.util.ArrayList;
12		import java.util.Arrays;
13		import java.util.Collection;
14		import java.util.Collections;
15	+	import java.util.ConcurrentModificationException;
16		import java.util.Deque;
17		import java.util.HashSet;
18		import java.util.Iterator;
#	Line 342 | Line 343 | public class Collection8Test extends JSR
343		}
344
345		/**
346	+	* All elements removed in the middle of CONCURRENT traversal.
347	+	*/
348	+	public void testElementRemovalDuringTraversal() {
349	+	Collection c = impl.emptyCollection();
350	+	ThreadLocalRandom rnd = ThreadLocalRandom.current();
351	+	int n = rnd.nextInt(6);
352	+	ArrayList copy = new ArrayList();
353	+	for (int i = 0; i < n; i++) {
354	+	Object x = impl.makeElement(i);
355	+	copy.add(x);
356	+	c.add(x);
357	+	}
358	+	ArrayList iterated = new ArrayList();
359	+	ArrayList spliterated = new ArrayList();
360	+	Spliterator s = c.spliterator();
361	+	Iterator it = c.iterator();
362	+	for (int i = rnd.nextInt(n + 1); --i >= 0; ) {
363	+	assertTrue(s.tryAdvance(spliterated::add));
364	+	if (rnd.nextBoolean()) assertTrue(it.hasNext());
365	+	iterated.add(it.next());
366	+	}
367	+	Consumer alwaysThrows = e -> { throw new AssertionError(); };
368	+	if (s.hasCharacteristics(Spliterator.CONCURRENT)) {
369	+	c.clear();          // TODO: many more removal methods
370	+	if (testImplementationDetails
371	+	&& !(c instanceof java.util.concurrent.ArrayBlockingQueue)) {
372	+	if (rnd.nextBoolean())
373	+	assertFalse(s.tryAdvance(alwaysThrows));
374	+	else
375	+	s.forEachRemaining(alwaysThrows);
376	+	}
377	+	if (it.hasNext()) iterated.add(it.next());
378	+	if (rnd.nextBoolean()) assertIteratorExhausted(it);
379	+	}
380	+	assertTrue(copy.containsAll(iterated));
381	+	assertTrue(copy.containsAll(spliterated));
382	+	}
383	+
384	+	/**
385	+	* Some elements randomly disappear in the middle of traversal.
386	+	*/
387	+	public void testRandomElementRemovalDuringTraversal() {
388	+	Collection c = impl.emptyCollection();
389	+	ThreadLocalRandom rnd = ThreadLocalRandom.current();
390	+	int n = rnd.nextInt(6);
391	+	ArrayList copy = new ArrayList();
392	+	for (int i = 0; i < n; i++) {
393	+	Object x = impl.makeElement(i);
394	+	copy.add(x);
395	+	c.add(x);
396	+	}
397	+	ArrayList iterated = new ArrayList();
398	+	ArrayList spliterated = new ArrayList();
399	+	ArrayList removed = new ArrayList();
400	+	Spliterator s = c.spliterator();
401	+	Iterator it = c.iterator();
402	+	if (! (s.hasCharacteristics(Spliterator.CONCURRENT) ||
403	+	s.hasCharacteristics(Spliterator.IMMUTABLE)))
404	+	return;
405	+	for (int i = rnd.nextInt(n + 1); --i >= 0; ) {
406	+	assertTrue(s.tryAdvance(e -> {}));
407	+	if (rnd.nextBoolean()) assertTrue(it.hasNext());
408	+	it.next();
409	+	}
410	+	Consumer alwaysThrows = e -> { throw new AssertionError(); };
411	+	// TODO: many more removal methods
412	+	if (rnd.nextBoolean()) {
413	+	for (Iterator z = c.iterator(); z.hasNext(); ) {
414	+	Object e = z.next();
415	+	if (rnd.nextBoolean()) {
416	+	try {
417	+	z.remove();
418	+	} catch (UnsupportedOperationException ok) { return; }
419	+	removed.add(e);
420	+	}
421	+	}
422	+	} else {
423	+	Predicate randomlyRemove = e -> {
424	+	if (rnd.nextBoolean()) { removed.add(e); return true; }
425	+	else return false;
426	+	};
427	+	c.removeIf(randomlyRemove);
428	+	}
429	+	s.forEachRemaining(spliterated::add);
430	+	while (it.hasNext())
431	+	iterated.add(it.next());
432	+	assertTrue(copy.containsAll(iterated));
433	+	assertTrue(copy.containsAll(spliterated));
434	+	assertTrue(copy.containsAll(removed));
435	+	if (s.hasCharacteristics(Spliterator.CONCURRENT)) {
436	+	ArrayList iteratedAndRemoved = new ArrayList(iterated);
437	+	ArrayList spliteratedAndRemoved = new ArrayList(spliterated);
438	+	iteratedAndRemoved.retainAll(removed);
439	+	spliteratedAndRemoved.retainAll(removed);
440	+	assertTrue(iteratedAndRemoved.size() <= 1);
441	+	assertTrue(spliteratedAndRemoved.size() <= 1);
442	+	if (testImplementationDetails
443	+	&& !(c instanceof java.util.concurrent.ArrayBlockingQueue))
444	+	assertTrue(spliteratedAndRemoved.isEmpty());
445	+	}
446	+	}
447	+
448	+	/**
449		* Various ways of traversing a collection yield same elements
450		*/
451	<	public void testIteratorEquivalence() {
451	>	public void testTraversalEquivalence() {
452		Collection c = impl.emptyCollection();
453		ThreadLocalRandom rnd = ThreadLocalRandom.current();
454		int n = rnd.nextInt(6);
#	Line 412 | Line 516 | public class Collection8Test extends JSR
516		}
517
518		/**
519	+	* Iterator.forEachRemaining has same behavior as Iterator's
520	+	* default implementation.
521	+	*/
522	+	public void testForEachRemainingConsistentWithDefaultImplementation() {
523	+	Collection c = impl.emptyCollection();
524	+	if (!testImplementationDetails
525	+	|| c.getClass() == java.util.LinkedList.class)
526	+	return;
527	+	ThreadLocalRandom rnd = ThreadLocalRandom.current();
528	+	int n = 1 + rnd.nextInt(3);
529	+	for (int i = 0; i < n; i++) c.add(impl.makeElement(i));
530	+	ArrayList iterated = new ArrayList();
531	+	ArrayList iteratedForEachRemaining = new ArrayList();
532	+	Iterator it1 = c.iterator();
533	+	Iterator it2 = c.iterator();
534	+	assertTrue(it1.hasNext());
535	+	assertTrue(it2.hasNext());
536	+	c.clear();
537	+	Object r1, r2;
538	+	try {
539	+	while (it1.hasNext()) iterated.add(it1.next());
540	+	r1 = iterated;
541	+	} catch (ConcurrentModificationException ex) {
542	+	r1 = ConcurrentModificationException.class;
543	+	assertFalse(impl.isConcurrent());
544	+	} catch (UnsupportedOperationException ex) {
545	+	r1 = UnsupportedOperationException.class;
546	+	}
547	+	try {
548	+	it2.forEachRemaining(iteratedForEachRemaining::add);
549	+	r2 = iteratedForEachRemaining;
550	+	} catch (ConcurrentModificationException ex) {
551	+	r2 = ConcurrentModificationException.class;
552	+	} catch (UnsupportedOperationException ex) {
553	+	r2 = UnsupportedOperationException.class;
554	+	}
555	+	assertEquals(r1, r2);
556	+	}
557	+
558	+	/**
559		* Calling Iterator#remove() after Iterator#forEachRemaining
560		* should (maybe) remove last element
561		*/
#	Line 550 | Line 694 | public class Collection8Test extends JSR
694		assertTrue(found.isEmpty());
695		}
696
697	+	/** TODO: promote to a common utility */
698	+	static <T> T chooseOne(T ... ts) {
699	+	return ts[ThreadLocalRandom.current().nextInt(ts.length)];
700	+	}
701	+
702	+	/** TODO: more random adders and removers */
703	+	static <E> Runnable adderRemover(Collection<E> c, E e) {
704	+	return chooseOne(
705	+	() -> {
706	+	assertTrue(c.add(e));
707	+	assertTrue(c.contains(e));
708	+	assertTrue(c.remove(e));
709	+	assertFalse(c.contains(e));
710	+	},
711	+	() -> {
712	+	assertTrue(c.add(e));
713	+	assertTrue(c.contains(e));
714	+	assertTrue(c.removeIf(x -> x == e));
715	+	assertFalse(c.contains(e));
716	+	},
717	+	() -> {
718	+	assertTrue(c.add(e));
719	+	assertTrue(c.contains(e));
720	+	for (Iterator it = c.iterator();; )
721	+	if (it.next() == e) {
722	+	try { it.remove(); }
723	+	catch (UnsupportedOperationException ok) {
724	+	c.remove(e);
725	+	}
726	+	break;
727	+	}
728	+	assertFalse(c.contains(e));
729	+	});
730	+	}
731	+
732		/**
733		* Motley crew of threads concurrently randomly hammer the collection.
734		*/
#	Line 557 | Line 736 | public class Collection8Test extends JSR
736		if (!impl.isConcurrent()) return;
737		final ThreadLocalRandom rnd = ThreadLocalRandom.current();
738		final Collection c = impl.emptyCollection();
739	<	final long testDurationMillis = timeoutMillis();
739	>	final long testDurationMillis
740	>	= expensiveTests ? LONG_DELAY_MS : timeoutMillis();
741		final AtomicBoolean done = new AtomicBoolean(false);
742		final Object one = impl.makeElement(1);
743		final Object two = impl.makeElement(2);
744	+	final Consumer checkSanity = x -> assertTrue(x == one || x == two);
745	+	final Consumer<Object[]> checkArraySanity = array -> {
746	+	// assertTrue(array.length <= 2); // duplicates are permitted
747	+	for (Object x : array) assertTrue(x == one || x == two);
748	+	};
749		final Object[] emptyArray =
750		(Object[]) java.lang.reflect.Array.newInstance(one.getClass(), 0);
751		final List<Future<?>> futures;
752		final Phaser threadsStarted = new Phaser(1); // register this thread
568	–	final Consumer checkSanity = x -> assertTrue(x == one || x == two);
753		final Runnable[] frobbers = {
754		() -> c.forEach(checkSanity),
755		() -> c.stream().forEach(checkSanity),
#	Line 581 | Line 765 | public class Collection8Test extends JSR
765		do {} while (s.tryAdvance(checkSanity));
766		},
767		() -> { for (Object x : c) checkSanity.accept(x); },
768	<	() -> { for (Object x : c.toArray()) checkSanity.accept(x); },
769	<	() -> { for (Object x : c.toArray(emptyArray)) checkSanity.accept(x); },
586	<	() -> {
587	<	assertTrue(c.add(one));
588	<	assertTrue(c.contains(one));
589	<	assertTrue(c.remove(one));
590	<	assertFalse(c.contains(one));
591	<	},
768	>	() -> checkArraySanity.accept(c.toArray()),
769	>	() -> checkArraySanity.accept(c.toArray(emptyArray)),
770		() -> {
771	<	assertTrue(c.add(two));
772	<	assertTrue(c.contains(two));
773	<	assertTrue(c.remove(two));
774	<	assertFalse(c.contains(two));
775	<	},
771	>	Object[] a = new Object[5];
772	>	Object three = impl.makeElement(3);
773	>	Arrays.fill(a, 0, a.length, three);
774	>	Object[] x = c.toArray(a);
775	>	if (x == a)
776	>	for (int i = 0; i < a.length && a[i] != null; i++)
777	>	checkSanity.accept(a[i]);
778	>	// A careful reading of the spec does not support:
779	>	// for (i++; i < a.length; i++) assertSame(three, a[i]);
780	>	else
781	>	checkArraySanity.accept(x);
782	>	},
783	>	adderRemover(c, one),
784	>	adderRemover(c, two),
785		};
786		final List<Runnable> tasks =
787		Arrays.stream(frobbers)
#	Line 652 | Line 839 | public class Collection8Test extends JSR
839		}
840		}
841
842	+	/**
843	+	* Spliterator.getComparator throws IllegalStateException iff the
844	+	* spliterator does not report SORTED.
845	+	*/
846	+	public void testGetComparator_IllegalStateException() {
847	+	Collection c = impl.emptyCollection();
848	+	Spliterator s = c.spliterator();
849	+	boolean reportsSorted = s.hasCharacteristics(Spliterator.SORTED);
850	+	try {
851	+	s.getComparator();
852	+	assertTrue(reportsSorted);
853	+	} catch (IllegalStateException ex) {
854	+	assertFalse(reportsSorted);
855	+	}
856	+	}
857	+
858		//     public void testCollection8DebugFail() {
859		//         fail(impl.klazz().getSimpleName());
860		//     }

Diff Legend

–	Removed lines
+	Added lines
<	Changed lines
>	Changed lines