diff options
Diffstat (limited to 'src')
49 files changed, 1072 insertions, 1395 deletions
diff --git a/src/main/java/org/apache/commons/lang3/Streams.java b/src/main/java/org/apache/commons/lang3/Streams.java index 84ffef643..7e92c97ef 100644 --- a/src/main/java/org/apache/commons/lang3/Streams.java +++ b/src/main/java/org/apache/commons/lang3/Streams.java @@ -332,7 +332,7 @@ public class Streams { * element to determine if it should be included. * @return the new stream */ - public FailableStream<O> filter(final FailablePredicate<O, ?> predicate){ + public FailableStream<O> filter(final FailablePredicate<O, ?> predicate) { assertNotTerminated(); stream = stream.filter(Functions.asPredicate(predicate)); return this; diff --git a/src/main/java/org/apache/commons/lang3/StringUtils.java b/src/main/java/org/apache/commons/lang3/StringUtils.java index 2937f7733..16dd46e96 100644 --- a/src/main/java/org/apache/commons/lang3/StringUtils.java +++ b/src/main/java/org/apache/commons/lang3/StringUtils.java @@ -361,7 +361,7 @@ public class StringUtils { if (strLen - offset < maxWidth - abbrevMarkerLength) { offset = strLen - (maxWidth - abbrevMarkerLength); } - if (offset <= abbrevMarkerLength+1) { + if (offset <= abbrevMarkerLength + 1) { return str.substring(0, maxWidth - abbrevMarkerLength) + abbrevMarker; } if (maxWidth < minAbbrevWidthOffset) { @@ -403,17 +403,13 @@ public class StringUtils { * @since 2.5 */ public static String abbreviateMiddle(final String str, final String middle, final int length) { - if (isAnyEmpty(str, middle) || length >= str.length() || length < middle.length()+2) { + if (isAnyEmpty(str, middle) || length >= str.length() || length < middle.length() + 2) { return str; } - - final int targetSting = length-middle.length(); - final int startOffset = targetSting/2+targetSting%2; - final int endOffset = str.length()-targetSting/2; - - return str.substring(0, startOffset) + - middle + - str.substring(endOffset); + final int targetSting = length - middle.length(); + final int startOffset = targetSting / 2 + targetSting % 2; + final int endOffset = str.length() - targetSting / 2; + return str.substring(0, startOffset) + middle + str.substring(endOffset); } /** @@ -9393,7 +9389,7 @@ public class StringUtils { * the string to be wrapped, may be {@code null} * @param wrapWith * the char that will wrap {@code str} - * @return the wrapped string, or {@code null} if {@code str==null} + * @return the wrapped string, or {@code null} if {@code str == null} * @since 3.4 */ public static String wrap(final String str, final char wrapWith) { @@ -9463,7 +9459,7 @@ public class StringUtils { * the string to be wrapped, may be {@code null} * @param wrapWith * the char that will wrap {@code str} - * @return the wrapped string, or {@code null} if {@code str==null} + * @return the wrapped string, or {@code null} if {@code str == null} * @since 3.5 */ public static String wrapIfMissing(final String str, final char wrapWith) { @@ -9513,7 +9509,7 @@ public class StringUtils { * the string to be wrapped, may be {@code null} * @param wrapWith * the string that will wrap {@code str} - * @return the wrapped string, or {@code null} if {@code str==null} + * @return the wrapped string, or {@code null} if {@code str == null} * @since 3.5 */ public static String wrapIfMissing(final String str, final String wrapWith) { diff --git a/src/main/java/org/apache/commons/lang3/builder/MultilineRecursiveToStringStyle.java b/src/main/java/org/apache/commons/lang3/builder/MultilineRecursiveToStringStyle.java index 3dacaea5d..861a55fe2 100644 --- a/src/main/java/org/apache/commons/lang3/builder/MultilineRecursiveToStringStyle.java +++ b/src/main/java/org/apache/commons/lang3/builder/MultilineRecursiveToStringStyle.java @@ -51,7 +51,7 @@ import org.apache.commons.lang3.ClassUtils; * <code>Person@7f54[ <br> * name=Stephen, <br> * age=29, <br> - * smoker=false, <br> + * smokealse, <br> * job=Job@43cd2[ <br> * title=Manager <br> * ] <br> diff --git a/src/main/java/org/apache/commons/lang3/compare/ComparableUtils.java b/src/main/java/org/apache/commons/lang3/compare/ComparableUtils.java index a8b1eb13a..5f6cbc13e 100644 --- a/src/main/java/org/apache/commons/lang3/compare/ComparableUtils.java +++ b/src/main/java/org/apache/commons/lang3/compare/ComparableUtils.java @@ -240,5 +240,7 @@ public class ComparableUtils { return ObjectUtils.compare(comparable1, comparable2, true) < 0 ? comparable1 : comparable2; } - private ComparableUtils() {} + private ComparableUtils() { + // empty + } } diff --git a/src/main/java/org/apache/commons/lang3/function/BooleanConsumer.java b/src/main/java/org/apache/commons/lang3/function/BooleanConsumer.java index eaed72d4f..c751f4ff1 100644 --- a/src/main/java/org/apache/commons/lang3/function/BooleanConsumer.java +++ b/src/main/java/org/apache/commons/lang3/function/BooleanConsumer.java @@ -30,7 +30,7 @@ import java.util.function.IntConsumer; public interface BooleanConsumer { /** NOP singleton */ - BooleanConsumer NOP = t -> {/* NOP */}; + BooleanConsumer NOP = t -> { /* NOP */ }; /** * Returns The NOP singleton. diff --git a/src/main/java/org/apache/commons/lang3/function/FailableBiConsumer.java b/src/main/java/org/apache/commons/lang3/function/FailableBiConsumer.java index e0c045d01..9f11bf2ba 100644 --- a/src/main/java/org/apache/commons/lang3/function/FailableBiConsumer.java +++ b/src/main/java/org/apache/commons/lang3/function/FailableBiConsumer.java @@ -33,7 +33,7 @@ public interface FailableBiConsumer<T, U, E extends Throwable> { /** NOP singleton */ @SuppressWarnings("rawtypes") - FailableBiConsumer NOP = (t, u) -> {/* NOP */}; + FailableBiConsumer NOP = (t, u) -> { /* NOP */ }; /** * Returns The NOP singleton. diff --git a/src/main/java/org/apache/commons/lang3/function/FailableDoubleConsumer.java b/src/main/java/org/apache/commons/lang3/function/FailableDoubleConsumer.java index 8e6814ef5..991790a53 100644 --- a/src/main/java/org/apache/commons/lang3/function/FailableDoubleConsumer.java +++ b/src/main/java/org/apache/commons/lang3/function/FailableDoubleConsumer.java @@ -31,7 +31,7 @@ public interface FailableDoubleConsumer<E extends Throwable> { /** NOP singleton */ @SuppressWarnings("rawtypes") - FailableDoubleConsumer NOP = t -> {/* NOP */}; + FailableDoubleConsumer NOP = t -> { /* NOP */ }; /** * Returns The NOP singleton. diff --git a/src/main/java/org/apache/commons/lang3/function/FailableIntConsumer.java b/src/main/java/org/apache/commons/lang3/function/FailableIntConsumer.java index c35e1afdd..e27cca351 100644 --- a/src/main/java/org/apache/commons/lang3/function/FailableIntConsumer.java +++ b/src/main/java/org/apache/commons/lang3/function/FailableIntConsumer.java @@ -31,7 +31,7 @@ public interface FailableIntConsumer<E extends Throwable> { /** NOP singleton */ @SuppressWarnings("rawtypes") - FailableIntConsumer NOP = t -> {/* NOP */}; + FailableIntConsumer NOP = t -> { /* NOP */ }; /** * Returns The NOP singleton. diff --git a/src/main/java/org/apache/commons/lang3/function/FailableLongConsumer.java b/src/main/java/org/apache/commons/lang3/function/FailableLongConsumer.java index bb23a6f86..59a41c914 100644 --- a/src/main/java/org/apache/commons/lang3/function/FailableLongConsumer.java +++ b/src/main/java/org/apache/commons/lang3/function/FailableLongConsumer.java @@ -31,7 +31,7 @@ public interface FailableLongConsumer<E extends Throwable> { /** NOP singleton */ @SuppressWarnings("rawtypes") - FailableLongConsumer NOP = t -> {/* NOP */}; + FailableLongConsumer NOP = t -> { /* NOP */ }; /** * Returns The NOP singleton. diff --git a/src/main/java/org/apache/commons/lang3/function/FailableObjDoubleConsumer.java b/src/main/java/org/apache/commons/lang3/function/FailableObjDoubleConsumer.java index b42e3ebc7..d0b3cd2d9 100644 --- a/src/main/java/org/apache/commons/lang3/function/FailableObjDoubleConsumer.java +++ b/src/main/java/org/apache/commons/lang3/function/FailableObjDoubleConsumer.java @@ -31,7 +31,7 @@ public interface FailableObjDoubleConsumer<T, E extends Throwable> { /** NOP singleton */ @SuppressWarnings("rawtypes") - FailableObjDoubleConsumer NOP = (t, u) -> {/* NOP */}; + FailableObjDoubleConsumer NOP = (t, u) -> { /* NOP */ }; /** * Returns The NOP singleton. diff --git a/src/main/java/org/apache/commons/lang3/function/FailableObjIntConsumer.java b/src/main/java/org/apache/commons/lang3/function/FailableObjIntConsumer.java index 427bbfe40..d4b3543f4 100644 --- a/src/main/java/org/apache/commons/lang3/function/FailableObjIntConsumer.java +++ b/src/main/java/org/apache/commons/lang3/function/FailableObjIntConsumer.java @@ -31,7 +31,7 @@ public interface FailableObjIntConsumer<T, E extends Throwable> { /** NOP singleton */ @SuppressWarnings("rawtypes") - FailableObjIntConsumer NOP = (t, u) -> {/* NOP */}; + FailableObjIntConsumer NOP = (t, u) -> { /* NOP */ }; /** * Returns The NOP singleton. diff --git a/src/main/java/org/apache/commons/lang3/function/FailableObjLongConsumer.java b/src/main/java/org/apache/commons/lang3/function/FailableObjLongConsumer.java index c6a6efd17..483d7e53f 100644 --- a/src/main/java/org/apache/commons/lang3/function/FailableObjLongConsumer.java +++ b/src/main/java/org/apache/commons/lang3/function/FailableObjLongConsumer.java @@ -31,7 +31,7 @@ public interface FailableObjLongConsumer<T, E extends Throwable> { /** NOP singleton */ @SuppressWarnings("rawtypes") - FailableObjLongConsumer NOP = (t, u) -> {/* NOP */}; + FailableObjLongConsumer NOP = (t, u) -> { /* NOP */ }; /** * Returns The NOP singleton. diff --git a/src/main/java/org/apache/commons/lang3/math/Fraction.java b/src/main/java/org/apache/commons/lang3/math/Fraction.java index 612c1b481..c1e8a8ee7 100644 --- a/src/main/java/org/apache/commons/lang3/math/Fraction.java +++ b/src/main/java/org/apache/commons/lang3/math/Fraction.java @@ -731,10 +731,10 @@ public final class Fraction extends Number implements Comparable<Fraction> { if (numerator == 0) { throw new ArithmeticException("Unable to invert zero."); } - if (numerator==Integer.MIN_VALUE) { + if (numerator == Integer.MIN_VALUE) { throw new ArithmeticException("overflow: can't negate numerator"); } - if (numerator<0) { + if (numerator < 0) { return new Fraction(-denominator, -numerator); } return new Fraction(denominator, numerator); @@ -770,8 +770,7 @@ public final class Fraction extends Number implements Comparable<Fraction> { // make sure we don't overflow unless the result *must* overflow. final int d1 = greatestCommonDivisor(numerator, fraction.denominator); final int d2 = greatestCommonDivisor(fraction.numerator, denominator); - return getReducedFraction(mulAndCheck(numerator / d1, fraction.numerator / d2), - mulPosAndCheck(denominator / d2, fraction.denominator / d1)); + return getReducedFraction(mulAndCheck(numerator / d1, fraction.numerator / d2), mulPosAndCheck(denominator / d2, fraction.denominator / d1)); } /** @@ -783,7 +782,7 @@ public final class Fraction extends Number implements Comparable<Fraction> { */ public Fraction negate() { // the positive range is one smaller than the negative range of an int. - if (numerator==Integer.MIN_VALUE) { + if (numerator == Integer.MIN_VALUE) { throw new ArithmeticException("overflow: too large to negate"); } return new Fraction(-numerator, denominator); @@ -875,7 +874,7 @@ public final class Fraction extends Number implements Comparable<Fraction> { } else if ((numerator > 0 ? -numerator : numerator) < -denominator) { // note that we do the magnitude comparison test above with // NEGATIVE (not positive) numbers, since negative numbers - // have a larger range. otherwise numerator==Integer.MIN_VALUE + // have a larger range. otherwise numerator == Integer.MIN_VALUE // is handled incorrectly. final int properNumerator = getProperNumerator(); if (properNumerator == 0) { diff --git a/src/main/java/org/apache/commons/lang3/mutable/MutableDouble.java b/src/main/java/org/apache/commons/lang3/mutable/MutableDouble.java index d4e352250..ab995c652 100644 --- a/src/main/java/org/apache/commons/lang3/mutable/MutableDouble.java +++ b/src/main/java/org/apache/commons/lang3/mutable/MutableDouble.java @@ -182,11 +182,11 @@ public class MutableDouble extends Number implements Comparable<MutableDouble>, * also has the value {@code true}. However, there are two exceptions: * <ul> * <li>If {@code d1} and {@code d2} both represent {@code Double.NaN}, then the - * {@code equals} method returns {@code true}, even though {@code Double.NaN==Double.NaN} has + * {@code equals} method returns {@code true}, even though {@code Double.NaN == Double.NaN} has * the value {@code false}. * <li>If {@code d1} represents {@code +0.0} while {@code d2} represents {@code -0.0}, * or vice versa, the {@code equal} test has the value {@code false}, even though - * {@code +0.0==-0.0} has the value {@code true}. This allows hashtables to operate properly. + * {@code +0.0 == -0.0} has the value {@code true}. This allows hashtables to operate properly. * </ul> * * @param obj the object to compare with, null returns false diff --git a/src/main/java/org/apache/commons/lang3/mutable/MutableFloat.java b/src/main/java/org/apache/commons/lang3/mutable/MutableFloat.java index 4b65aa01d..b78294603 100644 --- a/src/main/java/org/apache/commons/lang3/mutable/MutableFloat.java +++ b/src/main/java/org/apache/commons/lang3/mutable/MutableFloat.java @@ -182,11 +182,11 @@ public class MutableFloat extends Number implements Comparable<MutableFloat>, Mu * also has the value {@code true}. However, there are two exceptions: * <ul> * <li>If {@code f1} and {@code f2} both represent {@code Float.NaN}, then the - * {@code equals} method returns {@code true}, even though {@code Float.NaN==Float.NaN} has + * {@code equals} method returns {@code true}, even though {@code Float.NaN == Float.NaN} has * the value {@code false}. * <li>If {@code f1} represents {@code +0.0f} while {@code f2} represents {@code -0.0f}, * or vice versa, the {@code equal} test has the value {@code false}, even though - * {@code 0.0f==-0.0f} has the value {@code true}. + * {@code 0.0f == -0.0f} has the value {@code true}. * </ul> * This definition allows hashtables to operate properly. * diff --git a/src/main/java/org/apache/commons/lang3/reflect/ConstructorUtils.java b/src/main/java/org/apache/commons/lang3/reflect/ConstructorUtils.java index 5f4d35653..e7fde11a6 100644 --- a/src/main/java/org/apache/commons/lang3/reflect/ConstructorUtils.java +++ b/src/main/java/org/apache/commons/lang3/reflect/ConstructorUtils.java @@ -257,8 +257,7 @@ public class ConstructorUtils { parameterTypes = ArrayUtils.nullToEmpty(parameterTypes); final Constructor<T> ctor = getAccessibleConstructor(cls, parameterTypes); if (ctor == null) { - throw new NoSuchMethodException( - "No such accessible constructor on object: "+ cls.getName()); + throw new NoSuchMethodException("No such accessible constructor on object: " + cls.getName()); } return ctor.newInstance(args); } diff --git a/src/main/java/org/apache/commons/lang3/reflect/TypeUtils.java b/src/main/java/org/apache/commons/lang3/reflect/TypeUtils.java index 3f860e54b..9d4fbe1af 100644 --- a/src/main/java/org/apache/commons/lang3/reflect/TypeUtils.java +++ b/src/main/java/org/apache/commons/lang3/reflect/TypeUtils.java @@ -1713,7 +1713,7 @@ public class TypeUtils { buf.insert(0, c.getSimpleName()).insert(0, '.'); c = c.getEnclosingClass(); } - } else if (d instanceof Type) {// not possible as of now + } else if (d instanceof Type) { // not possible as of now buf.append(toString((Type) d)); } else { buf.append(d); diff --git a/src/main/java/org/apache/commons/lang3/stream/Streams.java b/src/main/java/org/apache/commons/lang3/stream/Streams.java index 3d5807828..57817e779 100644 --- a/src/main/java/org/apache/commons/lang3/stream/Streams.java +++ b/src/main/java/org/apache/commons/lang3/stream/Streams.java @@ -307,7 +307,6 @@ public class Streams { * Performs a mutable reduction operation on the elements of this FailableStream. A mutable reduction is one in which * the reduced value is a mutable result container, such as an {@link ArrayList}, and elements are incorporated by * updating the state of the result rather than by replacing the result. This produces a result equivalent to: - * * <pre> * {@code * R result = supplier.get(); @@ -316,29 +315,25 @@ public class Streams { * return result; * } * </pre> - * * <p> * Like {@link #reduce(Object, BinaryOperator)}, {@code collect} operations can be parallelized without requiring * additional synchronization. * </p> - * * <p> * This is a terminal operation. * </p> - * + * <p> * Note There are many existing classes in the JDK whose signatures are well-suited for use with method references as * arguments to {@code collect()}. For example, the following will accumulate strings into an {@link ArrayList}: - * + * </p> * <pre> * {@code * List<String> asList = stringStream.collect(ArrayList::new, ArrayList::add, ArrayList::addAll); * } * </pre> - * * <p> * The following will take a stream of strings and concatenates them into a single string: * </p> - * * <pre> * {@code * String concat = stringStream.collect(StringBuilder::new, StringBuilder::append, StringBuilder::append).toString(); @@ -362,7 +357,6 @@ public class Streams { /** * Returns a FailableStream consisting of the elements of this stream that match the given FailablePredicate. - * * <p> * This is an intermediate operation. * </p> @@ -379,11 +373,9 @@ public class Streams { /** * Performs an action for each element of this stream. - * * <p> * This is a terminal operation. * </p> - * * <p> * The behavior of this operation is explicitly nondeterministic. For parallel stream pipelines, this operation does * <em>not</em> guarantee to respect the encounter order of the stream, as doing so would sacrifice the benefit of diff --git a/src/main/java/org/apache/commons/lang3/text/translate/EntityArrays.java b/src/main/java/org/apache/commons/lang3/text/translate/EntityArrays.java index a157d67ea..5d08171d3 100644 --- a/src/main/java/org/apache/commons/lang3/text/translate/EntityArrays.java +++ b/src/main/java/org/apache/commons/lang3/text/translate/EntityArrays.java @@ -413,7 +413,7 @@ public class EntityArrays { */ public static String[][] invert(final String[][] array) { final String[][] newarray = new String[array.length][2]; - for (int i = 0; i<array.length; i++) { + for (int i = 0; i < array.length; i++) { newarray[i][0] = array[i][1]; newarray[i][1] = array[i][0]; } diff --git a/src/main/java/org/apache/commons/lang3/time/DurationFormatUtils.java b/src/main/java/org/apache/commons/lang3/time/DurationFormatUtils.java index a2c19450c..5e0fbccc8 100644 --- a/src/main/java/org/apache/commons/lang3/time/DurationFormatUtils.java +++ b/src/main/java/org/apache/commons/lang3/time/DurationFormatUtils.java @@ -310,7 +310,7 @@ public class DurationFormatUtils { lastOutputSeconds = false; } //as soon as we hit first nonliteral in optional, check for literal prefix - if (inOptional && !isLiteral && !firstOptionalNonLiteral){ + if (inOptional && !isLiteral && !firstOptionalNonLiteral) { firstOptionalNonLiteral = true; if (lastOutputZero) { buffer.delete(optionalStart, buffer.length()); @@ -680,14 +680,14 @@ public class DurationFormatUtils { // TODO: Need to handle escaping of ' case '[': if (inOptional) { - throw new IllegalArgumentException("Nested optional block at index: "+i); + throw new IllegalArgumentException("Nested optional block at index: " + i); } optionalIndex++; inOptional = true; break; case ']': if (!inOptional) { - throw new IllegalArgumentException("Attempting to close unopened optional block at index: "+i); + throw new IllegalArgumentException("Attempting to close unopened optional block at index: " + i); } inOptional = false; break; diff --git a/src/main/java/org/apache/commons/lang3/time/FastDatePrinter.java b/src/main/java/org/apache/commons/lang3/time/FastDatePrinter.java index 6b75ede7a..ea1b6d8cc 100644 --- a/src/main/java/org/apache/commons/lang3/time/FastDatePrinter.java +++ b/src/main/java/org/apache/commons/lang3/time/FastDatePrinter.java @@ -218,11 +218,11 @@ public class FastDatePrinter implements DatePrinter, Serializable { final int hours = offset / (60 * 60 * 1000); appendDigits(buffer, hours); - if (length<5) { + if (length < 5) { return; } - if (length==6) { + if (length == 6) { buffer.append(':'); } @@ -386,7 +386,7 @@ public class FastDatePrinter implements DatePrinter, Serializable { @Override public int estimateLength() { int max = 0; - for (int i=values.length; --i >= 0; ) { + for (int i = values.length; --i >= 0;) { final int len = values[i].length(); if (len > max) { max = len; diff --git a/src/site/resources/checkstyle/checkstyle.xml b/src/site/resources/checkstyle/checkstyle.xml index 3527cfa0d..90e7ba89e 100644 --- a/src/site/resources/checkstyle/checkstyle.xml +++ b/src/site/resources/checkstyle/checkstyle.xml @@ -15,11 +15,9 @@ WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the License for the specific language governing permissions and limitations under the License. --> - <!DOCTYPE module PUBLIC "-//Checkstyle//DTD Checkstyle Configuration 1.3//EN" "https://checkstyle.org/dtds/configuration_1_3.dtd"> - <!-- Apache Commons Lang customization of default Checkstyle behavior --> <module name="Checker"> <property name="localeLanguage" value="en"/> @@ -53,6 +51,8 @@ limitations under the License. <module name="RightCurly"/> <module name="GenericWhitespace"/> <module name="WhitespaceAfter"/> + <module name="WhitespaceAround"/> + <module name="WhitespaceAroundCheck"/> <module name="NoWhitespaceBefore"/> <module name="ImportOrder"> <property name="option" value="top"/> diff --git a/src/test/java/org/apache/commons/lang3/ArrayUtilsAddTest.java b/src/test/java/org/apache/commons/lang3/ArrayUtilsAddTest.java index f8ced87c4..0bc516efe 100644 --- a/src/test/java/org/apache/commons/lang3/ArrayUtilsAddTest.java +++ b/src/test/java/org/apache/commons/lang3/ArrayUtilsAddTest.java @@ -35,14 +35,14 @@ public class ArrayUtilsAddTest extends AbstractLangTest { public void testAddFirstBoolean() { boolean[] newArray; newArray = ArrayUtils.addFirst(null, false); - assertArrayEquals(new boolean[]{false}, newArray); + assertArrayEquals(new boolean[] { false }, newArray); assertEquals(Boolean.TYPE, newArray.getClass().getComponentType()); newArray = ArrayUtils.addFirst(null, true); - assertArrayEquals(new boolean[]{true}, newArray); + assertArrayEquals(new boolean[] { true }, newArray); assertEquals(Boolean.TYPE, newArray.getClass().getComponentType()); - final boolean[] array1 = {true, false, true}; + final boolean[] array1 = { true, false, true }; newArray = ArrayUtils.addFirst(array1, false); - assertArrayEquals(new boolean[]{false, true, false, true}, newArray); + assertArrayEquals(new boolean[] { false, true, false, true }, newArray); assertEquals(Boolean.TYPE, newArray.getClass().getComponentType()); } @@ -50,17 +50,17 @@ public class ArrayUtilsAddTest extends AbstractLangTest { public void testAddFirstByte() { byte[] newArray; newArray = ArrayUtils.addFirst((byte[]) null, (byte) 0); - assertArrayEquals(new byte[]{0}, newArray); + assertArrayEquals(new byte[] { 0 }, newArray); assertEquals(Byte.TYPE, newArray.getClass().getComponentType()); newArray = ArrayUtils.addFirst((byte[]) null, (byte) 1); - assertArrayEquals(new byte[]{1}, newArray); + assertArrayEquals(new byte[] { 1 }, newArray); assertEquals(Byte.TYPE, newArray.getClass().getComponentType()); - final byte[] array1 = {1, 2, 3}; + final byte[] array1 = { 1, 2, 3 }; newArray = ArrayUtils.addFirst(array1, (byte) 0); - assertArrayEquals(new byte[]{0, 1, 2, 3}, newArray); + assertArrayEquals(new byte[] { 0, 1, 2, 3 }, newArray); assertEquals(Byte.TYPE, newArray.getClass().getComponentType()); newArray = ArrayUtils.addFirst(array1, (byte) 4); - assertArrayEquals(new byte[]{4, 1, 2, 3}, newArray); + assertArrayEquals(new byte[] { 4, 1, 2, 3 }, newArray); assertEquals(Byte.TYPE, newArray.getClass().getComponentType()); } @@ -68,17 +68,17 @@ public class ArrayUtilsAddTest extends AbstractLangTest { public void testAddFirstChar() { char[] newArray; newArray = ArrayUtils.addFirst((char[]) null, (char) 0); - assertArrayEquals(new char[]{0}, newArray); + assertArrayEquals(new char[] { 0 }, newArray); assertEquals(Character.TYPE, newArray.getClass().getComponentType()); newArray = ArrayUtils.addFirst((char[]) null, (char) 1); - assertArrayEquals(new char[]{1}, newArray); + assertArrayEquals(new char[] { 1 }, newArray); assertEquals(Character.TYPE, newArray.getClass().getComponentType()); - final char[] array1 = {1, 2, 3}; + final char[] array1 = { 1, 2, 3 }; newArray = ArrayUtils.addFirst(array1, (char) 0); - assertArrayEquals(new char[]{0, 1, 2, 3}, newArray); + assertArrayEquals(new char[] { 0, 1, 2, 3 }, newArray); assertEquals(Character.TYPE, newArray.getClass().getComponentType()); newArray = ArrayUtils.addFirst(array1, (char) 4); - assertArrayEquals(new char[]{4, 1, 2, 3}, newArray); + assertArrayEquals(new char[] { 4, 1, 2, 3 }, newArray); assertEquals(Character.TYPE, newArray.getClass().getComponentType()); } @@ -86,17 +86,17 @@ public class ArrayUtilsAddTest extends AbstractLangTest { public void testAddFirstDouble() { double[] newArray; newArray = ArrayUtils.addFirst((double[]) null, 0); - assertArrayEquals(new double[]{0}, newArray); + assertArrayEquals(new double[] { 0 }, newArray); assertEquals(Double.TYPE, newArray.getClass().getComponentType()); newArray = ArrayUtils.addFirst((double[]) null, 1); - assertArrayEquals(new double[]{1}, newArray); + assertArrayEquals(new double[] { 1 }, newArray); assertEquals(Double.TYPE, newArray.getClass().getComponentType()); - final double[] array1 = {1, 2, 3}; + final double[] array1 = { 1, 2, 3 }; newArray = ArrayUtils.addFirst(array1, 0); - assertArrayEquals(new double[]{0, 1, 2, 3}, newArray); + assertArrayEquals(new double[] { 0, 1, 2, 3 }, newArray); assertEquals(Double.TYPE, newArray.getClass().getComponentType()); newArray = ArrayUtils.addFirst(array1, 4); - assertArrayEquals(new double[]{4, 1, 2, 3}, newArray); + assertArrayEquals(new double[] { 4, 1, 2, 3 }, newArray); assertEquals(Double.TYPE, newArray.getClass().getComponentType()); } @@ -104,17 +104,17 @@ public class ArrayUtilsAddTest extends AbstractLangTest { public void testAddFirstFloat() { float[] newArray; newArray = ArrayUtils.addFirst((float[]) null, 0); - assertArrayEquals(new float[]{0}, newArray); + assertArrayEquals(new float[] { 0 }, newArray); assertEquals(Float.TYPE, newArray.getClass().getComponentType()); newArray = ArrayUtils.addFirst((float[]) null, 1); - assertArrayEquals(new float[]{1}, newArray); + assertArrayEquals(new float[] { 1 }, newArray); assertEquals(Float.TYPE, newArray.getClass().getComponentType()); - final float[] array1 = {1, 2, 3}; + final float[] array1 = { 1, 2, 3 }; newArray = ArrayUtils.addFirst(array1, 0); - assertArrayEquals(new float[]{0, 1, 2, 3}, newArray); + assertArrayEquals(new float[] { 0, 1, 2, 3 }, newArray); assertEquals(Float.TYPE, newArray.getClass().getComponentType()); newArray = ArrayUtils.addFirst(array1, 4); - assertArrayEquals(new float[]{4, 1, 2, 3}, newArray); + assertArrayEquals(new float[] { 4, 1, 2, 3 }, newArray); assertEquals(Float.TYPE, newArray.getClass().getComponentType()); } @@ -122,17 +122,17 @@ public class ArrayUtilsAddTest extends AbstractLangTest { public void testAddFirstInt() { int[] newArray; newArray = ArrayUtils.addFirst((int[]) null, 0); - assertArrayEquals(new int[]{0}, newArray); + assertArrayEquals(new int[] { 0 }, newArray); assertEquals(Integer.TYPE, newArray.getClass().getComponentType()); newArray = ArrayUtils.addFirst((int[]) null, 1); - assertArrayEquals(new int[]{1}, newArray); + assertArrayEquals(new int[] { 1 }, newArray); assertEquals(Integer.TYPE, newArray.getClass().getComponentType()); - final int[] array1 = {1, 2, 3}; + final int[] array1 = { 1, 2, 3 }; newArray = ArrayUtils.addFirst(array1, 0); - assertArrayEquals(new int[]{0, 1, 2, 3}, newArray); + assertArrayEquals(new int[] { 0, 1, 2, 3 }, newArray); assertEquals(Integer.TYPE, newArray.getClass().getComponentType()); newArray = ArrayUtils.addFirst(array1, 4); - assertArrayEquals(new int[]{4, 1, 2, 3}, newArray); + assertArrayEquals(new int[] { 4, 1, 2, 3 }, newArray); assertEquals(Integer.TYPE, newArray.getClass().getComponentType()); } @@ -140,17 +140,17 @@ public class ArrayUtilsAddTest extends AbstractLangTest { public void testAddFirstLong() { long[] newArray; newArray = ArrayUtils.addFirst((long[]) null, 0); - assertArrayEquals(new long[]{0}, newArray); + assertArrayEquals(new long[] { 0 }, newArray); assertEquals(Long.TYPE, newArray.getClass().getComponentType()); newArray = ArrayUtils.addFirst((long[]) null, 1); - assertArrayEquals(new long[]{1}, newArray); + assertArrayEquals(new long[] { 1 }, newArray); assertEquals(Long.TYPE, newArray.getClass().getComponentType()); - final long[] array1 = {1, 2, 3}; + final long[] array1 = { 1, 2, 3 }; newArray = ArrayUtils.addFirst(array1, 0); - assertArrayEquals(new long[]{0, 1, 2, 3}, newArray); + assertArrayEquals(new long[] { 0, 1, 2, 3 }, newArray); assertEquals(Long.TYPE, newArray.getClass().getComponentType()); newArray = ArrayUtils.addFirst(array1, 4); - assertArrayEquals(new long[]{4, 1, 2, 3}, newArray); + assertArrayEquals(new long[] { 4, 1, 2, 3 }, newArray); assertEquals(Long.TYPE, newArray.getClass().getComponentType()); } @@ -158,16 +158,16 @@ public class ArrayUtilsAddTest extends AbstractLangTest { public void testAddFirstObject() { Object[] newArray; - //show that not casting is okay + // show that not casting is okay newArray = ArrayUtils.add((Object[]) null, "a"); - assertArrayEquals(new String[]{"a"}, newArray); - assertArrayEquals(new Object[]{"a"}, newArray); + assertArrayEquals(new String[] { "a" }, newArray); + assertArrayEquals(new Object[] { "a" }, newArray); assertEquals(String.class, newArray.getClass().getComponentType()); - //show that not casting to Object[] is okay and will assume String based on "a" + // show that not casting to Object[] is okay and will assume String based on "a" final String[] newStringArray = ArrayUtils.add(null, "a"); - assertArrayEquals(new String[]{"a"}, newStringArray); - assertArrayEquals(new Object[]{"a"}, newStringArray); + assertArrayEquals(new String[] { "a" }, newStringArray); + assertArrayEquals(new Object[] { "a" }, newStringArray); assertEquals(String.class, newStringArray.getClass().getComponentType()); final String[] stringArray1 = { "a", "b", "c" }; @@ -194,17 +194,17 @@ public class ArrayUtilsAddTest extends AbstractLangTest { public void testAddFirstShort() { short[] newArray; newArray = ArrayUtils.addFirst((short[]) null, (short) 0); - assertArrayEquals(new short[]{0}, newArray); + assertArrayEquals(new short[] { 0 }, newArray); assertEquals(Short.TYPE, newArray.getClass().getComponentType()); newArray = ArrayUtils.addFirst((short[]) null, (short) 1); - assertArrayEquals(new short[]{1}, newArray); + assertArrayEquals(new short[] { 1 }, newArray); assertEquals(Short.TYPE, newArray.getClass().getComponentType()); - final short[] array1 = {1, 2, 3}; + final short[] array1 = { 1, 2, 3 }; newArray = ArrayUtils.addFirst(array1, (short) 0); - assertArrayEquals(new short[]{0, 1, 2, 3}, newArray); + assertArrayEquals(new short[] { 0, 1, 2, 3 }, newArray); assertEquals(Short.TYPE, newArray.getClass().getComponentType()); newArray = ArrayUtils.addFirst(array1, (short) 4); - assertArrayEquals(new short[]{4, 1, 2, 3}, newArray); + assertArrayEquals(new short[] { 4, 1, 2, 3 }, newArray); assertEquals(Short.TYPE, newArray.getClass().getComponentType()); } @@ -212,14 +212,14 @@ public class ArrayUtilsAddTest extends AbstractLangTest { public void testAddObjectArrayBoolean() { boolean[] newArray; newArray = ArrayUtils.add(null, false); - assertArrayEquals(new boolean[]{false}, newArray); + assertArrayEquals(new boolean[] { false }, newArray); assertEquals(Boolean.TYPE, newArray.getClass().getComponentType()); newArray = ArrayUtils.add(null, true); - assertArrayEquals(new boolean[]{true}, newArray); + assertArrayEquals(new boolean[] { true }, newArray); assertEquals(Boolean.TYPE, newArray.getClass().getComponentType()); - final boolean[] array1 = {true, false, true}; + final boolean[] array1 = { true, false, true }; newArray = ArrayUtils.add(array1, false); - assertArrayEquals(new boolean[]{true, false, true, false}, newArray); + assertArrayEquals(new boolean[] { true, false, true, false }, newArray); assertEquals(Boolean.TYPE, newArray.getClass().getComponentType()); } @@ -227,17 +227,17 @@ public class ArrayUtilsAddTest extends AbstractLangTest { public void testAddObjectArrayByte() { byte[] newArray; newArray = ArrayUtils.add((byte[]) null, (byte) 0); - assertArrayEquals(new byte[]{0}, newArray); + assertArrayEquals(new byte[] { 0 }, newArray); assertEquals(Byte.TYPE, newArray.getClass().getComponentType()); newArray = ArrayUtils.add((byte[]) null, (byte) 1); - assertArrayEquals(new byte[]{1}, newArray); + assertArrayEquals(new byte[] { 1 }, newArray); assertEquals(Byte.TYPE, newArray.getClass().getComponentType()); - final byte[] array1 = {1, 2, 3}; + final byte[] array1 = { 1, 2, 3 }; newArray = ArrayUtils.add(array1, (byte) 0); - assertArrayEquals(new byte[]{1, 2, 3, 0}, newArray); + assertArrayEquals(new byte[] { 1, 2, 3, 0 }, newArray); assertEquals(Byte.TYPE, newArray.getClass().getComponentType()); newArray = ArrayUtils.add(array1, (byte) 4); - assertArrayEquals(new byte[]{1, 2, 3, 4}, newArray); + assertArrayEquals(new byte[] { 1, 2, 3, 4 }, newArray); assertEquals(Byte.TYPE, newArray.getClass().getComponentType()); } @@ -245,17 +245,17 @@ public class ArrayUtilsAddTest extends AbstractLangTest { public void testAddObjectArrayChar() { char[] newArray; newArray = ArrayUtils.add((char[]) null, (char) 0); - assertArrayEquals(new char[]{0}, newArray); + assertArrayEquals(new char[] { 0 }, newArray); assertEquals(Character.TYPE, newArray.getClass().getComponentType()); newArray = ArrayUtils.add((char[]) null, (char) 1); - assertArrayEquals(new char[]{1}, newArray); + assertArrayEquals(new char[] { 1 }, newArray); assertEquals(Character.TYPE, newArray.getClass().getComponentType()); - final char[] array1 = {1, 2, 3}; + final char[] array1 = { 1, 2, 3 }; newArray = ArrayUtils.add(array1, (char) 0); - assertArrayEquals(new char[]{1, 2, 3, 0}, newArray); + assertArrayEquals(new char[] { 1, 2, 3, 0 }, newArray); assertEquals(Character.TYPE, newArray.getClass().getComponentType()); newArray = ArrayUtils.add(array1, (char) 4); - assertArrayEquals(new char[]{1, 2, 3, 4}, newArray); + assertArrayEquals(new char[] { 1, 2, 3, 4 }, newArray); assertEquals(Character.TYPE, newArray.getClass().getComponentType()); } @@ -263,17 +263,17 @@ public class ArrayUtilsAddTest extends AbstractLangTest { public void testAddObjectArrayDouble() { double[] newArray; newArray = ArrayUtils.add((double[]) null, 0); - assertArrayEquals(new double[]{0}, newArray); + assertArrayEquals(new double[] { 0 }, newArray); assertEquals(Double.TYPE, newArray.getClass().getComponentType()); newArray = ArrayUtils.add((double[]) null, 1); - assertArrayEquals(new double[]{1}, newArray); + assertArrayEquals(new double[] { 1 }, newArray); assertEquals(Double.TYPE, newArray.getClass().getComponentType()); - final double[] array1 = {1, 2, 3}; + final double[] array1 = { 1, 2, 3 }; newArray = ArrayUtils.add(array1, 0); - assertArrayEquals(new double[]{1, 2, 3, 0}, newArray); + assertArrayEquals(new double[] { 1, 2, 3, 0 }, newArray); assertEquals(Double.TYPE, newArray.getClass().getComponentType()); newArray = ArrayUtils.add(array1, 4); - assertArrayEquals(new double[]{1, 2, 3, 4}, newArray); + assertArrayEquals(new double[] { 1, 2, 3, 4 }, newArray); assertEquals(Double.TYPE, newArray.getClass().getComponentType()); } @@ -281,17 +281,17 @@ public class ArrayUtilsAddTest extends AbstractLangTest { public void testAddObjectArrayFloat() { float[] newArray; newArray = ArrayUtils.add((float[]) null, 0); - assertArrayEquals(new float[]{0}, newArray); + assertArrayEquals(new float[] { 0 }, newArray); assertEquals(Float.TYPE, newArray.getClass().getComponentType()); newArray = ArrayUtils.add((float[]) null, 1); - assertArrayEquals(new float[]{1}, newArray); + assertArrayEquals(new float[] { 1 }, newArray); assertEquals(Float.TYPE, newArray.getClass().getComponentType()); - final float[] array1 = {1, 2, 3}; + final float[] array1 = { 1, 2, 3 }; newArray = ArrayUtils.add(array1, 0); - assertArrayEquals(new float[]{1, 2, 3, 0}, newArray); + assertArrayEquals(new float[] { 1, 2, 3, 0 }, newArray); assertEquals(Float.TYPE, newArray.getClass().getComponentType()); newArray = ArrayUtils.add(array1, 4); - assertArrayEquals(new float[]{1, 2, 3, 4}, newArray); + assertArrayEquals(new float[] { 1, 2, 3, 4 }, newArray); assertEquals(Float.TYPE, newArray.getClass().getComponentType()); } @@ -299,17 +299,17 @@ public class ArrayUtilsAddTest extends AbstractLangTest { public void testAddObjectArrayInt() { int[] newArray; newArray = ArrayUtils.add((int[]) null, 0); - assertArrayEquals(new int[]{0}, newArray); + assertArrayEquals(new int[] { 0 }, newArray); assertEquals(Integer.TYPE, newArray.getClass().getComponentType()); newArray = ArrayUtils.add((int[]) null, 1); - assertArrayEquals(new int[]{1}, newArray); + assertArrayEquals(new int[] { 1 }, newArray); assertEquals(Integer.TYPE, newArray.getClass().getComponentType()); - final int[] array1 = {1, 2, 3}; + final int[] array1 = { 1, 2, 3 }; newArray = ArrayUtils.add(array1, 0); - assertArrayEquals(new int[]{1, 2, 3, 0}, newArray); + assertArrayEquals(new int[] { 1, 2, 3, 0 }, newArray); assertEquals(Integer.TYPE, newArray.getClass().getComponentType()); newArray = ArrayUtils.add(array1, 4); - assertArrayEquals(new int[]{1, 2, 3, 4}, newArray); + assertArrayEquals(new int[] { 1, 2, 3, 4 }, newArray); assertEquals(Integer.TYPE, newArray.getClass().getComponentType()); } @@ -317,17 +317,17 @@ public class ArrayUtilsAddTest extends AbstractLangTest { public void testAddObjectArrayLong() { long[] newArray; newArray = ArrayUtils.add((long[]) null, 0); - assertArrayEquals(new long[]{0}, newArray); + assertArrayEquals(new long[] { 0 }, newArray); assertEquals(Long.TYPE, newArray.getClass().getComponentType()); newArray = ArrayUtils.add((long[]) null, 1); - assertArrayEquals(new long[]{1}, newArray); + assertArrayEquals(new long[] { 1 }, newArray); assertEquals(Long.TYPE, newArray.getClass().getComponentType()); - final long[] array1 = {1, 2, 3}; + final long[] array1 = { 1, 2, 3 }; newArray = ArrayUtils.add(array1, 0); - assertArrayEquals(new long[]{1, 2, 3, 0}, newArray); + assertArrayEquals(new long[] { 1, 2, 3, 0 }, newArray); assertEquals(Long.TYPE, newArray.getClass().getComponentType()); newArray = ArrayUtils.add(array1, 4); - assertArrayEquals(new long[]{1, 2, 3, 4}, newArray); + assertArrayEquals(new long[] { 1, 2, 3, 4 }, newArray); assertEquals(Long.TYPE, newArray.getClass().getComponentType()); } @@ -335,35 +335,35 @@ public class ArrayUtilsAddTest extends AbstractLangTest { public void testAddObjectArrayObject() { Object[] newArray; - //show that not casting is okay + // show that not casting is okay newArray = ArrayUtils.add((Object[]) null, "a"); - assertArrayEquals(new String[]{"a"}, newArray); - assertArrayEquals(new Object[]{"a"}, newArray); + assertArrayEquals(new String[] { "a" }, newArray); + assertArrayEquals(new Object[] { "a" }, newArray); assertEquals(String.class, newArray.getClass().getComponentType()); - //show that not casting to Object[] is okay and will assume String based on "a" + // show that not casting to Object[] is okay and will assume String based on "a" final String[] newStringArray = ArrayUtils.add(null, "a"); - assertArrayEquals(new String[]{"a"}, newStringArray); - assertArrayEquals(new Object[]{"a"}, newStringArray); + assertArrayEquals(new String[] { "a" }, newStringArray); + assertArrayEquals(new Object[] { "a" }, newStringArray); assertEquals(String.class, newStringArray.getClass().getComponentType()); - final String[] stringArray1 = {"a", "b", "c"}; + final String[] stringArray1 = { "a", "b", "c" }; newArray = ArrayUtils.add(stringArray1, null); - assertArrayEquals(new String[]{"a", "b", "c", null}, newArray); + assertArrayEquals(new String[] { "a", "b", "c", null }, newArray); assertEquals(String.class, newArray.getClass().getComponentType()); newArray = ArrayUtils.add(stringArray1, "d"); - assertArrayEquals(new String[]{"a", "b", "c", "d"}, newArray); + assertArrayEquals(new String[] { "a", "b", "c", "d" }, newArray); assertEquals(String.class, newArray.getClass().getComponentType()); - Number[] numberArray1 = {Integer.valueOf(1), Double.valueOf(2)}; + Number[] numberArray1 = { Integer.valueOf(1), Double.valueOf(2) }; newArray = ArrayUtils.add(numberArray1, Float.valueOf(3)); - assertArrayEquals(new Number[]{Integer.valueOf(1), Double.valueOf(2), Float.valueOf(3)}, newArray); + assertArrayEquals(new Number[] { Integer.valueOf(1), Double.valueOf(2), Float.valueOf(3) }, newArray); assertEquals(Number.class, newArray.getClass().getComponentType()); numberArray1 = null; newArray = ArrayUtils.add(numberArray1, Float.valueOf(3)); - assertArrayEquals(new Float[]{Float.valueOf(3)}, newArray); + assertArrayEquals(new Float[] { Float.valueOf(3) }, newArray); assertEquals(Float.class, newArray.getClass().getComponentType()); } @@ -371,17 +371,17 @@ public class ArrayUtilsAddTest extends AbstractLangTest { public void testAddObjectArrayShort() { short[] newArray; newArray = ArrayUtils.add((short[]) null, (short) 0); - assertArrayEquals(new short[]{0}, newArray); + assertArrayEquals(new short[] { 0 }, newArray); assertEquals(Short.TYPE, newArray.getClass().getComponentType()); newArray = ArrayUtils.add((short[]) null, (short) 1); - assertArrayEquals(new short[]{1}, newArray); + assertArrayEquals(new short[] { 1 }, newArray); assertEquals(Short.TYPE, newArray.getClass().getComponentType()); - final short[] array1 = {1, 2, 3}; + final short[] array1 = { 1, 2, 3 }; newArray = ArrayUtils.add(array1, (short) 0); - assertArrayEquals(new short[]{1, 2, 3, 0}, newArray); + assertArrayEquals(new short[] { 1, 2, 3, 0 }, newArray); assertEquals(Short.TYPE, newArray.getClass().getComponentType()); newArray = ArrayUtils.add(array1, (short) 4); - assertArrayEquals(new short[]{1, 2, 3, 4}, newArray); + assertArrayEquals(new short[] { 1, 2, 3, 4 }, newArray); assertEquals(Short.TYPE, newArray.getClass().getComponentType()); } @@ -389,93 +389,94 @@ public class ArrayUtilsAddTest extends AbstractLangTest { public void testAddObjectArrayToObjectArray() { assertNull(ArrayUtils.addAll(null, (Object[]) null)); Object[] newArray; - final String[] stringArray1 = {"a", "b", "c"}; - final String[] stringArray2 = {"1", "2", "3"}; + final String[] stringArray1 = { "a", "b", "c" }; + final String[] stringArray2 = { "1", "2", "3" }; newArray = ArrayUtils.addAll(stringArray1, (String[]) null); assertNotSame(stringArray1, newArray); assertArrayEquals(stringArray1, newArray); - assertArrayEquals(new String[]{"a", "b", "c"}, newArray); + assertArrayEquals(new String[] { "a", "b", "c" }, newArray); assertEquals(String.class, newArray.getClass().getComponentType()); newArray = ArrayUtils.addAll(null, stringArray2); assertNotSame(stringArray2, newArray); assertArrayEquals(stringArray2, newArray); - assertArrayEquals(new String[]{"1", "2", "3"}, newArray); + assertArrayEquals(new String[] { "1", "2", "3" }, newArray); assertEquals(String.class, newArray.getClass().getComponentType()); newArray = ArrayUtils.addAll(stringArray1, stringArray2); - assertArrayEquals(new String[]{"a", "b", "c", "1", "2", "3"}, newArray); + assertArrayEquals(new String[] { "a", "b", "c", "1", "2", "3" }, newArray); assertEquals(String.class, newArray.getClass().getComponentType()); newArray = ArrayUtils.addAll(ArrayUtils.EMPTY_STRING_ARRAY, (String[]) null); assertArrayEquals(ArrayUtils.EMPTY_STRING_ARRAY, newArray); - assertArrayEquals(new String[]{}, newArray); + assertArrayEquals(new String[] {}, newArray); assertEquals(String.class, newArray.getClass().getComponentType()); newArray = ArrayUtils.addAll(null, ArrayUtils.EMPTY_STRING_ARRAY); assertArrayEquals(ArrayUtils.EMPTY_STRING_ARRAY, newArray); - assertArrayEquals(new String[]{}, newArray); + assertArrayEquals(new String[] {}, newArray); assertEquals(String.class, newArray.getClass().getComponentType()); newArray = ArrayUtils.addAll(ArrayUtils.EMPTY_STRING_ARRAY, ArrayUtils.EMPTY_STRING_ARRAY); assertArrayEquals(ArrayUtils.EMPTY_STRING_ARRAY, newArray); - assertArrayEquals(new String[]{}, newArray); + assertArrayEquals(new String[] {}, newArray); assertEquals(String.class, newArray.getClass().getComponentType()); - final String[] stringArrayNull = {null}; + final String[] stringArrayNull = { null }; newArray = ArrayUtils.addAll(stringArrayNull, stringArrayNull); - assertArrayEquals(new String[]{null, null}, newArray); + assertArrayEquals(new String[] { null, null }, newArray); assertEquals(String.class, newArray.getClass().getComponentType()); // boolean - assertArrayEquals(new boolean[]{true, false, false, true}, ArrayUtils.addAll(new boolean[]{true, false}, false, true)); + assertArrayEquals(new boolean[] { true, false, false, true }, ArrayUtils.addAll(new boolean[] { true, false }, false, true)); - assertArrayEquals(new boolean[]{false, true}, ArrayUtils.addAll(null, new boolean[]{false, true})); + assertArrayEquals(new boolean[] { false, true }, ArrayUtils.addAll(null, new boolean[] { false, true })); - assertArrayEquals(new boolean[]{true, false}, ArrayUtils.addAll(new boolean[]{true, false}, null)); + assertArrayEquals(new boolean[] { true, false }, ArrayUtils.addAll(new boolean[] { true, false }, null)); // char - assertArrayEquals(new char[]{'a', 'b', 'c', 'd'}, ArrayUtils.addAll(new char[]{'a', 'b'}, 'c', 'd')); + assertArrayEquals(new char[] { 'a', 'b', 'c', 'd' }, ArrayUtils.addAll(new char[] { 'a', 'b' }, 'c', 'd')); - assertArrayEquals(new char[]{'c', 'd'}, ArrayUtils.addAll(null, new char[]{'c', 'd'})); + assertArrayEquals(new char[] { 'c', 'd' }, ArrayUtils.addAll(null, new char[] { 'c', 'd' })); - assertArrayEquals(new char[]{'a', 'b'}, ArrayUtils.addAll(new char[]{'a', 'b'}, null)); + assertArrayEquals(new char[] { 'a', 'b' }, ArrayUtils.addAll(new char[] { 'a', 'b' }, null)); // byte - assertArrayEquals(new byte[]{(byte) 0, (byte) 1, (byte) 2, (byte) 3}, ArrayUtils.addAll(new byte[]{(byte) 0, (byte) 1}, (byte) 2, (byte) 3)); + assertArrayEquals(new byte[] { (byte) 0, (byte) 1, (byte) 2, (byte) 3 }, ArrayUtils.addAll(new byte[] { (byte) 0, (byte) 1 }, (byte) 2, (byte) 3)); - assertArrayEquals(new byte[]{(byte) 2, (byte) 3}, ArrayUtils.addAll(null, new byte[]{(byte) 2, (byte) 3})); + assertArrayEquals(new byte[] { (byte) 2, (byte) 3 }, ArrayUtils.addAll(null, new byte[] { (byte) 2, (byte) 3 })); - assertArrayEquals(new byte[]{(byte) 0, (byte) 1}, ArrayUtils.addAll(new byte[]{(byte) 0, (byte) 1}, null)); + assertArrayEquals(new byte[] { (byte) 0, (byte) 1 }, ArrayUtils.addAll(new byte[] { (byte) 0, (byte) 1 }, null)); // short - assertArrayEquals(new short[]{(short) 10, (short) 20, (short) 30, (short) 40}, ArrayUtils.addAll(new short[]{(short) 10, (short) 20}, (short) 30, (short) 40)); + assertArrayEquals(new short[] { (short) 10, (short) 20, (short) 30, (short) 40 }, + ArrayUtils.addAll(new short[] { (short) 10, (short) 20 }, (short) 30, (short) 40)); - assertArrayEquals(new short[]{(short) 30, (short) 40}, ArrayUtils.addAll(null, new short[]{(short) 30, (short) 40})); + assertArrayEquals(new short[] { (short) 30, (short) 40 }, ArrayUtils.addAll(null, new short[] { (short) 30, (short) 40 })); - assertArrayEquals(new short[]{(short) 10, (short) 20}, ArrayUtils.addAll(new short[]{(short) 10, (short) 20}, null)); + assertArrayEquals(new short[] { (short) 10, (short) 20 }, ArrayUtils.addAll(new short[] { (short) 10, (short) 20 }, null)); // int - assertArrayEquals(new int[]{1, 1000, -1000, -1}, ArrayUtils.addAll(new int[]{1, 1000}, -1000, -1)); + assertArrayEquals(new int[] { 1, 1000, -1000, -1 }, ArrayUtils.addAll(new int[] { 1, 1000 }, -1000, -1)); - assertArrayEquals(new int[]{-1000, -1}, ArrayUtils.addAll(null, new int[]{-1000, -1})); + assertArrayEquals(new int[] { -1000, -1 }, ArrayUtils.addAll(null, new int[] { -1000, -1 })); - assertArrayEquals(new int[]{1, 1000}, ArrayUtils.addAll(new int[]{1, 1000}, null)); + assertArrayEquals(new int[] { 1, 1000 }, ArrayUtils.addAll(new int[] { 1, 1000 }, null)); // long - assertArrayEquals(new long[]{1L, -1L, 1000L, -1000L}, ArrayUtils.addAll(new long[]{1L, -1L}, 1000L, -1000L)); + assertArrayEquals(new long[] { 1L, -1L, 1000L, -1000L }, ArrayUtils.addAll(new long[] { 1L, -1L }, 1000L, -1000L)); - assertArrayEquals(new long[]{1000L, -1000L}, ArrayUtils.addAll(null, new long[]{1000L, -1000L})); + assertArrayEquals(new long[] { 1000L, -1000L }, ArrayUtils.addAll(null, new long[] { 1000L, -1000L })); - assertArrayEquals(new long[]{1L, -1L}, ArrayUtils.addAll(new long[]{1L, -1L}, null)); + assertArrayEquals(new long[] { 1L, -1L }, ArrayUtils.addAll(new long[] { 1L, -1L }, null)); // float - assertArrayEquals(new float[]{10.5f, 10.1f, 1.6f, 0.01f}, ArrayUtils.addAll(new float[]{10.5f, 10.1f}, 1.6f, 0.01f)); + assertArrayEquals(new float[] { 10.5f, 10.1f, 1.6f, 0.01f }, ArrayUtils.addAll(new float[] { 10.5f, 10.1f }, 1.6f, 0.01f)); - assertArrayEquals(new float[]{1.6f, 0.01f}, ArrayUtils.addAll(null, new float[]{1.6f, 0.01f})); + assertArrayEquals(new float[] { 1.6f, 0.01f }, ArrayUtils.addAll(null, new float[] { 1.6f, 0.01f })); - assertArrayEquals(new float[]{10.5f, 10.1f}, ArrayUtils.addAll(new float[]{10.5f, 10.1f}, null)); + assertArrayEquals(new float[] { 10.5f, 10.1f }, ArrayUtils.addAll(new float[] { 10.5f, 10.1f }, null)); // double - assertArrayEquals(new double[]{Math.PI, -Math.PI, 0, 9.99}, ArrayUtils.addAll(new double[]{Math.PI, -Math.PI}, 0, 9.99)); + assertArrayEquals(new double[] { Math.PI, -Math.PI, 0, 9.99 }, ArrayUtils.addAll(new double[] { Math.PI, -Math.PI }, 0, 9.99)); - assertArrayEquals(new double[]{0, 9.99}, ArrayUtils.addAll(null, new double[]{0, 9.99})); + assertArrayEquals(new double[] { 0, 9.99 }, ArrayUtils.addAll(null, new double[] { 0, 9.99 })); - assertArrayEquals(new double[]{Math.PI, -Math.PI}, ArrayUtils.addAll(new double[]{Math.PI, -Math.PI}, null)); + assertArrayEquals(new double[] { Math.PI, -Math.PI }, ArrayUtils.addAll(new double[] { Math.PI, -Math.PI }, null)); } @@ -484,25 +485,25 @@ public class ArrayUtilsAddTest extends AbstractLangTest { public void testAddObjectAtIndex() { Object[] newArray; newArray = ArrayUtils.add((Object[]) null, 0, "a"); - assertArrayEquals(new String[]{"a"}, newArray); - assertArrayEquals(new Object[]{"a"}, newArray); + assertArrayEquals(new String[] { "a" }, newArray); + assertArrayEquals(new Object[] { "a" }, newArray); assertEquals(String.class, newArray.getClass().getComponentType()); - final String[] stringArray1 = {"a", "b", "c"}; + final String[] stringArray1 = { "a", "b", "c" }; newArray = ArrayUtils.add(stringArray1, 0, null); - assertArrayEquals(new String[]{null, "a", "b", "c"}, newArray); + assertArrayEquals(new String[] { null, "a", "b", "c" }, newArray); assertEquals(String.class, newArray.getClass().getComponentType()); newArray = ArrayUtils.add(stringArray1, 1, null); - assertArrayEquals(new String[]{"a", null, "b", "c"}, newArray); + assertArrayEquals(new String[] { "a", null, "b", "c" }, newArray); assertEquals(String.class, newArray.getClass().getComponentType()); newArray = ArrayUtils.add(stringArray1, 3, null); - assertArrayEquals(new String[]{"a", "b", "c", null}, newArray); + assertArrayEquals(new String[] { "a", "b", "c", null }, newArray); assertEquals(String.class, newArray.getClass().getComponentType()); newArray = ArrayUtils.add(stringArray1, 3, "d"); - assertArrayEquals(new String[]{"a", "b", "c", "d"}, newArray); + assertArrayEquals(new String[] { "a", "b", "c", "d" }, newArray); assertEquals(String.class, newArray.getClass().getComponentType()); assertEquals(String.class, newArray.getClass().getComponentType()); - final Object[] o = {"1", "2", "4"}; + final Object[] o = { "1", "2", "4" }; final Object[] result = ArrayUtils.add(o, 2, "3"); final Object[] result2 = ArrayUtils.add(o, 3, "5"); @@ -520,134 +521,133 @@ public class ArrayUtilsAddTest extends AbstractLangTest { assertEquals("5", result2[3]); // boolean tests - boolean[] booleanArray = ArrayUtils.add( null, 0, true ); - assertArrayEquals(new boolean[]{true}, booleanArray); - IndexOutOfBoundsException e = - assertThrows(IndexOutOfBoundsException.class, () -> ArrayUtils.add( null, -1, true)); + boolean[] booleanArray = ArrayUtils.add(null, 0, true); + assertArrayEquals(new boolean[] { true }, booleanArray); + IndexOutOfBoundsException e = assertThrows(IndexOutOfBoundsException.class, () -> ArrayUtils.add(null, -1, true)); assertEquals("Index: -1, Length: 0", e.getMessage()); - booleanArray = ArrayUtils.add( new boolean[] { true }, 0, false); - assertArrayEquals(new boolean[]{false, true}, booleanArray); - booleanArray = ArrayUtils.add( new boolean[] { false }, 1, true); - assertArrayEquals(new boolean[]{false, true}, booleanArray); - booleanArray = ArrayUtils.add( new boolean[] { true, false }, 1, true); - assertArrayEquals(new boolean[]{true, true, false}, booleanArray); + booleanArray = ArrayUtils.add(new boolean[] { true }, 0, false); + assertArrayEquals(new boolean[] { false, true }, booleanArray); + booleanArray = ArrayUtils.add(new boolean[] { false }, 1, true); + assertArrayEquals(new boolean[] { false, true }, booleanArray); + booleanArray = ArrayUtils.add(new boolean[] { true, false }, 1, true); + assertArrayEquals(new boolean[] { true, true, false }, booleanArray); e = assertThrows(IndexOutOfBoundsException.class, () -> ArrayUtils.add(new boolean[] { true, false }, 4, true)); assertEquals("Index: 4, Length: 2", e.getMessage()); e = assertThrows(IndexOutOfBoundsException.class, () -> ArrayUtils.add(new boolean[] { true, false }, -1, true)); assertEquals("Index: -1, Length: 2", e.getMessage()); // char tests - char[] charArray = ArrayUtils.add( (char[]) null, 0, 'a' ); - assertArrayEquals(new char[]{'a'}, charArray); - e = assertThrows(IndexOutOfBoundsException.class, () -> ArrayUtils.add( (char[]) null, -1, 'a' )); + char[] charArray = ArrayUtils.add((char[]) null, 0, 'a'); + assertArrayEquals(new char[] { 'a' }, charArray); + e = assertThrows(IndexOutOfBoundsException.class, () -> ArrayUtils.add((char[]) null, -1, 'a')); assertEquals("Index: -1, Length: 0", e.getMessage()); - charArray = ArrayUtils.add( new char[] { 'a' }, 0, 'b'); - assertArrayEquals(new char[]{'b', 'a'}, charArray); - charArray = ArrayUtils.add( new char[] { 'a', 'b' }, 0, 'c'); - assertArrayEquals(new char[]{'c', 'a', 'b'}, charArray); - charArray = ArrayUtils.add( new char[] { 'a', 'b' }, 1, 'k'); - assertArrayEquals(new char[]{'a', 'k', 'b'}, charArray); - charArray = ArrayUtils.add( new char[] { 'a', 'b', 'c' }, 1, 't'); - assertArrayEquals(new char[]{'a', 't', 'b', 'c'}, charArray); - e = assertThrows(IndexOutOfBoundsException.class, () -> ArrayUtils.add( new char[] { 'a', 'b' }, 4, 'c')); + charArray = ArrayUtils.add(new char[] { 'a' }, 0, 'b'); + assertArrayEquals(new char[] { 'b', 'a' }, charArray); + charArray = ArrayUtils.add(new char[] { 'a', 'b' }, 0, 'c'); + assertArrayEquals(new char[] { 'c', 'a', 'b' }, charArray); + charArray = ArrayUtils.add(new char[] { 'a', 'b' }, 1, 'k'); + assertArrayEquals(new char[] { 'a', 'k', 'b' }, charArray); + charArray = ArrayUtils.add(new char[] { 'a', 'b', 'c' }, 1, 't'); + assertArrayEquals(new char[] { 'a', 't', 'b', 'c' }, charArray); + e = assertThrows(IndexOutOfBoundsException.class, () -> ArrayUtils.add(new char[] { 'a', 'b' }, 4, 'c')); assertEquals("Index: 4, Length: 2", e.getMessage()); - e = assertThrows(IndexOutOfBoundsException.class, () -> ArrayUtils.add( new char[] { 'a', 'b' }, -1, 'c')); + e = assertThrows(IndexOutOfBoundsException.class, () -> ArrayUtils.add(new char[] { 'a', 'b' }, -1, 'c')); assertEquals("Index: -1, Length: 2", e.getMessage()); // short tests - short[] shortArray = ArrayUtils.add( new short[] { 1 }, 0, (short) 2); - assertArrayEquals(new short[]{2, 1}, shortArray); - e = assertThrows(IndexOutOfBoundsException.class, () -> ArrayUtils.add( (short[]) null, -1, (short) 2)); + short[] shortArray = ArrayUtils.add(new short[] { 1 }, 0, (short) 2); + assertArrayEquals(new short[] { 2, 1 }, shortArray); + e = assertThrows(IndexOutOfBoundsException.class, () -> ArrayUtils.add((short[]) null, -1, (short) 2)); assertEquals("Index: -1, Length: 0", e.getMessage()); - shortArray = ArrayUtils.add( new short[] { 2, 6 }, 2, (short) 10); - assertArrayEquals(new short[]{2, 6, 10}, shortArray); - shortArray = ArrayUtils.add( new short[] { 2, 6 }, 0, (short) -4); - assertArrayEquals(new short[]{-4, 2, 6}, shortArray); - shortArray = ArrayUtils.add( new short[] { 2, 6, 3 }, 2, (short) 1); - assertArrayEquals(new short[]{2, 6, 1, 3}, shortArray); - e = assertThrows(IndexOutOfBoundsException.class, () -> ArrayUtils.add( new short[] { 2, 6 }, 4, (short) 10)); + shortArray = ArrayUtils.add(new short[] { 2, 6 }, 2, (short) 10); + assertArrayEquals(new short[] { 2, 6, 10 }, shortArray); + shortArray = ArrayUtils.add(new short[] { 2, 6 }, 0, (short) -4); + assertArrayEquals(new short[] { -4, 2, 6 }, shortArray); + shortArray = ArrayUtils.add(new short[] { 2, 6, 3 }, 2, (short) 1); + assertArrayEquals(new short[] { 2, 6, 1, 3 }, shortArray); + e = assertThrows(IndexOutOfBoundsException.class, () -> ArrayUtils.add(new short[] { 2, 6 }, 4, (short) 10)); assertEquals("Index: 4, Length: 2", e.getMessage()); - e = assertThrows(IndexOutOfBoundsException.class, () -> ArrayUtils.add( new short[] { 2, 6 }, -1, (short) 10)); + e = assertThrows(IndexOutOfBoundsException.class, () -> ArrayUtils.add(new short[] { 2, 6 }, -1, (short) 10)); assertEquals("Index: -1, Length: 2", e.getMessage()); // byte tests - byte[] byteArray = ArrayUtils.add( new byte[] { 1 }, 0, (byte) 2); - assertArrayEquals(new byte[]{2, 1}, byteArray); - e = assertThrows(IndexOutOfBoundsException.class, () -> ArrayUtils.add( (byte[]) null, -1, (byte) 2)); + byte[] byteArray = ArrayUtils.add(new byte[] { 1 }, 0, (byte) 2); + assertArrayEquals(new byte[] { 2, 1 }, byteArray); + e = assertThrows(IndexOutOfBoundsException.class, () -> ArrayUtils.add((byte[]) null, -1, (byte) 2)); assertEquals("Index: -1, Length: 0", e.getMessage()); - byteArray = ArrayUtils.add( new byte[] { 2, 6 }, 2, (byte) 3); - assertArrayEquals(new byte[]{2, 6, 3}, byteArray); - byteArray = ArrayUtils.add( new byte[] { 2, 6 }, 0, (byte) 1); - assertArrayEquals(new byte[]{1, 2, 6}, byteArray); - byteArray = ArrayUtils.add( new byte[] { 2, 6, 3 }, 2, (byte) 1); - assertArrayEquals(new byte[]{2, 6, 1, 3}, byteArray); - e = assertThrows(IndexOutOfBoundsException.class, () -> ArrayUtils.add( new byte[] { 2, 6 }, 4, (byte) 3)); + byteArray = ArrayUtils.add(new byte[] { 2, 6 }, 2, (byte) 3); + assertArrayEquals(new byte[] { 2, 6, 3 }, byteArray); + byteArray = ArrayUtils.add(new byte[] { 2, 6 }, 0, (byte) 1); + assertArrayEquals(new byte[] { 1, 2, 6 }, byteArray); + byteArray = ArrayUtils.add(new byte[] { 2, 6, 3 }, 2, (byte) 1); + assertArrayEquals(new byte[] { 2, 6, 1, 3 }, byteArray); + e = assertThrows(IndexOutOfBoundsException.class, () -> ArrayUtils.add(new byte[] { 2, 6 }, 4, (byte) 3)); assertEquals("Index: 4, Length: 2", e.getMessage()); - e = assertThrows(IndexOutOfBoundsException.class, () -> ArrayUtils.add( new byte[] { 2, 6 }, -1, (byte) 3)); + e = assertThrows(IndexOutOfBoundsException.class, () -> ArrayUtils.add(new byte[] { 2, 6 }, -1, (byte) 3)); assertEquals("Index: -1, Length: 2", e.getMessage()); // int tests - int[] intArray = ArrayUtils.add( new int[] { 1 }, 0, 2); - assertArrayEquals(new int[]{2, 1}, intArray); - e = assertThrows(IndexOutOfBoundsException.class, () -> ArrayUtils.add( (int[]) null, -1, 2)); + int[] intArray = ArrayUtils.add(new int[] { 1 }, 0, 2); + assertArrayEquals(new int[] { 2, 1 }, intArray); + e = assertThrows(IndexOutOfBoundsException.class, () -> ArrayUtils.add((int[]) null, -1, 2)); assertEquals("Index: -1, Length: 0", e.getMessage()); - intArray = ArrayUtils.add( new int[] { 2, 6 }, 2, 10); - assertArrayEquals(new int[]{2, 6, 10}, intArray); - intArray = ArrayUtils.add( new int[] { 2, 6 }, 0, -4); - assertArrayEquals(new int[]{-4, 2, 6}, intArray); - intArray = ArrayUtils.add( new int[] { 2, 6, 3 }, 2, 1); - assertArrayEquals(new int[]{2, 6, 1, 3}, intArray); - e = assertThrows(IndexOutOfBoundsException.class, () -> ArrayUtils.add( new int[] { 2, 6 }, 4, 10)); + intArray = ArrayUtils.add(new int[] { 2, 6 }, 2, 10); + assertArrayEquals(new int[] { 2, 6, 10 }, intArray); + intArray = ArrayUtils.add(new int[] { 2, 6 }, 0, -4); + assertArrayEquals(new int[] { -4, 2, 6 }, intArray); + intArray = ArrayUtils.add(new int[] { 2, 6, 3 }, 2, 1); + assertArrayEquals(new int[] { 2, 6, 1, 3 }, intArray); + e = assertThrows(IndexOutOfBoundsException.class, () -> ArrayUtils.add(new int[] { 2, 6 }, 4, 10)); assertEquals("Index: 4, Length: 2", e.getMessage()); - e = assertThrows(IndexOutOfBoundsException.class, () -> ArrayUtils.add( new int[] { 2, 6 }, -1, 10)); + e = assertThrows(IndexOutOfBoundsException.class, () -> ArrayUtils.add(new int[] { 2, 6 }, -1, 10)); assertEquals("Index: -1, Length: 2", e.getMessage()); // long tests - long[] longArray = ArrayUtils.add( new long[] { 1L }, 0, 2L); - assertArrayEquals(new long[]{2L, 1L}, longArray); - e = assertThrows(IndexOutOfBoundsException.class, () -> ArrayUtils.add( (long[]) null, -1, 2L)); + long[] longArray = ArrayUtils.add(new long[] { 1L }, 0, 2L); + assertArrayEquals(new long[] { 2L, 1L }, longArray); + e = assertThrows(IndexOutOfBoundsException.class, () -> ArrayUtils.add((long[]) null, -1, 2L)); assertEquals("Index: -1, Length: 0", e.getMessage()); - longArray = ArrayUtils.add( new long[] { 2L, 6L }, 2, 10L); - assertArrayEquals(new long[]{2L, 6L, 10L}, longArray); - longArray = ArrayUtils.add( new long[] { 2L, 6L }, 0, -4L); - assertArrayEquals(new long[]{-4L, 2L, 6L}, longArray); - longArray = ArrayUtils.add( new long[] { 2L, 6L, 3L }, 2, 1L); - assertArrayEquals(new long[]{2L, 6L, 1L, 3L}, longArray); - e = assertThrows(IndexOutOfBoundsException.class, () -> ArrayUtils.add( new long[] { 2L, 6L }, 4, 10L)); + longArray = ArrayUtils.add(new long[] { 2L, 6L }, 2, 10L); + assertArrayEquals(new long[] { 2L, 6L, 10L }, longArray); + longArray = ArrayUtils.add(new long[] { 2L, 6L }, 0, -4L); + assertArrayEquals(new long[] { -4L, 2L, 6L }, longArray); + longArray = ArrayUtils.add(new long[] { 2L, 6L, 3L }, 2, 1L); + assertArrayEquals(new long[] { 2L, 6L, 1L, 3L }, longArray); + e = assertThrows(IndexOutOfBoundsException.class, () -> ArrayUtils.add(new long[] { 2L, 6L }, 4, 10L)); assertEquals("Index: 4, Length: 2", e.getMessage()); - e = assertThrows(IndexOutOfBoundsException.class, () -> ArrayUtils.add( new long[] { 2L, 6L }, -1, 10L)); + e = assertThrows(IndexOutOfBoundsException.class, () -> ArrayUtils.add(new long[] { 2L, 6L }, -1, 10L)); assertEquals("Index: -1, Length: 2", e.getMessage()); // float tests - float[] floatArray = ArrayUtils.add( new float[] { 1.1f }, 0, 2.2f); - assertArrayEquals(new float[]{2.2f, 1.1f}, floatArray); - e = assertThrows(IndexOutOfBoundsException.class, () -> ArrayUtils.add( (float[]) null, -1, 2.2f)); + float[] floatArray = ArrayUtils.add(new float[] { 1.1f }, 0, 2.2f); + assertArrayEquals(new float[] { 2.2f, 1.1f }, floatArray); + e = assertThrows(IndexOutOfBoundsException.class, () -> ArrayUtils.add((float[]) null, -1, 2.2f)); assertEquals("Index: -1, Length: 0", e.getMessage()); - floatArray = ArrayUtils.add( new float[] { 2.3f, 6.4f }, 2, 10.5f); - assertArrayEquals(new float[]{2.3f, 6.4f, 10.5f}, floatArray); - floatArray = ArrayUtils.add( new float[] { 2.6f, 6.7f }, 0, -4.8f); - assertArrayEquals(new float[]{-4.8f, 2.6f, 6.7f}, floatArray); - floatArray = ArrayUtils.add( new float[] { 2.9f, 6.0f, 0.3f }, 2, 1.0f); - assertArrayEquals(new float[]{2.9f, 6.0f, 1.0f, 0.3f}, floatArray); - e = assertThrows(IndexOutOfBoundsException.class, () -> ArrayUtils.add( new float[] { 2.3f, 6.4f }, 4, 10.5f)); + floatArray = ArrayUtils.add(new float[] { 2.3f, 6.4f }, 2, 10.5f); + assertArrayEquals(new float[] { 2.3f, 6.4f, 10.5f }, floatArray); + floatArray = ArrayUtils.add(new float[] { 2.6f, 6.7f }, 0, -4.8f); + assertArrayEquals(new float[] { -4.8f, 2.6f, 6.7f }, floatArray); + floatArray = ArrayUtils.add(new float[] { 2.9f, 6.0f, 0.3f }, 2, 1.0f); + assertArrayEquals(new float[] { 2.9f, 6.0f, 1.0f, 0.3f }, floatArray); + e = assertThrows(IndexOutOfBoundsException.class, () -> ArrayUtils.add(new float[] { 2.3f, 6.4f }, 4, 10.5f)); assertEquals("Index: 4, Length: 2", e.getMessage()); - e = assertThrows(IndexOutOfBoundsException.class, () -> ArrayUtils.add( new float[] { 2.3f, 6.4f }, -1, 10.5f)); + e = assertThrows(IndexOutOfBoundsException.class, () -> ArrayUtils.add(new float[] { 2.3f, 6.4f }, -1, 10.5f)); assertEquals("Index: -1, Length: 2", e.getMessage()); // double tests - double[] doubleArray = ArrayUtils.add( new double[] { 1.1 }, 0, 2.2); - assertArrayEquals(new double[]{2.2, 1.1}, doubleArray); + double[] doubleArray = ArrayUtils.add(new double[] { 1.1 }, 0, 2.2); + assertArrayEquals(new double[] { 2.2, 1.1 }, doubleArray); e = assertThrows(IndexOutOfBoundsException.class, () -> ArrayUtils.add(null, -1, 2.2)); assertEquals("Index: -1, Length: 0", e.getMessage()); - doubleArray = ArrayUtils.add( new double[] { 2.3, 6.4 }, 2, 10.5); - assertArrayEquals(new double[]{2.3, 6.4, 10.5}, doubleArray); - doubleArray = ArrayUtils.add( new double[] { 2.6, 6.7 }, 0, -4.8); - assertArrayEquals(new double[]{-4.8, 2.6, 6.7}, doubleArray); - doubleArray = ArrayUtils.add( new double[] { 2.9, 6.0, 0.3 }, 2, 1.0); - assertArrayEquals(new double[]{2.9, 6.0, 1.0, 0.3}, doubleArray); - e = assertThrows(IndexOutOfBoundsException.class, () -> ArrayUtils.add( new double[] { 2.3, 6.4 }, 4, 10.5)); + doubleArray = ArrayUtils.add(new double[] { 2.3, 6.4 }, 2, 10.5); + assertArrayEquals(new double[] { 2.3, 6.4, 10.5 }, doubleArray); + doubleArray = ArrayUtils.add(new double[] { 2.6, 6.7 }, 0, -4.8); + assertArrayEquals(new double[] { -4.8, 2.6, 6.7 }, doubleArray); + doubleArray = ArrayUtils.add(new double[] { 2.9, 6.0, 0.3 }, 2, 1.0); + assertArrayEquals(new double[] { 2.9, 6.0, 1.0, 0.3 }, doubleArray); + e = assertThrows(IndexOutOfBoundsException.class, () -> ArrayUtils.add(new double[] { 2.3, 6.4 }, 4, 10.5)); assertEquals("Index: 4, Length: 2", e.getMessage()); - e = assertThrows(IndexOutOfBoundsException.class, () -> ArrayUtils.add( new double[] { 2.3, 6.4 }, -1, 10.5)); + e = assertThrows(IndexOutOfBoundsException.class, () -> ArrayUtils.add(new double[] { 2.3, 6.4 }, -1, 10.5)); assertEquals("Index: -1, Length: 2", e.getMessage()); } @@ -655,19 +655,18 @@ public class ArrayUtilsAddTest extends AbstractLangTest { public void testJira567() { final Number[] n; // Valid array construction - n = ArrayUtils.addAll(new Number[]{Integer.valueOf(1)}, new Long[]{Long.valueOf(2)}); + n = ArrayUtils.addAll(new Number[] { Integer.valueOf(1) }, new Long[] { Long.valueOf(2) }); assertEquals(2, n.length); assertEquals(Number.class, n.getClass().getComponentType()); // Invalid - can't store Long in Integer array - assertThrows(IllegalArgumentException.class, - () -> ArrayUtils.addAll(new Integer[]{Integer.valueOf(1)}, new Long[]{Long.valueOf(2)})); + assertThrows(IllegalArgumentException.class, () -> ArrayUtils.addAll(new Integer[] { Integer.valueOf(1) }, new Long[] { Long.valueOf(2) })); } @Test @SuppressWarnings("deprecation") public void testLANG571() { - final String[] stringArray=null; - final String aString=null; + final String[] stringArray = null; + final String aString = null; assertThrows(IllegalArgumentException.class, () -> ArrayUtils.add(stringArray, aString)); assertThrows(IllegalArgumentException.class, () -> ArrayUtils.add(stringArray, 0, aString)); } diff --git a/src/test/java/org/apache/commons/lang3/ArrayUtilsTest.java b/src/test/java/org/apache/commons/lang3/ArrayUtilsTest.java index cf3a2f176..017d4479b 100644 --- a/src/test/java/org/apache/commons/lang3/ArrayUtilsTest.java +++ b/src/test/java/org/apache/commons/lang3/ArrayUtilsTest.java @@ -5501,37 +5501,31 @@ public class ArrayUtilsTest extends AbstractLangTest { @Test public void testSubarrChar() { final char[] nullArray = null; - final char[] array = {'a', 'b', 'c', 'd', 'e', 'f'}; - final char[] leftSubarray = {'a', 'b', 'c', 'd'}; - final char[] midSubarray = {'b', 'c', 'd', 'e'}; - final char[] rightSubarray = {'c', 'd', 'e', 'f'}; + final char[] array = { 'a', 'b', 'c', 'd', 'e', 'f' }; + final char[] leftSubarray = { 'a', 'b', 'c', 'd' }; + final char[] midSubarray = { 'b', 'c', 'd', 'e' }; + final char[] rightSubarray = { 'c', 'd', 'e', 'f' }; assertTrue(ArrayUtils.isEquals(leftSubarray, ArrayUtils.subarray(array, 0, 4)), "0 start, mid end"); assertTrue(ArrayUtils.isEquals(array, ArrayUtils.subarray(array, 0, array.length)), "0 start, length end"); assertTrue(ArrayUtils.isEquals(midSubarray, ArrayUtils.subarray(array, 1, 5)), "mid start, mid end"); - assertTrue(ArrayUtils.isEquals(rightSubarray, ArrayUtils.subarray(array, 2, array.length)), - "mid start, length end"); + assertTrue(ArrayUtils.isEquals(rightSubarray, ArrayUtils.subarray(array, 2, array.length)), "mid start, length end"); assertNull(ArrayUtils.subarray(nullArray, 0, 3), "null input"); - assertEquals(ArrayUtils.EMPTY_CHAR_ARRAY, ArrayUtils.subarray(ArrayUtils.EMPTY_CHAR_ARRAY, 1, 2), - "empty array"); + assertEquals(ArrayUtils.EMPTY_CHAR_ARRAY, ArrayUtils.subarray(ArrayUtils.EMPTY_CHAR_ARRAY, 1, 2), "empty array"); assertEquals(ArrayUtils.EMPTY_CHAR_ARRAY, ArrayUtils.subarray(array, 4, 2), "start > end"); assertEquals(ArrayUtils.EMPTY_CHAR_ARRAY, ArrayUtils.subarray(array, 3, 3), "start == end"); - assertTrue(ArrayUtils.isEquals(leftSubarray, ArrayUtils.subarray(array, -2, 4)), - "start undershoot, normal end"); + assertTrue(ArrayUtils.isEquals(leftSubarray, ArrayUtils.subarray(array, -2, 4)), "start undershoot, normal end"); assertEquals(ArrayUtils.EMPTY_CHAR_ARRAY, ArrayUtils.subarray(array, 33, 4), "start overshoot, any end"); - assertTrue(ArrayUtils.isEquals(rightSubarray, ArrayUtils.subarray(array, 2, 33)), - "normal start, end overshoot"); + assertTrue(ArrayUtils.isEquals(rightSubarray, ArrayUtils.subarray(array, 2, 33)), "normal start, end overshoot"); assertTrue(ArrayUtils.isEquals(array, ArrayUtils.subarray(array, -2, 12)), "start undershoot, end overshoot"); // empty-return tests - assertSame(ArrayUtils.EMPTY_CHAR_ARRAY, ArrayUtils.subarray(ArrayUtils.EMPTY_CHAR_ARRAY, 1, 2), - "empty array, object test"); + assertSame(ArrayUtils.EMPTY_CHAR_ARRAY, ArrayUtils.subarray(ArrayUtils.EMPTY_CHAR_ARRAY, 1, 2), "empty array, object test"); assertSame(ArrayUtils.EMPTY_CHAR_ARRAY, ArrayUtils.subarray(array, 4, 1), "start > end, object test"); assertSame(ArrayUtils.EMPTY_CHAR_ARRAY, ArrayUtils.subarray(array, 3, 3), "start == end, object test"); - assertSame(ArrayUtils.EMPTY_CHAR_ARRAY, ArrayUtils.subarray(array, 8733, 4), - "start overshoot, any end, object test"); + assertSame(ArrayUtils.EMPTY_CHAR_ARRAY, ArrayUtils.subarray(array, 8733, 4), "start overshoot, any end, object test"); // array type tests @@ -5540,7 +5534,7 @@ public class ArrayUtilsTest extends AbstractLangTest { @Test public void testSwapBoolean() { - final boolean[] array = {true, false, false}; + final boolean[] array = { true, false, false }; ArrayUtils.swap(array, 0, 2); assertFalse(array[0]); assertFalse(array[1]); @@ -5549,38 +5543,38 @@ public class ArrayUtilsTest extends AbstractLangTest { @Test public void testSwapBooleanRange() { - boolean[] array = {false, false, true, true}; + boolean[] array = { false, false, true, true }; ArrayUtils.swap(array, 0, 2, 2); assertTrue(array[0]); assertTrue(array[1]); assertFalse(array[2]); assertFalse(array[3]); - array = new boolean[]{false, true, false}; + array = new boolean[] { false, true, false }; ArrayUtils.swap(array, 0, 3); assertFalse(array[0]); assertTrue(array[1]); assertFalse(array[2]); - array = new boolean[]{true, true, false}; + array = new boolean[] { true, true, false }; ArrayUtils.swap(array, 0, 2, 2); assertFalse(array[0]); assertTrue(array[1]); assertTrue(array[2]); - array = new boolean[]{true, true, false}; + array = new boolean[] { true, true, false }; ArrayUtils.swap(array, -1, 2, 2); assertFalse(array[0]); assertTrue(array[1]); assertTrue(array[2]); - array = new boolean[]{true, true, false}; + array = new boolean[] { true, true, false }; ArrayUtils.swap(array, 0, -1, 2); assertTrue(array[0]); assertTrue(array[1]); assertFalse(array[2]); - array = new boolean[]{true, true, false}; + array = new boolean[] { true, true, false }; ArrayUtils.swap(array, -1, -1, 2); assertTrue(array[0]); assertTrue(array[1]); @@ -5589,7 +5583,7 @@ public class ArrayUtilsTest extends AbstractLangTest { @Test public void testSwapByte() { - final byte[] array = {1, 2, 3}; + final byte[] array = { 1, 2, 3 }; ArrayUtils.swap(array, 0, 2); assertEquals(3, array[0]); assertEquals(2, array[1]); @@ -5598,38 +5592,38 @@ public class ArrayUtilsTest extends AbstractLangTest { @Test public void testSwapByteRange() { - byte[] array = {1, 2, 3, 4}; + byte[] array = { 1, 2, 3, 4 }; ArrayUtils.swap(array, 0, 2, 2); assertEquals(3, array[0]); assertEquals(4, array[1]); assertEquals(1, array[2]); assertEquals(2, array[3]); - array = new byte[]{1, 2, 3}; + array = new byte[] { 1, 2, 3 }; ArrayUtils.swap(array, 0, 3); assertEquals(1, array[0]); assertEquals(2, array[1]); assertEquals(3, array[2]); - array = new byte[]{1, 2, 3}; + array = new byte[] { 1, 2, 3 }; ArrayUtils.swap(array, 0, 2, 2); assertEquals(3, array[0]); assertEquals(2, array[1]); assertEquals(1, array[2]); - array = new byte[]{1, 2, 3}; + array = new byte[] { 1, 2, 3 }; ArrayUtils.swap(array, -1, 2, 2); assertEquals(3, array[0]); assertEquals(2, array[1]); assertEquals(1, array[2]); - array = new byte[]{1, 2, 3}; + array = new byte[] { 1, 2, 3 }; ArrayUtils.swap(array, 0, -1, 2); assertEquals(1, array[0]); assertEquals(2, array[1]); assertEquals(3, array[2]); - array = new byte[]{1, 2, 3}; + array = new byte[] { 1, 2, 3 }; ArrayUtils.swap(array, -1, -1, 2); assertEquals(1, array[0]); assertEquals(2, array[1]); @@ -5638,53 +5632,53 @@ public class ArrayUtilsTest extends AbstractLangTest { @Test public void testSwapChar() { - char[] array = {1, 2, 3}; + char[] array = { 1, 2, 3 }; ArrayUtils.swap(array, 0, 2); - assertArrayEquals(new char[]{3, 2, 1}, array); + assertArrayEquals(new char[] { 3, 2, 1 }, array); - array = new char[]{1, 2, 3}; + array = new char[] { 1, 2, 3 }; ArrayUtils.swap(array, 0, 0); - assertArrayEquals(new char[]{1, 2, 3}, array); + assertArrayEquals(new char[] { 1, 2, 3 }, array); - array = new char[]{1, 2, 3}; + array = new char[] { 1, 2, 3 }; ArrayUtils.swap(array, 1, 0); - assertArrayEquals(new char[]{2, 1, 3}, array); + assertArrayEquals(new char[] { 2, 1, 3 }, array); } @Test public void testSwapCharRange() { - char[] array = {1, 2, 3, 4}; + char[] array = { 1, 2, 3, 4 }; ArrayUtils.swap(array, 0, 2, 2); assertEquals(3, array[0]); assertEquals(4, array[1]); assertEquals(1, array[2]); assertEquals(2, array[3]); - array = new char[]{1, 2, 3}; + array = new char[] { 1, 2, 3 }; ArrayUtils.swap(array, 0, 3); assertEquals(1, array[0]); assertEquals(2, array[1]); assertEquals(3, array[2]); - array = new char[]{1, 2, 3}; + array = new char[] { 1, 2, 3 }; ArrayUtils.swap(array, 0, 2, 2); assertEquals(3, array[0]); assertEquals(2, array[1]); assertEquals(1, array[2]); - array = new char[]{1, 2, 3}; + array = new char[] { 1, 2, 3 }; ArrayUtils.swap(array, -1, 2, 2); assertEquals(3, array[0]); assertEquals(2, array[1]); assertEquals(1, array[2]); - array = new char[]{1, 2, 3}; + array = new char[] { 1, 2, 3 }; ArrayUtils.swap(array, 0, -1, 2); assertEquals(1, array[0]); assertEquals(2, array[1]); assertEquals(3, array[2]); - array = new char[]{1, 2, 3}; + array = new char[] { 1, 2, 3 }; ArrayUtils.swap(array, -1, -1, 2); assertEquals(1, array[0]); assertEquals(2, array[1]); @@ -5693,7 +5687,7 @@ public class ArrayUtilsTest extends AbstractLangTest { @Test public void testSwapDouble() { - final double[] array = {1, 2, 3}; + final double[] array = { 1, 2, 3 }; ArrayUtils.swap(array, 0, 2); assertEquals(3, array[0]); assertEquals(2, array[1]); @@ -5702,38 +5696,38 @@ public class ArrayUtilsTest extends AbstractLangTest { @Test public void testSwapDoubleRange() { - double[] array = {1, 2, 3, 4}; + double[] array = { 1, 2, 3, 4 }; ArrayUtils.swap(array, 0, 2, 2); assertEquals(3, array[0]); assertEquals(4, array[1]); assertEquals(1, array[2]); assertEquals(2, array[3]); - array = new double[]{1, 2, 3}; + array = new double[] { 1, 2, 3 }; ArrayUtils.swap(array, 0, 3); assertEquals(1, array[0]); assertEquals(2, array[1]); assertEquals(3, array[2]); - array = new double[]{1, 2, 3}; + array = new double[] { 1, 2, 3 }; ArrayUtils.swap(array, 0, 2, 2); assertEquals(3, array[0]); assertEquals(2, array[1]); assertEquals(1, array[2]); - array = new double[]{1, 2, 3}; + array = new double[] { 1, 2, 3 }; ArrayUtils.swap(array, -1, 2, 2); assertEquals(3, array[0]); assertEquals(2, array[1]); assertEquals(1, array[2]); - array = new double[]{1, 2, 3}; + array = new double[] { 1, 2, 3 }; ArrayUtils.swap(array, 0, -1, 2); assertEquals(1, array[0]); assertEquals(2, array[1]); assertEquals(3, array[2]); - array = new double[]{1, 2, 3}; + array = new double[] { 1, 2, 3 }; ArrayUtils.swap(array, -1, -1, 2); assertEquals(1, array[0]); assertEquals(2, array[1]); @@ -5805,7 +5799,7 @@ public class ArrayUtilsTest extends AbstractLangTest { @Test public void testSwapFloat() { - final float[] array = {1, 2, 3}; + final float[] array = { 1, 2, 3 }; ArrayUtils.swap(array, 0, 2); assertEquals(3, array[0]); assertEquals(2, array[1]); @@ -5814,38 +5808,38 @@ public class ArrayUtilsTest extends AbstractLangTest { @Test public void testSwapFloatRange() { - float[] array = {1, 2, 3, 4}; + float[] array = { 1, 2, 3, 4 }; ArrayUtils.swap(array, 0, 2, 2); assertEquals(3, array[0]); assertEquals(4, array[1]); assertEquals(1, array[2]); assertEquals(2, array[3]); - array = new float[]{1, 2, 3}; + array = new float[] { 1, 2, 3 }; ArrayUtils.swap(array, 0, 3); assertEquals(1, array[0]); assertEquals(2, array[1]); assertEquals(3, array[2]); - array = new float[]{1, 2, 3}; + array = new float[] { 1, 2, 3 }; ArrayUtils.swap(array, 0, 2, 2); assertEquals(3, array[0]); assertEquals(2, array[1]); assertEquals(1, array[2]); - array = new float[]{1, 2, 3}; + array = new float[] { 1, 2, 3 }; ArrayUtils.swap(array, -1, 2, 2); assertEquals(3, array[0]); assertEquals(2, array[1]); assertEquals(1, array[2]); - array = new float[]{1, 2, 3}; + array = new float[] { 1, 2, 3 }; ArrayUtils.swap(array, 0, -1, 2); assertEquals(1, array[0]); assertEquals(2, array[1]); assertEquals(3, array[2]); - array = new float[]{1, 2, 3}; + array = new float[] { 1, 2, 3 }; ArrayUtils.swap(array, -1, -1, 2); assertEquals(1, array[0]); assertEquals(2, array[1]); @@ -5854,7 +5848,7 @@ public class ArrayUtilsTest extends AbstractLangTest { @Test public void testSwapInt() { - final int[] array = {1, 2, 3}; + final int[] array = { 1, 2, 3 }; ArrayUtils.swap(array, 0, 2); assertEquals(3, array[0]); assertEquals(2, array[1]); @@ -5864,49 +5858,49 @@ public class ArrayUtilsTest extends AbstractLangTest { @Test public void testSwapIntExchangedOffsets() { int[] array; - array = new int[]{1, 2, 3}; + array = new int[] { 1, 2, 3 }; ArrayUtils.swap(array, 0, 1, 2); - assertArrayEquals(new int[]{2, 3, 1}, array); + assertArrayEquals(new int[] { 2, 3, 1 }, array); - array = new int[]{1, 2, 3}; + array = new int[] { 1, 2, 3 }; ArrayUtils.swap(array, 1, 0, 2); - assertArrayEquals(new int[]{2, 3, 1}, array); + assertArrayEquals(new int[] { 2, 3, 1 }, array); } @Test public void testSwapIntRange() { - int[] array = {1, 2, 3, 4}; + int[] array = { 1, 2, 3, 4 }; ArrayUtils.swap(array, 0, 2, 2); assertEquals(3, array[0]); assertEquals(4, array[1]); assertEquals(1, array[2]); assertEquals(2, array[3]); - array = new int[]{1, 2, 3}; + array = new int[] { 1, 2, 3 }; ArrayUtils.swap(array, 3, 0); assertEquals(1, array[0]); assertEquals(2, array[1]); assertEquals(3, array[2]); - array = new int[]{1, 2, 3}; + array = new int[] { 1, 2, 3 }; ArrayUtils.swap(array, 0, 2, 2); assertEquals(3, array[0]); assertEquals(2, array[1]); assertEquals(1, array[2]); - array = new int[]{1, 2, 3}; + array = new int[] { 1, 2, 3 }; ArrayUtils.swap(array, -1, 2, 2); assertEquals(3, array[0]); assertEquals(2, array[1]); assertEquals(1, array[2]); - array = new int[]{1, 2, 3}; + array = new int[] { 1, 2, 3 }; ArrayUtils.swap(array, 0, -1, 2); assertEquals(1, array[0]); assertEquals(2, array[1]); assertEquals(3, array[2]); - array = new int[]{1, 2, 3}; + array = new int[] { 1, 2, 3 }; ArrayUtils.swap(array, -1, -1, 2); assertEquals(1, array[0]); assertEquals(2, array[1]); @@ -5915,7 +5909,7 @@ public class ArrayUtilsTest extends AbstractLangTest { @Test public void testSwapLong() { - final long[] array = {1, 2, 3}; + final long[] array = { 1, 2, 3 }; ArrayUtils.swap(array, 0, 2); assertEquals(3, array[0]); assertEquals(2, array[1]); @@ -5924,38 +5918,38 @@ public class ArrayUtilsTest extends AbstractLangTest { @Test public void testSwapLongRange() { - long[] array = {1, 2, 3, 4}; + long[] array = { 1, 2, 3, 4 }; ArrayUtils.swap(array, 0, 2, 2); assertEquals(3, array[0]); assertEquals(4, array[1]); assertEquals(1, array[2]); assertEquals(2, array[3]); - array = new long[]{1, 2, 3}; + array = new long[] { 1, 2, 3 }; ArrayUtils.swap(array, 0, 3); assertEquals(1, array[0]); assertEquals(2, array[1]); assertEquals(3, array[2]); - array = new long[]{1, 2, 3}; + array = new long[] { 1, 2, 3 }; ArrayUtils.swap(array, 0, 2, 2); assertEquals(3, array[0]); assertEquals(2, array[1]); assertEquals(1, array[2]); - array = new long[]{1, 2, 3}; + array = new long[] { 1, 2, 3 }; ArrayUtils.swap(array, -1, 2, 2); assertEquals(3, array[0]); assertEquals(2, array[1]); assertEquals(1, array[2]); - array = new long[]{1, 2, 3}; + array = new long[] { 1, 2, 3 }; ArrayUtils.swap(array, 0, -1, 2); assertEquals(1, array[0]); assertEquals(2, array[1]); assertEquals(3, array[2]); - array = new long[]{1, 2, 3}; + array = new long[] { 1, 2, 3 }; ArrayUtils.swap(array, -1, -1, 2); assertEquals(1, array[0]); assertEquals(2, array[1]); @@ -6027,7 +6021,7 @@ public class ArrayUtilsTest extends AbstractLangTest { @Test public void testSwapObject() { - final String[] array = {"1", "2", "3"}; + final String[] array = { "1", "2", "3" }; ArrayUtils.swap(array, 0, 2); assertEquals("3", array[0]); assertEquals("2", array[1]); @@ -6036,21 +6030,21 @@ public class ArrayUtilsTest extends AbstractLangTest { @Test public void testSwapObjectRange() { - String[] array = {"1", "2", "3", "4"}; + String[] array = { "1", "2", "3", "4" }; ArrayUtils.swap(array, 0, 2, 2); assertEquals("3", array[0]); assertEquals("4", array[1]); assertEquals("1", array[2]); assertEquals("2", array[3]); - array = new String[]{"1", "2", "3", "4"}; + array = new String[] { "1", "2", "3", "4" }; ArrayUtils.swap(array, -1, 2, 3); assertEquals("3", array[0]); assertEquals("4", array[1]); assertEquals("1", array[2]); assertEquals("2", array[3]); - array = new String[]{"1", "2", "3", "4", "5"}; + array = new String[] { "1", "2", "3", "4", "5" }; ArrayUtils.swap(array, -3, 2, 3); assertEquals("3", array[0]); assertEquals("4", array[1]); @@ -6058,7 +6052,7 @@ public class ArrayUtilsTest extends AbstractLangTest { assertEquals("2", array[3]); assertEquals("1", array[4]); - array = new String[]{"1", "2", "3", "4", "5"}; + array = new String[] { "1", "2", "3", "4", "5" }; ArrayUtils.swap(array, 2, -2, 3); assertEquals("3", array[0]); assertEquals("4", array[1]); @@ -6077,7 +6071,7 @@ public class ArrayUtilsTest extends AbstractLangTest { @Test public void testSwapShort() { - final short[] array = {1, 2, 3}; + final short[] array = { 1, 2, 3 }; ArrayUtils.swap(array, 0, 2); assertEquals(3, array[0]); assertEquals(2, array[1]); @@ -6086,38 +6080,38 @@ public class ArrayUtilsTest extends AbstractLangTest { @Test public void testSwapShortRange() { - short[] array = {1, 2, 3, 4}; + short[] array = { 1, 2, 3, 4 }; ArrayUtils.swap(array, 0, 2, 2); assertEquals(3, array[0]); assertEquals(4, array[1]); assertEquals(1, array[2]); assertEquals(2, array[3]); - array = new short[]{1, 2, 3}; + array = new short[] { 1, 2, 3 }; ArrayUtils.swap(array, 3, 0); assertEquals(1, array[0]); assertEquals(2, array[1]); assertEquals(3, array[2]); - array = new short[]{1, 2, 3}; + array = new short[] { 1, 2, 3 }; ArrayUtils.swap(array, 0, 2, 2); assertEquals(3, array[0]); assertEquals(2, array[1]); assertEquals(1, array[2]); - array = new short[]{1, 2, 3}; + array = new short[] { 1, 2, 3 }; ArrayUtils.swap(array, -1, 2, 2); assertEquals(3, array[0]); assertEquals(2, array[1]); assertEquals(1, array[2]); - array = new short[]{1, 2, 3}; + array = new short[] { 1, 2, 3 }; ArrayUtils.swap(array, 0, -1, 2); assertEquals(1, array[0]); assertEquals(2, array[1]); assertEquals(3, array[2]); - array = new short[]{1, 2, 3}; + array = new short[] { 1, 2, 3 }; ArrayUtils.swap(array, -1, -1, 2); assertEquals(1, array[0]); assertEquals(2, array[1]); @@ -6130,7 +6124,7 @@ public class ArrayUtilsTest extends AbstractLangTest { final BitSet emptySet = new BitSet(); final BitSet testSet = new BitSet(); assertEquals(emptySet, ArrayUtils.indexesOf(array, 0)); - array = new int[]{0, 1, 2, 3, 0}; + array = new int[] { 0, 1, 2, 3, 0 }; testSet.set(0); testSet.set(4); assertEquals(testSet, ArrayUtils.indexesOf(array, 0)); @@ -6148,20 +6142,17 @@ public class ArrayUtilsTest extends AbstractLangTest { @Test public void testToMap() { - Map<?, ?> map = ArrayUtils.toMap(new String[][]{{"foo", "bar"}, {"hello", "world"}}); + Map<?, ?> map = ArrayUtils.toMap(new String[][] { { "foo", "bar" }, { "hello", "world" } }); assertEquals("bar", map.get("foo")); assertEquals("world", map.get("hello")); assertNull(ArrayUtils.toMap(null)); - assertThrows(IllegalArgumentException.class, () -> - ArrayUtils.toMap(new String[][]{{"foo", "bar"}, {"short"}})); - assertThrows(IllegalArgumentException.class, () -> - ArrayUtils.toMap(new Object[]{new Object[]{"foo", "bar"}, "illegal type"})); - assertThrows(IllegalArgumentException.class, () -> - ArrayUtils.toMap(new Object[]{new Object[]{"foo", "bar"}, null})); - - map = ArrayUtils.toMap(new Object[]{new Map.Entry<Object, Object>() { + assertThrows(IllegalArgumentException.class, () -> ArrayUtils.toMap(new String[][] { { "foo", "bar" }, { "short" } })); + assertThrows(IllegalArgumentException.class, () -> ArrayUtils.toMap(new Object[] { new Object[] { "foo", "bar" }, "illegal type" })); + assertThrows(IllegalArgumentException.class, () -> ArrayUtils.toMap(new Object[] { new Object[] { "foo", "bar" }, null })); + + map = ArrayUtils.toMap(new Object[] { new Map.Entry<Object, Object>() { @Override public boolean equals(final Object o) { throw new UnsupportedOperationException(); @@ -6186,18 +6177,18 @@ public class ArrayUtilsTest extends AbstractLangTest { public Object setValue(final Object value) { throw new UnsupportedOperationException(); } - }}); + } }); assertEquals("bar", map.get("foo")); // Return empty map when got input array with length = 0 assertEquals(Collections.emptyMap(), ArrayUtils.toMap(new Object[0])); // Test all null values - map = ArrayUtils.toMap(new Object[][] { {null, null}, {null, null} }); + map = ArrayUtils.toMap(new Object[][] { { null, null }, { null, null } }); assertEquals(Collections.singletonMap(null, null), map); // Test duplicate keys - map = ArrayUtils.toMap(new Object[][] { {"key", "value2"}, {"key", "value1"} }); + map = ArrayUtils.toMap(new Object[][] { { "key", "value2" }, { "key", "value1" } }); assertEquals(Collections.singletonMap("key", "value1"), map); } diff --git a/src/test/java/org/apache/commons/lang3/CharSequenceUtilsTest.java b/src/test/java/org/apache/commons/lang3/CharSequenceUtilsTest.java index e9022a465..2aa195335 100644 --- a/src/test/java/org/apache/commons/lang3/CharSequenceUtilsTest.java +++ b/src/test/java/org/apache/commons/lang3/CharSequenceUtilsTest.java @@ -56,7 +56,7 @@ public class CharSequenceUtilsTest extends AbstractLangTest { } - static class TestData{ + static class TestData { final String source; final boolean ignoreCase; final int toffset; @@ -65,8 +65,9 @@ public class CharSequenceUtilsTest extends AbstractLangTest { final int len; final boolean expected; final Class<? extends Throwable> throwable; - TestData(final String source, final boolean ignoreCase, final int toffset, - final String other, final int ooffset, final int len, final boolean expected) { + + TestData(final String source, final boolean ignoreCase, final int toffset, final String other, final int ooffset, final int len, + final boolean expected) { this.source = source; this.ignoreCase = ignoreCase; this.toffset = toffset; @@ -76,8 +77,9 @@ public class CharSequenceUtilsTest extends AbstractLangTest { this.expected = expected; this.throwable = null; } - TestData(final String source, final boolean ignoreCase, final int toffset, - final String other, final int ooffset, final int len, final Class<? extends Throwable> throwable) { + + TestData(final String source, final boolean ignoreCase, final int toffset, final String other, final int ooffset, final int len, + final Class<? extends Throwable> throwable) { this.source = source; this.ignoreCase = ignoreCase; this.toffset = toffset; @@ -87,11 +89,12 @@ public class CharSequenceUtilsTest extends AbstractLangTest { this.expected = false; this.throwable = throwable; } + @Override public String toString() { final StringBuilder sb = new StringBuilder(); sb.append(source).append("[").append(toffset).append("]"); - sb.append(ignoreCase? " caseblind ":" samecase "); + sb.append(ignoreCase ? " caseblind " : " samecase "); sb.append(other).append("[").append(ooffset).append("]"); sb.append(" ").append(len).append(" => "); if (throwable != null) { @@ -142,23 +145,26 @@ public class CharSequenceUtilsTest extends AbstractLangTest { } private static final TestData[] TEST_DATA = { - // Source IgnoreCase Offset Other Offset Length Result - new TestData("", true, -1, "", -1, -1, false), - new TestData("", true, 0, "", 0, 1, false), - new TestData("a", true, 0, "abc", 0, 0, true), - new TestData("a", true, 0, "abc", 0, 1, true), - new TestData("a", true, 0, null, 0, 0, NullPointerException.class), - new TestData(null, true, 0, null, 0, 0, NullPointerException.class), - new TestData(null, true, 0, "", 0, 0, NullPointerException.class), - new TestData("Abc", true, 0, "abc", 0, 3, true), - new TestData("Abc", false, 0, "abc", 0, 3, false), - new TestData("Abc", true, 1, "abc", 1, 2, true), - new TestData("Abc", false, 1, "abc", 1, 2, true), - new TestData("Abcd", true, 1, "abcD", 1, 2, true), - new TestData("Abcd", false, 1, "abcD", 1, 2, true), + // @formatter:off + // Source IgnoreCase Offset Other Offset Length Result + new TestData("", true, -1, "", -1, -1, false), + new TestData("", true, 0, "", 0, 1, false), + new TestData("a", true, 0, "abc", 0, 0, true), + new TestData("a", true, 0, "abc", 0, 1, true), + new TestData("a", true, 0, null, 0, 0, NullPointerException.class), + new TestData(null, true, 0, null, 0, 0, NullPointerException.class), + new TestData(null, true, 0, "", 0, 0, NullPointerException.class), + new TestData("Abc", true, 0, "abc", 0, 3, true), + new TestData("Abc", false, 0, "abc", 0, 3, false), + new TestData("Abc", true, 1, "abc", 1, 2, true), + new TestData("Abc", false, 1, "abc", 1, 2, true), + new TestData("Abcd", true, 1, "abcD", 1, 2, true), + new TestData("Abcd", false, 1, "abcD", 1, 2, true), + // @formatter:on }; static Stream<Arguments> lastIndexWithStandardCharSequence() { + // @formatter:off return Stream.of( arguments("abc", "b", 2, 1), arguments(new StringBuilder("abc"), "b", 2, 1), @@ -168,6 +174,7 @@ public class CharSequenceUtilsTest extends AbstractLangTest { arguments(new StringBuffer("abc"), new StringBuffer("b"), 2, 1), arguments(new StringBuilder("abc"), new StringBuffer("b"), 2, 1) ); + // @formatter:on } @Test @@ -182,8 +189,7 @@ public class CharSequenceUtilsTest extends AbstractLangTest { @ParameterizedTest @MethodSource("lastIndexWithStandardCharSequence") - public void testLastIndexOfWithDifferentCharSequences(final CharSequence cs, final CharSequence search, final int start, - final int expected) { + public void testLastIndexOfWithDifferentCharSequences(final CharSequence cs, final CharSequence search, final int start, final int expected) { assertEquals(expected, CharSequenceUtils.lastIndexOf(cs, search, start)); } @@ -240,11 +246,9 @@ public class CharSequenceUtilsTest extends AbstractLangTest { } private void testNewLastIndexOfSingleSingle(final CharSequence a, final CharSequence b, final int start) { - assertEquals( - a.toString().lastIndexOf(b.toString(), start), + assertEquals(a.toString().lastIndexOf(b.toString(), start), CharSequenceUtils.lastIndexOf(new WrapperString(a.toString()), new WrapperString(b.toString()), start), - "testNewLastIndexOf fails! original : " + a + " seg : " + b + " start : " + start - ); + "testNewLastIndexOf fails! original : " + a + " seg : " + b + " start : " + start); } @Test diff --git a/src/test/java/org/apache/commons/lang3/HashSetvBitSetTest.java b/src/test/java/org/apache/commons/lang3/HashSetvBitSetTest.java index a22195f8a..0d3f25553 100644 --- a/src/test/java/org/apache/commons/lang3/HashSetvBitSetTest.java +++ b/src/test/java/org/apache/commons/lang3/HashSetvBitSetTest.java @@ -40,8 +40,8 @@ public class HashSetvBitSetTest extends AbstractLangTest { private static int[] extractIndices(final BitSet coll) { final int[] result = new int[coll.cardinality()]; int i = 0; - int j=0; - while ((j=coll.nextSetBit(j)) != -1) { + int j = 0; + while ((j = coll.nextSetBit(j)) != -1) { result[i++] = j++; } return result; diff --git a/src/test/java/org/apache/commons/lang3/RandomStringUtilsTest.java b/src/test/java/org/apache/commons/lang3/RandomStringUtilsTest.java index c0b18783a..b4b8d6cae 100644 --- a/src/test/java/org/apache/commons/lang3/RandomStringUtilsTest.java +++ b/src/test/java/org/apache/commons/lang3/RandomStringUtilsTest.java @@ -46,6 +46,7 @@ public class RandomStringUtilsTest extends AbstractLangTest { /** * Computes Chi-Square statistic given observed and expected counts + * * @param observed array of observed frequency counts * @param expected array of expected frequency counts */ @@ -59,9 +60,7 @@ public class RandomStringUtilsTest extends AbstractLangTest { } /** - * Test for LANG-1286. Creates situation where old code would - * overflow a char and result in a code point outside the specified - * range. + * Test for LANG-1286. Creates situation where old code would overflow a char and result in a code point outside the specified range. */ @Test public void testCharOverflow() { @@ -69,8 +68,7 @@ public class RandomStringUtilsTest extends AbstractLangTest { final int end = Integer.MAX_VALUE; @SuppressWarnings("serial") - final - Random fixedRandom = new Random() { + final Random fixedRandom = new Random() { @Override public int nextInt(final int n) { // Prevents selection of 'start' as the character @@ -95,7 +93,7 @@ public class RandomStringUtilsTest extends AbstractLangTest { @Test public void testExceptions() { - final char[] DUMMY = {'a'}; // valid char array + final char[] DUMMY = { 'a' }; // valid char array assertThrows(IllegalArgumentException.class, () -> RandomStringUtils.random(-1)); assertThrows(IllegalArgumentException.class, () -> RandomStringUtils.random(-1, true, true)); assertThrows(IllegalArgumentException.class, () -> RandomStringUtils.random(-1, DUMMY)); @@ -104,9 +102,7 @@ public class RandomStringUtilsTest extends AbstractLangTest { assertThrows(IllegalArgumentException.class, () -> RandomStringUtils.random(-1, (String) null)); assertThrows(IllegalArgumentException.class, () -> RandomStringUtils.random(-1, 'a', 'z', false, false)); assertThrows(IllegalArgumentException.class, () -> RandomStringUtils.random(-1, 'a', 'z', false, false, DUMMY)); - assertThrows( - IllegalArgumentException.class, - () -> RandomStringUtils.random(-1, 'a', 'z', false, false, DUMMY, new Random())); + assertThrows(IllegalArgumentException.class, () -> RandomStringUtils.random(-1, 'a', 'z', false, false, DUMMY, new Random())); assertThrows(IllegalArgumentException.class, () -> RandomStringUtils.random(8, 32, 48, false, true)); assertThrows(IllegalArgumentException.class, () -> RandomStringUtils.random(8, 32, 65, true, false)); assertThrows(IllegalArgumentException.class, () -> RandomStringUtils.randomAlphabetic(-1)); @@ -117,8 +113,7 @@ public class RandomStringUtilsTest extends AbstractLangTest { } /** - * Checks if the string got by {@link RandomStringUtils#random(int)} - * can be converted to UTF-8 and back without loss. + * Checks if the string got by {@link RandomStringUtils#random(int)} can be converted to UTF-8 and back without loss. * * @see <a href="https://issues.apache.org/jira/browse/LANG-100">LANG-100</a> */ @@ -131,12 +126,11 @@ public class RandomStringUtilsTest extends AbstractLangTest { final String copy = new String(bytes, charset); // for a verbose compare: - for (int i=0; i < orig.length() && i < copy.length(); i++) { + for (int i = 0; i < orig.length() && i < copy.length(); i++) { final char o = orig.charAt(i); final char c = copy.charAt(i); - assertEquals(o, c, - "differs at " + i + "(" + Integer.toHexString(Character.valueOf(o).hashCode()) + "," + - Integer.toHexString(Character.valueOf(c).hashCode()) + ")"); + assertEquals(o, c, "differs at " + i + "(" + Integer.toHexString(Character.valueOf(o).hashCode()) + "," + + Integer.toHexString(Character.valueOf(c).hashCode()) + ")"); } // compare length also assertEquals(orig.length(), copy.length()); @@ -147,26 +141,24 @@ public class RandomStringUtilsTest extends AbstractLangTest { @Test public void testLANG805() { final long seedMillis = System.currentTimeMillis(); - assertEquals("aaa", RandomStringUtils.random(3, 0, 0, false, false, new char[]{'a'}, new Random(seedMillis))); + assertEquals("aaa", RandomStringUtils.random(3, 0, 0, false, false, new char[] { 'a' }, new Random(seedMillis))); } @Test public void testLANG807() { - final IllegalArgumentException ex = - assertThrows(IllegalArgumentException.class, () -> RandomStringUtils.random(3, 5, 5, false, false)); + final IllegalArgumentException ex = assertThrows(IllegalArgumentException.class, () -> RandomStringUtils.random(3, 5, 5, false, false)); final String msg = ex.getMessage(); assertTrue(msg.contains("start"), "Message (" + msg + ") must contain 'start'"); assertTrue(msg.contains("end"), "Message (" + msg + ") must contain 'end'"); } /** - * Make sure boundary alpha characters are generated by randomAlphabetic - * This test will fail randomly with probability = 4 * (51/52)**1000 ~ 1.58E-8 + * Make sure boundary alpha characters are generated by randomAlphabetic This test will fail randomly with probability = 4 * (51/52)**1000 ~ 1.58E-8 */ @Test public void testRandomAlphabetic() { - final char[] testChars = {'a', 'z', 'A', 'Z'}; - final boolean[] found = {false, false, false, false}; + final char[] testChars = { 'a', 'z', 'A', 'Z' }; + final boolean[] found = { false, false, false, false }; for (int i = 0; i < LOOP_COUNT; i++) { final String randString = RandomStringUtils.randomAlphabetic(10); for (int j = 0; j < testChars.length; j++) { @@ -176,9 +168,7 @@ public class RandomStringUtilsTest extends AbstractLangTest { } } for (int i = 0; i < testChars.length; i++) { - assertTrue(found[i], - "alphanumeric character not generated in 1000 attempts: " + testChars[i] + - " -- repeated failures indicate a problem "); + assertTrue(found[i], "alphanumeric character not generated in 1000 attempts: " + testChars[i] + " -- repeated failures indicate a problem "); } } @@ -208,13 +198,12 @@ public class RandomStringUtilsTest extends AbstractLangTest { } /** - * Make sure boundary alphanumeric characters are generated by randomAlphaNumeric - * This test will fail randomly with probability = 6 * (61/62)**1000 ~ 5.2E-7 + * Make sure boundary alphanumeric characters are generated by randomAlphaNumeric This test will fail randomly with probability = 6 * (61/62)**1000 ~ 5.2E-7 */ @Test public void testRandomAlphaNumeric() { - final char[] testChars = {'a', 'z', 'A', 'Z', '0', '9'}; - final boolean[] found = {false, false, false, false, false, false}; + final char[] testChars = { 'a', 'z', 'A', 'Z', '0', '9' }; + final boolean[] found = { false, false, false, false, false, false }; for (int i = 0; i < LOOP_COUNT; i++) { final String randString = RandomStringUtils.randomAlphanumeric(10); for (int j = 0; j < testChars.length; j++) { @@ -224,9 +213,7 @@ public class RandomStringUtilsTest extends AbstractLangTest { } } for (int i = 0; i < testChars.length; i++) { - assertTrue(found[i], - "alphanumeric character not generated in 1000 attempts: " + - testChars[i] + " -- repeated failures indicate a problem "); + assertTrue(found[i], "alphanumeric character not generated in 1000 attempts: " + testChars[i] + " -- repeated failures indicate a problem "); } } @@ -256,13 +243,12 @@ public class RandomStringUtilsTest extends AbstractLangTest { } /** - * Make sure 32 and 127 are generated by randomNumeric - * This test will fail randomly with probability = 2*(95/96)**1000 ~ 5.7E-5 + * Make sure 32 and 127 are generated by randomNumeric This test will fail randomly with probability = 2*(95/96)**1000 ~ 5.7E-5 */ @Test public void testRandomAscii() { - final char[] testChars = {(char) 32, (char) 126}; - final boolean[] found = {false, false}; + final char[] testChars = { (char) 32, (char) 126 }; + final boolean[] found = { false, false }; // Test failures have been observed on GitHub builds with a 100 limit. for (int i = 0; i < LOOP_COUNT; i++) { final String randString = RandomStringUtils.randomAscii(10); @@ -273,9 +259,7 @@ public class RandomStringUtilsTest extends AbstractLangTest { } } for (int i = 0; i < testChars.length; i++) { - assertTrue(found[i], - "ascii character not generated in 1000 attempts: " + (int) testChars[i] + - " -- repeated failures indicate a problem"); + assertTrue(found[i], "ascii character not generated in 1000 attempts: " + (int) testChars[i] + " -- repeated failures indicate a problem"); } } @@ -330,13 +314,12 @@ public class RandomStringUtilsTest extends AbstractLangTest { } /** - * Make sure '0' and '9' are generated by randomNumeric - * This test will fail randomly with probability = 2 * (9/10)**1000 ~ 3.5E-46 + * Make sure '0' and '9' are generated by randomNumeric This test will fail randomly with probability = 2 * (9/10)**1000 ~ 3.5E-46 */ @Test public void testRandomNumeric() { - final char[] testChars = {'0', '9'}; - final boolean[] found = {false, false}; + final char[] testChars = { '0', '9' }; + final boolean[] found = { false, false }; for (int i = 0; i < LOOP_COUNT; i++) { final String randString = RandomStringUtils.randomNumeric(10); for (int j = 0; j < testChars.length; j++) { @@ -346,9 +329,7 @@ public class RandomStringUtilsTest extends AbstractLangTest { } } for (int i = 0; i < testChars.length; i++) { - assertTrue(found[i], - "digit not generated in 1000 attempts: " + testChars[i] + - " -- repeated failures indicate a problem "); + assertTrue(found[i], "digit not generated in 1000 attempts: " + testChars[i] + " -- repeated failures indicate a problem "); } } @@ -501,36 +482,34 @@ public class RandomStringUtilsTest extends AbstractLangTest { } /** - * Test homogeneity of random strings generated -- - * i.e., test that characters show up with expected frequencies - * in generated strings. Will fail randomly about 1 in 100,000 times. - * Repeated failures indicate a problem. + * Test homogeneity of random strings generated -- i.e., test that characters show up with expected frequencies in generated strings. Will fail randomly + * about 1 in 100,000 times. Repeated failures indicate a problem. */ @Test public void testRandomStringUtilsHomog() { final String set = "abc"; final char[] chars = set.toCharArray(); - final int[] counts = {0, 0, 0}; - final int[] expected = {200, 200, 200}; + final int[] counts = { 0, 0, 0 }; + final int[] expected = { 200, 200, 200 }; for (int i = 0; i < 100; i++) { final String gen = RandomStringUtils.random(6, chars); for (int j = 0; j < 6; j++) { switch (gen.charAt(j)) { - case 'a': { - counts[0]++; - break; - } - case 'b': { - counts[1]++; - break; - } - case 'c': { - counts[2]++; - break; - } - default: { - fail("generated character not in set"); - } + case 'a': { + counts[0]++; + break; + } + case 'b': { + counts[1]++; + break; + } + case 'c': { + counts[2]++; + break; + } + default: { + fail("generated character not in set"); + } } } } @@ -540,4 +519,3 @@ public class RandomStringUtilsTest extends AbstractLangTest { assertThat("test homogeneity -- will fail about 1 in 100,000 times", chiSquare(expected, counts), lessThan(23.025850929940457d)); } } - diff --git a/src/test/java/org/apache/commons/lang3/StringEscapeUtilsTest.java b/src/test/java/org/apache/commons/lang3/StringEscapeUtilsTest.java index 8d9ce0d9d..c4b9c70aa 100644 --- a/src/test/java/org/apache/commons/lang3/StringEscapeUtilsTest.java +++ b/src/test/java/org/apache/commons/lang3/StringEscapeUtilsTest.java @@ -80,11 +80,9 @@ public class StringEscapeUtilsTest extends AbstractLangTest { final String actual = StringEscapeUtils.unescapeJava(original); assertEquals(expected, actual, - "unescape(String) failed" + - (message == null ? "" : ": " + message) + - ": expected '" + StringEscapeUtils.escapeJava(expected) + + "unescape(String) failed" + (message == null ? "" : ": " + message) + ": expected '" + StringEscapeUtils.escapeJava(expected) + // we escape this so we can see it in the error message - "' actual '" + StringEscapeUtils.escapeJava(actual) + "'"); + "' actual '" + StringEscapeUtils.escapeJava(actual) + "'"); final StringWriter writer = new StringWriter(); StringEscapeUtils.UNESCAPE_JAVA.translate(original, writer); @@ -122,23 +120,23 @@ public class StringEscapeUtilsTest extends AbstractLangTest { @Test public void testEscapeCsvString() { - assertEquals("foo.bar", StringEscapeUtils.escapeCsv("foo.bar")); - assertEquals("\"foo,bar\"", StringEscapeUtils.escapeCsv("foo,bar")); - assertEquals("\"foo\nbar\"", StringEscapeUtils.escapeCsv("foo\nbar")); - assertEquals("\"foo\rbar\"", StringEscapeUtils.escapeCsv("foo\rbar")); - assertEquals("\"foo\"\"bar\"", StringEscapeUtils.escapeCsv("foo\"bar")); + assertEquals("foo.bar", StringEscapeUtils.escapeCsv("foo.bar")); + assertEquals("\"foo,bar\"", StringEscapeUtils.escapeCsv("foo,bar")); + assertEquals("\"foo\nbar\"", StringEscapeUtils.escapeCsv("foo\nbar")); + assertEquals("\"foo\rbar\"", StringEscapeUtils.escapeCsv("foo\rbar")); + assertEquals("\"foo\"\"bar\"", StringEscapeUtils.escapeCsv("foo\"bar")); assertEquals("foo\uD84C\uDFB4bar", StringEscapeUtils.escapeCsv("foo\uD84C\uDFB4bar")); - assertEquals("", StringEscapeUtils.escapeCsv("")); + assertEquals("", StringEscapeUtils.escapeCsv("")); assertNull(StringEscapeUtils.escapeCsv(null)); } @Test public void testEscapeCsvWriter() throws Exception { - checkCsvEscapeWriter("foo.bar", "foo.bar"); - checkCsvEscapeWriter("\"foo,bar\"", "foo,bar"); - checkCsvEscapeWriter("\"foo\nbar\"", "foo\nbar"); - checkCsvEscapeWriter("\"foo\rbar\"", "foo\rbar"); - checkCsvEscapeWriter("\"foo\"\"bar\"", "foo\"bar"); + checkCsvEscapeWriter("foo.bar", "foo.bar"); + checkCsvEscapeWriter("\"foo,bar\"", "foo,bar"); + checkCsvEscapeWriter("\"foo\nbar\"", "foo\nbar"); + checkCsvEscapeWriter("\"foo\rbar\"", "foo\rbar"); + checkCsvEscapeWriter("\"foo\"\"bar\"", "foo\"bar"); checkCsvEscapeWriter("foo\uD84C\uDFB4bar", "foo\uD84C\uDFB4bar"); checkCsvEscapeWriter("", null); checkCsvEscapeWriter("", ""); @@ -163,10 +161,9 @@ public class StringEscapeUtilsTest extends AbstractLangTest { // Some random Japanese Unicode characters final String original = "\u304B\u304C\u3068"; final String escaped = StringEscapeUtils.escapeHtml4(original); - assertEquals(original, escaped, - "Hiragana character Unicode behavior should not be being escaped by escapeHtml4"); + assertEquals(original, escaped, "Hiragana character Unicode behavior should not be being escaped by escapeHtml4"); - final String unescaped = StringEscapeUtils.unescapeHtml4( escaped ); + final String unescaped = StringEscapeUtils.unescapeHtml4(escaped); assertEquals(escaped, unescaped, "Hiragana character Unicode behavior has changed - expected no unescaping"); } @@ -198,10 +195,10 @@ public class StringEscapeUtilsTest extends AbstractLangTest { final String original = new String(data, StandardCharsets.UTF_8); - final String escaped = StringEscapeUtils.escapeHtml4( original ); + final String escaped = StringEscapeUtils.escapeHtml4(original); assertEquals(original, escaped, "High Unicode should not have been escaped"); - final String unescaped = StringEscapeUtils.unescapeHtml4( escaped ); + final String unescaped = StringEscapeUtils.unescapeHtml4(escaped); assertEquals(original, unescaped, "High Unicode should have been unchanged"); // TODO: I think this should hold, needs further investigation @@ -235,12 +232,9 @@ public class StringEscapeUtilsTest extends AbstractLangTest { assertEscapeJava("\\u0001", "\u0001"); assertEscapeJava("Should use capitalized Unicode hex", "\\uABCD", "\uabcd"); - assertEscapeJava("He didn't say, \\\"stop!\\\"", - "He didn't say, \"stop!\""); - assertEscapeJava("non-breaking space", "This space is non-breaking:" + "\\u00A0", - "This space is non-breaking:\u00a0"); - assertEscapeJava("\\uABCD\\u1234\\u012C", - "\uABCD\u1234\u012C"); + assertEscapeJava("He didn't say, \\\"stop!\\\"", "He didn't say, \"stop!\""); + assertEscapeJava("non-breaking space", "This space is non-breaking:" + "\\u00A0", "This space is non-breaking:\u00a0"); + assertEscapeJava("\\uABCD\\u1234\\u012C", "\uABCD\u1234\u012C"); } /** @@ -254,8 +248,7 @@ public class StringEscapeUtilsTest extends AbstractLangTest { final String actual = StringEscapeUtils.escapeJava(input); /* - * In 2.4 StringEscapeUtils.escapeJava(String) escapes '/' characters, which are not a valid character to escape - * in a Java string. + * In 2.4 StringEscapeUtils.escapeJava(String) escapes '/' characters, which are not a valid character to escape in a Java string. */ assertEquals(expected, actual); } @@ -269,7 +262,7 @@ public class StringEscapeUtilsTest extends AbstractLangTest { assertEquals("He didn't say, \\\"stop!\\\"", StringEscapeUtils.escapeJson("He didn't say, \"stop!\"")); final String expected = "\\\"foo\\\" isn't \\\"bar\\\". specials: \\b\\r\\n\\f\\t\\\\\\/"; - final String input ="\"foo\" isn't \"bar\". specials: \b\r\n\f\t\\/"; + final String input = "\"foo\" isn't \"bar\". specials: \b\r\n\f\t\\/"; assertEquals(expected, StringEscapeUtils.escapeJson(input)); } @@ -281,12 +274,9 @@ public class StringEscapeUtilsTest extends AbstractLangTest { assertEquals("\u00A1", StringEscapeUtils.escapeXml("\u00A1"), "XML should not escape >0x7f values"); assertEquals("\u00A0", StringEscapeUtils.unescapeXml(" "), "XML should be able to unescape >0x7f values"); - assertEquals("\u00A0", StringEscapeUtils.unescapeXml(" "), - "XML should be able to unescape >0x7f values with one leading 0"); - assertEquals("\u00A0", StringEscapeUtils.unescapeXml(" "), - "XML should be able to unescape >0x7f values with two leading 0s"); - assertEquals("\u00A0", StringEscapeUtils.unescapeXml(" "), - "XML should be able to unescape >0x7f values with three leading 0s"); + assertEquals("\u00A0", StringEscapeUtils.unescapeXml(" "), "XML should be able to unescape >0x7f values with one leading 0"); + assertEquals("\u00A0", StringEscapeUtils.unescapeXml(" "), "XML should be able to unescape >0x7f values with two leading 0s"); + assertEquals("\u00A0", StringEscapeUtils.unescapeXml(" "), "XML should be able to unescape >0x7f values with three leading 0s"); assertEquals("ain't", StringEscapeUtils.unescapeXml("ain't")); assertEquals("ain't", StringEscapeUtils.escapeXml("ain't")); @@ -309,12 +299,9 @@ public class StringEscapeUtilsTest extends AbstractLangTest { assertEquals("a\tb\rc\nd", StringEscapeUtils.escapeXml10("a\tb\rc\nd"), "XML 1.0 should not escape \t \n \r"); assertEquals("ab", StringEscapeUtils.escapeXml10("a\u0000\u0001\u0008\u000b\u000c\u000e\u001fb"), "XML 1.0 should omit most #x0-x8 | #xb | #xc | #xe-#x19"); - assertEquals("a\ud7ff \ue000b", StringEscapeUtils.escapeXml10("a\ud7ff\ud800 \udfff \ue000b"), - "XML 1.0 should omit #xd800-#xdfff"); - assertEquals("a\ufffdb", StringEscapeUtils.escapeXml10("a\ufffd\ufffe\uffffb"), - "XML 1.0 should omit #xfffe | #xffff"); - assertEquals("a\u007e„\u0085†Ÿ\u00a0b", - StringEscapeUtils.escapeXml10("a\u007e\u007f\u0084\u0085\u0086\u009f\u00a0b"), + assertEquals("a\ud7ff \ue000b", StringEscapeUtils.escapeXml10("a\ud7ff\ud800 \udfff \ue000b"), "XML 1.0 should omit #xd800-#xdfff"); + assertEquals("a\ufffdb", StringEscapeUtils.escapeXml10("a\ufffd\ufffe\uffffb"), "XML 1.0 should omit #xfffe | #xffff"); + assertEquals("a\u007e„\u0085†Ÿ\u00a0b", StringEscapeUtils.escapeXml10("a\u007e\u007f\u0084\u0085\u0086\u009f\u00a0b"), "XML 1.0 should escape #x7f-#x84 | #x86 - #x9f, for XML 1.1 compatibility"); } @@ -323,16 +310,12 @@ public class StringEscapeUtilsTest extends AbstractLangTest { assertEquals("a<b>c"d'e&f", StringEscapeUtils.escapeXml11("a<b>c\"d'e&f")); assertEquals("a\tb\rc\nd", StringEscapeUtils.escapeXml11("a\tb\rc\nd"), "XML 1.1 should not escape \t \n \r"); assertEquals("ab", StringEscapeUtils.escapeXml11("a\u0000b"), "XML 1.1 should omit #x0"); - assertEquals("ab", - StringEscapeUtils.escapeXml11("a\u0001\u0008\u000b\u000c\u000e\u001fb"), + assertEquals("ab", StringEscapeUtils.escapeXml11("a\u0001\u0008\u000b\u000c\u000e\u001fb"), "XML 1.1 should escape #x1-x8 | #xb | #xc | #xe-#x19"); - assertEquals("a\u007e„\u0085†Ÿ\u00a0b", - StringEscapeUtils.escapeXml11("a\u007e\u007f\u0084\u0085\u0086\u009f\u00a0b"), + assertEquals("a\u007e„\u0085†Ÿ\u00a0b", StringEscapeUtils.escapeXml11("a\u007e\u007f\u0084\u0085\u0086\u009f\u00a0b"), "XML 1.1 should escape #x7F-#x84 | #x86-#x9F"); - assertEquals("a\ud7ff \ue000b", StringEscapeUtils.escapeXml11("a\ud7ff\ud800 \udfff \ue000b"), - "XML 1.1 should omit #xd800-#xdfff"); - assertEquals("a\ufffdb", StringEscapeUtils.escapeXml11("a\ufffd\ufffe\uffffb"), - "XML 1.1 should omit #xfffe | #xffff"); + assertEquals("a\ud7ff \ue000b", StringEscapeUtils.escapeXml11("a\ud7ff\ud800 \udfff \ue000b"), "XML 1.1 should omit #xd800-#xdfff"); + assertEquals("a\ufffdb", StringEscapeUtils.escapeXml11("a\ufffd\ufffe\uffffb"), "XML 1.1 should omit #xfffe | #xffff"); } @Test @@ -340,9 +323,9 @@ public class StringEscapeUtilsTest extends AbstractLangTest { // https://www.w3.org/TR/xml/#charsets says: // Char ::= #x9 | #xA | #xD | [#x20-#xD7FF] | [#xE000-#xFFFD] | [#x10000-#x10FFFF] /* any Unicode character, // excluding the surrogate blocks, FFFE, and FFFF. */ - final CharSequenceTranslator escapeXml = StringEscapeUtils.ESCAPE_XML - .with(NumericEntityEscaper.below(9), NumericEntityEscaper.between(0xB, 0xC), NumericEntityEscaper.between(0xE, 0x19), - NumericEntityEscaper.between(0xD800, 0xDFFF), NumericEntityEscaper.between(0xFFFE, 0xFFFF), NumericEntityEscaper.above(0x110000)); + final CharSequenceTranslator escapeXml = StringEscapeUtils.ESCAPE_XML.with(NumericEntityEscaper.below(9), NumericEntityEscaper.between(0xB, 0xC), + NumericEntityEscaper.between(0xE, 0x19), NumericEntityEscaper.between(0xD800, 0xDFFF), NumericEntityEscaper.between(0xFFFE, 0xFFFF), + NumericEntityEscaper.above(0x110000)); assertEquals("�", escapeXml.translate("\u0000\u0001\u0002\u0003\u0004\u0005\u0006\u0007\u0008")); assertEquals("\t", escapeXml.translate("\t")); // 0x9 @@ -358,23 +341,19 @@ public class StringEscapeUtilsTest extends AbstractLangTest { * <p> * From https://www.w3.org/International/questions/qa-escapes * </p> - * <blockquote> - * Supplementary characters are those Unicode characters that have code points higher than the characters in - * the Basic Multilingual Plane (BMP). In UTF-16 a supplementary character is encoded using two 16-bit surrogate code points from the - * BMP. Because of this, some people think that supplementary characters need to be represented using two escapes, but this is incorrect - * - you must use the single, code point value for that character. For example, use &#x233B4; rather than - * &#xD84C;&#xDFB4;. - * </blockquote> + * <blockquote> Supplementary characters are those Unicode characters that have code points higher than the characters in the Basic Multilingual Plane + * (BMP). In UTF-16 a supplementary character is encoded using two 16-bit surrogate code points from the BMP. Because of this, some people think that + * supplementary characters need to be represented using two escapes, but this is incorrect - you must use the single, code point value for that character. + * For example, use &#x233B4; rather than &#xD84C;&#xDFB4;. </blockquote> + * * @see <a href="https://www.w3.org/International/questions/qa-escapes">Using character escapes in markup and CSS</a> * @see <a href="https://issues.apache.org/jira/browse/LANG-728">LANG-728</a> */ @Test public void testEscapeXmlSupplementaryCharacters() { - final CharSequenceTranslator escapeXml = - StringEscapeUtils.ESCAPE_XML.with( NumericEntityEscaper.between(0x7f, Integer.MAX_VALUE) ); + final CharSequenceTranslator escapeXml = StringEscapeUtils.ESCAPE_XML.with(NumericEntityEscaper.between(0x7f, Integer.MAX_VALUE)); - assertEquals("𣎴", escapeXml.translate("\uD84C\uDFB4"), - "Supplementary character must be represented using a single escape"); + assertEquals("𣎴", escapeXml.translate("\uD84C\uDFB4"), "Supplementary character must be represented using a single escape"); assertEquals("a b c 𣎴", escapeXml.translate("a b c \uD84C\uDFB4"), "Supplementary characters mixed with basic characters should be encoded correctly"); @@ -388,8 +367,7 @@ public class StringEscapeUtilsTest extends AbstractLangTest { /** * Tests https://issues.apache.org/jira/browse/LANG-708 * - * @throws IOException - * if an I/O error occurs + * @throws IOException if an I/O error occurs */ @Test public void testLang708() throws IOException { @@ -441,30 +419,30 @@ public class StringEscapeUtilsTest extends AbstractLangTest { @Test public void testUnescapeCsvString() { - assertEquals("foo.bar", StringEscapeUtils.unescapeCsv("foo.bar")); - assertEquals("foo,bar", StringEscapeUtils.unescapeCsv("\"foo,bar\"")); - assertEquals("foo\nbar", StringEscapeUtils.unescapeCsv("\"foo\nbar\"")); - assertEquals("foo\rbar", StringEscapeUtils.unescapeCsv("\"foo\rbar\"")); - assertEquals("foo\"bar", StringEscapeUtils.unescapeCsv("\"foo\"\"bar\"")); - assertEquals("foo\uD84C\uDFB4bar", StringEscapeUtils.unescapeCsv("foo\uD84C\uDFB4bar")); - assertEquals("", StringEscapeUtils.unescapeCsv("")); + assertEquals("foo.bar", StringEscapeUtils.unescapeCsv("foo.bar")); + assertEquals("foo,bar", StringEscapeUtils.unescapeCsv("\"foo,bar\"")); + assertEquals("foo\nbar", StringEscapeUtils.unescapeCsv("\"foo\nbar\"")); + assertEquals("foo\rbar", StringEscapeUtils.unescapeCsv("\"foo\rbar\"")); + assertEquals("foo\"bar", StringEscapeUtils.unescapeCsv("\"foo\"\"bar\"")); + assertEquals("foo\uD84C\uDFB4bar", StringEscapeUtils.unescapeCsv("foo\uD84C\uDFB4bar")); + assertEquals("", StringEscapeUtils.unescapeCsv("")); assertNull(StringEscapeUtils.unescapeCsv(null)); - assertEquals("\"foo.bar\"", StringEscapeUtils.unescapeCsv("\"foo.bar\"")); + assertEquals("\"foo.bar\"", StringEscapeUtils.unescapeCsv("\"foo.bar\"")); } @Test public void testUnescapeCsvWriter() throws Exception { - checkCsvUnescapeWriter("foo.bar", "foo.bar"); - checkCsvUnescapeWriter("foo,bar", "\"foo,bar\""); - checkCsvUnescapeWriter("foo\nbar", "\"foo\nbar\""); - checkCsvUnescapeWriter("foo\rbar", "\"foo\rbar\""); - checkCsvUnescapeWriter("foo\"bar", "\"foo\"\"bar\""); + checkCsvUnescapeWriter("foo.bar", "foo.bar"); + checkCsvUnescapeWriter("foo,bar", "\"foo,bar\""); + checkCsvUnescapeWriter("foo\nbar", "\"foo\nbar\""); + checkCsvUnescapeWriter("foo\rbar", "\"foo\rbar\""); + checkCsvUnescapeWriter("foo\"bar", "\"foo\"\"bar\""); checkCsvUnescapeWriter("foo\uD84C\uDFB4bar", "foo\uD84C\uDFB4bar"); checkCsvUnescapeWriter("", null); checkCsvUnescapeWriter("", ""); - checkCsvUnescapeWriter("\"foo.bar\"", "\"foo.bar\""); + checkCsvUnescapeWriter("\"foo.bar\"", "\"foo.bar\""); } @Test @@ -486,7 +464,7 @@ public class StringEscapeUtilsTest extends AbstractLangTest { // Test all Character values: for (char i = Character.MIN_VALUE; i < Character.MAX_VALUE; i++) { final Character c1 = Character.valueOf(i); - final Character c2 = Character.valueOf((char) (i+1)); + final Character c2 = Character.valueOf((char) (i + 1)); final String expected = c1.toString() + c2; final String escapedC1 = "&#x" + Integer.toHexString(c1.charValue()) + ";"; final String escapedC2 = "&#x" + Integer.toHexString(c2.charValue()) + ";"; @@ -531,7 +509,7 @@ public class StringEscapeUtilsTest extends AbstractLangTest { assertUnescapeJava("\u123425foo\ntest\b", "\\u123425foo\\ntest\\b"); assertUnescapeJava("'\foo\teste\r", "\\'\\foo\\teste\\r"); assertUnescapeJava("", "\\"); - //foo + // foo assertUnescapeJava("lowercase Unicode", "\uABCDx", "\\uabcdx"); assertUnescapeJava("uppercase Unicode", "\uABCDx", "\\uABCDx"); assertUnescapeJava("Unicode as final character", "\uABCD", "\\uabcd"); @@ -545,7 +523,7 @@ public class StringEscapeUtilsTest extends AbstractLangTest { assertEquals("He didn't say, \"stop!\"", StringEscapeUtils.unescapeJson("He didn't say, \\\"stop!\\\"")); - final String expected ="\"foo\" isn't \"bar\". specials: \b\r\n\f\t\\/"; + final String expected = "\"foo\" isn't \"bar\". specials: \b\r\n\f\t\\/"; final String input = "\\\"foo\\\" isn't \\\"bar\\\". specials: \\b\\r\\n\\f\\t\\\\\\/"; assertEquals(expected, StringEscapeUtils.unescapeJson(input)); @@ -563,8 +541,7 @@ public class StringEscapeUtilsTest extends AbstractLangTest { */ @Test public void testUnescapeXmlSupplementaryCharacters() { - assertEquals("\uD84C\uDFB4", StringEscapeUtils.unescapeXml("𣎴"), - "Supplementary character must be represented using a single escape"); + assertEquals("\uD84C\uDFB4", StringEscapeUtils.unescapeXml("𣎴"), "Supplementary character must be represented using a single escape"); assertEquals("a b c \uD84C\uDFB4", StringEscapeUtils.unescapeXml("a b c 𣎴"), "Supplementary characters mixed with basic characters should be decoded correctly"); diff --git a/src/test/java/org/apache/commons/lang3/StringUtilsContainsTest.java b/src/test/java/org/apache/commons/lang3/StringUtilsContainsTest.java index 2c726b060..063085327 100644 --- a/src/test/java/org/apache/commons/lang3/StringUtilsContainsTest.java +++ b/src/test/java/org/apache/commons/lang3/StringUtilsContainsTest.java @@ -219,10 +219,10 @@ public class StringUtilsContainsTest extends AbstractLangTest { assertFalse(StringUtils.containsAnyIgnoreCase("", new String[] { "hello" })); assertFalse(StringUtils.containsAnyIgnoreCase("hello, goodbye", (String[]) null)); assertFalse(StringUtils.containsAnyIgnoreCase("hello, goodbye", new String[0])); - assertTrue(StringUtils.containsAnyIgnoreCase("hello, goodbye", new String[]{"hello", "goodbye"})); - assertTrue(StringUtils.containsAnyIgnoreCase("hello, goodbye", new String[]{"hello", "Goodbye"})); - assertTrue(StringUtils.containsAnyIgnoreCase("hello, goodbye", new String[]{"Hello", "Goodbye"})); - assertTrue(StringUtils.containsAnyIgnoreCase("hello, goodbye", new String[]{"Hello", null})); + assertTrue(StringUtils.containsAnyIgnoreCase("hello, goodbye", new String[] { "hello", "goodbye" })); + assertTrue(StringUtils.containsAnyIgnoreCase("hello, goodbye", new String[] { "hello", "Goodbye" })); + assertTrue(StringUtils.containsAnyIgnoreCase("hello, goodbye", new String[] { "Hello", "Goodbye" })); + assertTrue(StringUtils.containsAnyIgnoreCase("hello, goodbye", new String[] { "Hello", null })); assertTrue(StringUtils.containsAnyIgnoreCase("hello, null", new String[] { "Hello", null })); // Javadoc examples: assertTrue(StringUtils.containsAnyIgnoreCase("abcd", "ab", null)); @@ -235,20 +235,17 @@ public class StringUtilsContainsTest extends AbstractLangTest { public void testContainsIgnoreCase_LocaleIndependence() { final Locale[] locales = { Locale.ENGLISH, new Locale("tr"), Locale.getDefault() }; - final String[][] tdata = { { "i", "I" }, { "I", "i" }, { "\u03C2", "\u03C3" }, { "\u03A3", "\u03C2" }, - { "\u03A3", "\u03C3" }, }; + final String[][] tdata = { { "i", "I" }, { "I", "i" }, { "\u03C2", "\u03C3" }, { "\u03A3", "\u03C2" }, { "\u03A3", "\u03C3" }, }; final String[][] fdata = { { "\u00DF", "SS" }, }; for (final Locale testLocale : locales) { Locale.setDefault(testLocale); for (int j = 0; j < tdata.length; j++) { - assertTrue(StringUtils.containsIgnoreCase(tdata[j][0], tdata[j][1]), - Locale.getDefault() + ": " + j + " " + tdata[j][0] + " " + tdata[j][1]); + assertTrue(StringUtils.containsIgnoreCase(tdata[j][0], tdata[j][1]), Locale.getDefault() + ": " + j + " " + tdata[j][0] + " " + tdata[j][1]); } for (int j = 0; j < fdata.length; j++) { - assertFalse(StringUtils.containsIgnoreCase(fdata[j][0], fdata[j][1]), - Locale.getDefault() + ": " + j + " " + fdata[j][0] + " " + fdata[j][1]); + assertFalse(StringUtils.containsIgnoreCase(fdata[j][0], fdata[j][1]), Locale.getDefault() + ": " + j + " " + fdata[j][0] + " " + fdata[j][1]); } } } @@ -293,9 +290,9 @@ public class StringUtilsContainsTest extends AbstractLangTest { final String str1 = "a"; final String str2 = "b"; final String str3 = "ab."; - final char[] chars1= {'b'}; - final char[] chars2= {'.'}; - final char[] chars3= {'c', 'd'}; + final char[] chars1 = { 'b' }; + final char[] chars2 = { '.' }; + final char[] chars3 = { 'c', 'd' }; final char[] emptyChars = {}; assertTrue(StringUtils.containsNone(null, (char[]) null)); assertTrue(StringUtils.containsNone("", (char[]) null)); @@ -351,9 +348,9 @@ public class StringUtilsContainsTest extends AbstractLangTest { final String str1 = "a"; final String str2 = "b"; final String str3 = "ab."; - final String chars1= "b"; - final String chars2= "."; - final String chars3= "cd"; + final String chars1 = "b"; + final String chars2 = "."; + final String chars3 = "cd"; assertTrue(StringUtils.containsNone(null, (String) null)); assertTrue(StringUtils.containsNone("", (String) null)); assertTrue(StringUtils.containsNone(null, "")); @@ -408,9 +405,9 @@ public class StringUtilsContainsTest extends AbstractLangTest { final String str1 = "a"; final String str2 = "b"; final String str3 = "ab"; - final char[] chars1= {'b'}; - final char[] chars2= {'a'}; - final char[] chars3= {'a', 'b'}; + final char[] chars1 = { 'b' }; + final char[] chars2 = { 'a' }; + final char[] chars3 = { 'a', 'b' }; final char[] emptyChars = {}; assertFalse(StringUtils.containsOnly(null, (char[]) null)); assertFalse(StringUtils.containsOnly("", (char[]) null)); @@ -434,9 +431,9 @@ public class StringUtilsContainsTest extends AbstractLangTest { final String str1 = "a"; final String str2 = "b"; final String str3 = "ab"; - final String chars1= "b"; - final String chars2= "a"; - final String chars3= "ab"; + final String chars1 = "b"; + final String chars2 = "a"; + final String chars3 = "ab"; assertFalse(StringUtils.containsOnly(null, (String) null)); assertFalse(StringUtils.containsOnly("", (String) null)); assertFalse(StringUtils.containsOnly(null, "")); @@ -456,12 +453,12 @@ public class StringUtilsContainsTest extends AbstractLangTest { @Test public void testContainsWhitespace() { - assertFalse( StringUtils.containsWhitespace("") ); - assertTrue( StringUtils.containsWhitespace(" ") ); - assertFalse( StringUtils.containsWhitespace("a") ); - assertTrue( StringUtils.containsWhitespace("a ") ); - assertTrue( StringUtils.containsWhitespace(" a") ); - assertTrue( StringUtils.containsWhitespace("a\t") ); - assertTrue( StringUtils.containsWhitespace("\n") ); + assertFalse(StringUtils.containsWhitespace("")); + assertTrue(StringUtils.containsWhitespace(" ")); + assertFalse(StringUtils.containsWhitespace("a")); + assertTrue(StringUtils.containsWhitespace("a ")); + assertTrue(StringUtils.containsWhitespace(" a")); + assertTrue(StringUtils.containsWhitespace("a\t")); + assertTrue(StringUtils.containsWhitespace("\n")); } } diff --git a/src/test/java/org/apache/commons/lang3/StringUtilsTest.java b/src/test/java/org/apache/commons/lang3/StringUtilsTest.java index 1ee627347..db9dd55c0 100644 --- a/src/test/java/org/apache/commons/lang3/StringUtilsTest.java +++ b/src/test/java/org/apache/commons/lang3/StringUtilsTest.java @@ -359,7 +359,7 @@ public class StringUtilsTest extends AbstractLangTest { assertAbbreviateWithAbbrevMarkerAndOffset("abcdefghi*", "*", 2, 10); assertAbbreviateWithAbbrevMarkerAndOffset("abcdef{{{{", "{{{{", 4, 10); assertAbbreviateWithAbbrevMarkerAndOffset("abcdef____", "____", 5, 10); - assertAbbreviateWithAbbrevMarkerAndOffset("==fghijk==", "==", 5, 10); + assertAbbreviateWithAbbrevMarkerAndOffset(" == fghijk == ", " == ", 5, 10); assertAbbreviateWithAbbrevMarkerAndOffset("___ghij___", "___", 6, 10); assertAbbreviateWithAbbrevMarkerAndOffset("/ghijklmno", "/", 7, 10); assertAbbreviateWithAbbrevMarkerAndOffset("/ghijklmno", "/", 8, 10); @@ -1367,7 +1367,7 @@ public class StringUtilsTest extends AbstractLangTest { assertEquals("", StringUtils.join(EMPTY_STRING_LIST, null)); assertEquals("", StringUtils.join(EMPTY_STRING_LIST, "")); - assertEquals("", StringUtils.join(MIXED_STRING_LIST, "", 0, MIXED_STRING_LIST.size()- 1)); + assertEquals("", StringUtils.join(MIXED_STRING_LIST, "", 0, MIXED_STRING_LIST.size() - 1)); } @ParameterizedTest diff --git a/src/test/java/org/apache/commons/lang3/builder/HashCodeBuilderTest.java b/src/test/java/org/apache/commons/lang3/builder/HashCodeBuilderTest.java index 10bacf13b..86fcd148a 100644 --- a/src/test/java/org/apache/commons/lang3/builder/HashCodeBuilderTest.java +++ b/src/test/java/org/apache/commons/lang3/builder/HashCodeBuilderTest.java @@ -173,7 +173,7 @@ public class HashCodeBuilderTest extends AbstractLangTest { @Override public int hashCode() { - return b*17 + super.hashCode(); + return b * 17 + super.hashCode(); } } diff --git a/src/test/java/org/apache/commons/lang3/builder/JsonToStringStyleTest.java b/src/test/java/org/apache/commons/lang3/builder/JsonToStringStyleTest.java index 5370960bc..afd565baf 100644 --- a/src/test/java/org/apache/commons/lang3/builder/JsonToStringStyleTest.java +++ b/src/test/java/org/apache/commons/lang3/builder/JsonToStringStyleTest.java @@ -333,70 +333,57 @@ public class JsonToStringStyleTest extends AbstractLangTest { assertThrows(UnsupportedOperationException.class, () -> new ToStringBuilder(base).append(now).toString()); - assertEquals("{\"now\":\"" + now +"\"}", new ToStringBuilder(base).append("now", now) - .toString()); - assertEquals("{\"now\":\"" + now +"\",\"after\":\"" + afterNow + "\"}", new ToStringBuilder(base).append("now", now).append("after", afterNow) - .toString()); + assertEquals("{\"now\":\"" + now + "\"}", new ToStringBuilder(base).append("now", now).toString()); + assertEquals("{\"now\":\"" + now + "\",\"after\":\"" + afterNow + "\"}", + new ToStringBuilder(base).append("now", now).append("after", afterNow).toString()); } @Test public void testDoubleArray() { - final double[] array = {1, 2, -3, 4}; + final double[] array = { 1, 2, -3, 4 }; final ToStringBuilder toStringBuilder = new ToStringBuilder(base); assertThrows(UnsupportedOperationException.class, () -> toStringBuilder.append(array).toString()); - assertEquals("{\"doubleArray\":[1.0,2.0,-3.0,4.0]}", toStringBuilder.append("doubleArray", array) - .toString()); + assertEquals("{\"doubleArray\":[1.0,2.0,-3.0,4.0]}", toStringBuilder.append("doubleArray", array).toString()); - assertThrows( - UnsupportedOperationException.class, () -> toStringBuilder.append((Object) array).toString()); + assertThrows(UnsupportedOperationException.class, () -> toStringBuilder.append((Object) array).toString()); - assertThrows( - UnsupportedOperationException.class, () -> toStringBuilder.append((long[]) null).toString()); + assertThrows(UnsupportedOperationException.class, () -> toStringBuilder.append((long[]) null).toString()); - assertThrows( - UnsupportedOperationException.class, () -> toStringBuilder.append((Object) array).toString()); + assertThrows(UnsupportedOperationException.class, () -> toStringBuilder.append((Object) array).toString()); } @Test public void testFloatArray() { - final float[] array = {1, 2, -3, 4}; + final float[] array = { 1, 2, -3, 4 }; final ToStringBuilder toStringBuilder = new ToStringBuilder(base); assertThrows(UnsupportedOperationException.class, () -> toStringBuilder.append(array).toString()); - assertEquals("{\"floatArray\":[1.0,2.0,-3.0,4.0]}", toStringBuilder.append("floatArray", array) - .toString()); + assertEquals("{\"floatArray\":[1.0,2.0,-3.0,4.0]}", toStringBuilder.append("floatArray", array).toString()); - assertThrows( - UnsupportedOperationException.class, () -> toStringBuilder.append((Object) array).toString()); + assertThrows(UnsupportedOperationException.class, () -> toStringBuilder.append((Object) array).toString()); - assertThrows( - UnsupportedOperationException.class, () -> toStringBuilder.append((long[]) null).toString()); + assertThrows(UnsupportedOperationException.class, () -> toStringBuilder.append((long[]) null).toString()); - assertThrows( - UnsupportedOperationException.class, () -> toStringBuilder.append((Object) array).toString()); + assertThrows(UnsupportedOperationException.class, () -> toStringBuilder.append((Object) array).toString()); } @Test public void testIntArray() { - final int[] array = {1, 2, -3, 4}; + final int[] array = { 1, 2, -3, 4 }; final ToStringBuilder toStringBuilder = new ToStringBuilder(base); assertThrows(UnsupportedOperationException.class, () -> toStringBuilder.append(array).toString()); - assertEquals("{\"intArray\":[1,2,-3,4]}", toStringBuilder.append("intArray", array) - .toString()); + assertEquals("{\"intArray\":[1,2,-3,4]}", toStringBuilder.append("intArray", array).toString()); - assertThrows( - UnsupportedOperationException.class, () -> toStringBuilder.append((Object) array).toString()); + assertThrows(UnsupportedOperationException.class, () -> toStringBuilder.append((Object) array).toString()); - assertThrows( - UnsupportedOperationException.class, () -> toStringBuilder.append((long[]) null).toString()); + assertThrows(UnsupportedOperationException.class, () -> toStringBuilder.append((long[]) null).toString()); - assertThrows( - UnsupportedOperationException.class, () -> toStringBuilder.append((Object) array).toString()); + assertThrows(UnsupportedOperationException.class, () -> toStringBuilder.append((Object) array).toString()); } @Test @@ -444,39 +431,32 @@ public class JsonToStringStyleTest extends AbstractLangTest { @Test public void testLongArray() { - final long[] array = {1, 2, -3, 4}; + final long[] array = { 1, 2, -3, 4 }; final ToStringBuilder toStringBuilder = new ToStringBuilder(base); assertThrows(UnsupportedOperationException.class, () -> toStringBuilder.append(array).toString()); - assertEquals("{\"longArray\":[1,2,-3,4]}", toStringBuilder.append("longArray", array) - .toString()); + assertEquals("{\"longArray\":[1,2,-3,4]}", toStringBuilder.append("longArray", array).toString()); - assertThrows( - UnsupportedOperationException.class, () -> toStringBuilder.append((Object) array).toString()); + assertThrows(UnsupportedOperationException.class, () -> toStringBuilder.append((Object) array).toString()); - assertThrows( - UnsupportedOperationException.class, () -> toStringBuilder.append((long[]) null).toString()); + assertThrows(UnsupportedOperationException.class, () -> toStringBuilder.append((long[]) null).toString()); - assertThrows( - UnsupportedOperationException.class, () -> toStringBuilder.append((Object) array).toString()); + assertThrows(UnsupportedOperationException.class, () -> toStringBuilder.append((Object) array).toString()); } @Test public void testLongArrayArray() { - final long[][] array = {{1, 2}, null, {5}}; + final long[][] array = { { 1, 2 }, null, { 5 } }; final ToStringBuilder toStringBuilder = new ToStringBuilder(base); assertThrows(UnsupportedOperationException.class, () -> toStringBuilder.append(array).toString()); - assertThrows( - UnsupportedOperationException.class, () -> toStringBuilder.append((Object) array).toString()); + assertThrows(UnsupportedOperationException.class, () -> toStringBuilder.append((Object) array).toString()); - assertThrows( - UnsupportedOperationException.class, () -> toStringBuilder.append((long[][]) null).toString()); + assertThrows(UnsupportedOperationException.class, () -> toStringBuilder.append((long[][]) null).toString()); - assertThrows( - UnsupportedOperationException.class, () -> toStringBuilder.append((Object) array).toString()); + assertThrows(UnsupportedOperationException.class, () -> toStringBuilder.append((Object) array).toString()); } @Test @@ -498,18 +478,16 @@ public class JsonToStringStyleTest extends AbstractLangTest { baseMap.put("k1", "v1"); baseMap.put("k2", innerMap); - final InnerMapObject object = new InnerMapObject(){ + final InnerMapObject object = new InnerMapObject() { @Override public String toString() { - return new ToStringBuilder(this).append("pid", this.pid) - .append("map", this.map).toString(); + return new ToStringBuilder(this).append("pid", this.pid).append("map", this.map).toString(); } }; object.pid = "dummy-text"; object.map = baseMap; - assertEquals("{\"object\":{\"pid\":\"dummy-text\",\"map\":{\"k1\":\"v1\"," + - "\"k2\":{\"k2.1\":\"v2.1\",\"k2.2\":\"v2.2\"}}}}", + assertEquals("{\"object\":{\"pid\":\"dummy-text\",\"map\":{\"k1\":\"v1\"," + "\"k2\":{\"k2.1\":\"v2.1\",\"k2.2\":\"v2.2\"}}}}", new ToStringBuilder(base).append("object", object).toString()); } @@ -518,9 +496,7 @@ public class JsonToStringStyleTest extends AbstractLangTest { final Person p = new Person() { @Override public String toString() { - return new ToStringBuilder(this).append("name", this.name) - .append("age", this.age).append("smoker", this.smoker) - .toString(); + return new ToStringBuilder(this).append("name", this.name).append("age", this.age).append("smoker", this.smoker).toString(); } }; p.name = "Jane Doe"; @@ -528,14 +504,11 @@ public class JsonToStringStyleTest extends AbstractLangTest { p.smoker = true; final NestingPerson nestP = new NestingPerson(); - nestP.pid="#1@Jane"; + nestP.pid = "#1@Jane"; nestP.person = p; - assertEquals( - "{\"pid\":\"#1@Jane\",\"person\":{\"name\":\"Jane Doe\",\"age\":25,\"smoker\":true}}", - new ToStringBuilder(nestP).append("pid", nestP.pid) - .append("person", nestP.person) - .toString()); + assertEquals("{\"pid\":\"#1@Jane\",\"person\":{\"name\":\"Jane Doe\",\"age\":25,\"smoker\":true}}", + new ToStringBuilder(nestP).append("pid", nestP.pid).append("person", nestP.person).toString()); } @Test @@ -545,91 +518,55 @@ public class JsonToStringStyleTest extends AbstractLangTest { @Test public void testObject() { - final Integer i3 = Integer.valueOf(3); final Integer i4 = Integer.valueOf(4); - assertThrows( - UnsupportedOperationException.class, () -> new ToStringBuilder(base).append((Object) null).toString()); + assertThrows(UnsupportedOperationException.class, () -> new ToStringBuilder(base).append((Object) null).toString()); assertThrows(UnsupportedOperationException.class, () -> new ToStringBuilder(base).append(i3).toString()); - assertEquals("{\"a\":null}", - new ToStringBuilder(base).append("a", (Object) null).toString()); - assertEquals("{\"a\":3}", new ToStringBuilder(base).append("a", i3) - .toString()); - assertEquals("{\"a\":3,\"b\":4}", - new ToStringBuilder(base).append("a", i3).append("b", i4) - .toString()); + assertEquals("{\"a\":null}", new ToStringBuilder(base).append("a", (Object) null).toString()); + assertEquals("{\"a\":3}", new ToStringBuilder(base).append("a", i3).toString()); + assertEquals("{\"a\":3,\"b\":4}", new ToStringBuilder(base).append("a", i3).append("b", i4).toString()); - assertThrows( - UnsupportedOperationException.class, () -> new ToStringBuilder(base).append("a", i3, false).toString()); + assertThrows(UnsupportedOperationException.class, () -> new ToStringBuilder(base).append("a", i3, false).toString()); - assertThrows( - UnsupportedOperationException.class, - () -> new ToStringBuilder(base).append("a", new ArrayList<>(), false).toString()); + assertThrows(UnsupportedOperationException.class, () -> new ToStringBuilder(base).append("a", new ArrayList<>(), false).toString()); - assertEquals( - "{\"a\":[]}", - new ToStringBuilder(base).append("a", new ArrayList<>(), - true).toString()); + assertEquals("{\"a\":[]}", new ToStringBuilder(base).append("a", new ArrayList<>(), true).toString()); - assertThrows( - UnsupportedOperationException.class, - () -> new ToStringBuilder(base).append("a", new HashMap<>(), false).toString()); + assertThrows(UnsupportedOperationException.class, () -> new ToStringBuilder(base).append("a", new HashMap<>(), false).toString()); - assertEquals( - "{\"a\":{}}", - new ToStringBuilder(base).append("a", - new HashMap<>(), true).toString()); + assertEquals("{\"a\":{}}", new ToStringBuilder(base).append("a", new HashMap<>(), true).toString()); - assertThrows( - UnsupportedOperationException.class, - () -> new ToStringBuilder(base).append("a", (Object) new String[0], false).toString()); + assertThrows(UnsupportedOperationException.class, () -> new ToStringBuilder(base).append("a", (Object) new String[0], false).toString()); - assertEquals( - "{\"a\":[]}", - new ToStringBuilder(base).append("a", (Object) new String[0], - true).toString()); + assertEquals("{\"a\":[]}", new ToStringBuilder(base).append("a", (Object) new String[0], true).toString()); - assertThrows( - UnsupportedOperationException.class, - () -> new ToStringBuilder(base).append("a", (Object) new int[]{1, 2, 3}, false).toString()); + assertThrows(UnsupportedOperationException.class, () -> new ToStringBuilder(base).append("a", (Object) new int[] { 1, 2, 3 }, false).toString()); - assertEquals( - "{\"a\":[1,2,3]}", - new ToStringBuilder(base).append("a", - (Object) new int[]{1, 2, 3}, true).toString()); + assertEquals("{\"a\":[1,2,3]}", new ToStringBuilder(base).append("a", (Object) new int[] { 1, 2, 3 }, true).toString()); - assertThrows( - UnsupportedOperationException.class, - () -> new ToStringBuilder(base).append("a", (Object) new String[]{"v", "x", "y", "z"}, false).toString()); + assertThrows(UnsupportedOperationException.class, + () -> new ToStringBuilder(base).append("a", (Object) new String[] { "v", "x", "y", "z" }, false).toString()); - assertEquals( - "{\"a\":[\"v\",\"x\",\"y\",\"z\"]}", - new ToStringBuilder(base).append("a", - (Object) new String[]{"v", "x", "y", "z"}, true) - .toString()); + assertEquals("{\"a\":[\"v\",\"x\",\"y\",\"z\"]}", new ToStringBuilder(base).append("a", (Object) new String[] { "v", "x", "y", "z" }, true).toString()); } @Test public void testObjectArray() { - final Object[] array = {null, base, new int[]{3, 6}}; + final Object[] array = { null, base, new int[] { 3, 6 } }; final ToStringBuilder toStringBuilder = new ToStringBuilder(base); assertThrows(UnsupportedOperationException.class, () -> toStringBuilder.append(array).toString()); - assertEquals("{\"objectArray\":[null,5,[3,6]]}", toStringBuilder.append("objectArray", array) - .toString()); + assertEquals("{\"objectArray\":[null,5,[3,6]]}", toStringBuilder.append("objectArray", array).toString()); - assertThrows( - UnsupportedOperationException.class, () -> toStringBuilder.append((Object) array).toString()); + assertThrows(UnsupportedOperationException.class, () -> toStringBuilder.append((Object) array).toString()); - assertThrows( - UnsupportedOperationException.class, () -> toStringBuilder.append((Object[]) null).toString()); + assertThrows(UnsupportedOperationException.class, () -> toStringBuilder.append((Object[]) null).toString()); - assertThrows( - UnsupportedOperationException.class, () -> toStringBuilder.append((Object) array).toString()); + assertThrows(UnsupportedOperationException.class, () -> toStringBuilder.append((Object) array).toString()); } @Test @@ -638,11 +575,10 @@ public class JsonToStringStyleTest extends AbstractLangTest { map.put("k1", "value1"); map.put("k2", 2); - final InnerMapObject object = new InnerMapObject(){ + final InnerMapObject object = new InnerMapObject() { @Override public String toString() { - return new ToStringBuilder(this).append("pid", this.pid) - .append("map", this.map).toString(); + return new ToStringBuilder(this).append("pid", this.pid).append("map", this.map).toString(); } }; object.pid = "dummy-text"; diff --git a/src/test/java/org/apache/commons/lang3/builder/ToStringBuilderTest.java b/src/test/java/org/apache/commons/lang3/builder/ToStringBuilderTest.java index 6eb0bd561..e28b3c13b 100644 --- a/src/test/java/org/apache/commons/lang3/builder/ToStringBuilderTest.java +++ b/src/test/java/org/apache/commons/lang3/builder/ToStringBuilderTest.java @@ -41,17 +41,19 @@ public class ToStringBuilderTest extends AbstractLangTest { static final String staticString2 = "staticString2"; static final int staticInt2 = 67890; } + /** - * Points out failure to print anything from appendToString methods using MULTI_LINE_STYLE. - * See issue LANG-372. + * Points out failure to print anything from appendToString methods using MULTI_LINE_STYLE. See issue LANG-372. */ final class MultiLineTestObject { Integer i = Integer.valueOf(31337); + @Override public String toString() { return new ToStringBuilder(this).append("testInt", i).toString(); } } + static class ObjectCycle { Object obj; @@ -68,7 +70,9 @@ public class ToStringBuilderTest extends AbstractLangTest { return ToStringBuilder.reflectionToString(this); } } + Inner inner = new Inner(); + @Override public String toString() { return ToStringBuilder.reflectionToString(this); @@ -115,16 +119,16 @@ public class ToStringBuilderTest extends AbstractLangTest { static class ReflectionTestFixtureA { @SuppressWarnings("unused") - private final char a='a'; + private final char a = 'a'; @SuppressWarnings("unused") - private final transient char transientA='t'; + private final transient char transientA = 't'; } static class ReflectionTestFixtureB extends ReflectionTestFixtureA { @SuppressWarnings("unused") - private final char b='b'; + private final char b = 'b'; @SuppressWarnings("unused") - private final transient char transientB='t'; + private final transient char transientB = 't'; } private static final class SelfInstanceTwoVarsReflectionTestFixture { @@ -144,7 +148,7 @@ public class ToStringBuilderTest extends AbstractLangTest { public String toString() { return ToStringBuilder.reflectionToString(this); } - } + } private static final class SelfInstanceVarReflectionTestFixture { @SuppressWarnings("unused") @@ -158,7 +162,7 @@ public class ToStringBuilderTest extends AbstractLangTest { public String toString() { return ToStringBuilder.reflectionToString(this); } - } + } /** * Test fixture for ReflectionToStringBuilder.toString() for statics. @@ -194,7 +198,7 @@ public class ToStringBuilderTest extends AbstractLangTest { // // assertReflectionArray("<null>", array); // - // The expected value is not baseStr + "[<null>]" since array==null and is typed as Object. + // The expected value is not baseStr + "[<null>]" since array == null and is typed as Object. // The null array does not carry array type information. // If we added a primitive array type constructor and pile of associated methods, // then type declaring type information could be carried forward. IMHO, null is null. @@ -248,12 +252,9 @@ public class ToStringBuilderTest extends AbstractLangTest { final Boolean objectToAppend2 = Boolean.TRUE; final Object objectToAppend3 = new Object(); - assertEquals(baseStr + "[" + toBaseString(objectToAppend1) + "]", - new ToStringBuilder(base).appendAsObjectToString(objectToAppend1).toString()); - assertEquals(baseStr + "[" + toBaseString(objectToAppend2) + "]", - new ToStringBuilder(base).appendAsObjectToString(objectToAppend2).toString()); - assertEquals(baseStr + "[" + toBaseString(objectToAppend3) + "]", - new ToStringBuilder(base).appendAsObjectToString(objectToAppend3).toString()); + assertEquals(baseStr + "[" + toBaseString(objectToAppend1) + "]", new ToStringBuilder(base).appendAsObjectToString(objectToAppend1).toString()); + assertEquals(baseStr + "[" + toBaseString(objectToAppend2) + "]", new ToStringBuilder(base).appendAsObjectToString(objectToAppend2).toString()); + assertEquals(baseStr + "[" + toBaseString(objectToAppend3) + "]", new ToStringBuilder(base).appendAsObjectToString(objectToAppend3).toString()); } @Test @@ -266,10 +267,8 @@ public class ToStringBuilderTest extends AbstractLangTest { @Test public void testAppendBooleanArrayWithFieldName() { final boolean[] array = { true, false, false }; - assertEquals(baseStr + "[flags={true,false,false}]", - new ToStringBuilder(base).append("flags", array).toString()); - assertEquals(baseStr + "[flags=<null>]", - new ToStringBuilder(base).append("flags", (boolean[]) null).toString()); + assertEquals(baseStr + "[flags={true,false,false}]", new ToStringBuilder(base).append("flags", array).toString()); + assertEquals(baseStr + "[flags=<null>]", new ToStringBuilder(base).append("flags", (boolean[]) null).toString()); assertEquals(baseStr + "[<null>]", new ToStringBuilder(base).append(null, (boolean[]) null).toString()); assertEquals(baseStr + "[{true,false,false}]", new ToStringBuilder(base).append(null, array).toString()); } @@ -277,15 +276,12 @@ public class ToStringBuilderTest extends AbstractLangTest { @Test public void testAppendBooleanArrayWithFieldNameAndFullDetail() { final boolean[] array = { true, false, false }; - assertEquals(baseStr + "[flags={true,false,false}]", - new ToStringBuilder(base).append("flags", array, true).toString()); - assertEquals(baseStr + "[length=<size=3>]", - new ToStringBuilder(base).append("length", array, false).toString()); - assertEquals(baseStr + "[flags=<null>]", - new ToStringBuilder(base).append("flags", (boolean[]) null, true).toString()); + assertEquals(baseStr + "[flags={true,false,false}]", new ToStringBuilder(base).append("flags", array, true).toString()); + assertEquals(baseStr + "[length=<size=3>]", new ToStringBuilder(base).append("length", array, false).toString()); + assertEquals(baseStr + "[flags=<null>]", new ToStringBuilder(base).append("flags", (boolean[]) null, true).toString()); assertEquals(baseStr + "[<null>]", new ToStringBuilder(base).append(null, (boolean[]) null, false).toString()); assertEquals(baseStr + "[<size=3>]", new ToStringBuilder(base).append(null, array, false).toString()); - } + } // Reflection Array Array tests @@ -293,8 +289,7 @@ public class ToStringBuilderTest extends AbstractLangTest { public void testAppendByteArrayWithFieldName() { final byte[] array = { 1, 2, -3, 4 }; assertEquals(baseStr + "[values={1,2,-3,4}]", new ToStringBuilder(base).append("values", array).toString()); - assertEquals(baseStr + "[values=<null>]", - new ToStringBuilder(base).append("values", (boolean[]) null).toString()); + assertEquals(baseStr + "[values=<null>]", new ToStringBuilder(base).append("values", (boolean[]) null).toString()); assertEquals(baseStr + "[<null>]", new ToStringBuilder(base).append(null, (boolean[]) null).toString()); assertEquals(baseStr + "[{1,2,-3,4}]", new ToStringBuilder(base).append(null, array).toString()); } @@ -302,12 +297,9 @@ public class ToStringBuilderTest extends AbstractLangTest { @Test public void testAppendByteArrayWithFieldNameAndFullDetail() { final byte[] array = { 1, 2, -3, 4 }; - assertEquals(baseStr + "[values={1,2,-3,4}]", - new ToStringBuilder(base).append("values", array, true).toString()); - assertEquals(baseStr + "[length=<size=4>]", - new ToStringBuilder(base).append("length", array, false).toString()); - assertEquals(baseStr + "[values=<null>]", - new ToStringBuilder(base).append("values", (boolean[]) null, true).toString()); + assertEquals(baseStr + "[values={1,2,-3,4}]", new ToStringBuilder(base).append("values", array, true).toString()); + assertEquals(baseStr + "[length=<size=4>]", new ToStringBuilder(base).append("length", array, false).toString()); + assertEquals(baseStr + "[values=<null>]", new ToStringBuilder(base).append("values", (boolean[]) null, true).toString()); assertEquals(baseStr + "[<null>]", new ToStringBuilder(base).append(null, (boolean[]) null, false).toString()); assertEquals(baseStr + "[<size=4>]", new ToStringBuilder(base).append(null, array, false).toString()); } @@ -317,8 +309,7 @@ public class ToStringBuilderTest extends AbstractLangTest { final char[] array = { 'A', '2', '_', 'D' }; assertEquals(baseStr + "[chars={A,2,_,D}]", new ToStringBuilder(base).append("chars", array).toString()); assertEquals(baseStr + "[letters={A,2,_,D}]", new ToStringBuilder(base).append("letters", array).toString()); - assertEquals(baseStr + "[flags=<null>]", - new ToStringBuilder(base).append("flags", (boolean[]) null).toString()); + assertEquals(baseStr + "[flags=<null>]", new ToStringBuilder(base).append("flags", (boolean[]) null).toString()); assertEquals(baseStr + "[<null>]", new ToStringBuilder(base).append(null, (boolean[]) null).toString()); assertEquals(baseStr + "[{A,2,_,D}]", new ToStringBuilder(base).append(null, array).toString()); } @@ -327,10 +318,8 @@ public class ToStringBuilderTest extends AbstractLangTest { public void testAppendCharArrayWithFieldNameAndFullDetail() { final char[] array = { 'A', '2', '_', 'D' }; assertEquals(baseStr + "[chars={A,2,_,D}]", new ToStringBuilder(base).append("chars", array, true).toString()); - assertEquals(baseStr + "[letters=<size=4>]", - new ToStringBuilder(base).append("letters", array, false).toString()); - assertEquals(baseStr + "[flags=<null>]", - new ToStringBuilder(base).append("flags", (boolean[]) null, true).toString()); + assertEquals(baseStr + "[letters=<size=4>]", new ToStringBuilder(base).append("letters", array, false).toString()); + assertEquals(baseStr + "[flags=<null>]", new ToStringBuilder(base).append("flags", (boolean[]) null, true).toString()); assertEquals(baseStr + "[<null>]", new ToStringBuilder(base).append(null, (boolean[]) null, false).toString()); assertEquals(baseStr + "[<size=4>]", new ToStringBuilder(base).append(null, array, false).toString()); } @@ -338,10 +327,8 @@ public class ToStringBuilderTest extends AbstractLangTest { @Test public void testAppendDoubleArrayWithFieldName() { final double[] array = { 1.0, 2.9876, -3.00001, 4.3 }; - assertEquals(baseStr + "[values={1.0,2.9876,-3.00001,4.3}]", - new ToStringBuilder(base).append("values", array).toString()); - assertEquals(baseStr + "[values=<null>]", - new ToStringBuilder(base).append("values", (boolean[]) null).toString()); + assertEquals(baseStr + "[values={1.0,2.9876,-3.00001,4.3}]", new ToStringBuilder(base).append("values", array).toString()); + assertEquals(baseStr + "[values=<null>]", new ToStringBuilder(base).append("values", (boolean[]) null).toString()); assertEquals(baseStr + "[<null>]", new ToStringBuilder(base).append(null, (boolean[]) null).toString()); assertEquals(baseStr + "[{1.0,2.9876,-3.00001,4.3}]", new ToStringBuilder(base).append(null, array).toString()); } @@ -349,12 +336,9 @@ public class ToStringBuilderTest extends AbstractLangTest { @Test public void testAppendDoubleArrayWithFieldNameAndFullDetail() { final double[] array = { 1.0, 2.9876, -3.00001, 4.3 }; - assertEquals(baseStr + "[values={1.0,2.9876,-3.00001,4.3}]", - new ToStringBuilder(base).append("values", array, true).toString()); - assertEquals(baseStr + "[length=<size=4>]", - new ToStringBuilder(base).append("length", array, false).toString()); - assertEquals(baseStr + "[values=<null>]", - new ToStringBuilder(base).append("values", (boolean[]) null, true).toString()); + assertEquals(baseStr + "[values={1.0,2.9876,-3.00001,4.3}]", new ToStringBuilder(base).append("values", array, true).toString()); + assertEquals(baseStr + "[length=<size=4>]", new ToStringBuilder(base).append("length", array, false).toString()); + assertEquals(baseStr + "[values=<null>]", new ToStringBuilder(base).append("values", (boolean[]) null, true).toString()); assertEquals(baseStr + "[<null>]", new ToStringBuilder(base).append(null, (boolean[]) null, false).toString()); assertEquals(baseStr + "[<size=4>]", new ToStringBuilder(base).append(null, array, false).toString()); } @@ -362,10 +346,8 @@ public class ToStringBuilderTest extends AbstractLangTest { @Test public void testAppendFloatArrayWithFieldName() { final float[] array = { 1.0f, 2.9876f, -3.00001f, 4.3f }; - assertEquals(baseStr + "[values={1.0,2.9876,-3.00001,4.3}]", - new ToStringBuilder(base).append("values", array).toString()); - assertEquals(baseStr + "[values=<null>]", - new ToStringBuilder(base).append("values", (boolean[]) null).toString()); + assertEquals(baseStr + "[values={1.0,2.9876,-3.00001,4.3}]", new ToStringBuilder(base).append("values", array).toString()); + assertEquals(baseStr + "[values=<null>]", new ToStringBuilder(base).append("values", (boolean[]) null).toString()); assertEquals(baseStr + "[<null>]", new ToStringBuilder(base).append(null, (boolean[]) null).toString()); assertEquals(baseStr + "[{1.0,2.9876,-3.00001,4.3}]", new ToStringBuilder(base).append(null, array).toString()); } @@ -373,12 +355,9 @@ public class ToStringBuilderTest extends AbstractLangTest { @Test public void testAppendFloatArrayWithFieldNameAndFullDetail() { final float[] array = { 1.0f, 2.9876f, -3.00001f, 4.3f }; - assertEquals(baseStr + "[values={1.0,2.9876,-3.00001,4.3}]", - new ToStringBuilder(base).append("values", array, true).toString()); - assertEquals(baseStr + "[length=<size=4>]", - new ToStringBuilder(base).append("length", array, false).toString()); - assertEquals(baseStr + "[values=<null>]", - new ToStringBuilder(base).append("values", (boolean[]) null, true).toString()); + assertEquals(baseStr + "[values={1.0,2.9876,-3.00001,4.3}]", new ToStringBuilder(base).append("values", array, true).toString()); + assertEquals(baseStr + "[length=<size=4>]", new ToStringBuilder(base).append("length", array, false).toString()); + assertEquals(baseStr + "[values=<null>]", new ToStringBuilder(base).append("values", (boolean[]) null, true).toString()); assertEquals(baseStr + "[<null>]", new ToStringBuilder(base).append(null, (boolean[]) null, false).toString()); assertEquals(baseStr + "[<size=4>]", new ToStringBuilder(base).append(null, array, false).toString()); } @@ -387,8 +366,7 @@ public class ToStringBuilderTest extends AbstractLangTest { public void testAppendIntArrayWithFieldName() { final int[] array = { 1, 2, -3, 4 }; assertEquals(baseStr + "[values={1,2,-3,4}]", new ToStringBuilder(base).append("values", array).toString()); - assertEquals(baseStr + "[values=<null>]", - new ToStringBuilder(base).append("values", (boolean[]) null).toString()); + assertEquals(baseStr + "[values=<null>]", new ToStringBuilder(base).append("values", (boolean[]) null).toString()); assertEquals(baseStr + "[<null>]", new ToStringBuilder(base).append(null, (boolean[]) null).toString()); assertEquals(baseStr + "[{1,2,-3,4}]", new ToStringBuilder(base).append(null, array).toString()); } @@ -396,35 +374,28 @@ public class ToStringBuilderTest extends AbstractLangTest { @Test public void testAppendIntArrayWithFieldNameAndFullDetail() { final int[] array = { 1, 2, -3, 4 }; - assertEquals(baseStr + "[values={1,2,-3,4}]", - new ToStringBuilder(base).append("values", array, true).toString()); - assertEquals(baseStr + "[length=<size=4>]", - new ToStringBuilder(base).append("length", array, false).toString()); - assertEquals(baseStr + "[values=<null>]", - new ToStringBuilder(base).append("values", (boolean[]) null, true).toString()); + assertEquals(baseStr + "[values={1,2,-3,4}]", new ToStringBuilder(base).append("values", array, true).toString()); + assertEquals(baseStr + "[length=<size=4>]", new ToStringBuilder(base).append("length", array, false).toString()); + assertEquals(baseStr + "[values=<null>]", new ToStringBuilder(base).append("values", (boolean[]) null, true).toString()); assertEquals(baseStr + "[<null>]", new ToStringBuilder(base).append(null, (boolean[]) null, false).toString()); assertEquals(baseStr + "[<size=4>]", new ToStringBuilder(base).append(null, array, false).toString()); } @Test public void testAppendLongArrayWithFieldName() { - final long[] array = { 1, 2, -3, 4 }; - assertEquals(baseStr + "[values={1,2,-3,4}]", new ToStringBuilder(base).append("values", array).toString()); - assertEquals(baseStr + "[values=<null>]", - new ToStringBuilder(base).append("values", (boolean[]) null).toString()); - assertEquals(baseStr + "[<null>]", new ToStringBuilder(base).append(null, (boolean[]) null).toString()); - assertEquals(baseStr + "[{1,2,-3,4}]", new ToStringBuilder(base).append(null, array).toString()); + final long[] array = { 1, 2, -3, 4 }; + assertEquals(baseStr + "[values={1,2,-3,4}]", new ToStringBuilder(base).append("values", array).toString()); + assertEquals(baseStr + "[values=<null>]", new ToStringBuilder(base).append("values", (boolean[]) null).toString()); + assertEquals(baseStr + "[<null>]", new ToStringBuilder(base).append(null, (boolean[]) null).toString()); + assertEquals(baseStr + "[{1,2,-3,4}]", new ToStringBuilder(base).append(null, array).toString()); } @Test public void testAppendLongArrayWithFieldNameAndFullDetail() { final long[] array = { 1, 2, -3, 4 }; - assertEquals(baseStr + "[values={1,2,-3,4}]", - new ToStringBuilder(base).append("values", array, true).toString()); - assertEquals(baseStr + "[length=<size=4>]", - new ToStringBuilder(base).append("length", array, false).toString()); - assertEquals(baseStr + "[values=<null>]", - new ToStringBuilder(base).append("values", (boolean[]) null, true).toString()); + assertEquals(baseStr + "[values={1,2,-3,4}]", new ToStringBuilder(base).append("values", array, true).toString()); + assertEquals(baseStr + "[length=<size=4>]", new ToStringBuilder(base).append("length", array, false).toString()); + assertEquals(baseStr + "[values=<null>]", new ToStringBuilder(base).append("values", (boolean[]) null, true).toString()); assertEquals(baseStr + "[<null>]", new ToStringBuilder(base).append(null, (boolean[]) null, false).toString()); assertEquals(baseStr + "[<size=4>]", new ToStringBuilder(base).append(null, array, false).toString()); } @@ -432,25 +403,20 @@ public class ToStringBuilderTest extends AbstractLangTest { @Test public void testAppendObjectArrayWithFieldName() { final Object[] array = { null, base, new int[] { 3, 6 } }; - assertEquals(baseStr + "[values={<null>,5,{3,6}}]", - new ToStringBuilder(base).append("values", array).toString()); - assertEquals(baseStr + "[values=<null>]", - new ToStringBuilder(base).append("values", (boolean[]) null).toString()); + assertEquals(baseStr + "[values={<null>,5,{3,6}}]", new ToStringBuilder(base).append("values", array).toString()); + assertEquals(baseStr + "[values=<null>]", new ToStringBuilder(base).append("values", (boolean[]) null).toString()); assertEquals(baseStr + "[<null>]", new ToStringBuilder(base).append(null, (boolean[]) null).toString()); assertEquals(baseStr + "[{<null>,5,{3,6}}]", new ToStringBuilder(base).append(null, array).toString()); } @Test public void testAppendObjectArrayWithFieldNameAndFullDetail() { - final Object[] array = { null, base, new int[] { 3, 6 } }; - assertEquals(baseStr + "[values={<null>,5,{3,6}}]", - new ToStringBuilder(base).append("values", array, true).toString()); - assertEquals(baseStr + "[length=<size=3>]", - new ToStringBuilder(base).append("length", array, false).toString()); - assertEquals(baseStr + "[values=<null>]", - new ToStringBuilder(base).append("values", (boolean[]) null, true).toString()); - assertEquals(baseStr + "[<null>]", new ToStringBuilder(base).append(null, (boolean[]) null, false).toString()); - assertEquals(baseStr + "[<size=3>]", new ToStringBuilder(base).append(null, array, false).toString()); + final Object[] array = { null, base, new int[] { 3, 6 } }; + assertEquals(baseStr + "[values={<null>,5,{3,6}}]", new ToStringBuilder(base).append("values", array, true).toString()); + assertEquals(baseStr + "[length=<size=3>]", new ToStringBuilder(base).append("length", array, false).toString()); + assertEquals(baseStr + "[values=<null>]", new ToStringBuilder(base).append("values", (boolean[]) null, true).toString()); + assertEquals(baseStr + "[<null>]", new ToStringBuilder(base).append(null, (boolean[]) null, false).toString()); + assertEquals(baseStr + "[<size=3>]", new ToStringBuilder(base).append(null, array, false).toString()); } // Reflection cycle tests @@ -459,8 +425,7 @@ public class ToStringBuilderTest extends AbstractLangTest { public void testAppendShortArrayWithFieldName() { final short[] array = { 1, 2, -3, 4 }; assertEquals(baseStr + "[values={1,2,-3,4}]", new ToStringBuilder(base).append("values", array).toString()); - assertEquals(baseStr + "[values=<null>]", - new ToStringBuilder(base).append("values", (boolean[]) null).toString()); + assertEquals(baseStr + "[values=<null>]", new ToStringBuilder(base).append("values", (boolean[]) null).toString()); assertEquals(baseStr + "[<null>]", new ToStringBuilder(base).append(null, (boolean[]) null).toString()); assertEquals(baseStr + "[{1,2,-3,4}]", new ToStringBuilder(base).append(null, array).toString()); } @@ -468,12 +433,9 @@ public class ToStringBuilderTest extends AbstractLangTest { @Test public void testAppendShortArrayWithFieldNameAndFullDetail() { final short[] array = { 1, 2, -3, 4 }; - assertEquals(baseStr + "[values={1,2,-3,4}]", - new ToStringBuilder(base).append("values", array, true).toString()); - assertEquals(baseStr + "[length=<size=4>]", - new ToStringBuilder(base).append("length", array, false).toString()); - assertEquals(baseStr + "[values=<null>]", - new ToStringBuilder(base).append("values", (boolean[]) null, true).toString()); + assertEquals(baseStr + "[values={1,2,-3,4}]", new ToStringBuilder(base).append("values", array, true).toString()); + assertEquals(baseStr + "[length=<size=4>]", new ToStringBuilder(base).append("length", array, false).toString()); + assertEquals(baseStr + "[values=<null>]", new ToStringBuilder(base).append("values", (boolean[]) null, true).toString()); assertEquals(baseStr + "[<null>]", new ToStringBuilder(base).append(null, (boolean[]) null, false).toString()); assertEquals(baseStr + "[<size=4>]", new ToStringBuilder(base).append(null, array, false).toString()); } @@ -501,8 +463,7 @@ public class ToStringBuilderTest extends AbstractLangTest { @Test public void testAppendToStringUsingMultiLineStyle() { final MultiLineTestObject obj = new MultiLineTestObject(); - final ToStringBuilder testBuilder = new ToStringBuilder(this, ToStringStyle.MULTI_LINE_STYLE) - .appendToString(obj.toString()); + final ToStringBuilder testBuilder = new ToStringBuilder(this, ToStringStyle.MULTI_LINE_STYLE).appendToString(obj.toString()); assertEquals(-1, testBuilder.toString().indexOf("testInt=31337")); } @@ -520,7 +481,7 @@ public class ToStringBuilderTest extends AbstractLangTest { @Test public void testBooleanArray() { - boolean[] array = {true, false, false}; + boolean[] array = { true, false, false }; assertEquals(baseStr + "[{true,false,false}]", new ToStringBuilder(base).append(array).toString()); assertEquals(baseStr + "[{true,false,false}]", new ToStringBuilder(base).append((Object) array).toString()); array = null; @@ -530,7 +491,7 @@ public class ToStringBuilderTest extends AbstractLangTest { @Test public void testBooleanArrayArray() { - boolean[][] array = {{true, false}, null, {false}}; + boolean[][] array = { { true, false }, null, { false } }; assertEquals(baseStr + "[{{true,false},<null>,{false}}]", new ToStringBuilder(base).append(array).toString()); assertEquals(baseStr + "[{{true,false},<null>,{false}}]", new ToStringBuilder(base).append((Object) array).toString()); array = null; @@ -547,7 +508,7 @@ public class ToStringBuilderTest extends AbstractLangTest { @Test public void testByteArray() { - byte[] array = {1, 2, -3, 4}; + byte[] array = { 1, 2, -3, 4 }; assertEquals(baseStr + "[{1,2,-3,4}]", new ToStringBuilder(base).append(array).toString()); assertEquals(baseStr + "[{1,2,-3,4}]", new ToStringBuilder(base).append((Object) array).toString()); array = null; @@ -557,7 +518,7 @@ public class ToStringBuilderTest extends AbstractLangTest { @Test public void testByteArrayArray() { - byte[][] array = {{1, 2}, null, {5}}; + byte[][] array = { { 1, 2 }, null, { 5 } }; assertEquals(baseStr + "[{{1,2},<null>,{5}}]", new ToStringBuilder(base).append(array).toString()); assertEquals(baseStr + "[{{1,2},<null>,{5}}]", new ToStringBuilder(base).append((Object) array).toString()); array = null; @@ -574,7 +535,7 @@ public class ToStringBuilderTest extends AbstractLangTest { @Test public void testCharArray() { - char[] array = {'A', '2', '_', 'D'}; + char[] array = { 'A', '2', '_', 'D' }; assertEquals(baseStr + "[{A,2,_,D}]", new ToStringBuilder(base).append(array).toString()); assertEquals(baseStr + "[{A,2,_,D}]", new ToStringBuilder(base).append((Object) array).toString()); array = null; @@ -584,7 +545,7 @@ public class ToStringBuilderTest extends AbstractLangTest { @Test public void testCharArrayArray() { - char[][] array = {{'A', 'B'}, null, {'p'}}; + char[][] array = { { 'A', 'B' }, null, { 'p' } }; assertEquals(baseStr + "[{{A,B},<null>,{p}}]", new ToStringBuilder(base).append(array).toString()); assertEquals(baseStr + "[{{A,B},<null>,{p}}]", new ToStringBuilder(base).append((Object) array).toString()); array = null; @@ -631,7 +592,7 @@ public class ToStringBuilderTest extends AbstractLangTest { @Test public void testDoubleArray() { - double[] array = {1.0, 2.9876, -3.00001, 4.3}; + double[] array = { 1.0, 2.9876, -3.00001, 4.3 }; assertEquals(baseStr + "[{1.0,2.9876,-3.00001,4.3}]", new ToStringBuilder(base).append(array).toString()); assertEquals(baseStr + "[{1.0,2.9876,-3.00001,4.3}]", new ToStringBuilder(base).append((Object) array).toString()); array = null; @@ -641,7 +602,7 @@ public class ToStringBuilderTest extends AbstractLangTest { @Test public void testDoubleArrayArray() { - double[][] array = {{1.0, 2.29686}, null, {Double.NaN}}; + double[][] array = { { 1.0, 2.29686 }, null, { Double.NaN } }; assertEquals(baseStr + "[{{1.0,2.29686},<null>,{NaN}}]", new ToStringBuilder(base).append(array).toString()); assertEquals(baseStr + "[{{1.0,2.29686},<null>,{NaN}}]", new ToStringBuilder(base).append((Object) array).toString()); array = null; @@ -658,7 +619,7 @@ public class ToStringBuilderTest extends AbstractLangTest { @Test public void testFloatArray() { - float[] array = {1.0f, 2.9876f, -3.00001f, 4.3f}; + float[] array = { 1.0f, 2.9876f, -3.00001f, 4.3f }; assertEquals(baseStr + "[{1.0,2.9876,-3.00001,4.3}]", new ToStringBuilder(base).append(array).toString()); assertEquals(baseStr + "[{1.0,2.9876,-3.00001,4.3}]", new ToStringBuilder(base).append((Object) array).toString()); array = null; @@ -668,7 +629,7 @@ public class ToStringBuilderTest extends AbstractLangTest { @Test public void testFloatArrayArray() { - float[][] array = {{1.0f, 2.29686f}, null, {Float.NaN}}; + float[][] array = { { 1.0f, 2.29686f }, null, { Float.NaN } }; assertEquals(baseStr + "[{{1.0,2.29686},<null>,{NaN}}]", new ToStringBuilder(base).append(array).toString()); assertEquals(baseStr + "[{{1.0,2.29686},<null>,{NaN}}]", new ToStringBuilder(base).append((Object) array).toString()); array = null; @@ -693,18 +654,14 @@ public class ToStringBuilderTest extends AbstractLangTest { @Test public void testInheritedReflectionStatics() { final InheritedReflectionStaticFieldsFixture instance1 = new InheritedReflectionStaticFieldsFixture(); - assertEquals( - this.toBaseString(instance1) + "[staticInt2=67890,staticString2=staticString2]", - ReflectionToStringBuilder.toString(instance1, null, false, true, InheritedReflectionStaticFieldsFixture.class)); - assertEquals( - this.toBaseString(instance1) + "[staticInt2=67890,staticString2=staticString2,staticInt=12345,staticString=staticString]", - ReflectionToStringBuilder.toString(instance1, null, false, true, SimpleReflectionStaticFieldsFixture.class)); - assertEquals( - this.toBaseString(instance1) + "[staticInt2=67890,staticString2=staticString2,staticInt=12345,staticString=staticString]", - this.toStringWithStatics(instance1, null, SimpleReflectionStaticFieldsFixture.class)); - assertEquals( - this.toBaseString(instance1) + "[staticInt2=67890,staticString2=staticString2,staticInt=12345,staticString=staticString]", - this.toStringWithStatics(instance1, null, SimpleReflectionStaticFieldsFixture.class)); + assertEquals(this.toBaseString(instance1) + "[staticInt2=67890,staticString2=staticString2]", + ReflectionToStringBuilder.toString(instance1, null, false, true, InheritedReflectionStaticFieldsFixture.class)); + assertEquals(this.toBaseString(instance1) + "[staticInt2=67890,staticString2=staticString2,staticInt=12345,staticString=staticString]", + ReflectionToStringBuilder.toString(instance1, null, false, true, SimpleReflectionStaticFieldsFixture.class)); + assertEquals(this.toBaseString(instance1) + "[staticInt2=67890,staticString2=staticString2,staticInt=12345,staticString=staticString]", + this.toStringWithStatics(instance1, null, SimpleReflectionStaticFieldsFixture.class)); + assertEquals(this.toBaseString(instance1) + "[staticInt2=67890,staticString2=staticString2,staticInt=12345,staticString=staticString]", + this.toStringWithStatics(instance1, null, SimpleReflectionStaticFieldsFixture.class)); } @Test @@ -722,7 +679,7 @@ public class ToStringBuilderTest extends AbstractLangTest { @Test public void testIntArray() { - int[] array = {1, 2, -3, 4}; + int[] array = { 1, 2, -3, 4 }; assertEquals(baseStr + "[{1,2,-3,4}]", new ToStringBuilder(base).append(array).toString()); assertEquals(baseStr + "[{1,2,-3,4}]", new ToStringBuilder(base).append((Object) array).toString()); array = null; @@ -732,7 +689,7 @@ public class ToStringBuilderTest extends AbstractLangTest { @Test public void testIntArrayArray() { - int[][] array = {{1, 2}, null, {5}}; + int[][] array = { { 1, 2 }, null, { 5 } }; assertEquals(baseStr + "[{{1,2},<null>,{5}}]", new ToStringBuilder(base).append(array).toString()); assertEquals(baseStr + "[{{1,2},<null>,{5}}]", new ToStringBuilder(base).append((Object) array).toString()); array = null; @@ -749,7 +706,7 @@ public class ToStringBuilderTest extends AbstractLangTest { @Test public void testLongArray() { - long[] array = {1, 2, -3, 4}; + long[] array = { 1, 2, -3, 4 }; assertEquals(baseStr + "[{1,2,-3,4}]", new ToStringBuilder(base).append(array).toString()); assertEquals(baseStr + "[{1,2,-3,4}]", new ToStringBuilder(base).append((Object) array).toString()); array = null; @@ -759,7 +716,7 @@ public class ToStringBuilderTest extends AbstractLangTest { @Test public void testLongArrayArray() { - long[][] array = {{1, 2}, null, {5}}; + long[][] array = { { 1, 2 }, null, { 5 } }; assertEquals(baseStr + "[{{1,2},<null>,{5}}]", new ToStringBuilder(base).append(array).toString()); assertEquals(baseStr + "[{{1,2},<null>,{5}}]", new ToStringBuilder(base).append((Object) array).toString()); array = null; @@ -787,7 +744,7 @@ public class ToStringBuilderTest extends AbstractLangTest { @Test public void testObjectArray() { - Object[] array = {null, base, new int[] {3, 6}}; + Object[] array = { null, base, new int[] { 3, 6 } }; assertEquals(baseStr + "[{<null>,5,{3,6}}]", new ToStringBuilder(base).append(array).toString()); assertEquals(baseStr + "[{<null>,5,{3,6}}]", new ToStringBuilder(base).append((Object) array).toString()); array = null; @@ -825,29 +782,16 @@ public class ToStringBuilderTest extends AbstractLangTest { } /** - * Test a nasty combination of arrays and Objects pointing to each other. - * objects[0] -> SimpleReflectionTestFixture[ o -> objects ] + * Test a nasty combination of arrays and Objects pointing to each other. objects[0] -> SimpleReflectionTestFixture[ o -> objects ] */ @Test public void testReflectionArrayAndObjectCycle() { final Object[] objects = new Object[1]; final SimpleReflectionTestFixture simple = new SimpleReflectionTestFixture(objects); objects[0] = simple; - assertEquals( - this.toBaseString(objects) - + "[{" - + this.toBaseString(simple) - + "[o=" - + this.toBaseString(objects) - + "]" - + "}]", - ToStringBuilder.reflectionToString(objects)); - assertEquals( - this.toBaseString(simple) - + "[o={" - + this.toBaseString(simple) - + "}]", - ToStringBuilder.reflectionToString(simple)); + assertEquals(this.toBaseString(objects) + "[{" + this.toBaseString(simple) + "[o=" + this.toBaseString(objects) + "]" + "}]", + ToStringBuilder.reflectionToString(objects)); + assertEquals(this.toBaseString(simple) + "[o={" + this.toBaseString(simple) + "}]", ToStringBuilder.reflectionToString(simple)); } @Test @@ -858,18 +802,8 @@ public class ToStringBuilderTest extends AbstractLangTest { objects[1][0] = objects; objects[1][1] = objects; final String basicToString = this.toBaseString(objects); - assertEquals( - basicToString - + "[{{" - + basicToString - + "," - + basicToString - + "},{" - + basicToString - + "," - + basicToString - + "}}]", - ToStringBuilder.reflectionToString(objects)); + assertEquals(basicToString + "[{{" + basicToString + "," + basicToString + "},{" + basicToString + "," + basicToString + "}}]", + ToStringBuilder.reflectionToString(objects)); } /** @@ -879,9 +813,7 @@ public class ToStringBuilderTest extends AbstractLangTest { public void testReflectionArrayCycle() { final Object[] objects = new Object[1]; objects[0] = objects; - assertEquals( - this.toBaseString(objects) + "[{" + this.toBaseString(objects) + "}]", - ToStringBuilder.reflectionToString(objects)); + assertEquals(this.toBaseString(objects) + "[{" + this.toBaseString(objects) + "}]", ToStringBuilder.reflectionToString(objects)); } /** @@ -893,12 +825,8 @@ public class ToStringBuilderTest extends AbstractLangTest { final Object[] objectsLevel2 = new Object[1]; objects[0] = objectsLevel2; objectsLevel2[0] = objects; - assertEquals( - this.toBaseString(objects) + "[{{" + this.toBaseString(objects) + "}}]", - ToStringBuilder.reflectionToString(objects)); - assertEquals( - this.toBaseString(objectsLevel2) + "[{{" + this.toBaseString(objectsLevel2) + "}}]", - ToStringBuilder.reflectionToString(objectsLevel2)); + assertEquals(this.toBaseString(objects) + "[{{" + this.toBaseString(objects) + "}}]", ToStringBuilder.reflectionToString(objects)); + assertEquals(this.toBaseString(objectsLevel2) + "[{{" + this.toBaseString(objectsLevel2) + "}}]", ToStringBuilder.reflectionToString(objectsLevel2)); } /** @@ -1044,7 +972,8 @@ public class ToStringBuilderTest extends AbstractLangTest { // LANG-1337 without this, the generated string can differ depending on the JVM version/vendor final List<Object> list = new ArrayList<>(ARRAYLIST_INITIAL_CAPACITY); final String baseString = this.toBaseString(list); - final String expectedWithTransients = baseString + "[elementData={<null>,<null>,<null>,<null>,<null>,<null>,<null>,<null>,<null>,<null>},size=0,modCount=0]"; + final String expectedWithTransients = baseString + + "[elementData={<null>,<null>,<null>,<null>,<null>,<null>,<null>,<null>,<null>,<null>},size=0,modCount=0]"; final String toStringWithTransients = ToStringBuilder.reflectionToString(list, null, true); if (!expectedWithTransients.equals(toStringWithTransients)) { assertEquals(expectedWithTransients, toStringWithTransients); @@ -1123,9 +1052,7 @@ public class ToStringBuilderTest extends AbstractLangTest { final ReflectionTestCycleB b = new ReflectionTestCycleB(); a.b = b; b.a = a; - assertEquals( - this.toBaseString(a) + "[b=" + this.toBaseString(b) + "[a=" + this.toBaseString(a) + "]]", - a.toString()); + assertEquals(this.toBaseString(a) + "[b=" + this.toBaseString(b) + "[a=" + this.toBaseString(a) + "]]", a.toString()); } @Test @@ -1152,28 +1079,26 @@ public class ToStringBuilderTest extends AbstractLangTest { @Test public void testReflectionStatics() { final ReflectionStaticFieldsFixture instance1 = new ReflectionStaticFieldsFixture(); - assertEquals( - this.toBaseString(instance1) + "[instanceInt=67890,instanceString=instanceString,staticInt=12345,staticString=staticString]", - ReflectionToStringBuilder.toString(instance1, null, false, true, ReflectionStaticFieldsFixture.class)); - assertEquals( - this.toBaseString(instance1) + "[instanceInt=67890,instanceString=instanceString,staticInt=12345,staticString=staticString,staticTransientInt=54321,staticTransientString=staticTransientString,transientInt=98765,transientString=transientString]", - ReflectionToStringBuilder.toString(instance1, null, true, true, ReflectionStaticFieldsFixture.class)); - assertEquals( - this.toBaseString(instance1) + "[instanceInt=67890,instanceString=instanceString,staticInt=12345,staticString=staticString]", - this.toStringWithStatics(instance1, null, ReflectionStaticFieldsFixture.class)); - assertEquals( - this.toBaseString(instance1) + "[instanceInt=67890,instanceString=instanceString,staticInt=12345,staticString=staticString]", - this.toStringWithStatics(instance1, null, ReflectionStaticFieldsFixture.class)); + assertEquals(this.toBaseString(instance1) + "[instanceInt=67890,instanceString=instanceString,staticInt=12345,staticString=staticString]", + ReflectionToStringBuilder.toString(instance1, null, false, true, ReflectionStaticFieldsFixture.class)); + assertEquals(this.toBaseString(instance1) + + "[instanceInt=67890,instanceString=instanceString,staticInt=12345,staticString=staticString,staticTransientInt=54321,staticTransientString=staticTransientString,transientInt=98765,transientString=transientString]", + ReflectionToStringBuilder.toString(instance1, null, true, true, ReflectionStaticFieldsFixture.class)); + assertEquals(this.toBaseString(instance1) + "[instanceInt=67890,instanceString=instanceString,staticInt=12345,staticString=staticString]", + this.toStringWithStatics(instance1, null, ReflectionStaticFieldsFixture.class)); + assertEquals(this.toBaseString(instance1) + "[instanceInt=67890,instanceString=instanceString,staticInt=12345,staticString=staticString]", + this.toStringWithStatics(instance1, null, ReflectionStaticFieldsFixture.class)); } /** - * Test a class that defines an ivar pointing to itself. This test was - * created to show that handling cyclical object resulted in a missing endFieldSeparator call. + * Test a class that defines an ivar pointing to itself. This test was created to show that handling cyclical object resulted in a missing endFieldSeparator + * call. */ @Test public void testSelfInstanceTwoVarsReflectionObjectCycle() { final SelfInstanceTwoVarsReflectionTestFixture test = new SelfInstanceTwoVarsReflectionTestFixture(); - assertEquals(this.toBaseString(test) + "[otherType=" + test.getOtherType().toString() + ",typeIsSelf=" + this.toBaseString(test) + "]", test.toString()); + assertEquals(this.toBaseString(test) + "[otherType=" + test.getOtherType().toString() + ",typeIsSelf=" + this.toBaseString(test) + "]", + test.toString()); } /** @@ -1199,7 +1124,7 @@ public class ToStringBuilderTest extends AbstractLangTest { @Test public void testShortArray() { - short[] array = {1, 2, -3, 4}; + short[] array = { 1, 2, -3, 4 }; assertEquals(baseStr + "[{1,2,-3,4}]", new ToStringBuilder(base).append(array).toString()); assertEquals(baseStr + "[{1,2,-3,4}]", new ToStringBuilder(base).append((Object) array).toString()); array = null; @@ -1209,7 +1134,7 @@ public class ToStringBuilderTest extends AbstractLangTest { @Test public void testShortArrayArray() { - short[][] array = {{1, 2}, null, {5}}; + short[][] array = { { 1, 2 }, null, { 5 } }; assertEquals(baseStr + "[{{1,2},<null>,{5}}]", new ToStringBuilder(base).append(array).toString()); assertEquals(baseStr + "[{{1,2},<null>,{5}}]", new ToStringBuilder(base).append((Object) array).toString()); array = null; @@ -1230,22 +1155,19 @@ public class ToStringBuilderTest extends AbstractLangTest { @Test public void testSimpleReflectionStatics() { final SimpleReflectionStaticFieldsFixture instance1 = new SimpleReflectionStaticFieldsFixture(); - assertEquals( - this.toBaseString(instance1) + "[staticInt=12345,staticString=staticString]", - ReflectionToStringBuilder.toString(instance1, null, false, true, SimpleReflectionStaticFieldsFixture.class)); - assertEquals( - this.toBaseString(instance1) + "[staticInt=12345,staticString=staticString]", - ReflectionToStringBuilder.toString(instance1, null, true, true, SimpleReflectionStaticFieldsFixture.class)); - assertEquals( - this.toBaseString(instance1) + "[staticInt=12345,staticString=staticString]", - this.toStringWithStatics(instance1, null, SimpleReflectionStaticFieldsFixture.class)); - assertEquals( - this.toBaseString(instance1) + "[staticInt=12345,staticString=staticString]", - this.toStringWithStatics(instance1, null, SimpleReflectionStaticFieldsFixture.class)); + assertEquals(this.toBaseString(instance1) + "[staticInt=12345,staticString=staticString]", + ReflectionToStringBuilder.toString(instance1, null, false, true, SimpleReflectionStaticFieldsFixture.class)); + assertEquals(this.toBaseString(instance1) + "[staticInt=12345,staticString=staticString]", + ReflectionToStringBuilder.toString(instance1, null, true, true, SimpleReflectionStaticFieldsFixture.class)); + assertEquals(this.toBaseString(instance1) + "[staticInt=12345,staticString=staticString]", + this.toStringWithStatics(instance1, null, SimpleReflectionStaticFieldsFixture.class)); + assertEquals(this.toBaseString(instance1) + "[staticInt=12345,staticString=staticString]", + this.toStringWithStatics(instance1, null, SimpleReflectionStaticFieldsFixture.class)); } /** * Create the same toString() as Object.toString(). + * * @param o the object to create the string for. * @return a String in the Object.toString format. */ @@ -1254,28 +1176,25 @@ public class ToStringBuilderTest extends AbstractLangTest { } /** - * <p>This method uses reflection to build a suitable - * {@code toString} value which includes static fields.</p> - * - * <p>It uses {@code AccessibleObject.setAccessible} to gain access to private - * fields. This means that it will throw a security exception if run - * under a security manager, if the permissions are not set up correctly. - * It is also not as efficient as testing explicitly.</p> - * - * <p>Transient fields are not output.</p> - * - * <p>Superclass fields will be appended up to and including the specified superclass. - * A null superclass is treated as {@link Object}.</p> - * - * <p>If the style is {@code null}, the default - * {@code ToStringStyle} is used.</p> + * This method uses reflection to build a suitable {@code toString} value which includes static fields. + * <p> + * It uses {@code AccessibleObject.setAccessible} to gain access to private fields. This means that it will throw a security exception if run under a + * security manager, if the permissions are not set up correctly. It is also not as efficient as testing explicitly. + * </p> + * <p> + * Transient fields are not output. + * </p> + * <p> + * Superclass fields will be appended up to and including the specified superclass. A null superclass is treated as {@link Object}. + * </p> + * <p> + * If the style is {@code null}, the default {@code ToStringStyle} is used. + * </p> * - * @param <T> the type of the output object - * @param object the Object to be output - * @param style the style of the {@code toString} to create, - * may be {@code null} - * @param reflectUpToClass the superclass to reflect up to (inclusive), - * may be {@code null} + * @param <T> the type of the output object + * @param object the Object to be output + * @param style the style of the {@code toString} to create, may be {@code null} + * @param reflectUpToClass the superclass to reflect up to (inclusive), may be {@code null} * @return the String result * @throws IllegalArgumentException if the Object is {@code null} */ diff --git a/src/test/java/org/apache/commons/lang3/math/FractionTest.java b/src/test/java/org/apache/commons/lang3/math/FractionTest.java index 684a9e55c..a3e11227d 100644 --- a/src/test/java/org/apache/commons/lang3/math/FractionTest.java +++ b/src/test/java/org/apache/commons/lang3/math/FractionTest.java @@ -32,7 +32,7 @@ import org.junit.jupiter.api.Test; */ public class FractionTest extends AbstractLangTest { - private static final int SKIP = 500; //53 + private static final int SKIP = 500; // 53 @Test public void testAbs() { @@ -116,17 +116,17 @@ public class FractionTest extends AbstractLangTest { f = f2.add(f1); assertSame(f2, f); - f1 = Fraction.getFraction(-1, 13*13*2*2); - f2 = Fraction.getFraction(-2, 13*17*2); + f1 = Fraction.getFraction(-1, 13 * 13 * 2 * 2); + f2 = Fraction.getFraction(-2, 13 * 17 * 2); final Fraction fr = f1.add(f2); - assertEquals(13*13*17*2*2, fr.getDenominator()); - assertEquals(-17 - 2*13*2, fr.getNumerator()); + assertEquals(13 * 13 * 17 * 2 * 2, fr.getDenominator()); + assertEquals(-17 - 2 * 13 * 2, fr.getNumerator()); assertThrows(NullPointerException.class, () -> fr.add(null)); // if this fraction is added naively, it will overflow. // check that it doesn't. - f1 = Fraction.getFraction(1, 32768*3); + f1 = Fraction.getFraction(1, 32768 * 3); f2 = Fraction.getFraction(1, 59049); f = f1.add(f2); assertEquals(52451, f.getNumerator()); @@ -135,7 +135,7 @@ public class FractionTest extends AbstractLangTest { f1 = Fraction.getFraction(Integer.MIN_VALUE, 3); f2 = Fraction.ONE_THIRD; f = f1.add(f2); - assertEquals(Integer.MIN_VALUE+1, f.getNumerator()); + assertEquals(Integer.MIN_VALUE + 1, f.getNumerator()); assertEquals(3, f.getDenominator()); f1 = Fraction.getFraction(Integer.MAX_VALUE - 1, 1); @@ -148,9 +148,7 @@ public class FractionTest extends AbstractLangTest { assertThrows(ArithmeticException.class, () -> overflower.add(Fraction.ONE)); // should overflow // denominator should not be a multiple of 2 or 3 to trigger overflow - assertThrows( - ArithmeticException.class, - () -> Fraction.getFraction(Integer.MIN_VALUE, 5).add(Fraction.getFraction(-1, 5))); + assertThrows(ArithmeticException.class, () -> Fraction.getFraction(Integer.MIN_VALUE, 5).add(Fraction.getFraction(-1, 5))); final Fraction maxValue = Fraction.getFraction(-Integer.MAX_VALUE, 1); assertThrows(ArithmeticException.class, () -> maxValue.add(maxValue)); @@ -352,14 +350,14 @@ public class FractionTest extends AbstractLangTest { assertEquals(3, f.getDenominator()); // small - f = Fraction.getFraction(1.0d/10001d); + f = Fraction.getFraction(1.0d / 10001d); assertEquals(0, f.getNumerator()); assertEquals(1, f.getDenominator()); // normal Fraction f2; - for (int i = 1; i <= 100; i++) { // denominator - for (int j = 1; j <= i; j++) { // numerator + for (int i = 1; i <= 100; i++) { // denominator + for (int j = 1; j <= i; j++) { // numerator f = Fraction.getFraction((double) j / (double) i); f2 = Fraction.getReducedFraction(j, i); @@ -367,9 +365,9 @@ public class FractionTest extends AbstractLangTest { assertEquals(f2.getDenominator(), f.getDenominator()); } } - // save time by skipping some tests! ( - for (int i = 1001; i <= 10000; i+=SKIP) { // denominator - for (int j = 1; j <= i; j++) { // numerator + // save time by skipping some tests! ( + for (int i = 1001; i <= 10000; i += SKIP) { // denominator + for (int j = 1; j <= i; j++) { // numerator f = Fraction.getFraction((double) j / (double) i); f2 = Fraction.getReducedFraction(j, i); assertEquals(f2.getNumerator(), f.getNumerator()); @@ -726,9 +724,9 @@ public class FractionTest extends AbstractLangTest { assertEquals(75, f.getDenominator()); // large values - f = Fraction.getFraction(Integer.MAX_VALUE-1, Integer.MAX_VALUE); + f = Fraction.getFraction(Integer.MAX_VALUE - 1, Integer.MAX_VALUE); f = f.negate(); - assertEquals(Integer.MIN_VALUE+2, f.getNumerator()); + assertEquals(Integer.MIN_VALUE + 2, f.getNumerator()); assertEquals(Integer.MAX_VALUE, f.getDenominator()); assertThrows(ArithmeticException.class, () -> Fraction.getFraction(Integer.MIN_VALUE, 1).negate()); @@ -1014,7 +1012,7 @@ public class FractionTest extends AbstractLangTest { // if this fraction is subtracted naively, it will overflow. // check that it doesn't. - f1 = Fraction.getFraction(1, 32768*3); + f1 = Fraction.getFraction(1, 32768 * 3); f2 = Fraction.getFraction(1, 59049); f = f1.subtract(f2); assertEquals(-13085, f.getNumerator()); @@ -1023,36 +1021,27 @@ public class FractionTest extends AbstractLangTest { f1 = Fraction.getFraction(Integer.MIN_VALUE, 3); f2 = Fraction.ONE_THIRD.negate(); f = f1.subtract(f2); - assertEquals(Integer.MIN_VALUE+1, f.getNumerator()); + assertEquals(Integer.MIN_VALUE + 1, f.getNumerator()); assertEquals(3, f.getDenominator()); f1 = Fraction.getFraction(Integer.MAX_VALUE, 1); f2 = Fraction.ONE; f = f1.subtract(f2); - assertEquals(Integer.MAX_VALUE-1, f.getNumerator()); + assertEquals(Integer.MAX_VALUE - 1, f.getNumerator()); assertEquals(1, f.getDenominator()); // Should overflow - assertThrows( - ArithmeticException.class, - () -> Fraction.getFraction(1, Integer.MAX_VALUE).subtract(Fraction.getFraction(1, Integer.MAX_VALUE - 1))); + assertThrows(ArithmeticException.class, () -> Fraction.getFraction(1, Integer.MAX_VALUE).subtract(Fraction.getFraction(1, Integer.MAX_VALUE - 1))); // denominator should not be a multiple of 2 or 3 to trigger overflow - assertThrows( - ArithmeticException.class, - () -> Fraction.getFraction(Integer.MIN_VALUE, 5).subtract(Fraction.getFraction(1, 5))); + assertThrows(ArithmeticException.class, () -> Fraction.getFraction(Integer.MIN_VALUE, 5).subtract(Fraction.getFraction(1, 5))); - assertThrows( - ArithmeticException.class, () -> Fraction.getFraction(Integer.MIN_VALUE, 1).subtract(Fraction.ONE)); + assertThrows(ArithmeticException.class, () -> Fraction.getFraction(Integer.MIN_VALUE, 1).subtract(Fraction.ONE)); - assertThrows( - ArithmeticException.class, - () -> Fraction.getFraction(Integer.MAX_VALUE, 1).subtract(Fraction.ONE.negate())); + assertThrows(ArithmeticException.class, () -> Fraction.getFraction(Integer.MAX_VALUE, 1).subtract(Fraction.ONE.negate())); // Should overflow - assertThrows( - ArithmeticException.class, - () -> Fraction.getFraction(3, 327680).subtract(Fraction.getFraction(2, 59049))); + assertThrows(ArithmeticException.class, () -> Fraction.getFraction(3, 327680).subtract(Fraction.getFraction(2, 59049))); } @Test diff --git a/src/test/java/org/apache/commons/lang3/reflect/ConstructorUtilsTest.java b/src/test/java/org/apache/commons/lang3/reflect/ConstructorUtilsTest.java index 943c1ee81..866b1bc43 100644 --- a/src/test/java/org/apache/commons/lang3/reflect/ConstructorUtilsTest.java +++ b/src/test/java/org/apache/commons/lang3/reflect/ConstructorUtilsTest.java @@ -38,7 +38,8 @@ import org.junit.jupiter.api.Test; * Unit tests ConstructorUtils */ public class ConstructorUtilsTest extends AbstractLangTest { - private static class BaseClass {} + private static class BaseClass { + } static class PrivateClass { @SuppressWarnings("unused") @@ -52,7 +53,8 @@ public class ConstructorUtilsTest extends AbstractLangTest { } } - private static final class SubClass extends BaseClass {} + private static final class SubClass extends BaseClass { + } public static class TestBean { private final String toString; @@ -117,8 +119,8 @@ public class ConstructorUtilsTest extends AbstractLangTest { } void verify(final String str, final String[] args) { - assertEquals(str, toString); - assertArrayEquals(args, varArgs); + assertEquals(str, toString); + assertArrayEquals(args, varArgs); } } @@ -128,10 +130,8 @@ public class ConstructorUtilsTest extends AbstractLangTest { classCache = new HashMap<>(); } - private void expectMatchingAccessibleConstructorParameterTypes(final Class<?> cls, - final Class<?>[] requestTypes, final Class<?>[] actualTypes) { - final Constructor<?> c = ConstructorUtils.getMatchingAccessibleConstructor(cls, - requestTypes); + private void expectMatchingAccessibleConstructorParameterTypes(final Class<?> cls, final Class<?>[] requestTypes, final Class<?>[] actualTypes) { + final Constructor<?> c = ConstructorUtils.getMatchingAccessibleConstructor(cls, requestTypes); assertArrayEquals(actualTypes, c.getParameterTypes(), toString(c.getParameterTypes()) + " not equals " + toString(actualTypes)); } @@ -156,136 +156,86 @@ public class ConstructorUtilsTest extends AbstractLangTest { @Test public void testGetAccessibleConstructor() throws Exception { - assertNotNull(ConstructorUtils.getAccessibleConstructor(Object.class - .getConstructor(ArrayUtils.EMPTY_CLASS_ARRAY))); - assertNull(ConstructorUtils.getAccessibleConstructor(PrivateClass.class - .getConstructor(ArrayUtils.EMPTY_CLASS_ARRAY))); + assertNotNull(ConstructorUtils.getAccessibleConstructor(Object.class.getConstructor(ArrayUtils.EMPTY_CLASS_ARRAY))); + assertNull(ConstructorUtils.getAccessibleConstructor(PrivateClass.class.getConstructor(ArrayUtils.EMPTY_CLASS_ARRAY))); assertNull(ConstructorUtils.getAccessibleConstructor(PrivateClass.PublicInnerClass.class)); } @Test public void testGetAccessibleConstructorFromDescription() { - assertNotNull(ConstructorUtils.getAccessibleConstructor(Object.class, - ArrayUtils.EMPTY_CLASS_ARRAY)); - assertNull(ConstructorUtils.getAccessibleConstructor( - PrivateClass.class, ArrayUtils.EMPTY_CLASS_ARRAY)); + assertNotNull(ConstructorUtils.getAccessibleConstructor(Object.class, ArrayUtils.EMPTY_CLASS_ARRAY)); + assertNull(ConstructorUtils.getAccessibleConstructor(PrivateClass.class, ArrayUtils.EMPTY_CLASS_ARRAY)); } @Test public void testGetMatchingAccessibleMethod() { - expectMatchingAccessibleConstructorParameterTypes(TestBean.class, - ArrayUtils.EMPTY_CLASS_ARRAY, ArrayUtils.EMPTY_CLASS_ARRAY); - expectMatchingAccessibleConstructorParameterTypes(TestBean.class, null, - ArrayUtils.EMPTY_CLASS_ARRAY); - expectMatchingAccessibleConstructorParameterTypes(TestBean.class, - singletonArray(String.class), singletonArray(String.class)); - expectMatchingAccessibleConstructorParameterTypes(TestBean.class, - singletonArray(Object.class), singletonArray(Object.class)); - expectMatchingAccessibleConstructorParameterTypes(TestBean.class, - singletonArray(Boolean.class), singletonArray(Object.class)); - expectMatchingAccessibleConstructorParameterTypes(TestBean.class, - singletonArray(Byte.class), singletonArray(Integer.TYPE)); - expectMatchingAccessibleConstructorParameterTypes(TestBean.class, - singletonArray(Byte.TYPE), singletonArray(Integer.TYPE)); - expectMatchingAccessibleConstructorParameterTypes(TestBean.class, - singletonArray(Short.class), singletonArray(Integer.TYPE)); - expectMatchingAccessibleConstructorParameterTypes(TestBean.class, - singletonArray(Short.TYPE), singletonArray(Integer.TYPE)); - expectMatchingAccessibleConstructorParameterTypes(TestBean.class, - singletonArray(Character.class), singletonArray(Integer.TYPE)); - expectMatchingAccessibleConstructorParameterTypes(TestBean.class, - singletonArray(Character.TYPE), singletonArray(Integer.TYPE)); - expectMatchingAccessibleConstructorParameterTypes(TestBean.class, - singletonArray(Integer.class), singletonArray(Integer.class)); - expectMatchingAccessibleConstructorParameterTypes(TestBean.class, - singletonArray(Integer.TYPE), singletonArray(Integer.TYPE)); - expectMatchingAccessibleConstructorParameterTypes(TestBean.class, - singletonArray(Long.class), singletonArray(Double.TYPE)); - expectMatchingAccessibleConstructorParameterTypes(TestBean.class, - singletonArray(Long.TYPE), singletonArray(Double.TYPE)); - expectMatchingAccessibleConstructorParameterTypes(TestBean.class, - singletonArray(Float.class), singletonArray(Double.TYPE)); - expectMatchingAccessibleConstructorParameterTypes(TestBean.class, - singletonArray(Float.TYPE), singletonArray(Double.TYPE)); - expectMatchingAccessibleConstructorParameterTypes(TestBean.class, - singletonArray(Double.class), singletonArray(Double.TYPE)); - expectMatchingAccessibleConstructorParameterTypes(TestBean.class, - singletonArray(Double.TYPE), singletonArray(Double.TYPE)); - expectMatchingAccessibleConstructorParameterTypes(TestBean.class, - new Class<?>[]{SubClass.class, String[].class}, - new Class<?>[]{BaseClass.class, String[].class}); + expectMatchingAccessibleConstructorParameterTypes(TestBean.class, ArrayUtils.EMPTY_CLASS_ARRAY, ArrayUtils.EMPTY_CLASS_ARRAY); + expectMatchingAccessibleConstructorParameterTypes(TestBean.class, null, ArrayUtils.EMPTY_CLASS_ARRAY); + expectMatchingAccessibleConstructorParameterTypes(TestBean.class, singletonArray(String.class), singletonArray(String.class)); + expectMatchingAccessibleConstructorParameterTypes(TestBean.class, singletonArray(Object.class), singletonArray(Object.class)); + expectMatchingAccessibleConstructorParameterTypes(TestBean.class, singletonArray(Boolean.class), singletonArray(Object.class)); + expectMatchingAccessibleConstructorParameterTypes(TestBean.class, singletonArray(Byte.class), singletonArray(Integer.TYPE)); + expectMatchingAccessibleConstructorParameterTypes(TestBean.class, singletonArray(Byte.TYPE), singletonArray(Integer.TYPE)); + expectMatchingAccessibleConstructorParameterTypes(TestBean.class, singletonArray(Short.class), singletonArray(Integer.TYPE)); + expectMatchingAccessibleConstructorParameterTypes(TestBean.class, singletonArray(Short.TYPE), singletonArray(Integer.TYPE)); + expectMatchingAccessibleConstructorParameterTypes(TestBean.class, singletonArray(Character.class), singletonArray(Integer.TYPE)); + expectMatchingAccessibleConstructorParameterTypes(TestBean.class, singletonArray(Character.TYPE), singletonArray(Integer.TYPE)); + expectMatchingAccessibleConstructorParameterTypes(TestBean.class, singletonArray(Integer.class), singletonArray(Integer.class)); + expectMatchingAccessibleConstructorParameterTypes(TestBean.class, singletonArray(Integer.TYPE), singletonArray(Integer.TYPE)); + expectMatchingAccessibleConstructorParameterTypes(TestBean.class, singletonArray(Long.class), singletonArray(Double.TYPE)); + expectMatchingAccessibleConstructorParameterTypes(TestBean.class, singletonArray(Long.TYPE), singletonArray(Double.TYPE)); + expectMatchingAccessibleConstructorParameterTypes(TestBean.class, singletonArray(Float.class), singletonArray(Double.TYPE)); + expectMatchingAccessibleConstructorParameterTypes(TestBean.class, singletonArray(Float.TYPE), singletonArray(Double.TYPE)); + expectMatchingAccessibleConstructorParameterTypes(TestBean.class, singletonArray(Double.class), singletonArray(Double.TYPE)); + expectMatchingAccessibleConstructorParameterTypes(TestBean.class, singletonArray(Double.TYPE), singletonArray(Double.TYPE)); + expectMatchingAccessibleConstructorParameterTypes(TestBean.class, new Class<?>[] { SubClass.class, String[].class }, + new Class<?>[] { BaseClass.class, String[].class }); } @Test public void testInvokeConstructor() throws Exception { - assertEquals("()", ConstructorUtils.invokeConstructor(TestBean.class, - (Object[]) ArrayUtils.EMPTY_CLASS_ARRAY).toString()); - assertEquals("()", ConstructorUtils.invokeConstructor(TestBean.class, - (Object[]) null).toString()); + assertEquals("()", ConstructorUtils.invokeConstructor(TestBean.class, (Object[]) ArrayUtils.EMPTY_CLASS_ARRAY).toString()); + assertEquals("()", ConstructorUtils.invokeConstructor(TestBean.class, (Object[]) null).toString()); assertEquals("()", ConstructorUtils.invokeConstructor(TestBean.class).toString()); - assertEquals("(String)", ConstructorUtils.invokeConstructor( - TestBean.class, "").toString()); - assertEquals("(Object)", ConstructorUtils.invokeConstructor( - TestBean.class, new Object()).toString()); - assertEquals("(Object)", ConstructorUtils.invokeConstructor( - TestBean.class, Boolean.TRUE).toString()); - assertEquals("(Integer)", ConstructorUtils.invokeConstructor( - TestBean.class, NumberUtils.INTEGER_ONE).toString()); - assertEquals("(int)", ConstructorUtils.invokeConstructor( - TestBean.class, NumberUtils.BYTE_ONE).toString()); - assertEquals("(double)", ConstructorUtils.invokeConstructor( - TestBean.class, NumberUtils.LONG_ONE).toString()); - assertEquals("(double)", ConstructorUtils.invokeConstructor( - TestBean.class, NumberUtils.DOUBLE_ONE).toString()); - ConstructorUtils.invokeConstructor(TestBean.class, NumberUtils.INTEGER_ONE) - .verify("(Integer)", null); - ConstructorUtils.invokeConstructor(TestBean.class, "a", "b") - .verify("(String...)", new String[]{"a", "b"}); - ConstructorUtils.invokeConstructor(TestBean.class, NumberUtils.INTEGER_ONE, "a", "b") - .verify("(Integer, String...)", new String[]{"a", "b"}); - ConstructorUtils.invokeConstructor(TestBean.class, new SubClass(), new String[]{"a", "b"}) - .verify("(BaseClass, String...)", new String[]{"a", "b"}); + assertEquals("(String)", ConstructorUtils.invokeConstructor(TestBean.class, "").toString()); + assertEquals("(Object)", ConstructorUtils.invokeConstructor(TestBean.class, new Object()).toString()); + assertEquals("(Object)", ConstructorUtils.invokeConstructor(TestBean.class, Boolean.TRUE).toString()); + assertEquals("(Integer)", ConstructorUtils.invokeConstructor(TestBean.class, NumberUtils.INTEGER_ONE).toString()); + assertEquals("(int)", ConstructorUtils.invokeConstructor(TestBean.class, NumberUtils.BYTE_ONE).toString()); + assertEquals("(double)", ConstructorUtils.invokeConstructor(TestBean.class, NumberUtils.LONG_ONE).toString()); + assertEquals("(double)", ConstructorUtils.invokeConstructor(TestBean.class, NumberUtils.DOUBLE_ONE).toString()); + ConstructorUtils.invokeConstructor(TestBean.class, NumberUtils.INTEGER_ONE).verify("(Integer)", null); + ConstructorUtils.invokeConstructor(TestBean.class, "a", "b").verify("(String...)", new String[] { "a", "b" }); + ConstructorUtils.invokeConstructor(TestBean.class, NumberUtils.INTEGER_ONE, "a", "b").verify("(Integer, String...)", new String[] { "a", "b" }); + ConstructorUtils.invokeConstructor(TestBean.class, new SubClass(), new String[] { "a", "b" }).verify("(BaseClass, String...)", + new String[] { "a", "b" }); } @Test public void testInvokeExactConstructor() throws Exception { - assertEquals("()", ConstructorUtils.invokeExactConstructor( - TestBean.class, (Object[]) ArrayUtils.EMPTY_CLASS_ARRAY).toString()); - assertEquals("()", ConstructorUtils.invokeExactConstructor( - TestBean.class, (Object[]) null).toString()); - assertEquals("(String)", ConstructorUtils.invokeExactConstructor( - TestBean.class, "").toString()); - assertEquals("(Object)", ConstructorUtils.invokeExactConstructor( - TestBean.class, new Object()).toString()); - assertEquals("(Integer)", ConstructorUtils.invokeExactConstructor( - TestBean.class, NumberUtils.INTEGER_ONE).toString()); - assertEquals("(double)", ConstructorUtils.invokeExactConstructor( - TestBean.class, new Object[] { NumberUtils.DOUBLE_ONE }, - new Class[] { Double.TYPE }).toString()); - - assertThrows( - NoSuchMethodException.class, - () -> ConstructorUtils.invokeExactConstructor(TestBean.class, NumberUtils.BYTE_ONE)); - assertThrows( - NoSuchMethodException.class, - () -> ConstructorUtils.invokeExactConstructor(TestBean.class, NumberUtils.LONG_ONE)); - assertThrows( - NoSuchMethodException.class, - () -> ConstructorUtils.invokeExactConstructor(TestBean.class, Boolean.TRUE)); + assertEquals("()", ConstructorUtils.invokeExactConstructor(TestBean.class, (Object[]) ArrayUtils.EMPTY_CLASS_ARRAY).toString()); + assertEquals("()", ConstructorUtils.invokeExactConstructor(TestBean.class, (Object[]) null).toString()); + assertEquals("(String)", ConstructorUtils.invokeExactConstructor(TestBean.class, "").toString()); + assertEquals("(Object)", ConstructorUtils.invokeExactConstructor(TestBean.class, new Object()).toString()); + assertEquals("(Integer)", ConstructorUtils.invokeExactConstructor(TestBean.class, NumberUtils.INTEGER_ONE).toString()); + assertEquals("(double)", + ConstructorUtils.invokeExactConstructor(TestBean.class, new Object[] { NumberUtils.DOUBLE_ONE }, new Class[] { Double.TYPE }).toString()); + + assertThrows(NoSuchMethodException.class, () -> ConstructorUtils.invokeExactConstructor(TestBean.class, NumberUtils.BYTE_ONE)); + assertThrows(NoSuchMethodException.class, () -> ConstructorUtils.invokeExactConstructor(TestBean.class, NumberUtils.LONG_ONE)); + assertThrows(NoSuchMethodException.class, () -> ConstructorUtils.invokeExactConstructor(TestBean.class, Boolean.TRUE)); } @Test public void testNullArgument() { - expectMatchingAccessibleConstructorParameterTypes(MutableObject.class, - singletonArray(null), singletonArray(Object.class)); + expectMatchingAccessibleConstructorParameterTypes(MutableObject.class, singletonArray(null), singletonArray(Object.class)); } @Test public void testVarArgsUnboxing() throws Exception { - final TestBean testBean = ConstructorUtils.invokeConstructor( - TestBean.class, Integer.valueOf(1), Integer.valueOf(2), Integer.valueOf(3)); + final TestBean testBean = ConstructorUtils.invokeConstructor(TestBean.class, Integer.valueOf(1), Integer.valueOf(2), Integer.valueOf(3)); - assertArrayEquals(new String[]{"2", "3"}, testBean.varArgs); + assertArrayEquals(new String[] { "2", "3" }, testBean.varArgs); } private String toString(final Class<?>[] c) { diff --git a/src/test/java/org/apache/commons/lang3/reflect/TypeLiteralTest.java b/src/test/java/org/apache/commons/lang3/reflect/TypeLiteralTest.java index fd95d0467..b68120af6 100644 --- a/src/test/java/org/apache/commons/lang3/reflect/TypeLiteralTest.java +++ b/src/test/java/org/apache/commons/lang3/reflect/TypeLiteralTest.java @@ -30,29 +30,39 @@ public class TypeLiteralTest extends AbstractLangTest { @Test public void testBasic() { - assertTrue(TypeUtils.equals(String.class, new TypeLiteral<String>() {}.value)); - assertTrue(TypeUtils.equals(TypeUtils.parameterize(List.class, String.class), - new TypeLiteral<List<String>>() {}.value)); + assertTrue(TypeUtils.equals(String.class, new TypeLiteral<String>() { + }.value)); + assertTrue(TypeUtils.equals(TypeUtils.parameterize(List.class, String.class), new TypeLiteral<List<String>>() { + }.value)); } @Test public void testEquals() { - assertEquals(new TypeLiteral<String>() {}, new TypeLiteral<String>() {}); - assertEquals(new TypeLiteral<List<String>>() {}, new TypeLiteral<List<String>>() {}); - assertNotEquals(new TypeLiteral<String>() {}, new TypeLiteral<List<String>>() {}); + assertEquals(new TypeLiteral<String>() { + }, new TypeLiteral<String>() { + }); + assertEquals(new TypeLiteral<List<String>>() { + }, new TypeLiteral<List<String>>() { + }); + assertNotEquals(new TypeLiteral<String>() { + }, new TypeLiteral<List<String>>() { + }); } @SuppressWarnings("rawtypes") @Test public void testRaw() { - assertThrows(NullPointerException.class, () -> new TypeLiteral() {}); + assertThrows(NullPointerException.class, () -> new TypeLiteral() { + }); } @Test public void testTyped() { - final Typed<String> stringType = new TypeLiteral<String>() {}; + final Typed<String> stringType = new TypeLiteral<String>() { + }; assertTrue(TypeUtils.equals(String.class, stringType.getType())); - final Typed<List<String>> listOfStringType = new TypeLiteral<List<String>>() {}; + final Typed<List<String>> listOfStringType = new TypeLiteral<List<String>>() { + }; assertTrue(TypeUtils.equals(TypeUtils.parameterize(List.class, String.class), listOfStringType.getType())); } } diff --git a/src/test/java/org/apache/commons/lang3/test/NotVisibleExceptionFactory.java b/src/test/java/org/apache/commons/lang3/test/NotVisibleExceptionFactory.java index 3a77b96e3..763c8a551 100644 --- a/src/test/java/org/apache/commons/lang3/test/NotVisibleExceptionFactory.java +++ b/src/test/java/org/apache/commons/lang3/test/NotVisibleExceptionFactory.java @@ -17,36 +17,36 @@ package org.apache.commons.lang3.test; /** - * Allows for testing an exception that is not visible to - * {@link org.apache.commons.lang3.exception.ExceptionUtils} + * Allows for testing an exception that is not visible to {@link org.apache.commons.lang3.exception.ExceptionUtils} */ public class NotVisibleExceptionFactory { - private static final class NotVisibleException extends Exception { + private static final class NotVisibleException extends Exception { - private static final long serialVersionUID = 1L; // avoid warning + private static final long serialVersionUID = 1L; // avoid warning - private final Throwable cause; + private final Throwable cause; - private NotVisibleException(final Throwable cause) { - this.cause = cause; + private NotVisibleException(final Throwable cause) { + this.cause = cause; + } + + @Override + public synchronized Throwable getCause() { + return cause; + } + } + + /** + * Create a new Exception whose getCause method returns the provided cause. + * + * @param cause the cause of the exception + * @return a new {@link Exception} + */ + public static Exception createException(final Throwable cause) { + return new NotVisibleException(cause); } - @Override - public synchronized Throwable getCause() { - return cause; + private NotVisibleExceptionFactory() { } - } - - /** - * Create a new Exception whose getCause method returns the - * provided cause. - * @param cause the cause of the exception - * @return a new {@link Exception} - */ - public static Exception createException(final Throwable cause) { - return new NotVisibleException(cause); - } - - private NotVisibleExceptionFactory() {} } diff --git a/src/test/java/org/apache/commons/lang3/text/FormattableUtilsTest.java b/src/test/java/org/apache/commons/lang3/text/FormattableUtilsTest.java index bf6dddf17..6637c7911 100644 --- a/src/test/java/org/apache/commons/lang3/text/FormattableUtilsTest.java +++ b/src/test/java/org/apache/commons/lang3/text/FormattableUtilsTest.java @@ -33,7 +33,7 @@ public class FormattableUtilsTest extends AbstractLangTest { @Test public void testAlternatePadCharacter() { - final char pad='_'; + final char pad = '_'; assertEquals("foo", FormattableUtils.append("foo", new Formatter(), 0, -1, -1, pad).toString()); assertEquals("fo", FormattableUtils.append("foo", new Formatter(), 0, -1, 2, pad).toString()); assertEquals("_foo", FormattableUtils.append("foo", new Formatter(), 0, 4, -1, pad).toString()); diff --git a/src/test/java/org/apache/commons/lang3/text/translate/EntityArraysTest.java b/src/test/java/org/apache/commons/lang3/text/translate/EntityArraysTest.java index a777e90a8..60f1c5354 100644 --- a/src/test/java/org/apache/commons/lang3/text/translate/EntityArraysTest.java +++ b/src/test/java/org/apache/commons/lang3/text/translate/EntityArraysTest.java @@ -41,30 +41,30 @@ public class EntityArraysTest extends AbstractLangTest { public void testHTML40_EXTENDED_ESCAPE() { final Set<String> col0 = new HashSet<>(); final Set<String> col1 = new HashSet<>(); - final String [][] sa = EntityArrays.HTML40_EXTENDED_ESCAPE(); - for (int i =0; i <sa.length; i++) { - assertTrue(col0.add(sa[i][0]), "Already added entry 0: "+i+" "+sa[i][0]); - assertTrue(col1.add(sa[i][1]), "Already added entry 1: "+i+" "+sa[i][1]); + final String[][] sa = EntityArrays.HTML40_EXTENDED_ESCAPE(); + for (int i = 0; i < sa.length; i++) { + assertTrue(col0.add(sa[i][0]), "Already added entry 0: " + i + " " + sa[i][0]); + assertTrue(col1.add(sa[i][1]), "Already added entry 1: " + i + " " + sa[i][1]); } } - // LANG-658 - check arrays for duplicate entries + // LANG-658 - check arrays for duplicate entries @Test public void testISO8859_1_ESCAPE() { final Set<String> col0 = new HashSet<>(); final Set<String> col1 = new HashSet<>(); - final String [][] sa = EntityArrays.ISO8859_1_ESCAPE(); + final String[][] sa = EntityArrays.ISO8859_1_ESCAPE(); boolean success = true; - for (int i =0; i <sa.length; i++) { + for (int i = 0; i < sa.length; i++) { final boolean add0 = col0.add(sa[i][0]); final boolean add1 = col1.add(sa[i][1]); if (!add0) { success = false; - System.out.println("Already added entry 0: "+i+" "+sa[i][0]+" "+sa[i][1]); + System.out.println("Already added entry 0: " + i + " " + sa[i][0] + " " + sa[i][1]); } if (!add1) { success = false; - System.out.println("Already added entry 1: "+i+" "+sa[i][0]+" "+sa[i][1]); + System.out.println("Already added entry 1: " + i + " " + sa[i][0] + " " + sa[i][1]); } } assertTrue(success, "One or more errors detected"); diff --git a/src/test/java/org/apache/commons/lang3/time/DateUtilsFragmentTest.java b/src/test/java/org/apache/commons/lang3/time/DateUtilsFragmentTest.java index 35b5645d1..f3eae8ede 100644 --- a/src/test/java/org/apache/commons/lang3/time/DateUtilsFragmentTest.java +++ b/src/test/java/org/apache/commons/lang3/time/DateUtilsFragmentTest.java @@ -28,9 +28,9 @@ import org.junit.jupiter.api.Test; public class DateUtilsFragmentTest extends AbstractLangTest { - private static final int months = 7; // second final prime before 12 - private static final int days = 23; // second final prime before 31 (and valid) - private static final int hours = 19; // second final prime before 24 + private static final int months = 7; // second final prime before 12 + private static final int days = 23; // second final prime before 31 (and valid) + private static final int hours = 19; // second final prime before 24 private static final int minutes = 53; // second final prime before 60 private static final int seconds = 47; // third final prime before 60 private static final int millis = 991; // second final prime before 1000 @@ -127,25 +127,19 @@ public class DateUtilsFragmentTest extends AbstractLangTest { @Test public void testHoursOfMonthWithCalendar() { final long testResult = DateUtils.getFragmentInHours(aCalendar, Calendar.MONTH); - assertEquals( hours +(days - 1) * DateUtils.MILLIS_PER_DAY - / DateUtils.MILLIS_PER_HOUR, - testResult); + assertEquals(hours + (days - 1) * DateUtils.MILLIS_PER_DAY / DateUtils.MILLIS_PER_HOUR, testResult); } @Test public void testHoursOfMonthWithDate() { final long testResult = DateUtils.getFragmentInHours(aDate, Calendar.MONTH); - assertEquals(hours + (days - 1) * DateUtils.MILLIS_PER_DAY - / DateUtils.MILLIS_PER_HOUR, - testResult); + assertEquals(hours + (days - 1) * DateUtils.MILLIS_PER_DAY / DateUtils.MILLIS_PER_HOUR, testResult); } @Test public void testHoursOfYearWithCalendar() { final long testResult = DateUtils.getFragmentInHours(aCalendar, Calendar.YEAR); - assertEquals( hours +(aCalendar.get(Calendar.DAY_OF_YEAR) - 1) * DateUtils.MILLIS_PER_DAY - / DateUtils.MILLIS_PER_HOUR, - testResult); + assertEquals(hours + (aCalendar.get(Calendar.DAY_OF_YEAR) - 1) * DateUtils.MILLIS_PER_DAY / DateUtils.MILLIS_PER_HOUR, testResult); } @Test @@ -153,12 +147,10 @@ public class DateUtilsFragmentTest extends AbstractLangTest { final long testResult = DateUtils.getFragmentInHours(aDate, Calendar.YEAR); final Calendar cal = Calendar.getInstance(); cal.setTime(aDate); - assertEquals(hours + (cal.get(Calendar.DAY_OF_YEAR) - 1) * DateUtils.MILLIS_PER_DAY - / DateUtils.MILLIS_PER_HOUR, - testResult); + assertEquals(hours + (cal.get(Calendar.DAY_OF_YEAR) - 1) * DateUtils.MILLIS_PER_DAY / DateUtils.MILLIS_PER_HOUR, testResult); } - //Calendar.SECOND as useful fragment + // Calendar.SECOND as useful fragment @Test public void testInvalidFragmentWithCalendar() { @@ -178,7 +170,7 @@ public class DateUtilsFragmentTest extends AbstractLangTest { assertThrows(IllegalArgumentException.class, () -> DateUtils.getFragmentInDays(aDate, 0)); } - //Calendar.MINUTE as useful fragment + // Calendar.MINUTE as useful fragment @Test public void testMillisecondFragmentInLargerUnitWithCalendar() { @@ -207,7 +199,7 @@ public class DateUtilsFragmentTest extends AbstractLangTest { assertEquals(expectedValue, testresult); } - //Calendar.DATE and Calendar.DAY_OF_YEAR as useful fragment + // Calendar.DATE and Calendar.DAY_OF_YEAR as useful fragment @Test public void testMillisecondsOfDayWithDate() { long testresult = DateUtils.getFragmentInMilliseconds(aDate, Calendar.DATE); @@ -217,7 +209,7 @@ public class DateUtilsFragmentTest extends AbstractLangTest { assertEquals(expectedValue, testresult); } - //Calendar.HOUR_OF_DAY as useful fragment + // Calendar.HOUR_OF_DAY as useful fragment @Test public void testMillisecondsOfHourWithCalendar() { @@ -246,18 +238,16 @@ public class DateUtilsFragmentTest extends AbstractLangTest { @Test public void testMillisecondsOfMonthWithCalendar() { final long testResult = DateUtils.getFragmentInMilliseconds(aCalendar, Calendar.MONTH); - assertEquals(millis + seconds * DateUtils.MILLIS_PER_SECOND + minutes * DateUtils.MILLIS_PER_MINUTE - + hours * DateUtils.MILLIS_PER_HOUR + (days - 1) * DateUtils.MILLIS_PER_DAY, -testResult); + assertEquals(millis + seconds * DateUtils.MILLIS_PER_SECOND + minutes * DateUtils.MILLIS_PER_MINUTE + hours * DateUtils.MILLIS_PER_HOUR + + (days - 1) * DateUtils.MILLIS_PER_DAY, testResult); } - //Calendar.MONTH as useful fragment + // Calendar.MONTH as useful fragment @Test public void testMillisecondsOfMonthWithDate() { final long testResult = DateUtils.getFragmentInMilliseconds(aDate, Calendar.MONTH); - assertEquals(millis + seconds * DateUtils.MILLIS_PER_SECOND + minutes * DateUtils.MILLIS_PER_MINUTE - + hours * DateUtils.MILLIS_PER_HOUR + (days - 1) * DateUtils.MILLIS_PER_DAY, - testResult); + assertEquals(millis + seconds * DateUtils.MILLIS_PER_SECOND + minutes * DateUtils.MILLIS_PER_MINUTE + hours * DateUtils.MILLIS_PER_HOUR + + (days - 1) * DateUtils.MILLIS_PER_DAY, testResult); } @Test @@ -276,20 +266,18 @@ testResult); @Test public void testMillisecondsOfYearWithCalendar() { final long testResult = DateUtils.getFragmentInMilliseconds(aCalendar, Calendar.YEAR); - assertEquals(millis + seconds * DateUtils.MILLIS_PER_SECOND + minutes * DateUtils.MILLIS_PER_MINUTE - + hours * DateUtils.MILLIS_PER_HOUR + (aCalendar.get(Calendar.DAY_OF_YEAR) - 1) * DateUtils.MILLIS_PER_DAY, -testResult); + assertEquals(millis + seconds * DateUtils.MILLIS_PER_SECOND + minutes * DateUtils.MILLIS_PER_MINUTE + hours * DateUtils.MILLIS_PER_HOUR + + (aCalendar.get(Calendar.DAY_OF_YEAR) - 1) * DateUtils.MILLIS_PER_DAY, testResult); } - //Calendar.YEAR as useful fragment + // Calendar.YEAR as useful fragment @Test public void testMillisecondsOfYearWithDate() { final long testResult = DateUtils.getFragmentInMilliseconds(aDate, Calendar.YEAR); final Calendar cal = Calendar.getInstance(); cal.setTime(aDate); - assertEquals(millis + seconds * DateUtils.MILLIS_PER_SECOND + minutes * DateUtils.MILLIS_PER_MINUTE - + hours * DateUtils.MILLIS_PER_HOUR + (cal.get(Calendar.DAY_OF_YEAR) - 1)* DateUtils.MILLIS_PER_DAY, - testResult); + assertEquals(millis + seconds * DateUtils.MILLIS_PER_SECOND + minutes * DateUtils.MILLIS_PER_MINUTE + hours * DateUtils.MILLIS_PER_HOUR + + (cal.get(Calendar.DAY_OF_YEAR) - 1) * DateUtils.MILLIS_PER_DAY, testResult); } @Test @@ -309,7 +297,7 @@ testResult); @Test public void testMinutesOfDayWithCalendar() { long testResult = DateUtils.getFragmentInMinutes(aCalendar, Calendar.DATE); - final long expectedValue = minutes + hours * DateUtils.MILLIS_PER_HOUR/ DateUtils.MILLIS_PER_MINUTE; + final long expectedValue = minutes + hours * DateUtils.MILLIS_PER_HOUR / DateUtils.MILLIS_PER_MINUTE; assertEquals(expectedValue, testResult); testResult = DateUtils.getFragmentInMinutes(aCalendar, Calendar.DAY_OF_YEAR); assertEquals(expectedValue, testResult); @@ -318,7 +306,7 @@ testResult); @Test public void testMinutesOfDayWithDate() { long testResult = DateUtils.getFragmentInMinutes(aDate, Calendar.DATE); - final long expectedValue = minutes + hours * DateUtils.MILLIS_PER_HOUR/ DateUtils.MILLIS_PER_MINUTE; + final long expectedValue = minutes + hours * DateUtils.MILLIS_PER_HOUR / DateUtils.MILLIS_PER_MINUTE; assertEquals(expectedValue, testResult); testResult = DateUtils.getFragmentInMinutes(aDate, Calendar.DAY_OF_YEAR); assertEquals(expectedValue, testResult); @@ -339,25 +327,20 @@ testResult); @Test public void testMinutesOfMonthWithCalendar() { final long testResult = DateUtils.getFragmentInMinutes(aCalendar, Calendar.MONTH); - assertEquals( minutes +(hours * DateUtils.MILLIS_PER_HOUR + (days - 1) * DateUtils.MILLIS_PER_DAY) - / DateUtils.MILLIS_PER_MINUTE, - testResult); + assertEquals(minutes + (hours * DateUtils.MILLIS_PER_HOUR + (days - 1) * DateUtils.MILLIS_PER_DAY) / DateUtils.MILLIS_PER_MINUTE, testResult); } @Test public void testMinutesOfMonthWithDate() { final long testResult = DateUtils.getFragmentInMinutes(aDate, Calendar.MONTH); - assertEquals(minutes - + (hours * DateUtils.MILLIS_PER_HOUR + (days - 1) * DateUtils.MILLIS_PER_DAY) - / DateUtils.MILLIS_PER_MINUTE, - testResult); + assertEquals(minutes + (hours * DateUtils.MILLIS_PER_HOUR + (days - 1) * DateUtils.MILLIS_PER_DAY) / DateUtils.MILLIS_PER_MINUTE, testResult); } @Test public void testMinutesOfYearWithCalendar() { final long testResult = DateUtils.getFragmentInMinutes(aCalendar, Calendar.YEAR); - assertEquals( minutes +(hours * DateUtils.MILLIS_PER_HOUR + (aCalendar.get(Calendar.DAY_OF_YEAR) - 1) * DateUtils.MILLIS_PER_DAY) - / DateUtils.MILLIS_PER_MINUTE, + assertEquals(minutes + + (hours * DateUtils.MILLIS_PER_HOUR + (aCalendar.get(Calendar.DAY_OF_YEAR) - 1) * DateUtils.MILLIS_PER_DAY) / DateUtils.MILLIS_PER_MINUTE, testResult); } @@ -366,9 +349,8 @@ testResult); final long testResult = DateUtils.getFragmentInMinutes(aDate, Calendar.YEAR); final Calendar cal = Calendar.getInstance(); cal.setTime(aDate); - assertEquals(minutes - + (hours * DateUtils.MILLIS_PER_HOUR + (cal.get(Calendar.DAY_OF_YEAR) - 1) * DateUtils.MILLIS_PER_DAY) - / DateUtils.MILLIS_PER_MINUTE, + assertEquals( + minutes + (hours * DateUtils.MILLIS_PER_HOUR + (cal.get(Calendar.DAY_OF_YEAR) - 1) * DateUtils.MILLIS_PER_DAY) / DateUtils.MILLIS_PER_MINUTE, testResult); } @@ -382,7 +364,7 @@ testResult); c.set(Calendar.SECOND, 0); c.set(Calendar.MILLISECOND, 0); final long testResult = DateUtils.getFragmentInMinutes(c, Calendar.YEAR); - assertEquals( 0, testResult); + assertEquals(0, testResult); } @Test @@ -422,7 +404,7 @@ testResult); @Test public void testSecondsOfDayWithCalendar() { long testresult = DateUtils.getFragmentInSeconds(aCalendar, Calendar.DATE); - final long expectedValue = seconds + (minutes * DateUtils.MILLIS_PER_MINUTE + hours * DateUtils.MILLIS_PER_HOUR)/ DateUtils.MILLIS_PER_SECOND; + final long expectedValue = seconds + (minutes * DateUtils.MILLIS_PER_MINUTE + hours * DateUtils.MILLIS_PER_HOUR) / DateUtils.MILLIS_PER_SECOND; assertEquals(expectedValue, testresult); testresult = DateUtils.getFragmentInSeconds(aCalendar, Calendar.DAY_OF_YEAR); assertEquals(expectedValue, testresult); @@ -431,7 +413,7 @@ testResult); @Test public void testSecondsOfDayWithDate() { long testresult = DateUtils.getFragmentInSeconds(aDate, Calendar.DATE); - final long expectedValue = seconds + (minutes * DateUtils.MILLIS_PER_MINUTE + hours * DateUtils.MILLIS_PER_HOUR)/ DateUtils.MILLIS_PER_SECOND; + final long expectedValue = seconds + (minutes * DateUtils.MILLIS_PER_MINUTE + hours * DateUtils.MILLIS_PER_HOUR) / DateUtils.MILLIS_PER_SECOND; assertEquals(expectedValue, testresult); testresult = DateUtils.getFragmentInSeconds(aDate, Calendar.DAY_OF_YEAR); assertEquals(expectedValue, testresult); @@ -440,21 +422,13 @@ testResult); @Test public void testSecondsofHourWithCalendar() { final long testResult = DateUtils.getFragmentInSeconds(aCalendar, Calendar.HOUR_OF_DAY); - assertEquals( - seconds - + minutes - * DateUtils.MILLIS_PER_MINUTE / DateUtils.MILLIS_PER_SECOND, - testResult); + assertEquals(seconds + minutes * DateUtils.MILLIS_PER_MINUTE / DateUtils.MILLIS_PER_SECOND, testResult); } @Test public void testSecondsofHourWithDate() { final long testResult = DateUtils.getFragmentInSeconds(aDate, Calendar.HOUR_OF_DAY); - assertEquals( - seconds - + minutes - * DateUtils.MILLIS_PER_MINUTE / DateUtils.MILLIS_PER_SECOND, - testResult); + assertEquals(seconds + minutes * DateUtils.MILLIS_PER_MINUTE / DateUtils.MILLIS_PER_SECOND, testResult); } @Test @@ -473,34 +447,22 @@ testResult); @Test public void testSecondsOfMonthWithCalendar() { final long testResult = DateUtils.getFragmentInSeconds(aCalendar, Calendar.MONTH); - assertEquals( - seconds - + (minutes * DateUtils.MILLIS_PER_MINUTE - + hours * DateUtils.MILLIS_PER_HOUR + (days - 1) * DateUtils.MILLIS_PER_DAY) - / DateUtils.MILLIS_PER_SECOND, - testResult); + assertEquals(seconds + (minutes * DateUtils.MILLIS_PER_MINUTE + hours * DateUtils.MILLIS_PER_HOUR + (days - 1) * DateUtils.MILLIS_PER_DAY) + / DateUtils.MILLIS_PER_SECOND, testResult); } @Test public void testSecondsOfMonthWithDate() { final long testResult = DateUtils.getFragmentInSeconds(aDate, Calendar.MONTH); - assertEquals( - seconds - + (minutes * DateUtils.MILLIS_PER_MINUTE - + hours * DateUtils.MILLIS_PER_HOUR + (days - 1) * DateUtils.MILLIS_PER_DAY) - / DateUtils.MILLIS_PER_SECOND, - testResult); + assertEquals(seconds + (minutes * DateUtils.MILLIS_PER_MINUTE + hours * DateUtils.MILLIS_PER_HOUR + (days - 1) * DateUtils.MILLIS_PER_DAY) + / DateUtils.MILLIS_PER_SECOND, testResult); } @Test public void testSecondsOfYearWithCalendar() { final long testResult = DateUtils.getFragmentInSeconds(aCalendar, Calendar.YEAR); - assertEquals( - seconds - + (minutes * DateUtils.MILLIS_PER_MINUTE - + hours * DateUtils.MILLIS_PER_HOUR + (aCalendar.get(Calendar.DAY_OF_YEAR) - 1) * DateUtils.MILLIS_PER_DAY) - / DateUtils.MILLIS_PER_SECOND, - testResult); + assertEquals(seconds + (minutes * DateUtils.MILLIS_PER_MINUTE + hours * DateUtils.MILLIS_PER_HOUR + + (aCalendar.get(Calendar.DAY_OF_YEAR) - 1) * DateUtils.MILLIS_PER_DAY) / DateUtils.MILLIS_PER_SECOND, testResult); } @Test @@ -508,10 +470,8 @@ testResult); final long testResult = DateUtils.getFragmentInSeconds(aDate, Calendar.YEAR); final Calendar cal = Calendar.getInstance(); cal.setTime(aDate); - assertEquals( - seconds - + (minutes * DateUtils.MILLIS_PER_MINUTE - + hours * DateUtils.MILLIS_PER_HOUR + (cal.get(Calendar.DAY_OF_YEAR) - 1) * DateUtils.MILLIS_PER_DAY) + assertEquals(seconds + + (minutes * DateUtils.MILLIS_PER_MINUTE + hours * DateUtils.MILLIS_PER_HOUR + (cal.get(Calendar.DAY_OF_YEAR) - 1) * DateUtils.MILLIS_PER_DAY) / DateUtils.MILLIS_PER_SECOND, testResult); } diff --git a/src/test/java/org/apache/commons/lang3/time/DateUtilsRoundingTest.java b/src/test/java/org/apache/commons/lang3/time/DateUtilsRoundingTest.java index fef25d80f..750e409ed 100644 --- a/src/test/java/org/apache/commons/lang3/time/DateUtilsRoundingTest.java +++ b/src/test/java/org/apache/commons/lang3/time/DateUtilsRoundingTest.java @@ -70,13 +70,13 @@ public class DateUtilsRoundingTest extends AbstractLangTest { protected void baseRoundTest(final Date roundedUpDate, final Date roundedDownDate, final Date lastRoundDownDate, final int calendarField) { final Date firstRoundUpDate = DateUtils.addMilliseconds(lastRoundDownDate, 1); - //Date-comparison + // Date-comparison assertEquals(roundedDownDate, DateUtils.round(roundedDownDate, calendarField)); assertEquals(roundedUpDate, DateUtils.round(roundedUpDate, calendarField)); assertEquals(roundedDownDate, DateUtils.round(lastRoundDownDate, calendarField)); assertEquals(roundedUpDate, DateUtils.round(firstRoundUpDate, calendarField)); - //Calendar-initiations + // Calendar-initiations final Calendar roundedUpCalendar; final Calendar roundedDownCalendar; final Calendar lastRoundDownCalendar; @@ -90,13 +90,13 @@ public class DateUtilsRoundingTest extends AbstractLangTest { lastRoundDownCalendar.setTime(lastRoundDownDate); firstRoundUpCalendar.setTime(firstRoundUpDate); - //Calendar-comparison + // Calendar-comparison assertEquals(roundedDownCalendar, DateUtils.round(roundedDownCalendar, calendarField)); assertEquals(roundedUpCalendar, DateUtils.round(roundedUpCalendar, calendarField)); assertEquals(roundedDownCalendar, DateUtils.round(lastRoundDownCalendar, calendarField)); assertEquals(roundedUpCalendar, DateUtils.round(firstRoundUpCalendar, calendarField)); - //Object-comparison + // Object-comparison assertEquals(roundedDownDate, DateUtils.round((Object) roundedDownDate, calendarField)); assertEquals(roundedUpDate, DateUtils.round((Object) roundedUpDate, calendarField)); assertEquals(roundedDownDate, DateUtils.round((Object) lastRoundDownDate, calendarField)); @@ -120,12 +120,14 @@ public class DateUtilsRoundingTest extends AbstractLangTest { protected void baseTruncateTest(final Date truncatedDate, final Date lastTruncateDate, final int calendarField) { final Date nextTruncateDate = DateUtils.addMilliseconds(lastTruncateDate, 1); - //Date-comparison - assertEquals(truncatedDate, DateUtils.truncate(truncatedDate, calendarField), "Truncating "+ fdf.format(truncatedDate) +" as Date with CalendarField-value "+ calendarField +" must return itself"); + // Date-comparison + assertEquals(truncatedDate, DateUtils.truncate(truncatedDate, calendarField), + "Truncating " + fdf.format(truncatedDate) + " as Date with CalendarField-value " + calendarField + " must return itself"); assertEquals(truncatedDate, DateUtils.truncate(lastTruncateDate, calendarField)); - assertNotEquals(truncatedDate, DateUtils.truncate(nextTruncateDate, calendarField), fdf.format(lastTruncateDate) + " is not an extreme when truncating as Date with CalendarField-value " + calendarField); + assertNotEquals(truncatedDate, DateUtils.truncate(nextTruncateDate, calendarField), + fdf.format(lastTruncateDate) + " is not an extreme when truncating as Date with CalendarField-value " + calendarField); - //Calendar-initiations + // Calendar-initiations final Calendar truncatedCalendar; final Calendar lastTruncateCalendar; final Calendar nextTruncateCalendar; @@ -136,18 +138,24 @@ public class DateUtilsRoundingTest extends AbstractLangTest { lastTruncateCalendar.setTime(lastTruncateDate); nextTruncateCalendar.setTime(nextTruncateDate); - //Calendar-comparison - assertEquals(truncatedCalendar, DateUtils.truncate(truncatedCalendar, calendarField), "Truncating "+ fdf.format(truncatedCalendar) +" as Calendar with CalendarField-value "+ calendarField +" must return itself"); + // Calendar-comparison + assertEquals(truncatedCalendar, DateUtils.truncate(truncatedCalendar, calendarField), + "Truncating " + fdf.format(truncatedCalendar) + " as Calendar with CalendarField-value " + calendarField + " must return itself"); assertEquals(truncatedCalendar, DateUtils.truncate(lastTruncateCalendar, calendarField)); - assertNotEquals(truncatedCalendar, DateUtils.truncate(nextTruncateCalendar, calendarField), fdf.format(lastTruncateCalendar) + " is not an extreme when truncating as Calendar with CalendarField-value " + calendarField); + assertNotEquals(truncatedCalendar, DateUtils.truncate(nextTruncateCalendar, calendarField), + fdf.format(lastTruncateCalendar) + " is not an extreme when truncating as Calendar with CalendarField-value " + calendarField); - //Object-comparison - assertEquals(truncatedDate, DateUtils.truncate((Object) truncatedDate, calendarField), "Truncating "+ fdf.format(truncatedDate) +" as Date cast to Object with CalendarField-value "+ calendarField +" must return itself as Date"); + // Object-comparison + assertEquals(truncatedDate, DateUtils.truncate((Object) truncatedDate, calendarField), "Truncating " + fdf.format(truncatedDate) + + " as Date cast to Object with CalendarField-value " + calendarField + " must return itself as Date"); assertEquals(truncatedDate, DateUtils.truncate((Object) lastTruncateDate, calendarField)); - assertNotEquals(truncatedDate, DateUtils.truncate((Object) nextTruncateDate, calendarField), fdf.format(lastTruncateDate) + " is not an extreme when truncating as Date cast to Object with CalendarField-value " + calendarField); - assertEquals(truncatedDate, DateUtils.truncate((Object) truncatedCalendar, calendarField), "Truncating "+ fdf.format(truncatedCalendar) +" as Calendar cast to Object with CalendarField-value "+ calendarField +" must return itself as Date"); + assertNotEquals(truncatedDate, DateUtils.truncate((Object) nextTruncateDate, calendarField), + fdf.format(lastTruncateDate) + " is not an extreme when truncating as Date cast to Object with CalendarField-value " + calendarField); + assertEquals(truncatedDate, DateUtils.truncate((Object) truncatedCalendar, calendarField), "Truncating " + fdf.format(truncatedCalendar) + + " as Calendar cast to Object with CalendarField-value " + calendarField + " must return itself as Date"); assertEquals(truncatedDate, DateUtils.truncate((Object) lastTruncateCalendar, calendarField)); - assertNotEquals(truncatedDate, DateUtils.truncate((Object) nextTruncateCalendar, calendarField), fdf.format(lastTruncateCalendar) + " is not an extreme when truncating as Calendar cast to Object with CalendarField-value " + calendarField); + assertNotEquals(truncatedDate, DateUtils.truncate((Object) nextTruncateCalendar, calendarField), + fdf.format(lastTruncateCalendar) + " is not an extreme when truncating as Calendar cast to Object with CalendarField-value " + calendarField); } /** @@ -160,7 +168,8 @@ public class DateUtilsRoundingTest extends AbstractLangTest { * @param calendarField a Calendar.field value */ protected void roundToJanuaryFirst(final Date minDate, final Date maxDate, final int calendarField) { - assertEquals(januaryOneDate, DateUtils.round(januaryOneDate, calendarField), "Rounding "+ fdf.format(januaryOneDate) +" as Date with CalendarField-value "+ calendarField +" must return itself"); + assertEquals(januaryOneDate, DateUtils.round(januaryOneDate, calendarField), + "Rounding " + fdf.format(januaryOneDate) + " as Date with CalendarField-value " + calendarField + " must return itself"); assertEquals(januaryOneDate, DateUtils.round(minDate, calendarField)); assertEquals(januaryOneDate, DateUtils.round(maxDate, calendarField)); @@ -168,21 +177,26 @@ public class DateUtilsRoundingTest extends AbstractLangTest { minCalendar.setTime(minDate); final Calendar maxCalendar = Calendar.getInstance(); maxCalendar.setTime(maxDate); - assertEquals(januaryOneCalendar, DateUtils.round(januaryOneCalendar, calendarField), "Rounding "+ fdf.format(januaryOneCalendar) +" as Date with CalendarField-value "+ calendarField +" must return itself"); + assertEquals(januaryOneCalendar, DateUtils.round(januaryOneCalendar, calendarField), + "Rounding " + fdf.format(januaryOneCalendar) + " as Date with CalendarField-value " + calendarField + " must return itself"); assertEquals(januaryOneCalendar, DateUtils.round(minCalendar, calendarField)); assertEquals(januaryOneCalendar, DateUtils.round(maxCalendar, calendarField)); final Date toPrevRoundDate = DateUtils.addMilliseconds(minDate, -1); final Date toNextRoundDate = DateUtils.addMilliseconds(maxDate, 1); - assertNotEquals(januaryOneDate, DateUtils.round(toPrevRoundDate, calendarField), fdf.format(minDate) + " is not an lower-extreme when rounding as Date with CalendarField-value " + calendarField); - assertNotEquals(januaryOneDate, DateUtils.round(toNextRoundDate, calendarField), fdf.format(maxDate) + " is not an upper-extreme when rounding as Date with CalendarField-value " + calendarField); + assertNotEquals(januaryOneDate, DateUtils.round(toPrevRoundDate, calendarField), + fdf.format(minDate) + " is not an lower-extreme when rounding as Date with CalendarField-value " + calendarField); + assertNotEquals(januaryOneDate, DateUtils.round(toNextRoundDate, calendarField), + fdf.format(maxDate) + " is not an upper-extreme when rounding as Date with CalendarField-value " + calendarField); final Calendar toPrevRoundCalendar = Calendar.getInstance(); toPrevRoundCalendar.setTime(toPrevRoundDate); final Calendar toNextRoundCalendar = Calendar.getInstance(); toNextRoundCalendar.setTime(toNextRoundDate); - assertNotEquals(januaryOneDate, DateUtils.round(toPrevRoundDate, calendarField), fdf.format(minCalendar) + " is not an lower-extreme when rounding as Date with CalendarField-value " + calendarField); - assertNotEquals(januaryOneDate, DateUtils.round(toNextRoundDate, calendarField), fdf.format(maxCalendar) + " is not an upper-extreme when rounding as Date with CalendarField-value " + calendarField); + assertNotEquals(januaryOneDate, DateUtils.round(toPrevRoundDate, calendarField), + fdf.format(minCalendar) + " is not an lower-extreme when rounding as Date with CalendarField-value " + calendarField); + assertNotEquals(januaryOneDate, DateUtils.round(toNextRoundDate, calendarField), + fdf.format(maxCalendar) + " is not an upper-extreme when rounding as Date with CalendarField-value " + calendarField); } @BeforeEach @@ -192,13 +206,13 @@ public class DateUtilsRoundingTest extends AbstractLangTest { targetYearDate = dateTimeParser.parse("January 1, 2007 0:00:00.000"); targetDateDate = targetDayOfMonthDate = dateTimeParser.parse("June 1, 2008 0:00:00.000"); - targetAmDate = dateTimeParser.parse("June 1, 2008 0:00:00.000"); + targetAmDate = dateTimeParser.parse("June 1, 2008 0:00:00.000"); targetPmDate = dateTimeParser.parse("June 1, 2008 12:00:00.000"); targetHourDate = dateTimeParser.parse("June 1, 2008 8:00:00.000"); targetHourOfDayDate = dateTimeParser.parse("June 1, 2008 8:00:00.000"); - targetMinuteDate = dateTimeParser.parse("June 1, 2008 8:15:00.000"); - targetSecondDate = dateTimeParser.parse("June 1, 2008 8:15:14.000"); - targetMilliSecondDate = dateTimeParser.parse("June 1, 2008 8:15:14.231"); + targetMinuteDate = dateTimeParser.parse("June 1, 2008 8:15:00.000"); + targetSecondDate = dateTimeParser.parse("June 1, 2008 8:15:14.000"); + targetMilliSecondDate = dateTimeParser.parse("June 1, 2008 8:15:14.231"); januaryOneDate = dateTimeParser.parse("January 1, 2008 0:00:00.000"); januaryOneCalendar = Calendar.getInstance(); @@ -219,19 +233,19 @@ public class DateUtilsRoundingTest extends AbstractLangTest { final Date minDate; final Date maxDate; - //AM + // AM roundedUpDate = dateTimeParser.parse("June 1, 2008 12:00:00.000"); roundedDownDate = targetAmDate; lastRoundedDownDate = dateTimeParser.parse("June 1, 2008 5:59:59.999"); - baseRoundTest(roundedUpDate, roundedDownDate, lastRoundedDownDate, calendarField); + baseRoundTest(roundedUpDate, roundedDownDate, lastRoundedDownDate, calendarField); - //PM + // PM roundedUpDate = dateTimeParser.parse("June 2, 2008 0:00:00.000"); roundedDownDate = targetPmDate; lastRoundedDownDate = dateTimeParser.parse("June 1, 2008 17:59:59.999"); - baseRoundTest(roundedUpDate, roundedDownDate, lastRoundedDownDate, calendarField); + baseRoundTest(roundedUpDate, roundedDownDate, lastRoundedDownDate, calendarField); - //round to January 1 + // round to January 1 minDate = dateTimeParser.parse("December 31, 2007 18:00:00.000"); maxDate = dateTimeParser.parse("January 1, 2008 5:59:59.999"); roundToJanuaryFirst(minDate, maxDate, calendarField); @@ -256,9 +270,9 @@ public class DateUtilsRoundingTest extends AbstractLangTest { roundedUpDate = dateTimeParser.parse("June 2, 2008 0:00:00.000"); roundedDownDate = targetDateDate; lastRoundedDownDate = dateTimeParser.parse("June 1, 2008 11:59:59.999"); - baseRoundTest(roundedUpDate, roundedDownDate, lastRoundedDownDate, calendarField); + baseRoundTest(roundedUpDate, roundedDownDate, lastRoundedDownDate, calendarField); - //round to January 1 + // round to January 1 minDate = dateTimeParser.parse("December 31, 2007 12:00:00.000"); maxDate = dateTimeParser.parse("January 1, 2008 11:59:59.999"); roundToJanuaryFirst(minDate, maxDate, calendarField); @@ -283,9 +297,9 @@ public class DateUtilsRoundingTest extends AbstractLangTest { roundedUpDate = dateTimeParser.parse("June 2, 2008 0:00:00.000"); roundedDownDate = targetDayOfMonthDate; lastRoundedDownDate = dateTimeParser.parse("June 1, 2008 11:59:59.999"); - baseRoundTest(roundedUpDate, roundedDownDate, lastRoundedDownDate, calendarField); + baseRoundTest(roundedUpDate, roundedDownDate, lastRoundedDownDate, calendarField); - //round to January 1 + // round to January 1 minDate = dateTimeParser.parse("December 31, 2007 12:00:00.000"); maxDate = dateTimeParser.parse("January 1, 2008 11:59:59.999"); roundToJanuaryFirst(minDate, maxDate, calendarField); @@ -310,9 +324,9 @@ public class DateUtilsRoundingTest extends AbstractLangTest { roundedUpDate = dateTimeParser.parse("June 1, 2008 9:00:00.000"); roundedDownDate = targetHourDate; lastRoundedDownDate = dateTimeParser.parse("June 1, 2008 8:29:59.999"); - baseRoundTest(roundedUpDate, roundedDownDate, lastRoundedDownDate, calendarField); + baseRoundTest(roundedUpDate, roundedDownDate, lastRoundedDownDate, calendarField); - //round to January 1 + // round to January 1 minDate = dateTimeParser.parse("December 31, 2007 23:30:00.000"); maxDate = dateTimeParser.parse("January 1, 2008 0:29:59.999"); roundToJanuaryFirst(minDate, maxDate, calendarField); @@ -337,9 +351,9 @@ public class DateUtilsRoundingTest extends AbstractLangTest { roundedUpDate = dateTimeParser.parse("June 1, 2008 9:00:00.000"); roundedDownDate = targetHourOfDayDate; lastRoundedDownDate = dateTimeParser.parse("June 1, 2008 8:29:59.999"); - baseRoundTest(roundedUpDate, roundedDownDate, lastRoundedDownDate, calendarField); + baseRoundTest(roundedUpDate, roundedDownDate, lastRoundedDownDate, calendarField); - //round to January 1 + // round to January 1 minDate = dateTimeParser.parse("December 31, 2007 23:30:00.000"); maxDate = dateTimeParser.parse("January 1, 2008 0:29:59.999"); roundToJanuaryFirst(minDate, maxDate, calendarField); @@ -363,9 +377,9 @@ public class DateUtilsRoundingTest extends AbstractLangTest { roundedDownDate = lastRoundedDownDate = targetMilliSecondDate; roundedUpDate = dateTimeParser.parse("June 1, 2008 8:15:14.232"); - baseRoundTest(roundedUpDate, roundedDownDate, lastRoundedDownDate, calendarField); + baseRoundTest(roundedUpDate, roundedDownDate, lastRoundedDownDate, calendarField); - //round to January 1 + // round to January 1 minDate = maxDate = januaryOneDate; roundToJanuaryFirst(minDate, maxDate, calendarField); } @@ -389,9 +403,9 @@ public class DateUtilsRoundingTest extends AbstractLangTest { roundedUpDate = dateTimeParser.parse("June 1, 2008 8:16:00.000"); roundedDownDate = targetMinuteDate; lastRoundedDownDate = dateTimeParser.parse("June 1, 2008 8:15:29.999"); - baseRoundTest(roundedUpDate, roundedDownDate, lastRoundedDownDate, calendarField); + baseRoundTest(roundedUpDate, roundedDownDate, lastRoundedDownDate, calendarField); - //round to January 1 + // round to January 1 minDate = dateTimeParser.parse("December 31, 2007 23:59:30.000"); maxDate = dateTimeParser.parse("January 1, 2008 0:00:29.999"); roundToJanuaryFirst(minDate, maxDate, calendarField); @@ -411,31 +425,31 @@ public class DateUtilsRoundingTest extends AbstractLangTest { final Date minDate; final Date maxDate; - //month with 28 days + // month with 28 days roundedUpDate = dateTimeParser.parse("March 1, 2007 0:00:00.000"); roundedDownDate = dateTimeParser.parse("February 1, 2007 0:00:00.000"); lastRoundedDownDate = dateTimeParser.parse("February 14, 2007 23:59:59.999"); - baseRoundTest(roundedUpDate, roundedDownDate, lastRoundedDownDate, calendarField); + baseRoundTest(roundedUpDate, roundedDownDate, lastRoundedDownDate, calendarField); - //month with 29 days + // month with 29 days roundedUpDate = dateTimeParser.parse("March 1, 2008 0:00:00.000"); roundedDownDate = dateTimeParser.parse("February 1, 2008 0:00:00.000"); lastRoundedDownDate = dateTimeParser.parse("February 15, 2008 23:59:59.999"); - baseRoundTest(roundedUpDate, roundedDownDate, lastRoundedDownDate, calendarField); + baseRoundTest(roundedUpDate, roundedDownDate, lastRoundedDownDate, calendarField); - //month with 30 days + // month with 30 days roundedUpDate = dateTimeParser.parse("May 1, 2008 0:00:00.000"); roundedDownDate = dateTimeParser.parse("April 1, 2008 0:00:00.000"); lastRoundedDownDate = dateTimeParser.parse("April 15, 2008 23:59:59.999"); - baseRoundTest(roundedUpDate, roundedDownDate, lastRoundedDownDate, calendarField); + baseRoundTest(roundedUpDate, roundedDownDate, lastRoundedDownDate, calendarField); - //month with 31 days + // month with 31 days roundedUpDate = dateTimeParser.parse("June 1, 2008 0:00:00.000"); roundedDownDate = dateTimeParser.parse("May 1, 2008 0:00:00.000"); lastRoundedDownDate = dateTimeParser.parse("May 16, 2008 23:59:59.999"); - baseRoundTest(roundedUpDate, roundedDownDate, lastRoundedDownDate, calendarField); + baseRoundTest(roundedUpDate, roundedDownDate, lastRoundedDownDate, calendarField); - //round to January 1 + // round to January 1 minDate = dateTimeParser.parse("December 17, 2007 00:00:00.000"); maxDate = dateTimeParser.parse("January 16, 2008 23:59:59.999"); roundToJanuaryFirst(minDate, maxDate, calendarField); @@ -460,9 +474,9 @@ public class DateUtilsRoundingTest extends AbstractLangTest { roundedUpDate = dateTimeParser.parse("June 1, 2008 8:15:15.000"); roundedDownDate = targetSecondDate; lastRoundedDownDate = dateTimeParser.parse("June 1, 2008 8:15:14.499"); - baseRoundTest(roundedUpDate, roundedDownDate, lastRoundedDownDate, calendarField); + baseRoundTest(roundedUpDate, roundedDownDate, lastRoundedDownDate, calendarField); - //round to January 1 + // round to January 1 minDate = dateTimeParser.parse("December 31, 2007 23:59:59.500"); maxDate = dateTimeParser.parse("January 1, 2008 0:00:00.499"); roundToJanuaryFirst(minDate, maxDate, calendarField); @@ -482,55 +496,55 @@ public class DateUtilsRoundingTest extends AbstractLangTest { final Date minDate; final Date maxDate; - //month with 28 days (1) + // month with 28 days (1) roundedUpDate = dateTimeParser.parse("February 16, 2007 0:00:00.000"); roundedDownDate = dateTimeParser.parse("February 1, 2007 0:00:00.000"); lastRoundedDownDate = dateTimeParser.parse("February 8, 2007 23:59:59.999"); - baseRoundTest(roundedUpDate, roundedDownDate, lastRoundedDownDate, calendarField); + baseRoundTest(roundedUpDate, roundedDownDate, lastRoundedDownDate, calendarField); - //month with 28 days (2) + // month with 28 days (2) roundedUpDate = dateTimeParser.parse("March 1, 2007 0:00:00.000"); roundedDownDate = dateTimeParser.parse("February 16, 2007 0:00:00.000"); lastRoundedDownDate = dateTimeParser.parse("February 23, 2007 23:59:59.999"); - baseRoundTest(roundedUpDate, roundedDownDate, lastRoundedDownDate, calendarField); + baseRoundTest(roundedUpDate, roundedDownDate, lastRoundedDownDate, calendarField); - //month with 29 days (1) + // month with 29 days (1) roundedUpDate = dateTimeParser.parse("February 16, 2008 0:00:00.000"); roundedDownDate = dateTimeParser.parse("February 1, 2008 0:00:00.000"); lastRoundedDownDate = dateTimeParser.parse("February 8, 2008 23:59:59.999"); - baseRoundTest(roundedUpDate, roundedDownDate, lastRoundedDownDate, calendarField); + baseRoundTest(roundedUpDate, roundedDownDate, lastRoundedDownDate, calendarField); - //month with 29 days (2) + // month with 29 days (2) roundedUpDate = dateTimeParser.parse("March 1, 2008 0:00:00.000"); roundedDownDate = dateTimeParser.parse("February 16, 2008 0:00:00.000"); lastRoundedDownDate = dateTimeParser.parse("February 23, 2008 23:59:59.999"); - baseRoundTest(roundedUpDate, roundedDownDate, lastRoundedDownDate, calendarField); + baseRoundTest(roundedUpDate, roundedDownDate, lastRoundedDownDate, calendarField); - //month with 30 days (1) + // month with 30 days (1) roundedUpDate = dateTimeParser.parse("April 16, 2008 0:00:00.000"); roundedDownDate = dateTimeParser.parse("April 1, 2008 0:00:00.000"); lastRoundedDownDate = dateTimeParser.parse("April 8, 2008 23:59:59.999"); - baseRoundTest(roundedUpDate, roundedDownDate, lastRoundedDownDate, calendarField); + baseRoundTest(roundedUpDate, roundedDownDate, lastRoundedDownDate, calendarField); - //month with 30 days (2) + // month with 30 days (2) roundedUpDate = dateTimeParser.parse("May 1, 2008 0:00:00.000"); roundedDownDate = dateTimeParser.parse("April 16, 2008 0:00:00.000"); lastRoundedDownDate = dateTimeParser.parse("April 23, 2008 23:59:59.999"); - baseRoundTest(roundedUpDate, roundedDownDate, lastRoundedDownDate, calendarField); + baseRoundTest(roundedUpDate, roundedDownDate, lastRoundedDownDate, calendarField); - //month with 31 days (1) + // month with 31 days (1) roundedUpDate = dateTimeParser.parse("May 16, 2008 0:00:00.000"); roundedDownDate = dateTimeParser.parse("May 1, 2008 0:00:00.000"); lastRoundedDownDate = dateTimeParser.parse("May 8, 2008 23:59:59.999"); - baseRoundTest(roundedUpDate, roundedDownDate, lastRoundedDownDate, calendarField); + baseRoundTest(roundedUpDate, roundedDownDate, lastRoundedDownDate, calendarField); - //month with 31 days (2) + // month with 31 days (2) roundedUpDate = dateTimeParser.parse("June 1, 2008 0:00:00.000"); roundedDownDate = dateTimeParser.parse("May 16, 2008 0:00:00.000"); lastRoundedDownDate = dateTimeParser.parse("May 23, 2008 23:59:59.999"); - baseRoundTest(roundedUpDate, roundedDownDate, lastRoundedDownDate, calendarField); + baseRoundTest(roundedUpDate, roundedDownDate, lastRoundedDownDate, calendarField); - //round to January 1 + // round to January 1 minDate = dateTimeParser.parse("December 24, 2007 00:00:00.000"); maxDate = dateTimeParser.parse("January 8, 2008 23:59:59.999"); roundToJanuaryFirst(minDate, maxDate, calendarField); @@ -547,7 +561,7 @@ public class DateUtilsRoundingTest extends AbstractLangTest { final Date roundedUpDate = dateTimeParser.parse("January 1, 2008 0:00:00.000"); final Date roundedDownDate = targetYearDate; final Date lastRoundedDownDate = dateTimeParser.parse("June 30, 2007 23:59:59.999"); - baseRoundTest(roundedUpDate, roundedDownDate, lastRoundedDownDate, calendarField); + baseRoundTest(roundedUpDate, roundedDownDate, lastRoundedDownDate, calendarField); } /** @@ -559,12 +573,10 @@ public class DateUtilsRoundingTest extends AbstractLangTest { @Test public void testTruncateAmPm() throws Exception { final int calendarField = Calendar.AM_PM; - - //AM + // AM Date lastTruncateDate = dateTimeParser.parse("June 1, 2008 11:59:59.999"); baseTruncateTest(targetAmDate, lastTruncateDate, calendarField); - - //PM + // PM lastTruncateDate = dateTimeParser.parse("June 1, 2008 23:59:59.999"); baseTruncateTest(targetPmDate, lastTruncateDate, calendarField); } @@ -674,42 +686,42 @@ public class DateUtilsRoundingTest extends AbstractLangTest { final int calendarField = DateUtils.SEMI_MONTH; Date truncatedDate, lastTruncateDate; - //month with 28 days (1) + // month with 28 days (1) truncatedDate = dateTimeParser.parse("February 1, 2007 0:00:00.000"); lastTruncateDate = dateTimeParser.parse("February 15, 2007 23:59:59.999"); baseTruncateTest(truncatedDate, lastTruncateDate, calendarField); - //month with 28 days (2) + // month with 28 days (2) truncatedDate = dateTimeParser.parse("February 16, 2007 0:00:00.000"); lastTruncateDate = dateTimeParser.parse("February 28, 2007 23:59:59.999"); baseTruncateTest(truncatedDate, lastTruncateDate, calendarField); - //month with 29 days (1) + // month with 29 days (1) truncatedDate = dateTimeParser.parse("February 1, 2008 0:00:00.000"); lastTruncateDate = dateTimeParser.parse("February 15, 2008 23:59:59.999"); baseTruncateTest(truncatedDate, lastTruncateDate, calendarField); - //month with 29 days (2) + // month with 29 days (2) truncatedDate = dateTimeParser.parse("February 16, 2008 0:00:00.000"); lastTruncateDate = dateTimeParser.parse("February 29, 2008 23:59:59.999"); baseTruncateTest(truncatedDate, lastTruncateDate, calendarField); - //month with 30 days (1) + // month with 30 days (1) truncatedDate = dateTimeParser.parse("April 1, 2008 0:00:00.000"); lastTruncateDate = dateTimeParser.parse("April 15, 2008 23:59:59.999"); baseTruncateTest(truncatedDate, lastTruncateDate, calendarField); - //month with 30 days (2) + // month with 30 days (2) truncatedDate = dateTimeParser.parse("April 16, 2008 0:00:00.000"); lastTruncateDate = dateTimeParser.parse("April 30, 2008 23:59:59.999"); baseTruncateTest(truncatedDate, lastTruncateDate, calendarField); - //month with 31 days (1) + // month with 31 days (1) truncatedDate = dateTimeParser.parse("March 1, 2008 0:00:00.000"); lastTruncateDate = dateTimeParser.parse("March 15, 2008 23:59:59.999"); baseTruncateTest(truncatedDate, lastTruncateDate, calendarField); - //month with 31 days (2) + // month with 31 days (2) truncatedDate = dateTimeParser.parse("March 16, 2008 0:00:00.000"); lastTruncateDate = dateTimeParser.parse("March 31, 2008 23:59:59.999"); baseTruncateTest(truncatedDate, lastTruncateDate, calendarField); diff --git a/src/test/java/org/apache/commons/lang3/time/DateUtilsTest.java b/src/test/java/org/apache/commons/lang3/time/DateUtilsTest.java index 49bb8ae2a..61c6e80d1 100644 --- a/src/test/java/org/apache/commons/lang3/time/DateUtilsTest.java +++ b/src/test/java/org/apache/commons/lang3/time/DateUtilsTest.java @@ -1661,13 +1661,13 @@ public class DateUtilsTest extends AbstractLangTest { @Test public void testWeekIterator() { final Calendar now = Calendar.getInstance(); - for (int i = 0; i< 7; i++) { + for (int i = 0; i < 7; i++) { final Calendar today = DateUtils.truncate(now, Calendar.DATE); final Calendar sunday = DateUtils.truncate(now, Calendar.DATE); sunday.add(Calendar.DATE, 1 - sunday.get(Calendar.DAY_OF_WEEK)); final Calendar monday = DateUtils.truncate(now, Calendar.DATE); if (monday.get(Calendar.DAY_OF_WEEK) == 1) { - //This is sunday... roll back 6 days + // This is sunday... roll back 6 days monday.add(Calendar.DATE, -6); } else { monday.add(Calendar.DATE, 2 - monday.get(Calendar.DAY_OF_WEEK)); diff --git a/src/test/java/org/apache/commons/lang3/time/DurationFormatUtilsTest.java b/src/test/java/org/apache/commons/lang3/time/DurationFormatUtilsTest.java index 5c0878cf6..cbd4833e7 100644 --- a/src/test/java/org/apache/commons/lang3/time/DurationFormatUtilsTest.java +++ b/src/test/java/org/apache/commons/lang3/time/DurationFormatUtilsTest.java @@ -72,12 +72,12 @@ public class DurationFormatUtilsTest extends AbstractLangTest { c.set(year, month, day, 0, 0, 0); final int[] array1 = { year, month, day, 0, 0, 0 }; final int[] array2 = { year, month, day, 0, 0, 0 }; - for (int i=0; i < FOUR_YEARS; i++) { + for (int i = 0; i < FOUR_YEARS; i++) { array2[0] = c.get(Calendar.YEAR); array2[1] = c.get(Calendar.MONTH); array2[2] = c.get(Calendar.DAY_OF_MONTH); final String tmpMsg = msg + array2[0] + "-" + array2[1] + "-" + array2[2] + " at "; - assertEqualDuration(tmpMsg + i, Integer.toString(i), array1, array2, format ); + assertEqualDuration(tmpMsg + i, Integer.toString(i), array1, array2, format); c.add(calendarType, 1); } } @@ -93,37 +93,17 @@ public class DurationFormatUtilsTest extends AbstractLangTest { @Test public void testAlternatingLiteralOptionals() { final String format = "['d'dH'h'][m'm']['s's]['ms'S]"; - - assertEquals("d1", - DurationFormatUtils.formatDuration(Duration.ofDays(1).toMillis(), format)); - - assertEquals("1h", - DurationFormatUtils.formatDuration(Duration.ofHours(1).toMillis(), format)); - - assertEquals("1m", - DurationFormatUtils.formatDuration(Duration.ofMinutes(1).toMillis(), format)); - - assertEquals("s1", - DurationFormatUtils.formatDuration(Duration.ofSeconds(1).toMillis(), format)); - - assertEquals("ms001", - DurationFormatUtils.formatDuration(Duration.ofMillis(1).toMillis(), format)); - - assertEquals("d1s1", - DurationFormatUtils.formatDuration(Duration.ofDays(1).plusSeconds(1).toMillis(), format)); - - assertEquals("d11h", - DurationFormatUtils.formatDuration(Duration.ofDays(1).plusHours(1).toMillis(), format)); - - assertEquals("d11h1m", - DurationFormatUtils.formatDuration(Duration.ofDays(1).plusHours(1).plusMinutes(1).toMillis(), format)); - - assertEquals("d11h1ms1", - DurationFormatUtils.formatDuration(Duration.ofDays(1).plusHours(1).plusMinutes(1).plusSeconds(1).toMillis(), format)); - + assertEquals("d1", DurationFormatUtils.formatDuration(Duration.ofDays(1).toMillis(), format)); + assertEquals("1h", DurationFormatUtils.formatDuration(Duration.ofHours(1).toMillis(), format)); + assertEquals("1m", DurationFormatUtils.formatDuration(Duration.ofMinutes(1).toMillis(), format)); + assertEquals("s1", DurationFormatUtils.formatDuration(Duration.ofSeconds(1).toMillis(), format)); + assertEquals("ms001", DurationFormatUtils.formatDuration(Duration.ofMillis(1).toMillis(), format)); + assertEquals("d1s1", DurationFormatUtils.formatDuration(Duration.ofDays(1).plusSeconds(1).toMillis(), format)); + assertEquals("d11h", DurationFormatUtils.formatDuration(Duration.ofDays(1).plusHours(1).toMillis(), format)); + assertEquals("d11h1m", DurationFormatUtils.formatDuration(Duration.ofDays(1).plusHours(1).plusMinutes(1).toMillis(), format)); + assertEquals("d11h1ms1", DurationFormatUtils.formatDuration(Duration.ofDays(1).plusHours(1).plusMinutes(1).plusSeconds(1).toMillis(), format)); assertEquals("d11h1ms1ms001", - DurationFormatUtils.formatDuration(Duration.ofDays(1).plusHours(1).plusMinutes(1).plusSeconds(1).plusMillis(1).toMillis(), format)); - + DurationFormatUtils.formatDuration(Duration.ofDays(1).plusHours(1).plusMinutes(1).plusSeconds(1).plusMillis(1).toMillis(), format)); } /** See https://issues.apache.org/bugzilla/show_bug.cgi?id=38401 */ @@ -613,7 +593,7 @@ public class DurationFormatUtilsTest extends AbstractLangTest { public void testLANG984() { // Long durations assertEquals("0", DurationFormatUtils.formatDuration(0, "S")); assertEquals(Integer.toString(Integer.MAX_VALUE), DurationFormatUtils.formatDuration(Integer.MAX_VALUE, "S")); - long maxIntPlus=Integer.MAX_VALUE; + long maxIntPlus = Integer.MAX_VALUE; maxIntPlus++; assertEquals(Long.toString(maxIntPlus), DurationFormatUtils.formatDuration(maxIntPlus, "S")); assertEquals(Long.toString(Long.MAX_VALUE), DurationFormatUtils.formatDuration(Long.MAX_VALUE, "S")); @@ -622,27 +602,27 @@ public class DurationFormatUtilsTest extends AbstractLangTest { @Test public void testLexx() { // tests each constant - assertArrayEquals(new DurationFormatUtils.Token[]{ + assertArrayEquals(new DurationFormatUtils.Token[] { createTokenWithCount(DurationFormatUtils.y, 1), createTokenWithCount(DurationFormatUtils.M, 1), createTokenWithCount(DurationFormatUtils.d, 1), createTokenWithCount(DurationFormatUtils.H, 1), createTokenWithCount(DurationFormatUtils.m, 1), createTokenWithCount(DurationFormatUtils.s, 1), - createTokenWithCount(DurationFormatUtils.S, 1)}, DurationFormatUtils.lexx("yMdHmsS")); + createTokenWithCount(DurationFormatUtils.S, 1) }, DurationFormatUtils.lexx("yMdHmsS")); // tests the ISO 8601-like - assertArrayEquals(new DurationFormatUtils.Token[]{ + assertArrayEquals(new DurationFormatUtils.Token[] { createTokenWithCount(DurationFormatUtils.H, 2), createTokenWithCount(new StringBuilder(":"), 1), createTokenWithCount(DurationFormatUtils.m, 2), createTokenWithCount(new StringBuilder(":"), 1), createTokenWithCount(DurationFormatUtils.s, 2), createTokenWithCount(new StringBuilder("."), 1), - createTokenWithCount(DurationFormatUtils.S, 3)}, DurationFormatUtils.lexx("HH:mm:ss.SSS")); + createTokenWithCount(DurationFormatUtils.S, 3) }, DurationFormatUtils.lexx("HH:mm:ss.SSS")); // test the iso extended format - assertArrayEquals(new DurationFormatUtils.Token[]{ + assertArrayEquals(new DurationFormatUtils.Token[] { createTokenWithCount(new StringBuilder("P"), 1), createTokenWithCount(DurationFormatUtils.y, 4), createTokenWithCount(new StringBuilder("Y"), 1), @@ -657,8 +637,7 @@ public class DurationFormatUtilsTest extends AbstractLangTest { createTokenWithCount(DurationFormatUtils.s, 1), createTokenWithCount(new StringBuilder("."), 1), createTokenWithCount(DurationFormatUtils.S, 3), - createTokenWithCount(new StringBuilder("S"), 1)}, DurationFormatUtils - .lexx(DurationFormatUtils.ISO_EXTENDED_FORMAT_PATTERN)); + createTokenWithCount(new StringBuilder("S"), 1) }, DurationFormatUtils.lexx(DurationFormatUtils.ISO_EXTENDED_FORMAT_PATTERN)); // test failures in equals final DurationFormatUtils.Token token = createTokenWithCount(DurationFormatUtils.y, 4); @@ -671,25 +650,17 @@ public class DurationFormatUtilsTest extends AbstractLangTest { @Test public void testLiteralPrefixOptionalToken() { - assertEquals( - DurationFormatUtils.formatDuration(10000L, "s's'"), - DurationFormatUtils.formatDuration(10000L, "['['d']']['<'H'>']['{'m'}']s's'")); - assertEquals( - DurationFormatUtils.formatDuration(10000L, "s's'"), - DurationFormatUtils.formatDuration(10000L, "['{'m'}']s's'")); + assertEquals(DurationFormatUtils.formatDuration(10000L, "s's'"), DurationFormatUtils.formatDuration(10000L, "['['d']']['<'H'>']['{'m'}']s's'")); + assertEquals(DurationFormatUtils.formatDuration(10000L, "s's'"), DurationFormatUtils.formatDuration(10000L, "['{'m'}']s's'")); } // Testing the under a day range in DurationFormatUtils.formatPeriod @Test public void testLowDurations() { - for (int hr=0; hr < 24; hr++) { - for (int min=0; min < 60; min++) { - for (int sec=0; sec < 60; sec++) { - assertEqualDuration(hr + ":" + min + ":" + sec, - new int[] { 2000, 0, 1, 0, 0, 0, 0 }, - new int[] { 2000, 0, 1, hr, min, sec }, - "H:m:s" - ); + for (int hr = 0; hr < 24; hr++) { + for (int min = 0; min < 60; min++) { + for (int sec = 0; sec < 60; sec++) { + assertEqualDuration(hr + ":" + min + ":" + sec, new int[] { 2000, 0, 1, 0, 0, 0, 0 }, new int[] { 2000, 0, 1, hr, min, sec }, "H:m:s"); } } } @@ -697,14 +668,10 @@ public class DurationFormatUtilsTest extends AbstractLangTest { @Test public void testMultipleOptionalBlocks() { - - assertEquals( - DurationFormatUtils.formatDuration(Duration.ofHours(1).toMillis(), "'[['H']]'"), - DurationFormatUtils.formatDuration(Duration.ofHours(1).toMillis(), "['{'d'}']['[['H']]']")); - - assertEquals( - DurationFormatUtils.formatDuration(Duration.ofDays(1).toMillis(), "['{'d'}']"), - DurationFormatUtils.formatDuration(Duration.ofDays(1).toMillis(), "['{'d'}']['['H']']")); + assertEquals(DurationFormatUtils.formatDuration(Duration.ofHours(1).toMillis(), "'[['H']]'"), + DurationFormatUtils.formatDuration(Duration.ofHours(1).toMillis(), "['{'d'}']['[['H']]']")); + assertEquals(DurationFormatUtils.formatDuration(Duration.ofDays(1).toMillis(), "['{'d'}']"), + DurationFormatUtils.formatDuration(Duration.ofDays(1).toMillis(), "['{'d'}']['['H']']")); } @Test diff --git a/src/test/java/org/apache/commons/lang3/time/FastDateFormatTest.java b/src/test/java/org/apache/commons/lang3/time/FastDateFormatTest.java index f4d6dbd0f..aa13e2845 100644 --- a/src/test/java/org/apache/commons/lang3/time/FastDateFormatTest.java +++ b/src/test/java/org/apache/commons/lang3/time/FastDateFormatTest.java @@ -273,13 +273,11 @@ public class FastDateFormatTest extends AbstractLangTest { public void testParseSync() throws InterruptedException { final String pattern = "yyyy-MM-dd'T'HH:mm:ss.SSS"; final SimpleDateFormat inner = new SimpleDateFormat(pattern); - final Format sdf= new Format() { + final Format sdf = new Format() { private static final long serialVersionUID = 1L; @Override - public StringBuffer format(final Object obj, - final StringBuffer toAppendTo, - final FieldPosition fieldPosition) { + public StringBuffer format(final Object obj, final StringBuffer toAppendTo, final FieldPosition fieldPosition) { synchronized (this) { return inner.format(obj, toAppendTo, fieldPosition); } @@ -292,13 +290,13 @@ public class FastDateFormatTest extends AbstractLangTest { } } }; - final AtomicLongArray sdfTime= measureTime(sdf, sdf); + final AtomicLongArray sdfTime = measureTime(sdf, sdf); final Format fdf = FastDateFormat.getInstance(pattern); - final AtomicLongArray fdfTime= measureTime(fdf, fdf); + final AtomicLongArray fdfTime = measureTime(fdf, fdf); - //System.out.println(">>FastDateFormatTest: FastDatePrinter:"+fdfTime.get(0)+" SimpleDateFormat:"+sdfTime.get(0)); - //System.out.println(">>FastDateFormatTest: FastDateParser:"+fdfTime.get(1)+" SimpleDateFormat:"+sdfTime.get(1)); + // System.out.println(">>FastDateFormatTest: FastDatePrinter:"+fdfTime.get(0)+" SimpleDateFormat:"+sdfTime.get(0)); + // System.out.println(">>FastDateFormatTest: FastDateParser:"+fdfTime.get(1)+" SimpleDateFormat:"+sdfTime.get(1)); } @Test diff --git a/src/test/java/org/apache/commons/lang3/time/FastDateParserSDFTest.java b/src/test/java/org/apache/commons/lang3/time/FastDateParserSDFTest.java index fbff75e3a..31bf1db26 100644 --- a/src/test/java/org/apache/commons/lang3/time/FastDateParserSDFTest.java +++ b/src/test/java/org/apache/commons/lang3/time/FastDateParserSDFTest.java @@ -42,6 +42,7 @@ public class FastDateParserSDFTest extends AbstractLangTest { private static final TimeZone timeZone = TimeZone.getDefault(); public static Stream<Arguments> data() { + // @formatter:off return Stream.of( // General Time zone tests Arguments.of("z yyyy", "GMT 2010", Locale.UK, true), // no offset specified, but this is allowed as a TimeZone name @@ -103,13 +104,14 @@ public class FastDateParserSDFTest extends AbstractLangTest { Arguments.of( "hh", "48", Locale.UK, true), // Hour in am/pm (1-12), i.e. midday/midnight is 12, not 0 Arguments.of( "kk", "48", Locale.UK, true) // Hour in day (1-24), i.e. midnight is 24, not 0 ); + // @formatter:on } private void checkParse(final String formattedDate, final String format, final Locale locale, final boolean valid) { final SimpleDateFormat sdf = new SimpleDateFormat(format, locale); sdf.setTimeZone(timeZone); final DateParser fdf = new FastDateParser(format, timeZone, locale); - Date expectedTime=null; + Date expectedTime = null; Class<?> sdfE = null; try { expectedTime = sdf.parse(formattedDate); @@ -173,11 +175,10 @@ public class FastDateParserSDFTest extends AbstractLangTest { final int endIndex = fdfP.getIndex(); final int length = formattedDate.length(); assertEquals(length, endIndex, "Expected FDF to parse full string " + fdfP); - assertEquals(expectedTime, actualTime, locale +" "+formattedDate +"\n"); + assertEquals(expectedTime, actualTime, locale + " " + formattedDate + "\n"); } else { assertNotEquals(-1, fdferrorIndex, "Test data error: expected FDF parse to fail, but got " + actualTime); - assertTrue(sdferrorIndex - fdferrorIndex <= 4, - "FDF error index ("+ fdferrorIndex + ") should approximate SDF index (" + sdferrorIndex + ")"); + assertTrue(sdferrorIndex - fdferrorIndex <= 4, "FDF error index (" + fdferrorIndex + ") should approximate SDF index (" + sdferrorIndex + ")"); } } diff --git a/src/test/java/org/apache/commons/lang3/time/FastDatePrinterTest.java b/src/test/java/org/apache/commons/lang3/time/FastDatePrinterTest.java index e37b6925a..1bf938a0a 100644 --- a/src/test/java/org/apache/commons/lang3/time/FastDatePrinterTest.java +++ b/src/test/java/org/apache/commons/lang3/time/FastDatePrinterTest.java @@ -42,14 +42,14 @@ import org.junitpioneer.jupiter.DefaultTimeZone; public class FastDatePrinterTest extends AbstractLangTest { private enum Expected1806 { - India(INDIA, "+05", "+0530", "+05:30"), Greenwich(TimeZones.GMT, "Z", "Z", "Z"), NewYork( - NEW_YORK, "-05", "-0500", "-05:00"); + India(INDIA, "+05", "+0530", "+05:30"), Greenwich(TimeZones.GMT, "Z", "Z", "Z"), NewYork(NEW_YORK, "-05", "-0500", "-05:00"); final TimeZone zone; final String one; final String two; final String three; + Expected1806(final TimeZone zone, final String one, final String two, final String three) { this.zone = zone; this.one = one; @@ -57,6 +57,7 @@ public class FastDatePrinterTest extends AbstractLangTest { this.three = three; } } + private static final String YYYY_MM_DD = "yyyy/MM/dd"; private static final TimeZone NEW_YORK = TimeZone.getTimeZone("America/New_York"); private static final TimeZone INDIA = TimeZone.getTimeZone("Asia/Calcutta"); @@ -93,9 +94,10 @@ public class FastDatePrinterTest extends AbstractLangTest { /** * Override this method in derived tests to change the construction of instances - * @param format the format string to use + * + * @param format the format string to use * @param timeZone the time zone to use - * @param locale the locale to use + * @param locale the locale to use * @return the DatePrinter to use for testing */ protected DatePrinter getInstance(final String format, final TimeZone timeZone, final Locale locale) { @@ -117,6 +119,7 @@ public class FastDatePrinterTest extends AbstractLangTest { assertEquals(trial.three, printer.format(cal)); } } + @Test public void test1806Argument() { assertThrows(IllegalArgumentException.class, () -> getInstance("XXXX")); @@ -155,8 +158,8 @@ public class FastDatePrinterTest extends AbstractLangTest { @Test public void testEquals() { - final DatePrinter printer1= getInstance(YYYY_MM_DD); - final DatePrinter printer2= getInstance(YYYY_MM_DD); + final DatePrinter printer1 = getInstance(YYYY_MM_DD); + final DatePrinter printer2 = getInstance(YYYY_MM_DD); assertEquals(printer1, printer2); assertEquals(printer1.hashCode(), printer2.hashCode()); @@ -203,8 +206,8 @@ public class FastDatePrinterTest extends AbstractLangTest { assertEquals("-04:00", fdf.format(cal2)); assertEquals("-04:00", fdf.format(millis2)); - final String pattern = "GGGG GGG GG G yyyy yyy yy y MMMM MMM MM M LLLL LLL LL L" + - " dddd ddd dd d DDDD DDD DD D EEEE EEE EE E aaaa aaa aa a zzzz zzz zz z"; + final String pattern = "GGGG GGG GG G yyyy yyy yy y MMMM MMM MM M LLLL LLL LL L" + + " dddd ddd dd d DDDD DDD DD D EEEE EEE EE E aaaa aaa aa a zzzz zzz zz z"; fdf = getInstance(pattern); sdf = new SimpleDateFormat(pattern); // SDF bug fix starting with Java 7 @@ -279,8 +282,7 @@ public class FastDatePrinterTest extends AbstractLangTest { } /** - * According to LANG-916 (https://issues.apache.org/jira/browse/LANG-916), - * the format method did contain a bug: it did not use the TimeZone data. + * According to LANG-916 (https://issues.apache.org/jira/browse/LANG-916), the format method did contain a bug: it did not use the TimeZone data. * * This method test that the bug is fixed. */ @@ -308,7 +310,7 @@ public class FastDatePrinterTest extends AbstractLangTest { @Test public void testLocaleMatches() { - final DatePrinter printer= getInstance(YYYY_MM_DD, SWEDEN); + final DatePrinter printer = getInstance(YYYY_MM_DD, SWEDEN); assertEquals(SWEDEN, printer.getLocale()); } @@ -344,7 +346,7 @@ public class FastDatePrinterTest extends AbstractLangTest { @Test public void testPatternMatches() { - final DatePrinter printer= getInstance(YYYY_MM_DD); + final DatePrinter printer = getInstance(YYYY_MM_DD); assertEquals(YYYY_MM_DD, printer.getPattern()); } @@ -366,8 +368,7 @@ public class FastDatePrinterTest extends AbstractLangTest { } /** - * testLowYearPadding showed that the date was buggy - * This test confirms it, getting 366 back as a date + * testLowYearPadding showed that the date was buggy This test confirms it, getting 366 back as a date */ @Test public void testSimpleDate() { @@ -421,13 +422,13 @@ public class FastDatePrinterTest extends AbstractLangTest { @Test public void testTimeZoneMatches() { - final DatePrinter printer= getInstance(YYYY_MM_DD, NEW_YORK); + final DatePrinter printer = getInstance(YYYY_MM_DD, NEW_YORK); assertEquals(NEW_YORK, printer.getTimeZone()); } @Test public void testToStringContainsName() { - final DatePrinter printer= getInstance(YYYY_MM_DD); + final DatePrinter printer = getInstance(YYYY_MM_DD); assertTrue(printer.toString().startsWith("FastDate")); } diff --git a/src/test/java/org/apache/commons/lang3/time/FastTimeZoneTest.java b/src/test/java/org/apache/commons/lang3/time/FastTimeZoneTest.java index e66ed14d0..8aa9ccfdc 100644 --- a/src/test/java/org/apache/commons/lang3/time/FastTimeZoneTest.java +++ b/src/test/java/org/apache/commons/lang3/time/FastTimeZoneTest.java @@ -55,8 +55,8 @@ public class FastTimeZoneTest extends AbstractLangTest { assertEquals(HOURS_2, FastTimeZone.getGmtTimeZone("2:00").getRawOffset()); assertEquals(MINUTES_59, FastTimeZone.getGmtTimeZone("00:59").getRawOffset()); assertEquals(MINUTES_5, FastTimeZone.getGmtTimeZone("00:5").getRawOffset()); - assertEquals(HOURS_23+MINUTES_59, FastTimeZone.getGmtTimeZone("23:59").getRawOffset()); - assertEquals(HOURS_2+MINUTES_5, FastTimeZone.getGmtTimeZone("2:5").getRawOffset()); + assertEquals(HOURS_23 + MINUTES_59, FastTimeZone.getGmtTimeZone("23:59").getRawOffset()); + assertEquals(HOURS_2 + MINUTES_5, FastTimeZone.getGmtTimeZone("2:5").getRawOffset()); } @Test @@ -65,8 +65,8 @@ public class FastTimeZoneTest extends AbstractLangTest { assertEquals(HOURS_2, FastTimeZone.getGmtTimeZone("0200").getRawOffset()); assertEquals(MINUTES_59, FastTimeZone.getGmtTimeZone("0059").getRawOffset()); assertEquals(MINUTES_5, FastTimeZone.getGmtTimeZone("0005").getRawOffset()); - assertEquals(HOURS_23+MINUTES_59, FastTimeZone.getGmtTimeZone("2359").getRawOffset()); - assertEquals(HOURS_2+MINUTES_5, FastTimeZone.getGmtTimeZone("0205").getRawOffset()); + assertEquals(HOURS_23 + MINUTES_59, FastTimeZone.getGmtTimeZone("2359").getRawOffset()); + assertEquals(HOURS_2 + MINUTES_5, FastTimeZone.getGmtTimeZone("0205").getRawOffset()); } @Test diff --git a/src/test/java/org/apache/commons/lang3/tuple/ImmutablePairTest.java b/src/test/java/org/apache/commons/lang3/tuple/ImmutablePairTest.java index 2ac8ccc32..449d2e192 100644 --- a/src/test/java/org/apache/commons/lang3/tuple/ImmutablePairTest.java +++ b/src/test/java/org/apache/commons/lang3/tuple/ImmutablePairTest.java @@ -43,7 +43,7 @@ public class ImmutablePairTest extends AbstractLangTest { public void testBasic() { ImmutablePair<Integer, String> oldPair = new ImmutablePair<>(0, "foo"); ImmutablePair<Integer, String> nowPair; - for (int i=0; i<4; i++) { + for (int i = 0; i < 4; i++) { nowPair = ImmutablePair.of(oldPair); assertEquals(0, nowPair.left.intValue()); assertEquals(0, nowPair.getLeft().intValue()); @@ -55,7 +55,7 @@ public class ImmutablePairTest extends AbstractLangTest { ImmutablePair<Object, String> oldPair2 = new ImmutablePair<>(null, "bar"); ImmutablePair<Object, String> nowPair2; - for (int i=0; i<4; i++) { + for (int i = 0; i < 4; i++) { nowPair2 = ImmutablePair.of(oldPair2); assertNull(nowPair2.left); assertNull(nowPair2.getLeft()); diff --git a/src/test/java/org/apache/commons/lang3/tuple/MutablePairTest.java b/src/test/java/org/apache/commons/lang3/tuple/MutablePairTest.java index 13db4b363..a1972c9cd 100644 --- a/src/test/java/org/apache/commons/lang3/tuple/MutablePairTest.java +++ b/src/test/java/org/apache/commons/lang3/tuple/MutablePairTest.java @@ -37,7 +37,7 @@ public class MutablePairTest extends AbstractLangTest { public void testBasic() { MutablePair<Integer, String> oldPair = new MutablePair<>(0, "foo"); MutablePair<Integer, String> nowPair; - for (int i=0; i<4; i++) { + for (int i = 0; i < 4; i++) { nowPair = MutablePair.of(oldPair); assertEquals(0, nowPair.left.intValue()); assertEquals(0, nowPair.getLeft().intValue()); @@ -49,7 +49,7 @@ public class MutablePairTest extends AbstractLangTest { MutablePair<Object, String> oldPair2 = new MutablePair<>(null, "bar"); MutablePair<Object, String> nowPair2; - for (int i=0; i<4; i++) { + for (int i = 0; i < 4; i++) { nowPair2 = MutablePair.of(oldPair2); assertNull(nowPair2.left); assertNull(nowPair2.getLeft()); |