garydgregory commented on a change in pull request #492:
style(org.apache.commons.lang3): rename method from removeAllOccurenc…
URL: https://github.com/apache/commons-lang/pull/492#discussion_r380220692
##########
File path: src/test/java/org/apache/commons/lang3/ArrayUtilsRemoveTest.java
##########
@@ -403,189 +403,189 @@ public void testRemoveElementShortArray() {
@Test
public void testRemoveAllBooleanOccurences() {
boolean[] a = null;
- assertNull(ArrayUtils.removeAllOccurences(a, true));
+ assertNull(ArrayUtils.removeAllOccurrences(a, true));
Review comment:
If you create a new API (see my other comment) with the corrected name,
either test both the new and old APIs next to each other, or, clone the test
methods and also correct their names.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
With regards,
Apache Git Services
