ViewVC Help
View File | Revision Log | Show Annotations | Download File | Root Listing
root/jsr166/jsr166/src/test/tck/Collection8Test.java
(Generate patch)

Comparing jsr166/src/test/tck/Collection8Test.java (file contents):
Revision 1.52 by jsr166, Sat May 5 18:29:53 2018 UTC vs.
Revision 1.55 by jsr166, Tue May 22 15:51:31 2018 UTC

# Line 589 | Line 589 | public class Collection8Test extends JSR
589      public void testRemoveAfterForEachRemaining() {
590          Collection c = impl.emptyCollection();
591          ThreadLocalRandom rnd = ThreadLocalRandom.current();
592 +        ArrayList copy = new ArrayList();
593 +        boolean ordered = c.spliterator().hasCharacteristics(Spliterator.ORDERED);
594          testCollection: {
595              int n = 3 + rnd.nextInt(2);
596 <            for (int i = 0; i < n; i++) c.add(impl.makeElement(i));
596 >            for (int i = 0; i < n; i++) {
597 >                Object x = impl.makeElement(i);
598 >                c.add(x);
599 >                copy.add(x);
600 >            }
601              Iterator it = c.iterator();
602 <            assertTrue(it.hasNext());
603 <            assertEquals(impl.makeElement(0), it.next());
604 <            assertTrue(it.hasNext());
605 <            assertEquals(impl.makeElement(1), it.next());
606 <            it.forEachRemaining(e -> assertTrue(c.contains(e)));
602 >            if (ordered) {
603 >                if (rnd.nextBoolean()) assertTrue(it.hasNext());
604 >                assertEquals(impl.makeElement(0), it.next());
605 >                if (rnd.nextBoolean()) assertTrue(it.hasNext());
606 >                assertEquals(impl.makeElement(1), it.next());
607 >            } else {
608 >                if (rnd.nextBoolean()) assertTrue(it.hasNext());
609 >                assertTrue(copy.contains(it.next()));
610 >                if (rnd.nextBoolean()) assertTrue(it.hasNext());
611 >                assertTrue(copy.contains(it.next()));
612 >            }
613 >            if (rnd.nextBoolean()) assertTrue(it.hasNext());
614 >            it.forEachRemaining(
615 >                e -> {
616 >                    assertTrue(c.contains(e));
617 >                    assertTrue(copy.contains(e));});
618              if (testImplementationDetails) {
619                  if (c instanceof java.util.concurrent.ArrayBlockingQueue) {
620                      assertIteratorExhausted(it);
# Line 607 | Line 624 | public class Collection8Test extends JSR
624                          break testCollection;
625                      }
626                      assertEquals(n - 1, c.size());
627 <                    for (int i = 0; i < n - 1; i++)
628 <                        assertTrue(c.contains(impl.makeElement(i)));
629 <                    assertFalse(c.contains(impl.makeElement(n - 1)));
627 >                    if (ordered) {
628 >                        for (int i = 0; i < n - 1; i++)
629 >                            assertTrue(c.contains(impl.makeElement(i)));
630 >                        assertFalse(c.contains(impl.makeElement(n - 1)));
631 >                    }
632                  }
633              }
634          }
635          if (c instanceof Deque) {
636              Deque d = (Deque) impl.emptyCollection();
637 +            assertTrue(ordered);
638              int n = 3 + rnd.nextInt(2);
639              for (int i = 0; i < n; i++) d.add(impl.makeElement(i));
640              Iterator it = d.descendingIterator();
# Line 932 | Line 952 | public class Collection8Test extends JSR
952          } catch (java.io.NotSerializableException acceptable) {}
953      }
954  
955 <    public void testReplaceAllIsNotStructuralModification() {
955 >    public void DISABLED_testReplaceAllIsNotStructuralModification() {
956          Collection c = impl.emptyCollection();
957          if (!(c instanceof List))
958              return;

Diff Legend

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