You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@commons.apache.org by gg...@apache.org on 2024/03/31 13:52:59 UTC
(commons-collections) 01/02: Reuse StringUtils.EMPTY in tests
This is an automated email from the ASF dual-hosted git repository.
ggregory pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/commons-collections.git
commit 52cdb17e69bc21a3ac7554a6c7eb5c7357064f7a
Author: Gary Gregory <ga...@gmail.com>
AuthorDate: Sun Mar 31 09:52:46 2024 -0400
Reuse StringUtils.EMPTY in tests
---
.../commons/collections4/IterableUtilsTest.java | 33 +++++++++++-----------
.../apache/commons/collections4/ListUtilsTest.java | 3 +-
.../apache/commons/collections4/MapUtilsTest.java | 5 ++--
.../apache/commons/collections4/MockTestCase.java | 3 +-
.../commons/collections4/TransformerUtilsTest.java | 3 +-
.../commons/collections4/bag/AbstractBagTest.java | 3 +-
.../collection/AbstractCollectionTest.java | 3 +-
.../collections4/keyvalue/MultiKeyTest.java | 3 +-
.../collections4/list/AbstractListTest.java | 3 +-
.../collections4/list/SetUniqueListTest.java | 3 +-
.../commons/collections4/map/AbstractMapTest.java | 5 ++--
.../collections4/map/CaseInsensitiveMapTest.java | 3 +-
.../collections4/map/ListOrderedMapTest.java | 9 +++---
.../multimap/AbstractMultiValuedMapTest.java | 11 ++++----
.../multiset/AbstractMultiSetTest.java | 3 +-
.../properties/EmptyPropertiesTest.java | 3 +-
.../sequence/SequencesComparatorTest.java | 5 ++--
.../splitmap/TransformedSplitMapTest.java | 3 +-
.../collections4/trie/PatriciaTrieTest.java | 7 +++--
19 files changed, 65 insertions(+), 46 deletions(-)
diff --git a/src/test/java/org/apache/commons/collections4/IterableUtilsTest.java b/src/test/java/org/apache/commons/collections4/IterableUtilsTest.java
index 746d93643..94ea67884 100644
--- a/src/test/java/org/apache/commons/collections4/IterableUtilsTest.java
+++ b/src/test/java/org/apache/commons/collections4/IterableUtilsTest.java
@@ -37,6 +37,7 @@ import java.util.List;
import java.util.Set;
import org.apache.commons.collections4.bag.HashBag;
+import org.apache.commons.lang3.StringUtils;
import org.junit.jupiter.api.BeforeEach;
import org.junit.jupiter.api.Test;
@@ -446,13 +447,13 @@ public class IterableUtilsTest {
result = IterableUtils.toString(new ArrayList<>(), input -> {
fail("not supposed to reach here");
- return "";
+ return StringUtils.EMPTY;
});
assertEquals("[]", result);
result = IterableUtils.toString(null, input -> {
fail("not supposed to reach here");
- return "";
+ return StringUtils.EMPTY;
});
assertEquals("[]", result);
}
@@ -462,13 +463,13 @@ public class IterableUtilsTest {
final Transformer<Integer, String> transformer = input -> Integer.toString(input * 2);
- String result = IterableUtils.toString(iterableA, transformer, "", "", "");
+ String result = IterableUtils.toString(iterableA, transformer, StringUtils.EMPTY, StringUtils.EMPTY, StringUtils.EMPTY);
assertEquals("2446668888", result);
- result = IterableUtils.toString(iterableA, transformer, ",", "", "");
+ result = IterableUtils.toString(iterableA, transformer, ",", StringUtils.EMPTY, StringUtils.EMPTY);
assertEquals("2,4,4,6,6,6,8,8,8,8", result);
- result = IterableUtils.toString(iterableA, transformer, "", "[", "]");
+ result = IterableUtils.toString(iterableA, transformer, StringUtils.EMPTY, "[", "]");
assertEquals("[2446668888]", result);
result = IterableUtils.toString(iterableA, transformer, ",", "[", "]");
@@ -483,38 +484,38 @@ public class IterableUtilsTest {
result = IterableUtils.toString(iterableA, transformer, ",,", "((", "))");
assertEquals("((2,,4,,4,,6,,6,,6,,8,,8,,8,,8))", result);
- result = IterableUtils.toString(new ArrayList<>(), transformer, "", "(", ")");
+ result = IterableUtils.toString(new ArrayList<>(), transformer, StringUtils.EMPTY, "(", ")");
assertEquals("()", result);
- result = IterableUtils.toString(new ArrayList<>(), transformer, "", "", "");
- assertEquals("", result);
+ result = IterableUtils.toString(new ArrayList<>(), transformer, StringUtils.EMPTY, StringUtils.EMPTY, StringUtils.EMPTY);
+ assertEquals(StringUtils.EMPTY, result);
}
@Test
public void testToStringWithNullArguments() {
final String result = IterableUtils.toString(null, input -> {
fail("not supposed to reach here");
- return "";
- }, "", "(", ")");
+ return StringUtils.EMPTY;
+ }, StringUtils.EMPTY, "(", ")");
assertEquals("()", result);
assertAll(
- () -> assertThrows(NullPointerException.class, () -> IterableUtils.toString(new ArrayList<>(), null, "", "(", ")"),
+ () -> assertThrows(NullPointerException.class, () -> IterableUtils.toString(new ArrayList<>(), null, StringUtils.EMPTY, "(", ")"),
"expecting NullPointerException"),
() -> assertThrows(NullPointerException.class, () ->
IterableUtils.toString(new ArrayList<>(), input -> {
fail("not supposed to reach here");
- return "";
+ return StringUtils.EMPTY;
}, null, "(", ")"),
"expecting NullPointerException"),
() -> assertThrows(NullPointerException.class, () -> IterableUtils.toString(new ArrayList<>(), input -> {
fail("not supposed to reach here");
- return "";
- }, "", null, ")"),
+ return StringUtils.EMPTY;
+ }, StringUtils.EMPTY, null, ")"),
"expecting NullPointerException"),
() -> assertThrows(NullPointerException.class, () -> IterableUtils.toString(new ArrayList<>(), input -> {
fail("not supposed to reach here");
- return "";
- }, "", "(", null),
+ return StringUtils.EMPTY;
+ }, StringUtils.EMPTY, "(", null),
"expecting NullPointerException")
);
}
diff --git a/src/test/java/org/apache/commons/collections4/ListUtilsTest.java b/src/test/java/org/apache/commons/collections4/ListUtilsTest.java
index c909937a0..44f8b1513 100644
--- a/src/test/java/org/apache/commons/collections4/ListUtilsTest.java
+++ b/src/test/java/org/apache/commons/collections4/ListUtilsTest.java
@@ -34,6 +34,7 @@ import java.util.List;
import org.apache.commons.collections4.functors.EqualPredicate;
import org.apache.commons.collections4.list.PredicatedList;
+import org.apache.commons.lang3.StringUtils;
import org.junit.jupiter.api.BeforeEach;
import org.junit.jupiter.api.Test;
@@ -294,7 +295,7 @@ public class ListUtilsTest {
"failed to check for null argument")
);
- String lcs = ListUtils.longestCommonSubsequence("", "");
+ String lcs = ListUtils.longestCommonSubsequence(StringUtils.EMPTY, StringUtils.EMPTY);
assertEquals(0, lcs.length());
final String banana = "BANANA";
diff --git a/src/test/java/org/apache/commons/collections4/MapUtilsTest.java b/src/test/java/org/apache/commons/collections4/MapUtilsTest.java
index 10e70d1c4..9b8ecb9e2 100644
--- a/src/test/java/org/apache/commons/collections4/MapUtilsTest.java
+++ b/src/test/java/org/apache/commons/collections4/MapUtilsTest.java
@@ -50,6 +50,7 @@ import org.apache.commons.collections4.map.HashedMap;
import org.apache.commons.collections4.map.LazyMap;
import org.apache.commons.collections4.map.MultiValueMap;
import org.apache.commons.collections4.map.PredicatedMap;
+import org.apache.commons.lang3.StringUtils;
import org.junit.jupiter.api.Test;
/**
@@ -607,7 +608,7 @@ public class MapUtilsTest {
if ("noKey".equals(key)) {
return "default";
}
- return "";
+ return StringUtils.EMPTY;
}));
assertEquals("default", MapUtils.getString(null, "noKey", "default"));
}
@@ -990,7 +991,7 @@ public class MapUtilsTest {
MapUtils.safeAddToMap(inMap, "key1", "value1");
MapUtils.safeAddToMap(inMap, "key2", null);
assertEquals("value1", inMap.get("key1"));
- assertEquals("", inMap.get("key2"));
+ assertEquals(StringUtils.EMPTY, inMap.get("key2"));
}
@Test
diff --git a/src/test/java/org/apache/commons/collections4/MockTestCase.java b/src/test/java/org/apache/commons/collections4/MockTestCase.java
index e7d2ec90c..98c4e8b8f 100644
--- a/src/test/java/org/apache/commons/collections4/MockTestCase.java
+++ b/src/test/java/org/apache/commons/collections4/MockTestCase.java
@@ -20,6 +20,7 @@ import java.util.ArrayList;
import java.util.List;
import java.util.ListIterator;
+import org.apache.commons.lang3.StringUtils;
import org.easymock.EasyMock;
import org.easymock.IExpectationSetters;
@@ -57,7 +58,7 @@ public abstract class MockTestCase {
try {
EasyMock.verify(i.next());
} catch (final AssertionError e) {
- throw new AssertionError(i.previousIndex() + 1 + ""
+ throw new AssertionError(i.previousIndex() + 1 + StringUtils.EMPTY
+ e.getMessage());
}
}
diff --git a/src/test/java/org/apache/commons/collections4/TransformerUtilsTest.java b/src/test/java/org/apache/commons/collections4/TransformerUtilsTest.java
index a2b1df177..42ff8ffdf 100644
--- a/src/test/java/org/apache/commons/collections4/TransformerUtilsTest.java
+++ b/src/test/java/org/apache/commons/collections4/TransformerUtilsTest.java
@@ -38,6 +38,7 @@ import org.apache.commons.collections4.functors.FalsePredicate;
import org.apache.commons.collections4.functors.NOPTransformer;
import org.apache.commons.collections4.functors.StringValueTransformer;
import org.apache.commons.collections4.functors.TruePredicate;
+import org.apache.commons.lang3.StringUtils;
import org.junit.jupiter.api.Test;
/**
@@ -166,7 +167,7 @@ public class TransformerUtilsTest {
assertThrows(FunctorException.class, () -> finalTrans.transform(String.class));
trans = TransformerUtils.instantiateTransformer();
- assertEquals("", trans.transform(String.class));
+ assertEquals(StringUtils.EMPTY, trans.transform(String.class));
trans = TransformerUtils.instantiateTransformer(new Class[] { Long.TYPE }, new Object[] {1000L});
assertEquals(new Date(1000L), trans.transform(Date.class));
diff --git a/src/test/java/org/apache/commons/collections4/bag/AbstractBagTest.java b/src/test/java/org/apache/commons/collections4/bag/AbstractBagTest.java
index e6e03aa09..939d7a43d 100644
--- a/src/test/java/org/apache/commons/collections4/bag/AbstractBagTest.java
+++ b/src/test/java/org/apache/commons/collections4/bag/AbstractBagTest.java
@@ -39,6 +39,7 @@ import org.apache.commons.collections4.BulkTest;
import org.apache.commons.collections4.collection.AbstractCollectionTest;
import org.apache.commons.collections4.set.AbstractSetTest;
import org.apache.commons.lang3.ArrayUtils;
+import org.apache.commons.lang3.StringUtils;
import org.junit.jupiter.api.Test;
/**
@@ -71,7 +72,7 @@ public abstract class AbstractBagTest<T> extends AbstractCollectionTest<T> {
public class TestBagUniqueSet extends AbstractSetTest<T> {
public TestBagUniqueSet() {
- super("");
+ super(StringUtils.EMPTY);
}
@Override
diff --git a/src/test/java/org/apache/commons/collections4/collection/AbstractCollectionTest.java b/src/test/java/org/apache/commons/collections4/collection/AbstractCollectionTest.java
index 8abbf7a4e..06f59830f 100644
--- a/src/test/java/org/apache/commons/collections4/collection/AbstractCollectionTest.java
+++ b/src/test/java/org/apache/commons/collections4/collection/AbstractCollectionTest.java
@@ -46,6 +46,7 @@ import java.util.function.Predicate;
import org.apache.commons.collections4.AbstractObjectTest;
import org.apache.commons.lang3.ArrayUtils;
+import org.apache.commons.lang3.StringUtils;
import org.junit.jupiter.api.Test;
/**
@@ -341,7 +342,7 @@ public abstract class AbstractCollectionTest<E> extends AbstractObjectTest {
@SuppressWarnings("unchecked")
public E[] getFullNonNullElements() {
return (E[]) new Object[] {
- "",
+ StringUtils.EMPTY,
"One",
Integer.valueOf(2),
"Three",
diff --git a/src/test/java/org/apache/commons/collections4/keyvalue/MultiKeyTest.java b/src/test/java/org/apache/commons/collections4/keyvalue/MultiKeyTest.java
index 47dd29534..2ebba4727 100644
--- a/src/test/java/org/apache/commons/collections4/keyvalue/MultiKeyTest.java
+++ b/src/test/java/org/apache/commons/collections4/keyvalue/MultiKeyTest.java
@@ -37,6 +37,7 @@ import java.util.Collection;
import java.util.HashMap;
import java.util.Map;
+import org.apache.commons.lang3.StringUtils;
import org.junit.jupiter.api.DynamicTest;
import org.junit.jupiter.api.Test;
import org.junit.jupiter.api.TestFactory;
@@ -170,7 +171,7 @@ public class MultiKeyTest {
assertEquals(mk1, mk1);
assertEquals(mk1, mk2);
assertNotEquals(mk1, mk3);
- assertNotEquals("", mk1);
+ assertNotEquals(StringUtils.EMPTY, mk1);
assertNotEquals(null, mk1);
}
diff --git a/src/test/java/org/apache/commons/collections4/list/AbstractListTest.java b/src/test/java/org/apache/commons/collections4/list/AbstractListTest.java
index 14e62e20e..d1750bed4 100644
--- a/src/test/java/org/apache/commons/collections4/list/AbstractListTest.java
+++ b/src/test/java/org/apache/commons/collections4/list/AbstractListTest.java
@@ -39,6 +39,7 @@ import java.util.NoSuchElementException;
import org.apache.commons.collections4.BulkTest;
import org.apache.commons.collections4.collection.AbstractCollectionTest;
import org.apache.commons.collections4.iterators.AbstractListIteratorTest;
+import org.apache.commons.lang3.StringUtils;
import org.junit.jupiter.api.Test;
/**
@@ -59,7 +60,7 @@ public abstract class AbstractListTest<E> extends AbstractCollectionTest<E> {
private final AbstractListTest<E> outer;
public BulkTestSubList(final AbstractListTest<E> outer) {
- super("");
+ super(StringUtils.EMPTY);
this.outer = outer;
}
diff --git a/src/test/java/org/apache/commons/collections4/list/SetUniqueListTest.java b/src/test/java/org/apache/commons/collections4/list/SetUniqueListTest.java
index b1ceed831..c13cba61b 100644
--- a/src/test/java/org/apache/commons/collections4/list/SetUniqueListTest.java
+++ b/src/test/java/org/apache/commons/collections4/list/SetUniqueListTest.java
@@ -33,6 +33,7 @@ import java.util.Set;
import java.util.TreeSet;
import org.apache.commons.collections4.set.UnmodifiableSet;
+import org.apache.commons.lang3.StringUtils;
import org.junit.jupiter.api.Test;
/**
@@ -67,7 +68,7 @@ public class SetUniqueListTest<E> extends AbstractListTest<E> {
public E[] getFullNonNullElements() {
// override to avoid duplicate "One"
return (E[]) new Object[] {
- "",
+ StringUtils.EMPTY,
"One",
Integer.valueOf(2),
"Three",
diff --git a/src/test/java/org/apache/commons/collections4/map/AbstractMapTest.java b/src/test/java/org/apache/commons/collections4/map/AbstractMapTest.java
index b1f095295..4ae9b1514 100644
--- a/src/test/java/org/apache/commons/collections4/map/AbstractMapTest.java
+++ b/src/test/java/org/apache/commons/collections4/map/AbstractMapTest.java
@@ -44,6 +44,7 @@ import org.apache.commons.collections4.CollectionUtils;
import org.apache.commons.collections4.collection.AbstractCollectionTest;
import org.apache.commons.collections4.keyvalue.DefaultMapEntry;
import org.apache.commons.collections4.set.AbstractSetTest;
+import org.apache.commons.lang3.StringUtils;
import org.junit.jupiter.api.AfterEach;
import org.junit.jupiter.api.Test;
@@ -330,7 +331,7 @@ public abstract class AbstractMapTest<K, V> extends AbstractObjectTest {
}
public class TestMapKeySet extends AbstractSetTest<K> {
public TestMapKeySet() {
- super("");
+ super(StringUtils.EMPTY);
}
@Override
@@ -410,7 +411,7 @@ public abstract class AbstractMapTest<K, V> extends AbstractObjectTest {
public class TestMapValues extends AbstractCollectionTest<V> {
public TestMapValues() {
- super("");
+ super(StringUtils.EMPTY);
}
@Override
diff --git a/src/test/java/org/apache/commons/collections4/map/CaseInsensitiveMapTest.java b/src/test/java/org/apache/commons/collections4/map/CaseInsensitiveMapTest.java
index d8324ecd0..71417bb87 100644
--- a/src/test/java/org/apache/commons/collections4/map/CaseInsensitiveMapTest.java
+++ b/src/test/java/org/apache/commons/collections4/map/CaseInsensitiveMapTest.java
@@ -25,6 +25,7 @@ import java.util.Locale;
import java.util.Map;
import java.util.Set;
+import org.apache.commons.lang3.StringUtils;
import org.junit.jupiter.api.Test;
/**
@@ -82,7 +83,7 @@ public class CaseInsensitiveMapTest<K, V> extends AbstractIterableMapTest<K, V>
public void testLocaleIndependence() {
final Locale orig = Locale.getDefault();
- final Locale[] locales = { Locale.ENGLISH, new Locale("tr", "", ""), Locale.getDefault() };
+ final Locale[] locales = { Locale.ENGLISH, new Locale("tr", StringUtils.EMPTY, StringUtils.EMPTY), Locale.getDefault() };
final String[][] data = {
{ "i", "I" },
diff --git a/src/test/java/org/apache/commons/collections4/map/ListOrderedMapTest.java b/src/test/java/org/apache/commons/collections4/map/ListOrderedMapTest.java
index b1a6504e0..353a3c48a 100644
--- a/src/test/java/org/apache/commons/collections4/map/ListOrderedMapTest.java
+++ b/src/test/java/org/apache/commons/collections4/map/ListOrderedMapTest.java
@@ -31,6 +31,7 @@ import java.util.TreeMap;
import org.apache.commons.collections4.BulkTest;
import org.apache.commons.collections4.MapIterator;
import org.apache.commons.collections4.list.AbstractListTest;
+import org.apache.commons.lang3.StringUtils;
import org.junit.jupiter.api.Test;
/**
@@ -482,19 +483,19 @@ public class ListOrderedMapTest<K, V> extends AbstractOrderedMapTest<K, V> {
resetEmpty();
ListOrderedMap<K, V> lom = getMap();
try {
- lom.setValue(0, (V) "");
+ lom.setValue(0, (V) StringUtils.EMPTY);
} catch (final IndexOutOfBoundsException ex) {}
try {
- lom.setValue(-1, (V) "");
+ lom.setValue(-1, (V) StringUtils.EMPTY);
} catch (final IndexOutOfBoundsException ex) {}
resetFull();
lom = getMap();
try {
- lom.setValue(-1, (V) "");
+ lom.setValue(-1, (V) StringUtils.EMPTY);
} catch (final IndexOutOfBoundsException ex) {}
try {
- lom.setValue(lom.size(), (V) "");
+ lom.setValue(lom.size(), (V) StringUtils.EMPTY);
} catch (final IndexOutOfBoundsException ex) {}
for (int i = 0; i < lom.size(); i++) {
diff --git a/src/test/java/org/apache/commons/collections4/multimap/AbstractMultiValuedMapTest.java b/src/test/java/org/apache/commons/collections4/multimap/AbstractMultiValuedMapTest.java
index 0908ad7f6..b9243162c 100644
--- a/src/test/java/org/apache/commons/collections4/multimap/AbstractMultiValuedMapTest.java
+++ b/src/test/java/org/apache/commons/collections4/multimap/AbstractMultiValuedMapTest.java
@@ -48,6 +48,7 @@ import org.apache.commons.collections4.collection.AbstractCollectionTest;
import org.apache.commons.collections4.map.AbstractMapTest;
import org.apache.commons.collections4.multiset.AbstractMultiSetTest;
import org.apache.commons.collections4.set.AbstractSetTest;
+import org.apache.commons.lang3.StringUtils;
import org.junit.jupiter.api.Disabled;
import org.junit.jupiter.api.Test;
@@ -63,7 +64,7 @@ public abstract class AbstractMultiValuedMapTest<K, V> extends AbstractObjectTes
public class TestMultiValuedMapAsMap extends AbstractMapTest<K, Collection<V>> {
public TestMultiValuedMapAsMap() {
- super("");
+ super(StringUtils.EMPTY);
}
@Override
@@ -177,7 +178,7 @@ public abstract class AbstractMultiValuedMapTest<K, V> extends AbstractObjectTes
public class TestMultiValuedMapEntries extends AbstractCollectionTest<Entry<K, V>> {
public TestMultiValuedMapEntries() {
- super("");
+ super(StringUtils.EMPTY);
}
@SuppressWarnings("unchecked")
@@ -253,7 +254,7 @@ public abstract class AbstractMultiValuedMapTest<K, V> extends AbstractObjectTes
public class TestMultiValuedMapKeys extends AbstractMultiSetTest<K> {
public TestMultiValuedMapKeys() {
- super("");
+ super(StringUtils.EMPTY);
}
@Override
@@ -313,7 +314,7 @@ public abstract class AbstractMultiValuedMapTest<K, V> extends AbstractObjectTes
public class TestMultiValuedMapKeySet extends AbstractSetTest<K> {
public TestMultiValuedMapKeySet() {
- super("");
+ super(StringUtils.EMPTY);
}
@SuppressWarnings("unchecked")
@@ -360,7 +361,7 @@ public abstract class AbstractMultiValuedMapTest<K, V> extends AbstractObjectTes
public class TestMultiValuedMapValues extends AbstractCollectionTest<V> {
public TestMultiValuedMapValues() {
- super("");
+ super(StringUtils.EMPTY);
}
@Override
diff --git a/src/test/java/org/apache/commons/collections4/multiset/AbstractMultiSetTest.java b/src/test/java/org/apache/commons/collections4/multiset/AbstractMultiSetTest.java
index 66e9f8333..54010d11a 100644
--- a/src/test/java/org/apache/commons/collections4/multiset/AbstractMultiSetTest.java
+++ b/src/test/java/org/apache/commons/collections4/multiset/AbstractMultiSetTest.java
@@ -38,6 +38,7 @@ import org.apache.commons.collections4.MultiSet;
import org.apache.commons.collections4.collection.AbstractCollectionTest;
import org.apache.commons.collections4.set.AbstractSetTest;
import org.apache.commons.lang3.ArrayUtils;
+import org.apache.commons.lang3.StringUtils;
import org.junit.jupiter.api.Test;
/**
@@ -64,7 +65,7 @@ public abstract class AbstractMultiSetTest<T> extends AbstractCollectionTest<T>
public class TestMultiSetUniqueSet extends AbstractSetTest<T> {
public TestMultiSetUniqueSet() {
- super("");
+ super(StringUtils.EMPTY);
}
@Override
diff --git a/src/test/java/org/apache/commons/collections4/properties/EmptyPropertiesTest.java b/src/test/java/org/apache/commons/collections4/properties/EmptyPropertiesTest.java
index d5c509a87..9b56d2b49 100644
--- a/src/test/java/org/apache/commons/collections4/properties/EmptyPropertiesTest.java
+++ b/src/test/java/org/apache/commons/collections4/properties/EmptyPropertiesTest.java
@@ -38,6 +38,7 @@ import java.util.Properties;
import org.apache.commons.io.input.NullReader;
import org.apache.commons.lang3.ArrayUtils;
+import org.apache.commons.lang3.StringUtils;
import org.junit.jupiter.api.Test;
public class EmptyPropertiesTest {
@@ -62,7 +63,7 @@ public class EmptyPropertiesTest {
private String removeLine2(final String x) {
final String[] s = x.split("\\R", 2);
- return s[0] + System.lineSeparator() + (s.length > 2 ? s[2] : "");
+ return s[0] + System.lineSeparator() + (s.length > 2 ? s[2] : StringUtils.EMPTY);
}
@Test
diff --git a/src/test/java/org/apache/commons/collections4/sequence/SequencesComparatorTest.java b/src/test/java/org/apache/commons/collections4/sequence/SequencesComparatorTest.java
index cec0f8ad9..2a599109c 100644
--- a/src/test/java/org/apache/commons/collections4/sequence/SequencesComparatorTest.java
+++ b/src/test/java/org/apache/commons/collections4/sequence/SequencesComparatorTest.java
@@ -24,6 +24,7 @@ import java.util.Arrays;
import java.util.List;
import java.util.Random;
+import org.apache.commons.lang3.StringUtils;
import org.junit.jupiter.api.AfterEach;
import org.junit.jupiter.api.BeforeEach;
import org.junit.jupiter.api.Test;
@@ -83,7 +84,7 @@ public class SequencesComparatorTest {
before = Arrays.asList(
"bottle",
"nematode knowledge",
- "",
+ StringUtils.EMPTY,
"aa",
"prefixed string",
"ABCABBA",
@@ -94,7 +95,7 @@ public class SequencesComparatorTest {
after = Arrays.asList(
"noodle",
"empty bottle",
- "",
+ StringUtils.EMPTY,
"C",
"prefix",
"CBABAC",
diff --git a/src/test/java/org/apache/commons/collections4/splitmap/TransformedSplitMapTest.java b/src/test/java/org/apache/commons/collections4/splitmap/TransformedSplitMapTest.java
index ea05e8eee..bb7b185e9 100644
--- a/src/test/java/org/apache/commons/collections4/splitmap/TransformedSplitMapTest.java
+++ b/src/test/java/org/apache/commons/collections4/splitmap/TransformedSplitMapTest.java
@@ -31,6 +31,7 @@ import org.apache.commons.collections4.BulkTest;
import org.apache.commons.collections4.MapIterator;
import org.apache.commons.collections4.Transformer;
import org.apache.commons.collections4.functors.NOPTransformer;
+import org.apache.commons.lang3.StringUtils;
import org.junit.jupiter.api.Test;
/**
@@ -110,7 +111,7 @@ public class TransformedSplitMapTest extends BulkTest {
new HashMap<>(), intToString, objectToClass);
final Integer[] k = { 0, 1, 2, 3, 4, 5, 6 };
- final Object[] v = { "", new Object(), new HashMap<>(), 0, BigInteger.TEN, null,
+ final Object[] v = { StringUtils.EMPTY, new Object(), new HashMap<>(), 0, BigInteger.TEN, null,
new Object[0] };
assertEquals(0, map.size());
diff --git a/src/test/java/org/apache/commons/collections4/trie/PatriciaTrieTest.java b/src/test/java/org/apache/commons/collections4/trie/PatriciaTrieTest.java
index bef6b2e76..d74592de3 100755
--- a/src/test/java/org/apache/commons/collections4/trie/PatriciaTrieTest.java
+++ b/src/test/java/org/apache/commons/collections4/trie/PatriciaTrieTest.java
@@ -36,6 +36,7 @@ import java.util.SortedMap;
import org.apache.commons.collections4.Trie;
import org.apache.commons.collections4.map.AbstractSortedMapTest;
+import org.apache.commons.lang3.StringUtils;
import org.junit.jupiter.api.Test;
/**
@@ -67,7 +68,7 @@ public class PatriciaTrieTest<V> extends AbstractSortedMapTest<String, V> {
final PatriciaTrie<String> trie = new PatriciaTrie<>();
final String[] keys = {
- "",
+ StringUtils.EMPTY,
"Albert", "Xavier", "XyZ", "Anna", "Alien", "Alberto",
"Alberts", "Allie", "Alliese", "Alabama", "Banane",
"Blabla", "Amber", "Ammun", "Akka", "Akko", "Albertoo",
@@ -272,7 +273,7 @@ public class PatriciaTrieTest<V> extends AbstractSortedMapTest<String, V> {
iterator = map.values().iterator();
assertFalse(iterator.hasNext());
- map = trie.prefixMap("");
+ map = trie.prefixMap(StringUtils.EMPTY);
assertSame(trie, map); // stricter than necessary, but a good check
map = trie.prefixMap("\0");
@@ -423,7 +424,7 @@ public class PatriciaTrieTest<V> extends AbstractSortedMapTest<String, V> {
final char char_b = 'b'; // 1100010
final PatriciaTrie<String> trie = new PatriciaTrie<>();
- final String prefixString = "" + char_b;
+ final String prefixString = StringUtils.EMPTY + char_b;
final String longerString = prefixString + u8000;
assertEquals(1, prefixString.length());