test/java/util/Collection/ListDefaults.java updates

Akhil Arora akhil.arora at oracle.com
Mon May 13 16:58:28 UTC 2013


Looks good to me. I have had to add similar print statements when 
debugging, but I deleted them before committing, to keep the default 
test output clean.

Minor - it would be good to have a little more consistency... some tests 
print the class name always, some only on failure. Also would be good to 
have similar changes in the CollectionDefaults and IteratorDefaults tests.

On 05/13/2013 06:25 AM, Chris Hegarty wrote:
> Akil, Mike,
>
> I've recently been diagnosing failures with ListDefaults.java, when
> working on a separate issue. I found it difficult to determine which
> collection type was having problems, in some failure cases.
>
> The diffs below are what I had to implement in my local repo to help
> identify the cause of several failures. It is probably not complete in
> terms of updating all the possible error conditions, but what I have
> sitting in my local repo, and possibly useful.
>
> If you agree with the changes, I can file a bug and push them.
>
> diff --git a/test/java/util/Collection/ListDefaults.java
> b/test/java/util/Collection/ListDefaults.java
> old mode 100644
> new mode 100755
> --- a/test/java/util/Collection/ListDefaults.java
> +++ b/test/java/util/Collection/ListDefaults.java
> @@ -117,17 +117,18 @@ public class ListDefaults {
>
>       @Test(dataProvider = "listProvider")
>       public void testProvidedWithNull(final List<Integer> list) throws
> Exception {
> +        final String cn = list.getClass().getName();
>           try {
>               list.forEach(null);
> -            fail("expected NPE not thrown");
> +            fail("expected NPE not thrown; " + cn + ".forEach()");
>           } catch (NullPointerException npe) {}
>           try {
>               list.replaceAll(null);
> -            fail("expected NPE not thrown");
> +            fail("expected NPE not thrown; " + cn + ".replaceAll()");
>           } catch (NullPointerException npe) {}
>           try {
>               list.removeIf(null);
> -            fail("expected NPE not thrown");
> +            fail("expected NPE not thrown; " + cn + ".removeIf()");
>           } catch (NullPointerException npe) {}
>       }
>
> @@ -183,8 +184,9 @@ public class ListDefaults {
>               final List<Integer> list = ((List<Integer>) test.collection);
>
>               try {
> +                final String cn = list.getClass().getName();
>                   list.removeIf(null);
> -                fail("expected NPE not thrown");
> +                fail("expected NPE not thrown;" + cn + ".removeIf()");
>               } catch (NullPointerException npe) {}
>               CollectionAsserts.assertContents(list, original);
>
> @@ -215,10 +217,12 @@ public class ListDefaults {
>               final List<Integer> list = ((List<Integer>) test.collection);
>               final List<Integer> listCopy = new ArrayList<>(list);
>               if (original.size() > SUBLIST_SIZE) {
> +                System.out.println("Testcase.name: " + test.name + ",
> classname:" + test.className);
>                   final List<Integer> subList =
> list.subList(SUBLIST_FROM, SUBLIST_TO);
>                   final List<Integer> subListCopy = new
> ArrayList<>(subList);
>                   listCopy.removeAll(subList);
>                   subList.removeIf(pOdd);
> +                System.out.println("Testcase.name: " + test.name + ",
> classname:" + test.className + " COMPLETE");
>                   for (int i : subList) {
>                       assertTrue((i % 2) == 0);
>                   }
> @@ -274,8 +278,9 @@ public class ListDefaults {
>               final List<Integer> list = ((List<Integer>) test.collection);
>
>               try {
> +                final String cn = list.getClass().getName();
>                   list.replaceAll(null);
> -                fail("expected NPE not thrown");
> +                fail("expected NPE not thrown;" + cn + ".replaceAll()");
>               } catch (NullPointerException npe) {}
>               CollectionAsserts.assertContents(list, original);
>
> -Chris.




More information about the core-libs-dev mailing list