Skip to content

Commit 2de5281

Browse files
committed
Add missing test coverage for
org.apache.commons.lang3.ArrayUtils.shift(boolean[], int, int, int)
1 parent f690d2e commit 2de5281

File tree

1 file changed

+40
-5
lines changed

1 file changed

+40
-5
lines changed

src/test/java/org/apache/commons/lang3/ArrayUtilsTest.java

Lines changed: 40 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -4504,33 +4504,68 @@ void testShiftAllShort() {
45044504

45054505
@Test
45064506
void testShiftBoolean() {
4507-
final boolean[] array = {true, true, false, false};
4508-
4507+
final boolean[] array = { true, true, false, false };
45094508
ArrayUtils.shift(array, 1);
45104509
assertFalse(array[0]);
45114510
assertTrue(array[1]);
45124511
assertTrue(array[2]);
45134512
assertFalse(array[3]);
4514-
45154513
ArrayUtils.shift(array, -1);
45164514
assertTrue(array[0]);
45174515
assertTrue(array[1]);
45184516
assertFalse(array[2]);
45194517
assertFalse(array[3]);
4520-
45214518
ArrayUtils.shift(array, 5);
45224519
assertFalse(array[0]);
45234520
assertTrue(array[1]);
45244521
assertTrue(array[2]);
45254522
assertFalse(array[3]);
4526-
45274523
ArrayUtils.shift(array, -3);
45284524
assertFalse(array[0]);
45294525
assertFalse(array[1]);
45304526
assertTrue(array[2]);
45314527
assertTrue(array[3]);
45324528
}
45334529

4530+
@Test
4531+
void testShiftBooleanParams() {
4532+
// edge cases where nothing happens
4533+
// (1) array == null
4534+
ArrayUtils.shift((boolean[]) null, 0, 0, 0);
4535+
// (2) startIndexInclusive >= array.length - 1
4536+
ArrayUtils.shift(new boolean[0], 100, 0, 0);
4537+
// (3) endIndexExclusive <= 0
4538+
boolean[] array1 = { true };
4539+
ArrayUtils.shift(array1, -1, 0, 0);
4540+
assertArrayEquals(new boolean[] { true }, array1);
4541+
// (4) n <= 1
4542+
boolean[] array2 = { true, false, true };
4543+
ArrayUtils.shift(array2, 1, 1, 0);
4544+
assertArrayEquals(new boolean[] { true, false, true }, array2);
4545+
// tests
4546+
boolean[] array = { true, true, false, false };
4547+
ArrayUtils.shift(array, 0, array.length, 1);
4548+
assertFalse(array[0]);
4549+
assertTrue(array[1]);
4550+
assertTrue(array[2]);
4551+
assertFalse(array[3]);
4552+
ArrayUtils.shift(array, 0, array.length, -1);
4553+
assertTrue(array[0]);
4554+
assertTrue(array[1]);
4555+
assertFalse(array[2]);
4556+
assertFalse(array[3]);
4557+
ArrayUtils.shift(array, 0, array.length, 5);
4558+
assertFalse(array[0]);
4559+
assertTrue(array[1]);
4560+
assertTrue(array[2]);
4561+
assertFalse(array[3]);
4562+
ArrayUtils.shift(array, 0, array.length, -3);
4563+
assertFalse(array[0]);
4564+
assertFalse(array[1]);
4565+
assertTrue(array[2]);
4566+
assertTrue(array[3]);
4567+
}
4568+
45344569
@Test
45354570
void testShiftByte() {
45364571
final byte[] array = {1, 2, 3, 4};

0 commit comments

Comments
 (0)