메시지 전송 api 구현

[so3500]

🐣 변경 사항

POST /api/rooms/{roomId}/messages 구현

⚽ 기능 설명

채팅방에서 주고받는 메시지 저장 기능

🏷 연관 이슈

#19

🧢 체크 리스트

• 테스트 코드를 포함하고 있나요?
• 공통된 코드 스타일을 따르 있나요?

[so3500]

이 메서드를 MessageCreationTest 에 추가한 이유

이 테스트를 RoomControllerIntegrationTest 에 추가할 때 RoomControllerIntegrationTest#shouldGetMessageList_inRoomIdWith1 가 영향 받음

하나의 클래스에서 처리할 수 없을지 고민 필요

response

MockHttpServletResponse:
           Status = 200
    Error message = null
          Headers = [Content-Type:"application/json;charset=UTF-8"]
     Content type = application/json
             Body = [{"id":1,"contents":"foo","createdAt":[2020,8,1,0,0],"messageType":"TEXT"},{"id":3,"contents":"hi hello","createdAt":[2020,8,30,1,5,8,114499000],"messageType":"TEXT"},{"id":4,"contents":"foo","createdAt":[2020,8,1,0,0],"messageType":"TEXT"}]
    Forwarded URL = null
   Redirected URL = null
          Cookies = []

error log

JSON path "$.*"
Expected: a collection with size <1>
     but: collection size was <3>
java.lang.AssertionError: JSON path "$.*"
Expected: a collection with size <1>
     but: collection size was <3>
	at org.hamcrest.MatcherAssert.assertThat(MatcherAssert.java:20)
	at org.springframework.test.util.JsonPathExpectationsHelper.assertValue(JsonPathExpectationsHelper.java:73)
	at org.springframework.test.web.servlet.result.JsonPathResultMatchers.lambda$value$0(JsonPathResultMatchers.java:87)
	at org.springframework.test.web.servlet.MockMvc$1.andExpect(MockMvc.java:196)
	at web.chat.backend.controller.RoomControllerIntegrationTest.shouldGetMessageList_inRoomIdWith1(RoomControllerIntegrationTest.java:58)
	at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
	at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
	at 
...

[so3500]

#25 에서 고민해보기

[so3500]

유효하지 않는 roomId 를 받았을 때 null 을 리턴하지 않고 NotFoundException 을 발생시켜 각 계층 간 null을 주고받는 상황을 방지

[raccoonback]

exists() 사용하지 않고 Optional 으로 하니까 깔끔하네 굿굿 👍 👍

web-chat-backend/src/main/java/web/chat/backend/service/MessageService.java

Lines 17 to 23 in ce7b639

	public List<Message> getMessagesBy(Long roomId) {
	if (!messageRepository.existsByRoomId(roomId)) {
	throw new NotFoundException(roomId);
	}
	return messageRepository.findAllByRoomId(roomId);
	}

[so3500]

JPA Auditing 적용

c691006 에서 누락된 작업 보강

ref. https://www.baeldung.com/database-auditing-jpa
ref. https://docs.spring.io/spring-data/jpa/docs/current/reference/html/#jpa.auditing

[raccoonback]

ref. https://gist.github.com/dlxotn216/94c34a2debf848396cf82a7f21a32abe

[so3500]

👍 나중에 공통 프로퍼티는 BaseEntity 클래스로 빼는 방법도 고려해봐야겠다

[raccoonback]

👍 👍

[raccoonback]

exists() 사용하지 않고 Optional 으로 하니까 깔끔하네 굿굿 👍 👍

web-chat-backend/src/main/java/web/chat/backend/service/MessageService.java

Lines 17 to 23 in ce7b639

	public List<Message> getMessagesBy(Long roomId) {
	if (!messageRepository.existsByRoomId(roomId)) {
	throw new NotFoundException(roomId);
	}
	return messageRepository.findAllByRoomId(roomId);
	}