Skip to content

Commit

Permalink
MINOR: Remove unnecessary assertDoesNotThrow (apache#9854)
Browse files Browse the repository at this point in the history
Reviewers: Chia-Ping Tsai <[email protected]>
  • Loading branch information
g1geordie authored Jan 13, 2021
1 parent 2996642 commit 1ad7cea
Showing 1 changed file with 2 additions and 3 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -36,7 +36,6 @@
import java.util.concurrent.Future;
import java.util.concurrent.TimeoutException;

import static org.junit.jupiter.api.Assertions.assertDoesNotThrow;
import static org.junit.jupiter.api.Assertions.assertEquals;
import static org.junit.jupiter.api.Assertions.assertThrows;

Expand Down Expand Up @@ -140,15 +139,15 @@ public void testSyncSendWithoutIgnoringExceptionsShouldNotThrowException() {
MockKafkaLog4jAppender mockKafkaLog4jAppender = getMockKafkaLog4jAppender();
replaceProducerWithMocked(mockKafkaLog4jAppender, true);

assertDoesNotThrow(() -> logger.error(getMessage(0)));
logger.error(getMessage(0));
}

@Test
public void testRealProducerConfigWithSyncSendShouldNotThrowException() {
Properties props = getLog4jConfigWithRealProducer(true);
PropertyConfigurator.configure(props);

assertDoesNotThrow(() -> logger.error(getMessage(0)));
logger.error(getMessage(0));
}

@Test
Expand Down

0 comments on commit 1ad7cea

Please sign in to comment.