Skip to content

Commit 8b9fe3c

Browse files
committed
Use static imports only for JUnit
1 parent 3eea737 commit 8b9fe3c

2 files changed

Lines changed: 11 additions & 8 deletions

File tree

src/test/java/org/apache/commons/collections4/bloomfilter/IndexExtractorTest.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -16,13 +16,13 @@
1616
*/
1717
package org.apache.commons.collections4.bloomfilter;
1818

19+
import static org.junit.jupiter.api.Assertions.assertArrayEquals;
1920
import static org.junit.jupiter.api.Assertions.assertEquals;
2021

2122
import java.util.ArrayList;
2223
import java.util.List;
2324
import java.util.function.LongPredicate;
2425

25-
import org.junit.jupiter.api.Assertions;
2626
import org.junit.jupiter.api.Test;
2727
import org.junit.jupiter.params.ParameterizedTest;
2828
import org.junit.jupiter.params.provider.ValueSource;
@@ -57,7 +57,7 @@ void testAsIndexArray(final int n) {
5757
}
5858
return true;
5959
};
60-
Assertions.assertArrayEquals(new int[n], ip.asIndexArray());
60+
assertArrayEquals(new int[n], ip.asIndexArray());
6161
}
6262

6363
@Test

src/test/java/org/apache/commons/collections4/map/MultiKeyMapCompress672Test.java

Lines changed: 9 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -17,9 +17,12 @@
1717

1818
package org.apache.commons.collections4.map;
1919

20+
import static org.junit.Assert.assertNull;
21+
import static org.junit.jupiter.api.Assertions.assertEquals;
22+
import static org.junit.jupiter.api.Assertions.assertNotEquals;
23+
2024
import java.util.Objects;
2125

22-
import org.junit.jupiter.api.Assertions;
2326
import org.junit.jupiter.api.Test;
2427

2528
/**
@@ -85,16 +88,16 @@ void testMutateKey() {
8588
final KeyFixture keyFixture2Ro = new KeyFixture();
8689
final KeyFixture keyFixture2Rw = new KeyFixture();
8790
// Put original mapping
88-
Assertions.assertNull(multiKeyMap.put(KEY_1, keyFixture2Rw, "value"));
91+
assertNull(multiKeyMap.put(KEY_1, keyFixture2Rw, "value"));
8992
// Both mappings are correct
90-
Assertions.assertEquals("value", multiKeyMap.get(KEY_1, keyFixture2Rw));
91-
Assertions.assertEquals("value", multiKeyMap.get(KEY_1, keyFixture2Ro));
93+
assertEquals("value", multiKeyMap.get(KEY_1, keyFixture2Rw));
94+
assertEquals("value", multiKeyMap.get(KEY_1, keyFixture2Ro));
9295
// Modify 'fixture2'
9396
keyFixture2Rw.setValue("newValue");
9497
// Modified mapping SHOULD NOT work
95-
Assertions.assertNull(multiKeyMap.get(KEY_1, keyFixture2Rw));
98+
assertNull(multiKeyMap.get(KEY_1, keyFixture2Rw));
9699
// Claim: Original mapping SHOULD work
97100
// COUNTER CLAIM: The class is documented to use MultiKey which documents key elements as "The keys should be immutable".
98-
Assertions.assertNotEquals("value", multiKeyMap.get(KEY_1, keyFixture2Ro));
101+
assertNotEquals("value", multiKeyMap.get(KEY_1, keyFixture2Ro));
99102
}
100103
}

0 commit comments

Comments
 (0)