Skip to content

Commit 1ad7cea

Browse files
authored
MINOR: Remove unnecessary assertDoesNotThrow (#9854)
Reviewers: Chia-Ping Tsai <[email protected]>
1 parent 2996642 commit 1ad7cea

File tree

1 file changed

+2
-3
lines changed

1 file changed

+2
-3
lines changed

log4j-appender/src/test/java/org/apache/kafka/log4jappender/KafkaLog4jAppenderTest.java

+2-3
Original file line numberDiff line numberDiff line change
@@ -36,7 +36,6 @@
3636
import java.util.concurrent.Future;
3737
import java.util.concurrent.TimeoutException;
3838

39-
import static org.junit.jupiter.api.Assertions.assertDoesNotThrow;
4039
import static org.junit.jupiter.api.Assertions.assertEquals;
4140
import static org.junit.jupiter.api.Assertions.assertThrows;
4241

@@ -140,15 +139,15 @@ public void testSyncSendWithoutIgnoringExceptionsShouldNotThrowException() {
140139
MockKafkaLog4jAppender mockKafkaLog4jAppender = getMockKafkaLog4jAppender();
141140
replaceProducerWithMocked(mockKafkaLog4jAppender, true);
142141

143-
assertDoesNotThrow(() -> logger.error(getMessage(0)));
142+
logger.error(getMessage(0));
144143
}
145144

146145
@Test
147146
public void testRealProducerConfigWithSyncSendShouldNotThrowException() {
148147
Properties props = getLog4jConfigWithRealProducer(true);
149148
PropertyConfigurator.configure(props);
150149

151-
assertDoesNotThrow(() -> logger.error(getMessage(0)));
150+
logger.error(getMessage(0));
152151
}
153152

154153
@Test

0 commit comments

Comments
 (0)