Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[BE] feat: 놀이터 참여하기 #637

Merged
merged 52 commits into from
Oct 18, 2024
Merged
Show file tree
Hide file tree
Changes from 7 commits
Commits
Show all changes
52 commits
Select commit Hold shift + click to select a range
2fc980c
feat: 내 강아지 존재 유무 기능 구현
ehtjsv2 Oct 3, 2024
045894b
feat: 놀이터 참여하기 구현
ehtjsv2 Oct 4, 2024
31e8e80
feat: 놀이터 참여시, 이미 놀이터 참여중이면 예외 구현
ehtjsv2 Oct 7, 2024
c11fd8f
refactor: record 정적팩토리메서드 삭제, 부생성자로 변경
ehtjsv2 Oct 7, 2024
055c1af
Merge branch 'feature/be-playground-post' of https://github.com/woowa…
ehtjsv2 Oct 7, 2024
6a808ea
test: 개행제거
ehtjsv2 Oct 7, 2024
5b7bbe9
test: 겹치는 놀이터 생성 테스트 위도차이 테스트 추가
ehtjsv2 Oct 7, 2024
aa5c128
test: 오타수정
ehtjsv2 Oct 7, 2024
b2a3e87
refactor: 펫 존재 유무 네이밍 변경
ehtjsv2 Oct 7, 2024
4f25a42
refactor: location의 거리계산 메소드 GeoCalculator로 이동
ehtjsv2 Oct 7, 2024
37425d8
test: 오타 수정
ehtjsv2 Oct 7, 2024
a4cadf2
refactor: 상수명 변경 및
ehtjsv2 Oct 7, 2024
5c16474
refactor: 위도 경도 차이 구하는 메서드 plus, minus함수 분리
ehtjsv2 Oct 7, 2024
bcb4a2c
test: hasSize 활용
ehtjsv2 Oct 7, 2024
2ba707b
fix: validateOverlapPlayground 논리 오류 수정
ehtjsv2 Oct 8, 2024
90ed35c
merge: 이전 기능과 merge
ehtjsv2 Oct 8, 2024
bde6064
feat: 놀이터 나가기 기능 구현
ehtjsv2 Oct 8, 2024
01cf148
feat: 놀이터 나갔을 때, 참여하고 있는 멤버가 없을 경우 놀이터가 삭제되는 기능 구현
ehtjsv2 Oct 8, 2024
90608b7
refactor: 놀이터 참여 response 데이터 변경
ehtjsv2 Oct 9, 2024
e63edc3
feat: 놀이터 요약 API 구현
ehtjsv2 Oct 10, 2024
4299649
refactor: 놀이터 요약 pet image 5개만 가져오도록 구현
ehtjsv2 Oct 10, 2024
a50e432
refactor: 겹치는 놀이터 생성 시 커스텀 예외코드 적용
ehtjsv2 Oct 10, 2024
3d021ad
feat: 놀이터 요약 API 도착한 강아지 우선으로 이미지 조회 구현
ehtjsv2 Oct 10, 2024
c9bf99f
test: 놀이터 요약 API 펫이미지 두번째 우선순위 참여한 펫 구현
ehtjsv2 Oct 10, 2024
249b4cc
refactor: 정렬하는 코드 repository계층으로 이동
ehtjsv2 Oct 10, 2024
463a2c0
refactor: 놀이터 전체조회 api N+1문제 해결
ehtjsv2 Oct 10, 2024
fe3aa24
refactor: PlaygroundMember EqualsAndHashCode 재정의
ehtjsv2 Oct 10, 2024
1f9ab71
refactor: PlaygroundMember EqualsAndHashCode제거
ehtjsv2 Oct 10, 2024
708c111
merge: 충돌해결
ehtjsv2 Oct 10, 2024
70b2196
refactor: 사용하지 않는 메서드 삭제
ehtjsv2 Oct 11, 2024
4d15892
feat: 도착 여부 수정 api 구현
ehtjsv2 Oct 11, 2024
4ab0cc7
test: 도착 여부 수정 api docs작성
ehtjsv2 Oct 11, 2024
9be8b42
feat: 상태메세지 변경 mock api생성
ehtjsv2 Oct 11, 2024
83b2101
feat: 상태메세지 변경 api 구현
ehtjsv2 Oct 11, 2024
9ea15ca
feat: member 상태메세지 nullable false로 변경
ehtjsv2 Oct 12, 2024
5f11f8a
refactor: 이미 놀이터 참여했을 경우 에러코드 설정
ehtjsv2 Oct 12, 2024
90b70c5
refactor: 참여중인 놀이터 없을 때, 예외코드 적용
ehtjsv2 Oct 12, 2024
a794b59
feat: 놀이터 범위 밖으로 나가면, 나간시간 기록 구현
ehtjsv2 Oct 12, 2024
395a76d
fix: messgae수정 api message null오는 버그 수정
ehtjsv2 Oct 15, 2024
66420f4
feat: 놀이터 상세정보 정렬 적용
ehtjsv2 Oct 15, 2024
773b5c6
refactor: 정적 팩토리 메서드 -> 부생성자
ehtjsv2 Oct 17, 2024
604df8c
refactor: 필요없는 상수 제거
ehtjsv2 Oct 17, 2024
109929d
refactor: 놀이터 반경 150으로 변경
ehtjsv2 Oct 17, 2024
6419a9b
test: 터지는 테스트 수정
ehtjsv2 Oct 17, 2024
eb3b167
refactor: 오타수정
ehtjsv2 Oct 18, 2024
53ceac5
test: isEmpty로 변경
ehtjsv2 Oct 18, 2024
bdbb5a0
test: test 숫자 조절
ehtjsv2 Oct 18, 2024
dcaa255
test: 오타수정
ehtjsv2 Oct 18, 2024
c4af4fd
test: docs 더미 명확한 이름으로 변경
ehtjsv2 Oct 18, 2024
e23735b
refactor: 상수명 변경
ehtjsv2 Oct 18, 2024
b24c0fb
refactor: 상태메세지 변경 request 검증추가
ehtjsv2 Oct 18, 2024
6408617
fix: 이미 자르는 로직 버그 수정
ehtjsv2 Oct 18, 2024
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Original file line number Diff line number Diff line change
@@ -1,7 +1,11 @@
package com.happy.friendogly.playground.dto.request;

import jakarta.validation.constraints.NotBlank;

public record UpdatePlaygroundMemberMessageRequest(
String message //todo: 검증추가

@NotBlank(message = "상태메세지는 빈 문자열이나 null을 입력할 수 없습니다.")
String message
) {

}
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,7 @@
@Transactional(readOnly = true)
public class PlaygroundQueryService {

private static final int MAX_PET_IMAGE_COUNT = 5;
private static final int MAX_PET_PREVIEW_IMAGE_COUNT = 5;

private final PlaygroundRepository playgroundRepository;
private final PlaygroundMemberRepository playgroundMemberRepository;
Expand Down Expand Up @@ -120,7 +120,7 @@ public FindPlaygroundSummaryResponse findSummary(Long playgroundId) {
.toList()
);

petImages = cutPeImagesCount(petImages);
petImages = cutPetImagesCount(petImages);
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

오타 수정 굿

}

return new FindPlaygroundSummaryResponse(
Expand All @@ -131,9 +131,9 @@ public FindPlaygroundSummaryResponse findSummary(Long playgroundId) {
);
}

private List<String> cutPeImagesCount(List<String> petImageUrls) {
if (petImageUrls.size() > MAX_PET_IMAGE_COUNT) {
return petImageUrls.subList(0, MAX_PET_IMAGE_COUNT);
private List<String> cutPetImagesCount(List<String> petImageUrls) {
if (petImageUrls.size() > MAX_PET_PREVIEW_IMAGE_COUNT) {
return petImageUrls.subList(0, MAX_PET_PREVIEW_IMAGE_COUNT);
}
return petImageUrls;
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -256,7 +256,7 @@ void findSummary() throws Exception {
1L,
3,
1,
List.of("url1", "url2", "url3")
List.of("http://image1.jpg", "http://image2.jpg", "http://image3.jpg")
);

when(playgroundQueryService.findSummary(anyLong()))
Expand Down Expand Up @@ -349,7 +349,7 @@ void leavePlayground() throws Exception {
.andExpect(status().isNoContent());
}

@DisplayName("놀이터에 참여한 멤벙 메세지 수정")
@DisplayName("놀이터에 참여한 멤버 메세지 수정")
@Test
void updateMessage() throws Exception {

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -100,7 +100,7 @@ void throwExceptionWhenOverlapPlaygroundScopeWithLatitudeDiff() {
Member member = saveMember("김도선");
double latitudeA = 37.516382;
double longitude = 127.120040;
double latitude299mFromA = GeoCalculator.calculateLatitudeOffset(latitudeA, 1);
double latitude299mFromA = GeoCalculator.calculateLatitudeOffset(latitudeA, 299);
savePlayground(latitudeA, longitude);
SavePlaygroundRequest request = new SavePlaygroundRequest(latitude299mFromA, longitude);

Expand Down Expand Up @@ -147,10 +147,9 @@ void leavePlayground() {

// when
playgroundCommandService.leavePlayground(member.getId());
boolean isPlaygroundMemberPresent = playgroundMemberRepository.findById(playgroundMember.getId()).isPresent();

// then
assertThat(isPlaygroundMemberPresent).isFalse();
assertThat(playgroundMemberRepository.findById(playgroundMember.getId())).isEmpty();
}

@DisplayName("놀이터를 나갔을 때, 놀이터에 참여중인 멤버가 하나도 없으면 놀이터는 삭제된다.")
Expand Down
Loading