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.

Sign up for GitHub

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

Jump to bottom

[OMCT-410] jwt,content 단위 테스트, feedReader 상세 조회 테스트 #218

Merged
merged 15 commits into from
Jan 8, 2024
Merged

[OMCT-410] jwt,content 단위 테스트, feedReader 상세 조회 테스트 #218

merged 15 commits into from
Jan 8, 2024

Conversation

HandmadeCloud
Copy link
Member

@HandmadeCloud HandmadeCloud commented Dec 31, 2023 •
edited
Loading

📌 PR 종류

어떤 종류의 PR인지 아래 항목 중에 체크 해주세요.

  • 🐛 버그 수정
  • ✨ 기능 추가
  • ✅ 테스트 추가
  • 🎨 코드 스타일 변경 (formatting, local variables)
  • 🔨 리팩토링 (기능 변경 X)
  • 💚 빌드 관련 수정
  • 📝 문서 내용 수정
  • 그 외, 어떤 종류인지 기입 바람:

📌 어떤 기능이 추가 되었나요?

Issue Number

410

기능 설명

  • jwt토큰이 정상적으로 발행되는지 테스트함(발행 성공, 유효성검사 성공, 유효성 검사 실패(만료시간)) 512a75e

  • content 내용 테스트 및 최소 글자수 검증에 대한 로직을 추가함 6a0a24b

  • 피드 상세조회 테스트를 진행함 ae22a55

📌 기존에 있던 기능에 영향을 주나요?

  • 네
  • 아니요

@HandmadeCloud HandmadeCloud added the test Good for newcomers label Dec 31, 2023
@HandmadeCloud HandmadeCloud self-assigned this Dec 31, 2023
@HandmadeCloud HandmadeCloud changed the title [OMCT-410] jet,content 단위 테스트, feedReader 상세 조회 테스트 [OMCT-410] jwt,content 단위 테스트, feedReader 상세 조회 테스트 Jan 1, 2024
Yiseull
Yiseull reviewed Jan 8, 2024
View reviewed changes
Copy link
Contributor

@Yiseull Yiseull left a comment

Choose a reason for hiding this comment

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

테스트 작성하느라 고생 많으셨어요! 코드에 대해서 궁금한 점 몇 가지 리뷰 달아놨습니다 ㅎㅎ

...in/src/testFixtures/java/com/programmers/bucketback/domains/member/domain/MemberBuilder.java Outdated
Comment on lines 26 to 47
public static Member build() {
Member member = Member.builder()
.nickname("nickname")
.email("email@naver.com")
.password("password")
.role(Role.USER)
.build();

setMemberId(member);

return member;
}

private static void setMemberId(
final Member member
) {
ReflectionTestUtils.setField(
member,
"id",
1L
);
}
Copy link
Contributor

Choose a reason for hiding this comment

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

p3;
34째 줄에서 기존 setMemberId를 활용하여 setMemberId(member, 1L); 이렇게 작성하는 것은 어떤가요 ??
멤버만 받는 setMemberId 메서드는 꼭 새로 만들 필요는 없는 것 같아용!

Copy link
Member Author

Choose a reason for hiding this comment

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

MemberBuilder가 다른 분이 만들기 전이어서 임시로 만들고 사용했는데, 이제 바꿔도 될 것 같습니다.

...k-domain/src/test/java/com/programmers/bucketback/domains/comment/domain/vo/ContentTest.java
void successValidateContentLength(int count) {
String fill = "a";
Content content = new Content(fill.repeat(count));
assertThat(content).isNotNull();
Copy link
Contributor

Choose a reason for hiding this comment

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

p1;
isNotNull 보다는 예외가 터지지 않는 것을 확인하는 메서드니까 doesNotThrowAnyException를 사용하는 것은 어떤가요??

Copy link
Member Author

@HandmadeCloud HandmadeCloud Jan 8, 2024 •
edited
Loading

Choose a reason for hiding this comment

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

예외가 터지지 않는다 = 값이 정상적으로 잘 생성되었다. 이렇게 볼 수 있을 것 같아요
테스트를 예외 중심으로 바라봐는 시각과 생성 중심으로 바라보냐의 차이로 생각했을 때 저는 후자를 선택했습니다.

...ain/src/test/java/com/programmers/bucketback/domains/feed/implementation/FeedReaderTest.java Outdated
Comment on lines 67 to 78
FeedInfo feedInfo = new FeedInfo(
feed.getId(),
feed.getHobby().getHobbyValue(),
feed.getFeedContent(),
feed.getBucketInfo().getName(),
feed.getBucketInfo().getBudget(),
feed.getTotalPrice(),
feed.getCreatedAt(),
feed.hasAdoptedComment(),
feed.getLikes().size(),
true
);
Copy link
Contributor

Choose a reason for hiding this comment

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

p1;
FeedInfo의 of 메서드나 빌더 패턴을 사용하면 좋을 것 같아요!

...-api/src/test/java/com/programmers/bucketback/global/config/security/jwt/JwtServiceTest.java Outdated
Comment on lines 46 to 47
Member member = buildMember();
ReflectionTestUtils.setField(member, "id", 1L);
Copy link
Contributor

Choose a reason for hiding this comment

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

P1;
MemberBuilder를 사용하면 좋을 것 같아요!

Copy link
Member Author

Choose a reason for hiding this comment

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

MemberBuilder가 다른 분이 만들기 전이어서 임시로 만들고 사용했는데, 이제 바꿔도 될 것 같습니다.

...-api/src/test/java/com/programmers/bucketback/global/config/security/jwt/JwtServiceTest.java Outdated
Comment on lines 53 to 58
byte[] keyBytes = Decoders.BASE64.decode(jwtConfig.accessSecretKey());
SecretKey decodedSecretKey = Keys.hmacShaKeyFor(keyBytes);
Claims claims = Jwts.parserBuilder().setSigningKey(decodedSecretKey).build().parseClaimsJws(token).getBody();

//then
assertThat(claims.getExpiration()).isAfter(new Date());
Copy link
Contributor

Choose a reason for hiding this comment

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

P5;
JwtService의 extractExpiration 메서드를 사용하지 않고 직접 작성하신 이유가 있을까요 ???

Copy link
Member Author

Choose a reason for hiding this comment

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

private메소드로 생성된 내용이어서 직접 구현해서 사용했습니다.
public 메소드인 isAccessTokenValid로 만들어서 해결했습니다.

@HandmadeCloud HandmadeCloud merged commit aa26c7a into main Jan 8, 2024
HandmadeCloud added a commit that referenced this pull request Jan 30, 2024
@HandmadeCloud
Squashed commit of the following:
a046851 
commit 20a56ef
Merge: 4e1a308 68b64f8
Author: JoJaeHyeon <77893164+HandmadeCloud@users.noreply.github.com>
Date:   Wed Jan 10 11:52:07 2024 +0900

    Merge pull request #222 from bucket-back/OMCT-408-alarm-sse

    [OMCT-408] emitter 생명주기를 짧게 수정함, securityConfiguration에 subscribe 예외 처리

commit 4e1a308
Merge: aa26c7a 43e60eb
Author: Yiseul Park <omjl5123@catholic.ac.kr>
Date:   Mon Jan 8 20:33:35 2024 +0900

    Merge pull request #220 from bucket-back/OMCT-345-test-vote-reader

    [OMCT-345] �VoteReader 단위테스트 추가

commit aa26c7a
Merge: 6b98be2 c7f56ec
Author: JoJaeHyeon <77893164+HandmadeCloud@users.noreply.github.com>
Date:   Mon Jan 8 16:51:59 2024 +0900

    Merge pull request #218 from bucket-back/OMCT-410--domain-layer-test

    [OMCT-410] jwt,content 단위 테스트, feedReader 상세 조회 테스트

commit c7f56ec
Author: JoJaeHyeon <joje1003@naver.com>
Date:   Mon Jan 8 16:49:43 2024 +0900

    test : 기존 memberBuilder 삭제

commit ff18c37
Author: JoJaeHyeon <joje1003@naver.com>
Date:   Mon Jan 8 16:49:19 2024 +0900

    test : 피드 상세 조회 로직 memberBuilder 삭제, private메소드 => public메소드 테스트로 수정함

commit 4e67f97
Author: JoJaeHyeon <joje1003@naver.com>
Date:   Mon Jan 8 16:49:05 2024 +0900

    test : 피드 상세 조회 로직 memberBuilder 삭제

commit 6b98be2
Merge: 34a1665 675ae43
Author: Yiseul Park <omjl5123@catholic.ac.kr>
Date:   Mon Jan 8 16:25:13 2024 +0900

    Merge pull request #221 from bucket-back/OMCT-265-test-member-implemenatation

    [OMCT-265] Meber Implementation 단위테스트 추가

commit 34a1665
Merge: cd96d75 b9b3cfd
Author: ByeongGon Kang <kangbk4182@naver.com>
Date:   Mon Jan 8 15:35:59 2024 +0900

    Merge pull request #175 from bucket-back/OMCT-347-item-addItem-test

    [OMCT-347] 아이템 추가 통합 테스트

commit b9b3cfd
Merge: f2ddfa0 cd96d75
Author: ByeongGon Kang <kangbk4182@naver.com>
Date:   Mon Jan 8 15:35:52 2024 +0900

    Merge branch 'main' into OMCT-347-item-addItem-test

commit cd96d75
Merge: a908fa4 db32aca
Author: JoJaeHyeon <77893164+HandmadeCloud@users.noreply.github.com>
Date:   Mon Jan 8 15:28:51 2024 +0900

    Merge pull request #219 from bucket-back/OMCT-408-alarm-sse

    [OMCT-408] 알림 전송 로직 수정

commit 674da17
Author: JoJaeHyeon <joje1003@naver.com>
Date:   Sun Jan 7 18:45:53 2024 +0900

    refactoring : toString ->  string valueOf 변경

commit 512a75e
Author: JoJaeHyeon <joje1003@naver.com>
Date:   Sun Jan 7 18:43:27 2024 +0900

    test : refresh Token 적용으로 인한 단위테스트 수정

commit 7ab802f
Merge: 701973a a908fa4
Author: JoJaeHyeon <joje1003@naver.com>
Date:   Sun Jan 7 18:27:56 2024 +0900

    Merge branch 'main' into OMCT-410--domain-layer-test

commit 675ae43
Author: Yiseul Park <omjl5123@catholic.ac.kr>
Date:   Sat Jan 6 17:06:47 2024 +0900

    test: 닉네임으로 회원 조회 테스트

commit 3eea6de
Author: Yiseul Park <omjl5123@catholic.ac.kr>
Date:   Sat Jan 6 17:04:45 2024 +0900

    test: 이메일로 회원 조회 테스트

commit d745986
Author: Yiseul Park <omjl5123@catholic.ac.kr>
Date:   Sat Jan 6 17:04:06 2024 +0900

    test: memberId로 회원 조회 테스트 추가

commit 95626a9
Author: Yiseul Park <omjl5123@catholic.ac.kr>
Date:   Sat Jan 6 16:53:28 2024 +0900

    test: MemberModifier 단위테스트 추가

commit 4286f1d
Author: Yiseul Park <omjl5123@catholic.ac.kr>
Date:   Sat Jan 6 16:15:21 2024 +0900

    test: MemberRemover 단위테스트 추가

commit e6bbf20
Author: Yiseul Park <omjl5123@catholic.ac.kr>
Date:   Sat Jan 6 16:12:12 2024 +0900

    style: 파일 끝 개행 추가

commit 2a218ac
Author: Yiseul Park <omjl5123@catholic.ac.kr>
Date:   Sat Jan 6 16:09:53 2024 +0900

    test: 이메일 중복 확인 테스트 추가

commit 8794ff7
Author: Yiseul Park <omjl5123@catholic.ac.kr>
Date:   Sat Jan 6 16:08:26 2024 +0900

    test: 닉네임 중복 확인 테스트 추가

commit ba949ac
Author: Yiseul Park <omjl5123@catholic.ac.kr>
Date:   Sat Jan 6 15:51:15 2024 +0900

    test: MemberAppender 단위테스트 추가

commit 43e60eb
Author: Yiseul Park <omjl5123@catholic.ac.kr>
Date:   Fri Jan 5 22:58:28 2024 +0900

    test: 검색 조건에 맞는 투표 목록 조회 테스트 추가

commit ca8c462
Author: Yiseul Park <omjl5123@catholic.ac.kr>
Date:   Fri Jan 5 16:01:11 2024 +0900

    test: 투표 키워드 조회 테스트 추가

commit a908fa4
Merge: 905d439 3266bee
Author: Yiseul Park <omjl5123@catholic.ac.kr>
Date:   Fri Jan 5 14:16:10 2024 +0900

    Merge pull request #217 from bucket-back/OMCT-411-refresh-token

    [OMCT-411] 로그인 연장 기능 추가 - Refresh token으로 Access token 재발급

commit 3266bee
Author: Yiseul Park <omjl5123@catholic.ac.kr>
Date:   Thu Jan 4 12:25:17 2024 +0900

    refactor: extractClaim 제거하고 extractAllClaim 직접 사용

commit 49e8fae
Author: Yiseul Park <omjl5123@catholic.ac.kr>
Date:   Wed Jan 3 15:42:36 2024 +0900

    style: 매개변수 2개 이상 개행

commit 561850b
Author: Yiseul Park <omjl5123@catholic.ac.kr>
Date:   Wed Jan 3 15:39:17 2024 +0900

    refactor: 매직넘버 제거

commit cb10734
Author: Yiseul Park <omjl5123@catholic.ac.kr>
Date:   Wed Jan 3 15:25:32 2024 +0900

    refactor: 캐시 설정 변경

commit 963cce7
Author: Yiseul Park <omjl5123@catholic.ac.kr>
Date:   Tue Jan 2 17:54:11 2024 +0900

    refactor: JWT 토큰 관련 리팩토링

    1. extractAccessTokenUsername -> extractUsername 메서드명 변경
    2. generateAccessToken 메서드에 불필요한 매개변수 제거
    3. isAccessTokenValid, isRefreshTokenValid 커스텀 에러코드를 위한 예외처리
    4. Refresh Token의 밸류값 null 체크문 제거

commit 359cba6
Author: Yiseul Park <omjl5123@catholic.ac.kr>
Date:   Tue Jan 2 17:50:09 2024 +0900

    feat: Access Token, Refresh Token 예외 처리 추가

commit 905d439
Merge: badd8a1 517d046
Author: ByeongGon Kang <kangbk4182@naver.com>
Date:   Tue Jan 2 14:06:02 2024 +0900

    Merge pull request #203 from bucket-back/OMCT-271-feed-ranking-redis

    [OMCT-271] 피드 랭킹 기능 추가

commit 701973a
Author: JoJaeHyeon <joje1003@naver.com>
Date:   Sun Dec 31 23:19:16 2023 +0900

    test : 인터페이스에 의존하도록 롤백

commit d406cf7
Author: JoJaeHyeon <joje1003@naver.com>
Date:   Sun Dec 31 23:15:48 2023 +0900

    test : memberBuilder 생성 로직 추가

commit ae22a55
Author: JoJaeHyeon <joje1003@naver.com>
Date:   Sun Dec 31 23:15:34 2023 +0900

    test : 피드 상세 조회 테스트

commit 53584ba
Author: JoJaeHyeon <joje1003@naver.com>
Date:   Sun Dec 31 01:18:11 2023 +0900

    chore : 테스트 이름 설명 추가

commit 165fb9f
Author: JoJaeHyeon <joje1003@naver.com>
Date:   Sun Dec 31 01:14:54 2023 +0900

    test : 댓글 내용 도메인 테스트

commit 6a0a24b
Author: JoJaeHyeon <joje1003@naver.com>
Date:   Sun Dec 31 01:14:41 2023 +0900

    refactor : content 댓글 최소 수 검증 로직 추가

commit 52d6b02
Author: Yiseul Park <omjl5123@catholic.ac.kr>
Date:   Sat Dec 30 00:01:12 2023 +0900

    feat: Refresh token으로 Access Token 재발급 기능 추가

commit a22c434
Author: Yiseul Park <omjl5123@catholic.ac.kr>
Date:   Fri Dec 29 20:02:35 2023 +0900

    feat: 로그아웃, 회원탈퇴 시 저장해둔 리프레쉬 토큰 삭제

commit 9efe9aa
Author: Yiseul Park <omjl5123@catholic.ac.kr>
Date:   Fri Dec 29 17:21:05 2023 +0900

    !BREAKING CHANGE: 리프레쉬 토큰 바디 대신 쿠키로 전달

commit 5ed18db
Author: Yiseul Park <omjl5123@catholic.ac.kr>
Date:   Fri Dec 29 16:19:13 2023 +0900

    feat: 로그인 시 리프레쉬 토큰 저장

commit eb24b1d
Author: Yiseul Park <omjl5123@catholic.ac.kr>
Date:   Fri Dec 29 16:13:25 2023 +0900

    feat: cache 관련 설정 추가

commit 44d3a8e
Author: Yiseul Park <omjl5123@catholic.ac.kr>
Date:   Fri Dec 29 16:11:28 2023 +0900

    chore: Spring cache, caffeine 의존성 추가

commit badd8a1
Merge: 86a6bfe 4e48721
Author: Yiseul Park <omjl5123@catholic.ac.kr>
Date:   Fri Dec 29 14:04:53 2023 +0900

    Merge pull request #214 from bucket-back/OMCT-409-limit-vote-participation

    [OMCT-409] 최대 투표자 수를 제한하는 요구사항 추가

commit 86a6bfe
Merge: 7ea011a d4b2b87
Author: JoJaeHyeon <77893164+HandmadeCloud@users.noreply.github.com>
Date:   Fri Dec 29 01:10:42 2023 +0900

    Merge pull request #215 from bucket-back/OMCT-408-alarm-sse

    [OMCT-408] 댓글 생성에 대한 알림 기능 구현

commit 5fce082
Author: JoJaeHyeon <joje1003@naver.com>
Date:   Thu Dec 28 23:54:05 2023 +0900

    refactor : jwt 토큰 검사 로직 수정

commit 35c0fe5
Author: JoJaeHyeon <joje1003@naver.com>
Date:   Thu Dec 28 23:53:54 2023 +0900

    test : jwt 토큰 검사 테스트 구현

commit 0dcbe57
Merge: 3874afd 903c93f
Author: JoJaeHyeon <joje1003@naver.com>
Date:   Thu Dec 28 00:14:33 2023 +0900

    Merge branch 'main' into OMCT-410--domain-layer-test

commit 7ea011a
Merge: 903c93f 3874afd
Author: JoJaeHyeon <77893164+HandmadeCloud@users.noreply.github.com>
Date:   Thu Dec 28 00:13:30 2023 +0900

    Merge pull request #213 from bucket-back/OMCT-410--domain-layer-test

    [OMCT-410]  domain validation test 및 refactoring

commit 3084c34
Author: Yiseul Park <omjl5123@catholic.ac.kr>
Date:   Wed Dec 27 16:40:33 2023 +0900

    feat: 로그인할 때 리프레쉬 토큰도 함께 반환

commit d7c59f7
Author: Yiseul Park <omjl5123@catholic.ac.kr>
Date:   Wed Dec 27 14:31:48 2023 +0900

    refactor: generateToken -> generateAccessToken 메서드명 변경, 매개변수로 memberId 받도록 변경

commit 3874afd
Author: JoJaeHyeon <joje1003@naver.com>
Date:   Wed Dec 27 01:20:33 2023 +0900

    chore : 테스트 메세지 변경

commit 4e48721
Author: Yiseul Park <omjl5123@catholic.ac.kr>
Date:   Tue Dec 26 15:46:09 2023 +0900

    fix: 최대 투표자수 확인 시 null 체크

commit 94251bd
Author: Yiseul Park <omjl5123@catholic.ac.kr>
Date:   Tue Dec 26 14:53:00 2023 +0900

    feat: 투표자 수 제한하는 요구사항 추가

commit 4745a6a
Author: Yiseul Park <omjl5123@catholic.ac.kr>
Date:   Tue Dec 26 14:41:10 2023 +0900

    refactor: 로그인된 사용자를 얻을 때 삭제된 회원인지 검증하는 로직 삭제

commit 08f4321
Author: Yiseul Park <omjl5123@catholic.ac.kr>
Date:   Tue Dec 26 08:12:42 2023 +0900

    refactor: JWT 토큰 인증할 때 DB에 접근 안하도록 변경

commit 903c93f
Merge: e6a7553 7397954
Author: ByeongGon Kang <kangbk4182@naver.com>
Date:   Thu Dec 21 23:01:07 2023 +0900

    Merge pull request #202 from bucket-back/OMCT-380-feed-modifier-test

    [OMCT-380] 피드 수정 테스트 기능 추가

commit e6a7553
Merge: 1836424 e36fce0
Author: JoJaeHyeon <77893164+HandmadeCloud@users.noreply.github.com>
Date:   Thu Dec 21 20:33:43 2023 +0900

    Merge pull request #212 from bucket-back/OMCT-352-test-inventory-reader

    [OMCT-352] inventoryReader 테스트 기능 구현

commit 8091ada
Author: JoJaeHyeon <joje1003@naver.com>
Date:   Wed Dec 20 01:19:01 2023 +0900

    refactor : bucketInfo 검증 로직 수정

commit 208c497
Author: JoJaeHyeon <joje1003@naver.com>
Date:   Wed Dec 20 01:18:43 2023 +0900

    test : bucketInfo 검증 테스트 구현

commit 1992f0a
Author: JoJaeHyeon <joje1003@naver.com>
Date:   Wed Dec 20 00:50:43 2023 +0900

    test : feedContent 검증 테스트 구현

commit e36fce0
Author: JoJaeHyeon <joje1003@naver.com>
Date:   Tue Dec 19 01:23:48 2023 +0900

    test : reviewBuilder test-fixture 구현

commit 2089843
Merge: 647dc48 1836424
Author: JoJaeHyeon <77893164+HandmadeCloud@users.noreply.github.com>
Date:   Tue Dec 19 01:19:02 2023 +0900

    Merge branch 'main' into OMCT-352-test-inventory-reader

commit 647dc48
Author: JoJaeHyeon <joje1003@naver.com>
Date:   Tue Dec 19 01:14:36 2023 +0900

    test : 마이페이지 테스트 검증 내용 보완, 및 로직 정리

commit c57ca75
Author: JoJaeHyeon <joje1003@naver.com>
Date:   Tue Dec 19 01:09:50 2023 +0900

    test : inventoryReader 아이템 목록 조회, 마이페이지 조회 테스트 구현

commit 096051d
Author: JoJaeHyeon <joje1003@naver.com>
Date:   Tue Dec 19 01:09:07 2023 +0900

    test : 사용하지 않는 의존성 삭제

commit 8e2a8fe
Author: JoJaeHyeon <joje1003@naver.com>
Date:   Mon Dec 18 23:06:13 2023 +0900

    test : inventory 아이템 상세조회

commit 11d321f
Author: JoJaeHyeon <joje1003@naver.com>
Date:   Mon Dec 18 23:05:58 2023 +0900

    test : inventory test-fixture 메소드 수정

commit 13e17e4
Merge: 37a31d2 1836424
Author: Yiseul Park <omjl5123@catholic.ac.kr>
Date:   Tue Dec 12 07:21:23 2023 +0900

    Merge remote-tracking branch 'origin/main' into OMCT-345-test-vote-reader

commit 517d046
Author: Curry4182 <kangbk4182@naver.com>
Date:   Thu Nov 30 08:41:56 2023 +0900

    feat: hash 기능을 이용하여 big o 1에 탐색할 수 있도록 기능 변경

commit 88a3ce1
Author: Curry4182 <kangbk4182@naver.com>
Date:   Thu Nov 30 08:41:14 2023 +0900

    feat: redis template에서 hash에 대해서도 serializer 할 수 있도록 기능 추가

commit 725748d
Author: Curry4182 <kangbk4182@naver.com>
Date:   Thu Nov 30 07:52:07 2023 +0900

    refactor: 하나의 메서드로 사용할 수 있는 기능은 합치는 리팩토링

commit 275105e
Author: Curry4182 <kangbk4182@naver.com>
Date:   Thu Nov 30 07:42:29 2023 +0900

    refactor: 이미 존재하는 피드 id 인지 확인하고 없으면 생성할 수 있도록 기능 수정

commit 22e9ce9
Author: Curry4182 <kangbk4182@naver.com>
Date:   Thu Nov 30 07:40:54 2023 +0900

    feat: 이미 존재하는 feedId 인지 확인하는 기능 추가

commit eb17b6e
Author: Curry4182 <kangbk4182@naver.com>
Date:   Thu Nov 30 07:25:30 2023 +0900

    feat: 피드 싫어요 할 경우 랭킹이 낮아 지도록 기능 추가

commit 89c6fa3
Author: Curry4182 <kangbk4182@naver.com>
Date:   Thu Nov 30 07:22:05 2023 +0900

    feat: 피드 좋아요 할 경우 랭킹이 오르는 기능 추가

commit a21e46b
Author: Curry4182 <kangbk4182@naver.com>
Date:   Thu Nov 30 06:48:49 2023 +0900

    feat: 피드 상세 조회 할 경우 랭킹이 오르는 기능 추가

commit ce75d6b
Author: Curry4182 <kangbk4182@naver.com>
Date:   Thu Nov 30 06:48:13 2023 +0900

    feat: 피드 랭킹 정보 조회 기능 추가

commit 1c208e6
Author: Curry4182 <kangbk4182@naver.com>
Date:   Thu Nov 30 06:44:42 2023 +0900

    feat: 피드 랭킹 정보를 레디스에 저장 하는 기능 추가

commit f2ddfa0
Author: Curry4182 <kangbk4182@naver.com>
Date:   Thu Nov 30 04:50:40 2023 +0900

    refactor: 테스트 관련 기능을 gradle에 잘 보이도록 수정

commit 1df200e
Author: Curry4182 <kangbk4182@naver.com>
Date:   Thu Nov 30 04:50:04 2023 +0900

    fix: 중요 제약 설정 테이블 삭제 후 다시 복구 되도록 수정

commit f600b8a
Author: Curry4182 <kangbk4182@naver.com>
Date:   Thu Nov 30 04:47:20 2023 +0900

    feat: 테스트 하기전 테이블 삭제 하기 위한 sql 추가

commit 7397954
Author: Curry4182 <kangbk4182@naver.com>
Date:   Thu Nov 30 04:23:46 2023 +0900

    test: 피드 수정 테스트 기능 추가

commit 37a31d2
Author: Yiseul Park <omjl5123@catholic.ac.kr>
Date:   Wed Nov 29 14:59:56 2023 +0900

    test: 투표 상세 조회 테스트 추가

commit 0943dc0
Merge: 09e455f d7fceac
Author: JoJaeHyeon <joje1003@naver.com>
Date:   Tue Nov 28 11:57:41 2023 +0900

    Merge branch 'main' into OMCT-352-test-inventory-reader

commit 9cf9872
Author: Yiseul Park <omjl5123@catholic.ac.kr>
Date:   Mon Nov 27 00:04:35 2023 +0900

    refactor: 투표 상세조회 메서드명 변경

commit 886b86f
Author: Yiseul Park <omjl5123@catholic.ac.kr>
Date:   Mon Nov 27 00:02:58 2023 +0900

    test: 투표 엔티티 조회 테스트 추가

commit 70f5a7e
Author: Yiseul Park <omjl5123@catholic.ac.kr>
Date:   Sun Nov 26 23:52:35 2023 +0900

    fixture: final 추가, buildMany range 변경

commit e9b1fdd
Author: Curry4182 <kangbk4182@naver.com>
Date:   Sun Nov 26 05:17:03 2023 +0900

     test: 테스트할 필요가 없는 클래스에 Disabled 어노테이션 추가

commit 6e6c37b
Author: Curry4182 <kangbk4182@naver.com>
Date:   Sun Nov 26 04:52:51 2023 +0900

     test: 나의 아이템 추가 통합 테스트

commit 29dd259
Author: Curry4182 <kangbk4182@naver.com>
Date:   Sun Nov 26 04:51:28 2023 +0900

     test: 통합 테스트에 사용되는 IntegrationTest 클래스 추가

commit e9463cf
Author: Curry4182 <kangbk4182@naver.com>
Date:   Sun Nov 26 04:50:42 2023 +0900

     fixture: Item을 데이터베이스에 저장하는 ItemSetup 추가

commit 31cc1d0
Author: Curry4182 <kangbk4182@naver.com>
Date:   Sun Nov 26 04:49:48 2023 +0900

    chore: 도메인에 setup 클래스에서 jpa repository를 사용하기 위한 의존성 추가
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Curry4182 Curry4182

@Yiseull Yiseull

Assignees

@HandmadeCloud HandmadeCloud

Labels
test Good for newcomers
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@HandmadeCloud @Curry4182 @Yiseull