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

Comparing jsr166/src/test/tck/ConcurrentSkipListSetTest.java (file contents):
Revision 1.19 by jsr166, Thu Nov 18 20:21:53 2010 UTC vs.
Revision 1.44 by jsr166, Thu Sep 15 16:43:56 2016 UTC

# Line 1 | Line 1
1   /*
2   * Written by Doug Lea with assistance from members of JCP JSR-166
3   * Expert Group and released to the public domain, as explained at
4 < * http://creativecommons.org/licenses/publicdomain
4 > * http://creativecommons.org/publicdomain/zero/1.0/
5   */
6  
7 < import junit.framework.*;
8 < import java.util.*;
9 < import java.util.concurrent.*;
10 < import java.io.*;
7 > import java.util.Arrays;
8 > import java.util.BitSet;
9 > import java.util.Collection;
10 > import java.util.Comparator;
11 > import java.util.Iterator;
12 > import java.util.NavigableSet;
13 > import java.util.NoSuchElementException;
14 > import java.util.Random;
15 > import java.util.Set;
16 > import java.util.SortedSet;
17 > import java.util.concurrent.ConcurrentSkipListSet;
18 >
19 > import junit.framework.Test;
20 > import junit.framework.TestSuite;
21  
22   public class ConcurrentSkipListSetTest extends JSR166TestCase {
23      public static void main(String[] args) {
24 <        junit.textui.TestRunner.run(suite());
24 >        main(suite(), args);
25      }
26      public static Test suite() {
27          return new TestSuite(ConcurrentSkipListSetTest.class);
# Line 24 | Line 34 | public class ConcurrentSkipListSetTest e
34      }
35  
36      /**
37 <     * Create a set of given size containing consecutive
37 >     * Returns a new set of given size containing consecutive
38       * Integers 0 ... n.
39       */
40      private ConcurrentSkipListSet<Integer> populatedSet(int n) {
41          ConcurrentSkipListSet<Integer> q =
42              new ConcurrentSkipListSet<Integer>();
43          assertTrue(q.isEmpty());
44 <        for (int i = n-1; i >= 0; i-=2)
44 >        for (int i = n - 1; i >= 0; i -= 2)
45              assertTrue(q.add(new Integer(i)));
46 <        for (int i = (n & 1); i < n; i+=2)
46 >        for (int i = (n & 1); i < n; i += 2)
47              assertTrue(q.add(new Integer(i)));
48          assertFalse(q.isEmpty());
49          assertEquals(n, q.size());
# Line 41 | Line 51 | public class ConcurrentSkipListSetTest e
51      }
52  
53      /**
54 <     * Create set of first 5 ints
54 >     * Returns a new set of first 5 ints.
55       */
56      private ConcurrentSkipListSet set5() {
57          ConcurrentSkipListSet q = new ConcurrentSkipListSet();
# Line 67 | Line 77 | public class ConcurrentSkipListSetTest e
77       */
78      public void testConstructor3() {
79          try {
80 <            ConcurrentSkipListSet q = new ConcurrentSkipListSet((Collection)null);
80 >            new ConcurrentSkipListSet((Collection)null);
81              shouldThrow();
82          } catch (NullPointerException success) {}
83      }
# Line 77 | Line 87 | public class ConcurrentSkipListSetTest e
87       */
88      public void testConstructor4() {
89          try {
90 <            Integer[] ints = new Integer[SIZE];
81 <            ConcurrentSkipListSet q = new ConcurrentSkipListSet(Arrays.asList(ints));
90 >            new ConcurrentSkipListSet(Arrays.asList(new Integer[SIZE]));
91              shouldThrow();
92          } catch (NullPointerException success) {}
93      }
# Line 87 | Line 96 | public class ConcurrentSkipListSetTest e
96       * Initializing from Collection with some null elements throws NPE
97       */
98      public void testConstructor5() {
99 +        Integer[] ints = new Integer[SIZE];
100 +        for (int i = 0; i < SIZE - 1; ++i)
101 +            ints[i] = new Integer(i);
102          try {
103 <            Integer[] ints = new Integer[SIZE];
92 <            for (int i = 0; i < SIZE-1; ++i)
93 <                ints[i] = new Integer(i);
94 <            ConcurrentSkipListSet q = new ConcurrentSkipListSet(Arrays.asList(ints));
103 >            new ConcurrentSkipListSet(Arrays.asList(ints));
104              shouldThrow();
105          } catch (NullPointerException success) {}
106      }
# Line 119 | Line 128 | public class ConcurrentSkipListSetTest e
128          for (int i = 0; i < SIZE; ++i)
129              ints[i] = new Integer(i);
130          q.addAll(Arrays.asList(ints));
131 <        for (int i = SIZE-1; i >= 0; --i)
131 >        for (int i = SIZE - 1; i >= 0; --i)
132              assertEquals(ints[i], q.pollFirst());
133      }
134  
# Line 143 | Line 152 | public class ConcurrentSkipListSetTest e
152      public void testSize() {
153          ConcurrentSkipListSet q = populatedSet(SIZE);
154          for (int i = 0; i < SIZE; ++i) {
155 <            assertEquals(SIZE-i, q.size());
155 >            assertEquals(SIZE - i, q.size());
156              q.pollFirst();
157          }
158          for (int i = 0; i < SIZE; ++i) {
# Line 156 | Line 165 | public class ConcurrentSkipListSetTest e
165       * add(null) throws NPE
166       */
167      public void testAddNull() {
168 +        ConcurrentSkipListSet q = new ConcurrentSkipListSet();
169          try {
160            ConcurrentSkipListSet q = new ConcurrentSkipListSet();
170              q.add(null);
171              shouldThrow();
172          } catch (NullPointerException success) {}
# Line 185 | Line 194 | public class ConcurrentSkipListSetTest e
194       * Add of non-Comparable throws CCE
195       */
196      public void testAddNonComparable() {
197 +        ConcurrentSkipListSet q = new ConcurrentSkipListSet();
198          try {
189            ConcurrentSkipListSet q = new ConcurrentSkipListSet();
190            q.add(new Object());
199              q.add(new Object());
200              q.add(new Object());
201              shouldThrow();
202 <        } catch (ClassCastException success) {}
202 >        } catch (ClassCastException success) {
203 >            assertTrue(q.size() < 2);
204 >            for (int i = 0, size = q.size(); i < size; i++)
205 >                assertTrue(q.pollFirst().getClass() == Object.class);
206 >            assertNull(q.pollFirst());
207 >            assertTrue(q.isEmpty());
208 >            assertEquals(0, q.size());
209 >        }
210      }
211  
212      /**
213       * addAll(null) throws NPE
214       */
215      public void testAddAll1() {
216 +        ConcurrentSkipListSet q = new ConcurrentSkipListSet();
217          try {
202            ConcurrentSkipListSet q = new ConcurrentSkipListSet();
218              q.addAll(null);
219              shouldThrow();
220          } catch (NullPointerException success) {}
# Line 209 | Line 224 | public class ConcurrentSkipListSetTest e
224       * addAll of a collection with null elements throws NPE
225       */
226      public void testAddAll2() {
227 +        ConcurrentSkipListSet q = new ConcurrentSkipListSet();
228 +        Integer[] ints = new Integer[SIZE];
229          try {
213            ConcurrentSkipListSet q = new ConcurrentSkipListSet();
214            Integer[] ints = new Integer[SIZE];
230              q.addAll(Arrays.asList(ints));
231              shouldThrow();
232          } catch (NullPointerException success) {}
# Line 222 | Line 237 | public class ConcurrentSkipListSetTest e
237       * possibly adding some elements
238       */
239      public void testAddAll3() {
240 +        ConcurrentSkipListSet q = new ConcurrentSkipListSet();
241 +        Integer[] ints = new Integer[SIZE];
242 +        for (int i = 0; i < SIZE - 1; ++i)
243 +            ints[i] = new Integer(i);
244          try {
226            ConcurrentSkipListSet q = new ConcurrentSkipListSet();
227            Integer[] ints = new Integer[SIZE];
228            for (int i = 0; i < SIZE-1; ++i)
229                ints[i] = new Integer(i);
245              q.addAll(Arrays.asList(ints));
246              shouldThrow();
247          } catch (NullPointerException success) {}
# Line 239 | Line 254 | public class ConcurrentSkipListSetTest e
254          Integer[] empty = new Integer[0];
255          Integer[] ints = new Integer[SIZE];
256          for (int i = 0; i < SIZE; ++i)
257 <            ints[i] = new Integer(SIZE-1-i);
257 >            ints[i] = new Integer(SIZE - 1 - i);
258          ConcurrentSkipListSet q = new ConcurrentSkipListSet();
259          assertFalse(q.addAll(Arrays.asList(empty)));
260          assertTrue(q.addAll(Arrays.asList(ints)));
# Line 263 | Line 278 | public class ConcurrentSkipListSetTest e
278       */
279      public void testPollLast() {
280          ConcurrentSkipListSet q = populatedSet(SIZE);
281 <        for (int i = SIZE-1; i >= 0; --i) {
281 >        for (int i = SIZE - 1; i >= 0; --i) {
282              assertEquals(i, q.pollLast());
283          }
284          assertNull(q.pollFirst());
285      }
286  
272
287      /**
288       * remove(x) removes x and returns true if present
289       */
290      public void testRemoveElement() {
291          ConcurrentSkipListSet q = populatedSet(SIZE);
292 <        for (int i = 1; i < SIZE; i+=2) {
292 >        for (int i = 1; i < SIZE; i += 2) {
293              assertTrue(q.contains(i));
294              assertTrue(q.remove(i));
295              assertFalse(q.contains(i));
296 <            assertTrue(q.contains(i-1));
296 >            assertTrue(q.contains(i - 1));
297          }
298 <        for (int i = 0; i < SIZE; i+=2) {
298 >        for (int i = 0; i < SIZE; i += 2) {
299              assertTrue(q.contains(i));
300              assertTrue(q.remove(i));
301              assertFalse(q.contains(i));
302 <            assertFalse(q.remove(i+1));
303 <            assertFalse(q.contains(i+1));
302 >            assertFalse(q.remove(i + 1));
303 >            assertFalse(q.contains(i + 1));
304          }
305          assertTrue(q.isEmpty());
306      }
# Line 345 | Line 359 | public class ConcurrentSkipListSetTest e
359                  assertTrue(changed);
360  
361              assertTrue(q.containsAll(p));
362 <            assertEquals(SIZE-i, q.size());
362 >            assertEquals(SIZE - i, q.size());
363              p.pollFirst();
364          }
365      }
# Line 358 | Line 372 | public class ConcurrentSkipListSetTest e
372              ConcurrentSkipListSet q = populatedSet(SIZE);
373              ConcurrentSkipListSet p = populatedSet(i);
374              assertTrue(q.removeAll(p));
375 <            assertEquals(SIZE-i, q.size());
375 >            assertEquals(SIZE - i, q.size());
376              for (int j = 0; j < i; ++j) {
377 <                Integer I = (Integer)(p.pollFirst());
378 <                assertFalse(q.contains(I));
377 >                Integer x = (Integer)(p.pollFirst());
378 >                assertFalse(q.contains(x));
379              }
380          }
381      }
382  
369
370
383      /**
384       * lower returns preceding element
385       */
# Line 456 | Line 468 | public class ConcurrentSkipListSetTest e
468      public void testToArray2() {
469          ConcurrentSkipListSet<Integer> q = populatedSet(SIZE);
470          Integer[] ints = new Integer[SIZE];
471 <        Integer[] array = q.toArray(ints);
460 <        assertSame(ints, array);
471 >        assertSame(ints, q.toArray(ints));
472          for (int i = 0; i < ints.length; i++)
473              assertSame(ints[i], q.pollFirst());
474      }
# Line 467 | Line 478 | public class ConcurrentSkipListSetTest e
478       */
479      public void testIterator() {
480          ConcurrentSkipListSet q = populatedSet(SIZE);
470        int i = 0;
481          Iterator it = q.iterator();
482 <        while (it.hasNext()) {
482 >        int i;
483 >        for (i = 0; it.hasNext(); i++)
484              assertTrue(q.contains(it.next()));
474            ++i;
475        }
485          assertEquals(i, SIZE);
486 +        assertIteratorExhausted(it);
487      }
488  
489      /**
490       * iterator of empty set has no elements
491       */
492      public void testEmptyIterator() {
493 <        ConcurrentSkipListSet q = new ConcurrentSkipListSet();
494 <        int i = 0;
495 <        Iterator it = q.iterator();
486 <        while (it.hasNext()) {
487 <            assertTrue(q.contains(it.next()));
488 <            ++i;
489 <        }
490 <        assertEquals(i, 0);
493 >        NavigableSet s = new ConcurrentSkipListSet();
494 >        assertIteratorExhausted(s.iterator());
495 >        assertIteratorExhausted(s.descendingSet().iterator());
496      }
497  
498      /**
# Line 509 | Line 514 | public class ConcurrentSkipListSetTest e
514          assertFalse(it.hasNext());
515      }
516  
512
517      /**
518       * toString contains toStrings of elements
519       */
# Line 517 | Line 521 | public class ConcurrentSkipListSetTest e
521          ConcurrentSkipListSet q = populatedSet(SIZE);
522          String s = q.toString();
523          for (int i = 0; i < SIZE; ++i) {
524 <            assertTrue(s.indexOf(String.valueOf(i)) >= 0);
524 >            assertTrue(s.contains(String.valueOf(i)));
525          }
526      }
527  
# Line 525 | Line 529 | public class ConcurrentSkipListSetTest e
529       * A deserialized serialized set has same elements
530       */
531      public void testSerialization() throws Exception {
532 <        ConcurrentSkipListSet q = populatedSet(SIZE);
533 <        ByteArrayOutputStream bout = new ByteArrayOutputStream(10000);
534 <        ObjectOutputStream out = new ObjectOutputStream(new BufferedOutputStream(bout));
535 <        out.writeObject(q);
536 <        out.close();
537 <
538 <        ByteArrayInputStream bin = new ByteArrayInputStream(bout.toByteArray());
539 <        ObjectInputStream in = new ObjectInputStream(new BufferedInputStream(bin));
540 <        ConcurrentSkipListSet r = (ConcurrentSkipListSet)in.readObject();
541 <        assertEquals(q.size(), r.size());
542 <        while (!q.isEmpty())
543 <            assertEquals(q.pollFirst(), r.pollFirst());
532 >        NavigableSet x = populatedSet(SIZE);
533 >        NavigableSet y = serialClone(x);
534 >
535 >        assertNotSame(x, y);
536 >        assertEquals(x.size(), y.size());
537 >        assertEquals(x, y);
538 >        assertEquals(y, x);
539 >        while (!x.isEmpty()) {
540 >            assertFalse(y.isEmpty());
541 >            assertEquals(x.pollFirst(), y.pollFirst());
542 >        }
543 >        assertTrue(y.isEmpty());
544      }
545  
546      /**
# Line 659 | Line 663 | public class ConcurrentSkipListSetTest e
663      }
664  
665      Random rnd = new Random(666);
662    BitSet bs;
666  
667      /**
668       * Subsets of subsets subdivide correctly
# Line 669 | Line 672 | public class ConcurrentSkipListSetTest e
672          Class cl = ConcurrentSkipListSet.class;
673  
674          NavigableSet<Integer> set = newSet(cl);
675 <        bs = new BitSet(setSize);
675 >        BitSet bs = new BitSet(setSize);
676  
677 <        populate(set, setSize);
678 <        check(set,                 0, setSize - 1, true);
679 <        check(set.descendingSet(), 0, setSize - 1, false);
680 <
681 <        mutateSet(set, 0, setSize - 1);
682 <        check(set,                 0, setSize - 1, true);
683 <        check(set.descendingSet(), 0, setSize - 1, false);
677 >        populate(set, setSize, bs);
678 >        check(set,                 0, setSize - 1, true, bs);
679 >        check(set.descendingSet(), 0, setSize - 1, false, bs);
680 >
681 >        mutateSet(set, 0, setSize - 1, bs);
682 >        check(set,                 0, setSize - 1, true, bs);
683 >        check(set.descendingSet(), 0, setSize - 1, false, bs);
684  
685          bashSubSet(set.subSet(0, true, setSize, false),
686 <                   0, setSize - 1, true);
686 >                   0, setSize - 1, true, bs);
687 >    }
688 >
689 >    /**
690 >     * addAll is idempotent
691 >     */
692 >    public void testAddAll_idempotent() throws Exception {
693 >        Set x = populatedSet(SIZE);
694 >        Set y = new ConcurrentSkipListSet(x);
695 >        y.addAll(x);
696 >        assertEquals(x, y);
697 >        assertEquals(y, x);
698      }
699  
700      static NavigableSet<Integer> newSet(Class cl) throws Exception {
701 <        NavigableSet<Integer> result = (NavigableSet<Integer>) cl.newInstance();
702 <        assertEquals(result.size(), 0);
701 >        NavigableSet<Integer> result =
702 >            (NavigableSet<Integer>) cl.getConstructor().newInstance();
703 >        assertEquals(0, result.size());
704          assertFalse(result.iterator().hasNext());
705          return result;
706      }
707  
708 <    void populate(NavigableSet<Integer> set, int limit) {
708 >    void populate(NavigableSet<Integer> set, int limit, BitSet bs) {
709          for (int i = 0, n = 2 * limit / 3; i < n; i++) {
710              int element = rnd.nextInt(limit);
711 <            put(set, element);
711 >            put(set, element, bs);
712          }
713      }
714  
715 <    void mutateSet(NavigableSet<Integer> set, int min, int max) {
715 >    void mutateSet(NavigableSet<Integer> set, int min, int max, BitSet bs) {
716          int size = set.size();
717          int rangeSize = max - min + 1;
718  
719          // Remove a bunch of entries directly
720          for (int i = 0, n = rangeSize / 2; i < n; i++) {
721 <            remove(set, min - 5 + rnd.nextInt(rangeSize + 10));
721 >            remove(set, min - 5 + rnd.nextInt(rangeSize + 10), bs);
722          }
723  
724          // Remove a bunch of entries with iterator
# Line 717 | Line 732 | public class ConcurrentSkipListSetTest e
732          // Add entries till we're back to original size
733          while (set.size() < size) {
734              int element = min + rnd.nextInt(rangeSize);
735 <            assertTrue(element >= min && element<= max);
736 <            put(set, element);
735 >            assertTrue(element >= min && element <= max);
736 >            put(set, element, bs);
737          }
738      }
739  
740 <    void mutateSubSet(NavigableSet<Integer> set, int min, int max) {
740 >    void mutateSubSet(NavigableSet<Integer> set, int min, int max,
741 >                      BitSet bs) {
742          int size = set.size();
743          int rangeSize = max - min + 1;
744  
745          // Remove a bunch of entries directly
746          for (int i = 0, n = rangeSize / 2; i < n; i++) {
747 <            remove(set, min - 5 + rnd.nextInt(rangeSize + 10));
747 >            remove(set, min - 5 + rnd.nextInt(rangeSize + 10), bs);
748          }
749  
750          // Remove a bunch of entries with iterator
# Line 742 | Line 758 | public class ConcurrentSkipListSetTest e
758          // Add entries till we're back to original size
759          while (set.size() < size) {
760              int element = min - 5 + rnd.nextInt(rangeSize + 10);
761 <            if (element >= min && element<= max) {
762 <                put(set, element);
761 >            if (element >= min && element <= max) {
762 >                put(set, element, bs);
763              } else {
764                  try {
765                      set.add(element);
# Line 753 | Line 769 | public class ConcurrentSkipListSetTest e
769          }
770      }
771  
772 <    void put(NavigableSet<Integer> set, int element) {
772 >    void put(NavigableSet<Integer> set, int element, BitSet bs) {
773          if (set.add(element))
774              bs.set(element);
775      }
776  
777 <    void remove(NavigableSet<Integer> set, int element) {
777 >    void remove(NavigableSet<Integer> set, int element, BitSet bs) {
778          if (set.remove(element))
779              bs.clear(element);
780      }
781  
782      void bashSubSet(NavigableSet<Integer> set,
783 <                    int min, int max, boolean ascending) {
784 <        check(set, min, max, ascending);
785 <        check(set.descendingSet(), min, max, !ascending);
786 <
787 <        mutateSubSet(set, min, max);
788 <        check(set, min, max, ascending);
789 <        check(set.descendingSet(), min, max, !ascending);
783 >                    int min, int max, boolean ascending,
784 >                    BitSet bs) {
785 >        check(set, min, max, ascending, bs);
786 >        check(set.descendingSet(), min, max, !ascending, bs);
787 >
788 >        mutateSubSet(set, min, max, bs);
789 >        check(set, min, max, ascending, bs);
790 >        check(set.descendingSet(), min, max, !ascending, bs);
791  
792          // Recurse
793          if (max - min < 2)
# Line 782 | Line 799 | public class ConcurrentSkipListSetTest e
799          NavigableSet<Integer> hm = set.headSet(midPoint, incl);
800          if (ascending) {
801              if (rnd.nextBoolean())
802 <                bashSubSet(hm, min, midPoint - (incl ? 0 : 1), true);
802 >                bashSubSet(hm, min, midPoint - (incl ? 0 : 1), true, bs);
803              else
804                  bashSubSet(hm.descendingSet(), min, midPoint - (incl ? 0 : 1),
805 <                           false);
805 >                           false, bs);
806          } else {
807              if (rnd.nextBoolean())
808 <                bashSubSet(hm, midPoint + (incl ? 0 : 1), max, false);
808 >                bashSubSet(hm, midPoint + (incl ? 0 : 1), max, false, bs);
809              else
810                  bashSubSet(hm.descendingSet(), midPoint + (incl ? 0 : 1), max,
811 <                           true);
811 >                           true, bs);
812          }
813  
814          // tailSet - pick direction and endpoint inclusion randomly
# Line 799 | Line 816 | public class ConcurrentSkipListSetTest e
816          NavigableSet<Integer> tm = set.tailSet(midPoint,incl);
817          if (ascending) {
818              if (rnd.nextBoolean())
819 <                bashSubSet(tm, midPoint + (incl ? 0 : 1), max, true);
819 >                bashSubSet(tm, midPoint + (incl ? 0 : 1), max, true, bs);
820              else
821                  bashSubSet(tm.descendingSet(), midPoint + (incl ? 0 : 1), max,
822 <                           false);
822 >                           false, bs);
823          } else {
824              if (rnd.nextBoolean()) {
825 <                bashSubSet(tm, min, midPoint - (incl ? 0 : 1), false);
825 >                bashSubSet(tm, min, midPoint - (incl ? 0 : 1), false, bs);
826              } else {
827                  bashSubSet(tm.descendingSet(), min, midPoint - (incl ? 0 : 1),
828 <                           true);
828 >                           true, bs);
829              }
830          }
831  
# Line 825 | Line 842 | public class ConcurrentSkipListSetTest e
842                  endpoints[0], lowIncl, endpoints[1], highIncl);
843              if (rnd.nextBoolean())
844                  bashSubSet(sm, endpoints[0] + (lowIncl ? 0 : 1),
845 <                           endpoints[1] - (highIncl ? 0 : 1), true);
845 >                           endpoints[1] - (highIncl ? 0 : 1), true, bs);
846              else
847                  bashSubSet(sm.descendingSet(), endpoints[0] + (lowIncl ? 0 : 1),
848 <                           endpoints[1] - (highIncl ? 0 : 1), false);
848 >                           endpoints[1] - (highIncl ? 0 : 1), false, bs);
849          } else {
850              NavigableSet<Integer> sm = set.subSet(
851                  endpoints[1], highIncl, endpoints[0], lowIncl);
852              if (rnd.nextBoolean())
853                  bashSubSet(sm, endpoints[0] + (lowIncl ? 0 : 1),
854 <                           endpoints[1] - (highIncl ? 0 : 1), false);
854 >                           endpoints[1] - (highIncl ? 0 : 1), false, bs);
855              else
856                  bashSubSet(sm.descendingSet(), endpoints[0] + (lowIncl ? 0 : 1),
857 <                           endpoints[1] - (highIncl ? 0 : 1), true);
857 >                           endpoints[1] - (highIncl ? 0 : 1), true, bs);
858          }
859      }
860  
# Line 845 | Line 862 | public class ConcurrentSkipListSetTest e
862       * min and max are both inclusive.  If max < min, interval is empty.
863       */
864      void check(NavigableSet<Integer> set,
865 <                      final int min, final int max, final boolean ascending) {
866 <       class ReferenceSet {
865 >               final int min, final int max, final boolean ascending,
866 >               final BitSet bs) {
867 >        class ReferenceSet {
868              int lower(int element) {
869                  return ascending ?
870                      lowerAscending(element) : higherAscending(element);
# Line 916 | Line 934 | public class ConcurrentSkipListSetTest e
934              if (bsContainsI)
935                  size++;
936          }
937 <        assertEquals(set.size(), size);
937 >        assertEquals(size, set.size());
938  
939          // Test contents using contains elementSet iterator
940          int size2 = 0;
# Line 964 | Line 982 | public class ConcurrentSkipListSetTest e
982      }
983  
984      static boolean eq(Integer i, int j) {
985 <        return i == null ? j == -1 : i == j;
985 >        return (i == null) ? j == -1 : i == j;
986      }
987  
988   }

Diff Legend

Removed lines
+ Added lines
< Changed lines
> Changed lines