Skip to content

Commit 99407e2

Browse files
committed
test: 테스트 수정
1 parent f8a49f7 commit 99407e2

File tree

2 files changed

+23
-3
lines changed

2 files changed

+23
-3
lines changed

src/test/java/eatda/config/LoggingInterceptorTest.java

Lines changed: 21 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,14 @@
11
package eatda.config;
22

3+
import static org.assertj.core.api.Assertions.assertThat;
4+
5+
import ch.qos.logback.classic.Level;
6+
import ch.qos.logback.classic.Logger;
7+
import ch.qos.logback.classic.spi.ILoggingEvent;
8+
import ch.qos.logback.core.read.ListAppender;
39
import org.junit.jupiter.api.Nested;
410
import org.junit.jupiter.api.Test;
11+
import org.slf4j.LoggerFactory;
512
import org.springframework.mock.web.MockHttpServletRequest;
613
import org.springframework.mock.web.MockHttpServletResponse;
714

@@ -13,11 +20,24 @@ class LoggingInterceptorTest {
1320
class afterCompletion {
1421

1522
@Test
16-
void preHandle_없이_afterCompletion만_호출되면_duration_unknown_로그가_남는다() {
23+
void preHandle_없이_afterCompletion만_호출되면_duration_unknown_로그가_남는다() throws Exception {
1724
MockHttpServletRequest request = new MockHttpServletRequest();
1825
MockHttpServletResponse response = new MockHttpServletResponse();
1926

27+
Logger logger = (Logger) LoggerFactory.getLogger(LoggingInterceptor.class);
28+
ListAppender<ILoggingEvent> listAppender = new ListAppender<>();
29+
listAppender.start();
30+
logger.addAppender(listAppender);
31+
2032
interceptor.afterCompletion(request, response, new Object(), null);
33+
34+
assertThat(listAppender.list).anySatisfy(event -> {
35+
assertThat(event.getLevel()).isEqualTo(Level.WARN);
36+
assertThat(event.getFormattedMessage()).contains("duration unknown - preHandle not called");
37+
});
38+
39+
listAppender.stop();
40+
logger.detachAppender(listAppender);
2141
}
2242
}
2343
}

src/test/java/eatda/exception/GlobalExceptionHandlerTest.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -205,14 +205,14 @@ class handleExceptions {
205205
void 제약조건_위반은_400() throws Exception {
206206
mockMvc.perform(get("/test/constraint?name="))
207207
.andExpect(status().isBadRequest())
208-
.andExpect(jsonPath("$.errorCode", equalTo(EtcErrorCode.CLIENT_REQUEST_ERROR.getCode())));
208+
.andExpect(jsonPath("$.errorCode", equalTo(EtcErrorCode.VALIDATION_ERROR.getCode())));
209209
}
210210

211211
@Test
212212
void HandlerMethodValidationException은_400() throws Exception {
213213
mockMvc.perform(get("/test/handler-validation"))
214214
.andExpect(status().isBadRequest())
215-
.andExpect(jsonPath("$.errorCode", equalTo(EtcErrorCode.VALIDATION_ERROR.getCode())));
215+
.andExpect(jsonPath("$.errorCode", equalTo(EtcErrorCode.CLIENT_REQUEST_ERROR.getCode())));
216216
}
217217
}
218218
}

0 commit comments

Comments
 (0)