🚀 좋아요 요청 및 취소 기능 작성 #55

shoeone96 · 2024-01-08T14:16:14Z

✅ 요구사항

좋아요 요청 및 취소 api 작성
좋아요 기능 test 코드 작성

🚀 주요 변경사항

api 추가
- api 요청 시 처음 좋아요를 누르면 좋아요 객체를 생성, 두 번째 이후부터는 좋아요의 status를 수정하는 방식으로 적용
- LikeController 생성
  - LikeRequest 생성
  - LikeResponse 생성
- LikeService 생성
- LikeRepository 생성
- Like entity class 생성
상황별로 발생할 수 있는 test 코드 작성
- LikeControllerTest
- LikeServiceTest
- LikeRepositoryTest

💡 기타사항

추후 like count와 isLike 여부를 post에 담아 보내줄 수 있게 refactoring 예정
동시성 문제가 likeCount를 Post table에 넣고 업데이트 할 때 발생하는데 현재의 로직(매 요청 시 like table에서 like count를 읽는)에서는 발생하지 않을 것으로 짐작됨

github-actions · 2024-01-08T14:31:45Z

📑 Jacoco 테스트 결과

Overall Project	96.49% `-0.22%`	🍏
Files changed	96.62%	🍏

File	Coverage
LikeRequest.java	100%	🍏
LikeResponse.java	100%	🍏
LikeController.java	100%	🍏
Like.java	100%	🍏
LikeService.java	92.65% `-7.35%`	🍏

Sehee-Lee-01

깔끔하게 작성해주셔서 보기 편했던 것 같습니다! 몇 개 코맨트만 남겨놓았습니다!

Sehee-Lee-01 · 2024-01-09T03:30:35Z

src/test/java/com/backendoori/ootw/like/domain/LikeTest.java

+
+    @ParameterizedTest
+    @NullSource
+    @DisplayName("post가 null 인 경우 좋아요 객체를 생성 시 예외가 발생한다.")


DisplayName에 post가 아니라 user가 들어가야하는 것 같습니다!

Sehee-Lee-01 · 2024-01-09T03:35:23Z

src/test/java/com/backendoori/ootw/like/service/LikeServiceTest.java

+    }
+
+    private WeatherDto weatherDtoGenerator() {
+        return new WeatherDto(0.0, -10.0, 10.0, 1, 1);


weatherDtoGenerator() 중복 줄이기에 좋은 것 같습니다👍
post 테스트 부분도 개선해보도록 하겠습니다!

혹시 저는 저 WeatherDto를 상수로 놓았는데 혹시 이 방법은 어떻게 생각하시나요? 수정 제안이 아니라 의견이 궁금해서 여쭤봅니다!

혹시 다른 방법이 안보이는데 어디를 참고하면 될까요?

앗 코드랑 같이 보여드릴 걸 그랬네요! 지금 수정 중인 코드에서 작성한 코드입니다!

static final TemperatureArrangeDto TEMPERATURE_ARRANGE_DTO = new TemperatureArrangeDto(-10.0, 10.0); static final TemperatureArrange TEMPERATURE_ARRANGE = TemperatureArrange.from(TEMPERATURE_ARRANGE_DTO);

저는 이런 방식으로 작성을 해놓았습니다!

지금은 WeatherDto가 TemperatureArrangeDto로 이름이 변경되었습니다!

괜찮은 방법인 것 같아 도입하고 push 했습니다!

앗!!!!!! 이따가 이야기 나눠보죠,,전 중원님 방식도 괜찮은 것 같아서,,, 제 방식만 공유해본 것이엇는데 수정한다면 제가 맞추겠습니다!

github-actions · 2024-01-09T04:47:30Z

📑 Jacoco 테스트 결과

Overall Project	96.49% `-0.22%`	🍏
Files changed	96.62%	🍏

File	Coverage
LikeRequest.java	100%	🍏
LikeResponse.java	100%	🍏
LikeController.java	100%	🍏
Like.java	100%	🍏
LikeService.java	92.65% `-7.35%`	🍏

github-actions · 2024-01-09T08:39:50Z

📑 Jacoco 테스트 결과

Overall Project	96.49% `-0.22%`	🍏
Files changed	96.62%	🍏

File	Coverage
LikeRequest.java	100%	🍏
LikeResponse.java	100%	🍏
LikeController.java	100%	🍏
Like.java	100%	🍏
LikeService.java	92.65% `-7.35%`	🍏

github-actions · 2024-01-09T08:47:38Z

📑 Jacoco 테스트 결과

Overall Project	96.49% `-0.22%`	🍏
Files changed	96.62%	🍏

File	Coverage
LikeRequest.java	100%	🍏
LikeResponse.java	100%	🍏
LikeController.java	100%	🍏
Like.java	100%	🍏
LikeService.java	92.65% `-7.35%`	🍏

ASak1104

고생하셨습니다! 스타일만 조금 수정하면 좋을 것 같아요

ASak1104 · 2024-01-09T11:17:38Z

src/main/java/com/backendoori/ootw/like/controller/LikeController.java

+
+    private final LikeService likeService;
+
+    @PostMapping("")


mapping이 별도로 필요 없으면 안쓰는 것도 좋을 것 같아요

Suggested change

@PostMapping("")

@PostMapping

ASak1104 · 2024-01-09T11:18:35Z

src/main/java/com/backendoori/ootw/like/dto/controller/LikeRequest.java

+import jakarta.validation.constraints.NotNull;
+
+public record LikeRequest(
+    @Min(value = 1)


@Positive를 쓰는 것도 좋을 것 같아요

ASak1104 · 2024-01-09T11:19:02Z

src/main/java/com/backendoori/ootw/like/service/LikeService.java

+
+    @Transactional
+    public LikeResponse requestLike(Long userId, Long postId) {
+


여기 개행은 없어도 될 것 같아요!

ASak1104 · 2024-01-09T11:19:38Z

src/test/java/com/backendoori/ootw/like/controller/LikeControllerTest.java

+    static Faker faker = new Faker();
+    static final WeatherDto weatherDto = new WeatherDto(0.0, -10.0, 10.0, 1, 1);


통일성을 위해 faker에도 final 키워드 쓰는 것도 좋을 것 같아요

ASak1104 · 2024-01-09T11:20:17Z

src/test/java/com/backendoori/ootw/like/service/LikeServiceTest.java

+
+

개행을 한 줄만 해줘도 좋을 것 같아요

github-actions · 2024-01-09T11:25:48Z

📑 Jacoco 테스트 결과

Overall Project	96.49% `-0.22%`	🍏
Files changed	96.62%	🍏

File	Coverage
LikeRequest.java	100%	🍏
LikeResponse.java	100%	🍏
LikeController.java	100%	🍏
Like.java	100%	🍏
LikeService.java	92.65% `-7.35%`	🍏

shoeone96 added 9 commits January 8, 2024 22:54

feat: like schema 추가

26ff537

feat: Like entity 추가

21cf1fd

feat: 좋아요 요청과 취소를 위한 api 개발

2adc901

feat: 좋아요 요청과 삭제 시 전달되는 데이터 정의

be4aca6

feat: 좋아요 db 연결을 위한 repository 작성

3efe23f

feat: 좋아요 서비스 로직 작성

73e4061

test: like 요청 api test

48706da

feat: like 요청 서비스 로직 테스트

76b2cf6

test: like entity test 코드

d0b8b0a

shoeone96 added the enhancement New feature or request label Jan 8, 2024

shoeone96 added this to the Should Have 서비스 구현 milestone Jan 8, 2024

shoeone96 self-assigned this Jan 8, 2024

shoeone96 requested review from ASak1104 and Sehee-Lee-01 as code owners January 8, 2024 14:16

shoeone96 had a problem deploying to build January 8, 2024 14:16 — with GitHub Actions Failure

fix: sql error 구문 수정

4cf1359

shoeone96 temporarily deployed to build January 8, 2024 14:29 — with GitHub Actions Inactive

Sehee-Lee-01 approved these changes Jan 9, 2024

View reviewed changes

style: test DisplayName 수정

8352b24

shoeone96 temporarily deployed to build January 9, 2024 04:45 — with GitHub Actions Inactive

refactor: WeatherDto 상수로 선언

d9dcc5f

shoeone96 temporarily deployed to build January 9, 2024 08:38 — with GitHub Actions Inactive

style: style check

73c7193

shoeone96 temporarily deployed to build January 9, 2024 08:45 — with GitHub Actions Inactive

ASak1104 approved these changes Jan 9, 2024

View reviewed changes

style: 공통 스타일 수정

153bcfc

shoeone96 temporarily deployed to build January 9, 2024 11:24 — with GitHub Actions Inactive

Sehee-Lee-01 merged commit d9932a7 into feature Jan 9, 2024
1 check passed

Sehee-Lee-01 deleted the feature-like branch January 9, 2024 11:26

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

🚀 좋아요 요청 및 취소 기능 작성 #55

🚀 좋아요 요청 및 취소 기능 작성 #55

shoeone96 commented Jan 8, 2024

github-actions bot commented Jan 8, 2024

Sehee-Lee-01 left a comment

Sehee-Lee-01 Jan 9, 2024

Sehee-Lee-01 Jan 9, 2024

shoeone96 Jan 9, 2024

Sehee-Lee-01 Jan 9, 2024 •

edited

Loading

Sehee-Lee-01 Jan 9, 2024

shoeone96 Jan 9, 2024

Sehee-Lee-01 Jan 9, 2024 •

edited

Loading

github-actions bot commented Jan 9, 2024

github-actions bot commented Jan 9, 2024

github-actions bot commented Jan 9, 2024

ASak1104 left a comment

ASak1104 Jan 9, 2024

ASak1104 Jan 9, 2024

ASak1104 Jan 9, 2024

ASak1104 Jan 9, 2024

ASak1104 Jan 9, 2024

github-actions bot commented Jan 9, 2024


		@Transactional
		public LikeResponse requestLike(Long userId, Long postId) {

		static Faker faker = new Faker();
		static final WeatherDto weatherDto = new WeatherDto(0.0, -10.0, 10.0, 1, 1);

🚀 좋아요 요청 및 취소 기능 작성 #55

🚀 좋아요 요청 및 취소 기능 작성 #55

Conversation

shoeone96 commented Jan 8, 2024

✅ 요구사항

🚀 주요 변경사항

💡 기타사항

github-actions bot commented Jan 8, 2024

📑 Jacoco 테스트 결과

Sehee-Lee-01 left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Sehee-Lee-01 Jan 9, 2024 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Sehee-Lee-01 Jan 9, 2024 • edited Loading

Choose a reason for hiding this comment

github-actions bot commented Jan 9, 2024

📑 Jacoco 테스트 결과

github-actions bot commented Jan 9, 2024

📑 Jacoco 테스트 결과

github-actions bot commented Jan 9, 2024

📑 Jacoco 테스트 결과

ASak1104 left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

github-actions bot commented Jan 9, 2024

📑 Jacoco 테스트 결과

Sehee-Lee-01 Jan 9, 2024 •

edited

Loading

Sehee-Lee-01 Jan 9, 2024 •

edited

Loading