Skip to content

Commit e9bd183

Browse files
committed
Simplify test and add missing cases
1 parent 35cc96c commit e9bd183

File tree

1 file changed

+6
-21
lines changed

1 file changed

+6
-21
lines changed

src/test/java/org/apache/commons/lang3/function/ConsumersTest.java

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

1818
package org.apache.commons.lang3.function;
1919

20-
import static org.junit.jupiter.api.Assertions.assertEquals;
2120
import static org.junit.jupiter.api.Assertions.assertTrue;
2221

22+
import java.util.concurrent.atomic.AtomicBoolean;
2323
import java.util.function.Consumer;
2424
import java.util.stream.Stream;
2525

@@ -31,31 +31,16 @@
3131
*/
3232
public class ConsumersTest extends AbstractLangTest {
3333

34-
private static final class TestConsumer<T> implements Consumer<T> {
35-
private boolean isCalled;
36-
37-
@Override
38-
public void accept(T t) {
39-
isCalled = true;
40-
}
41-
}
42-
4334
/**
4435
* Tests {@link Consumers#accept(Consumer, Object)}.
4536
*/
4637
@Test
4738
public void testAccept() {
48-
final StringBuilder builder = new StringBuilder("foo");
49-
Consumers.accept(sb -> sb.append("-bar"), builder);
50-
assertEquals("foo-bar", builder.toString());
51-
52-
final TestConsumer<String> consumer = new TestConsumer<>();
53-
Consumers.accept(consumer, null);
54-
assertTrue(consumer.isCalled);
55-
56-
final StringBuilder builder2 = new StringBuilder("foo");
57-
Consumers.accept(null, builder2);
58-
assertEquals("foo", builder2.toString());
39+
final AtomicBoolean bool = new AtomicBoolean();
40+
Consumers.accept(bool::set, true);
41+
assertTrue(bool.get());
42+
Consumers.accept(null, "");
43+
Consumers.accept(null, null);
5944
}
6045

6146
/**

0 commit comments

Comments
 (0)