--- old/test/jdk/java/util/RandomAccess/Basic.java 2019-09-24 18:03:09.000000000 +0100 +++ new/test/jdk/java/util/RandomAccess/Basic.java 2019-09-24 18:03:09.000000000 +0100 @@ -1,5 +1,5 @@ /* - * Copyright (c) 2000, 2018, Oracle and/or its affiliates. All rights reserved. + * Copyright (c) 2000, 2019, Oracle and/or its affiliates. All rights reserved. * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER. * * This code is free software; you can redistribute it and/or modify it @@ -21,12 +21,18 @@ * questions. */ -/* +/** * @test * @bug 4327164 * @summary Basic test for new RandomAccess interface + * @run testng Basic */ +import org.testng.annotations.DataProvider; +import org.testng.annotations.Test; + +import static org.testng.Assert.assertEquals; + import java.util.ArrayList; import java.util.Arrays; import java.util.Collections; @@ -34,98 +40,108 @@ import java.util.List; import java.util.Random; import java.util.RandomAccess; +import java.util.Stack; import java.util.Vector; +import java.util.concurrent.CopyOnWriteArrayList; public class Basic { - public static void main(String[] args) throws Exception { - List a0 = Arrays.asList(new String[] { "a", "b", "c" }); - List a[] = { a0, new ArrayList(a0), new LinkedList(a0), - new Vector(a0) }; - - if (!(a[0] instanceof RandomAccess)) - throw new Exception("Arrays.asList doesn't implement RandomAccess"); - if (!(a[1] instanceof RandomAccess)) - throw new Exception("ArrayList doesn't implement RandomAccess"); - if (a[2] instanceof RandomAccess) - throw new Exception("LinkedList implements RandomAccess"); - if (!(a[3] instanceof RandomAccess)) - throw new Exception("Vector doesn't implement RandomAccess"); - - for (int i = 0; i < a.length; i++) { - List t = a[i]; - List ut = Collections.unmodifiableList(t); - List st = Collections.synchronizedList(t); - - boolean random = t instanceof RandomAccess; - if ((ut instanceof RandomAccess) != random) - throw new Exception( - "Unmodifiable fails to preserve RandomAccess: " + i); - if ((st instanceof RandomAccess) != random) - throw new Exception( - "Synchronized fails to preserve RandomAccess: " + i); - - while (t.size() > 0) { - t = t.subList(0, t.size() - 1); - if ((t instanceof RandomAccess) != random) - throw new Exception( - "SubList fails to preserve RandomAccess: " + i - + ", " + t.size()); - - ut = ut.subList(0, ut.size() - 1); - if ((ut instanceof RandomAccess) != random) - throw new Exception( - "SubList(unmodifiable) fails to preserve RandomAccess: " - + i + ", " + ut.size()); - - st = st.subList(0, st.size() - 1); - if ((st instanceof RandomAccess) != random) - throw new Exception( - "SubList(synchronized) fails to preserve RandomAccess: " - + i + ", " + st.size()); - } + + @DataProvider(name = "testLists") + public Object[][] testData() { + var list = Arrays.asList(new String[]{"a", "b", "c"}); + var stack = new Stack(); + stack.push("a"); + stack.push("b"); + stack.push("c"); + return new Object[][]{ + {list, true, "Arrays.asList"}, + {stack, true, "Stack"}, + {new ArrayList<>(list), true, "ArrayList"}, + {new LinkedList<>(list), false, "LinkedList"}, + {new Vector<>(list), true, "Vector"}, + {new CopyOnWriteArrayList<>(list), true, "CopyOnWriteArrayList"} + }; + } + + @Test(dataProvider = "testLists") + public void testRandomAccess(List list, boolean expectedRA, String failMsg) { + + var actualRA = list instanceof RandomAccess; + assertEquals(actualRA, expectedRA, failMsg); + + List unmodList = Collections.unmodifiableList(list); + List syncList = Collections.synchronizedList(list); + assertEquals((unmodList instanceof RandomAccess), actualRA, + "Unmodifiable fails to preserve RandomAccess"); + assertEquals((syncList instanceof RandomAccess), actualRA, + "Synchronized fails to preserve RandomAccess"); + + while (list.size() > 0) { + list = list.subList(0, list.size() - 1); + assertEquals((list instanceof RandomAccess), actualRA, + "SubList fails to preserve RandomAccess: " + list.size()); + + unmodList = unmodList.subList(0, unmodList.size() - 1); + assertEquals((unmodList instanceof RandomAccess), actualRA, + "SubList(unmodifiable) fails to preserve RandomAccess: " + + unmodList.size()); + + syncList = syncList.subList(0, syncList.size() - 1); + assertEquals((syncList instanceof RandomAccess), actualRA, + "SubList(synchronized) fails to preserve RandomAccess: " + + syncList.size()); } + } + + @Test + public void testListCopy() { + ArrayList testCollection = new ArrayList<>(Collections.nCopies(100, "test")); + ArrayList arrayListStr = new ArrayList<>(Collections.nCopies(100, "")); + List linkedListStr = new LinkedList<>(arrayListStr); + // Test that copy works on random & sequential access + Collections.copy(arrayListStr, testCollection); + Collections.copy(linkedListStr, testCollection); + assertEquals(arrayListStr, testCollection, "ArrayList copy failed"); + assertEquals(linkedListStr, testCollection, "LinkedList copy failed"); + } + + @Test + public void testListFill() { + ArrayList testCollection = new ArrayList<>(Collections.nCopies(100, "test")); + ArrayList arrayListStr = new ArrayList<>(Collections.nCopies(100, "")); + List linkedListStr = new LinkedList<>(arrayListStr); + // Test that fill works on random & sequential access + Collections.fill(arrayListStr, "test"); + Collections.fill(linkedListStr, "test"); + assertEquals(arrayListStr, testCollection, "ArrayList fill failed"); + assertEquals(linkedListStr, testCollection, "LinkedList fill failed"); + } + @Test + public void testListShuffle() { // Test that shuffle works the same on random and sequential access - List al = new ArrayList(); + ArrayList al = new ArrayList<>(); for (int j = 0; j < 100; j++) - al.add(Integer.valueOf(2 * j)); - List ll = new LinkedList(al); - Random r1 = new Random(666), r2 = new Random(666); + al.add(Integer.valueOf(2 * j)); + List ll = new LinkedList<>(al); for (int i = 0; i < 100; i++) { - Collections.shuffle(al, r1); - Collections.shuffle(ll, r2); - if (!al.equals(ll)) - throw new Exception("Shuffle failed: " + i); + Collections.shuffle(al, new Random(666)); + Collections.shuffle(ll, new Random(666)); + assertEquals(al, ll, "Shuffle failed: " + i); } + } - // Test that fill works on random & sequential access - List gumbyParade = Collections.nCopies(100, "gumby"); - Collections.fill(al, "gumby"); - if (!al.equals(gumbyParade)) - throw new Exception("ArrayList fill failed"); - Collections.fill(ll, "gumby"); - if (!ll.equals(gumbyParade)) - throw new Exception("LinkedList fill failed"); - - // Test that copy works on random & sequential access - List pokeyParade = Collections.nCopies(100, "pokey"); - Collections.copy(al, pokeyParade); - if (!al.equals(pokeyParade)) - throw new Exception("ArrayList copy failed"); - Collections.copy(ll, pokeyParade); - if (!ll.equals(pokeyParade)) - throw new Exception("LinkedList copy failed"); - + @Test + public void testListBinarySearch() { // Test that binarySearch works the same on random & sequential access - al = new ArrayList(); + ArrayList al = new ArrayList<>(); for (int i = 0; i < 10000; i++) al.add(Integer.valueOf(2 * i)); - ll = new LinkedList(al); + List ll = new LinkedList<>(al); for (int i = 0; i < 500; i++) { - Integer key = Integer.valueOf(r1.nextInt(20000)); - if (Collections.binarySearch(al, key) != Collections - .binarySearch(ll, key)) - throw new Exception("Binary search failed: " + i); + Integer key = Integer.valueOf(new Random(666).nextInt(20000)); + assertEquals(Collections.binarySearch(al, key), Collections + .binarySearch(ll, key), "Binary search failed: " + i); } } }