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.30 by jsr166, Wed Nov 23 00:25:16 2016 UTC vs.
Revision 1.44 by jsr166, Sun Dec 18 20:47:28 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 21 | Line 22 | import java.util.Queue;
22		import java.util.Spliterator;
23		import java.util.concurrent.BlockingDeque;
24		import java.util.concurrent.BlockingQueue;
25	+	import java.util.concurrent.ConcurrentLinkedQueue;
26		import java.util.concurrent.CountDownLatch;
27		import java.util.concurrent.Executors;
28		import java.util.concurrent.ExecutorService;
#	Line 341 | 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 352 | Line 457 | public class Collection8Test extends JSR
457		ArrayList iteratedForEachRemaining = new ArrayList();
458		ArrayList tryAdvanced = new ArrayList();
459		ArrayList spliterated = new ArrayList();
460	+	ArrayList splitonced = new ArrayList();
461		ArrayList forEached = new ArrayList();
462	+	ArrayList streamForEached = new ArrayList();
463	+	ConcurrentLinkedQueue parallelStreamForEached = new ConcurrentLinkedQueue();
464		ArrayList removeIfed = new ArrayList();
465		for (Object x : c) iterated.add(x);
466		c.iterator().forEachRemaining(iteratedForEachRemaining::add);
467		for (Spliterator s = c.spliterator();
468		s.tryAdvance(tryAdvanced::add); ) {}
469		c.spliterator().forEachRemaining(spliterated::add);
470	+	{                       // trySplit returns "strict prefix"
471	+	Spliterator s1 = c.spliterator(), s2 = s1.trySplit();
472	+	if (s2 != null) s2.forEachRemaining(splitonced::add);
473	+	s1.forEachRemaining(splitonced::add);
474	+	}
475		c.forEach(forEached::add);
476	+	c.stream().forEach(streamForEached::add);
477	+	c.parallelStream().forEach(parallelStreamForEached::add);
478		c.removeIf(e -> { removeIfed.add(e); return false; });
479		boolean ordered =
480		c.spliterator().hasCharacteristics(Spliterator.ORDERED);
481		if (c instanceof List || c instanceof Deque)
482		assertTrue(ordered);
483	+	HashSet cset = new HashSet(c);
484	+	assertEquals(cset, new HashSet(parallelStreamForEached));
485		if (ordered) {
486		assertEquals(iterated, iteratedForEachRemaining);
487		assertEquals(iterated, tryAdvanced);
488		assertEquals(iterated, spliterated);
489	+	assertEquals(iterated, splitonced);
490		assertEquals(iterated, forEached);
491	+	assertEquals(iterated, streamForEached);
492		assertEquals(iterated, removeIfed);
493		} else {
375	–	HashSet cset = new HashSet(c);
494		assertEquals(cset, new HashSet(iterated));
495		assertEquals(cset, new HashSet(iteratedForEachRemaining));
496		assertEquals(cset, new HashSet(tryAdvanced));
497		assertEquals(cset, new HashSet(spliterated));
498	+	assertEquals(cset, new HashSet(splitonced));
499		assertEquals(cset, new HashSet(forEached));
500	+	assertEquals(cset, new HashSet(streamForEached));
501		assertEquals(cset, new HashSet(removeIfed));
502		}
503		if (c instanceof Deque) {
#	Line 396 | 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	+	}
545	+	try {
546	+	it2.forEachRemaining(iteratedForEachRemaining::add);
547	+	r2 = iteratedForEachRemaining;
548	+	} catch (ConcurrentModificationException ex) {
549	+	r2 = ConcurrentModificationException.class;
550	+	assertFalse(impl.isConcurrent());
551	+	}
552	+	assertEquals(r1, r2);
553	+	}
554	+
555	+	/**
556		* Calling Iterator#remove() after Iterator#forEachRemaining
557		* should (maybe) remove last element
558		*/
#	Line 534 | Line 691 | public class Collection8Test extends JSR
691		assertTrue(found.isEmpty());
692		}
693
694	+	/** TODO: promote to a common utility */
695	+	static <T> T chooseOne(T ... ts) {
696	+	return ts[ThreadLocalRandom.current().nextInt(ts.length)];
697	+	}
698	+
699	+	/** TODO: more random adders and removers */
700	+	static <E> Runnable adderRemover(Collection<E> c, E e) {
701	+	return chooseOne(
702	+	() -> {
703	+	assertTrue(c.add(e));
704	+	assertTrue(c.contains(e));
705	+	assertTrue(c.remove(e));
706	+	assertFalse(c.contains(e));
707	+	},
708	+	() -> {
709	+	assertTrue(c.add(e));
710	+	assertTrue(c.contains(e));
711	+	assertTrue(c.removeIf(x -> x == e));
712	+	assertFalse(c.contains(e));
713	+	},
714	+	() -> {
715	+	assertTrue(c.add(e));
716	+	assertTrue(c.contains(e));
717	+	for (Iterator it = c.iterator();; )
718	+	if (it.next() == e) {
719	+	try { it.remove(); }
720	+	catch (UnsupportedOperationException ok) {
721	+	c.remove(e);
722	+	}
723	+	break;
724	+	}
725	+	assertFalse(c.contains(e));
726	+	});
727	+	}
728	+
729		/**
730		* Motley crew of threads concurrently randomly hammer the collection.
731		*/
#	Line 541 | Line 733 | public class Collection8Test extends JSR
733		if (!impl.isConcurrent()) return;
734		final ThreadLocalRandom rnd = ThreadLocalRandom.current();
735		final Collection c = impl.emptyCollection();
736	<	final long testDurationMillis = timeoutMillis();
736	>	final long testDurationMillis
737	>	= expensiveTests ? LONG_DELAY_MS : timeoutMillis();
738		final AtomicBoolean done = new AtomicBoolean(false);
739		final Object one = impl.makeElement(1);
740		final Object two = impl.makeElement(2);
741	+	final Consumer checkSanity = x -> assertTrue(x == one || x == two);
742	+	final Consumer<Object[]> checkArraySanity = array -> {
743	+	// assertTrue(array.length <= 2); // duplicates are permitted
744	+	for (Object x : array) assertTrue(x == one || x == two);
745	+	};
746		final Object[] emptyArray =
747		(Object[]) java.lang.reflect.Array.newInstance(one.getClass(), 0);
748		final List<Future<?>> futures;
749		final Phaser threadsStarted = new Phaser(1); // register this thread
750		final Runnable[] frobbers = {
751	<	() -> c.forEach(x -> assertTrue(x == one || x == two)),
752	<	() -> c.stream().forEach(x -> assertTrue(x == one || x == two)),
751	>	() -> c.forEach(checkSanity),
752	>	() -> c.stream().forEach(checkSanity),
753	>	() -> c.parallelStream().forEach(checkSanity),
754		() -> c.spliterator().trySplit(),
755		() -> {
756		Spliterator s = c.spliterator();
757	<	s.tryAdvance(x -> assertTrue(x == one || x == two));
757	>	s.tryAdvance(checkSanity);
758		s.trySplit();
759		},
760		() -> {
761		Spliterator s = c.spliterator();
762	<	do {} while (s.tryAdvance(x -> assertTrue(x == one || x == two)));
564	<	},
565	<	() -> {
566	<	for (Object x : c) assertTrue(x == one || x == two);
762	>	do {} while (s.tryAdvance(checkSanity));
763		},
764	+	() -> { for (Object x : c) checkSanity.accept(x); },
765	+	() -> checkArraySanity.accept(c.toArray()),
766	+	() -> checkArraySanity.accept(c.toArray(emptyArray)),
767		() -> {
768	<	for (Object x : c.toArray()) assertTrue(x == one || x == two);
769	<	},
770	<	() -> {
771	<	for (Object x : c.toArray(emptyArray)) assertTrue(x == one || x == two);
772	<	},
773	<	() -> {
774	<	assertTrue(c.add(one));
775	<	assertTrue(c.contains(one));
776	<	assertTrue(c.remove(one));
777	<	assertFalse(c.contains(one));
778	<	},
779	<	() -> {
780	<	assertTrue(c.add(two));
781	<	assertTrue(c.contains(two));
583	<	assertTrue(c.remove(two));
584	<	assertFalse(c.contains(two));
585	<	},
768	>	Object[] a = new Object[5];
769	>	Object three = impl.makeElement(3);
770	>	Arrays.fill(a, 0, a.length, three);
771	>	Object[] x = c.toArray(a);
772	>	if (x == a)
773	>	for (int i = 0; i < a.length && a[i] != null; i++)
774	>	checkSanity.accept(a[i]);
775	>	// A careful reading of the spec does not support:
776	>	// for (i++; i < a.length; i++) assertSame(three, a[i]);
777	>	else
778	>	checkArraySanity.accept(x);
779	>	},
780	>	adderRemover(c, one),
781	>	adderRemover(c, two),
782		};
783		final List<Runnable> tasks =
784		Arrays.stream(frobbers)
#	Line 613 | Line 809 | public class Collection8Test extends JSR
809		*/
810		public void testLateBindingStyle() {
811		if (!testImplementationDetails) return;
812	+	if (impl.klazz() == ArrayList.class) return; // for jdk8
813		// Immutable (snapshot) spliterators are exempt
814		if (impl.emptyCollection().spliterator()
815		.hasCharacteristics(Spliterator.IMMUTABLE))
#	Line 639 | Line 836 | public class Collection8Test extends JSR
836		}
837		}
838
839	+	/**
840	+	* Spliterator.getComparator throws IllegalStateException iff the
841	+	* spliterator does not report SORTED.
842	+	*/
843	+	public void testGetComparator_IllegalStateException() {
844	+	Collection c = impl.emptyCollection();
845	+	Spliterator s = c.spliterator();
846	+	boolean reportsSorted = s.hasCharacteristics(Spliterator.SORTED);
847	+	try {
848	+	s.getComparator();
849	+	assertTrue(reportsSorted);
850	+	} catch (IllegalStateException ex) {
851	+	assertFalse(reportsSorted);
852	+	}
853	+	}
854	+
855		//     public void testCollection8DebugFail() {
856		//         fail(impl.klazz().getSimpleName());
857		//     }

Diff Legend

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