[ https://issues.apache.org/jira/browse/COLLECTIONS-738?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Bruno P. Kinoshita updated COLLECTIONS-738: ------------------------------------------- Assignee: Bruno P. Kinoshita > The last assertNull in IterableUtilsTest.find is redundant > ---------------------------------------------------------- > > Key: COLLECTIONS-738 > URL: https://issues.apache.org/jira/browse/COLLECTIONS-738 > Project: Commons Collections > Issue Type: Test > Components: Iterator > Reporter: Pengyu Nie > Assignee: Bruno P. Kinoshita > Priority: Major > Time Spent: 10m > Remaining Estimate: 0h > > The last assertNull in IterableUtilsTest.find (line 277, as shown > below), is redundant and confusing; it will not be invoked because > IterableUtils.find(*, null) should not return any value but just throw > NullPointerException. I will make a PR to clarify this. > {code:java} > try { > assertNull(IterableUtils.find(iterableA, null)); > fail("expecting NullPointerException"); > } catch (final NullPointerException npe) { > // expected > } > {code} -- This message was sent by Atlassian Jira (v8.3.4#803005)