MemberJpaRepository 테스트 구현 #624

kyum-q · 2024-09-10T04:39:03Z

⚡️ 관련 이슈

close #620

📍주요 변경 사항

MemberJpaRepository 관련 테스트 구현

메서드 이름 및 쿼리를 직접 작성한 메서드에 대해 테스트를 작성

🎸기타

문제 있었음 -> 결론 제우스가 도와줌 내 yml 파일 설정 문제 !

테스트 처음 실행 시 정상 실행 (DB에 테이블 없는 상태에서)

두번 째 실행부터 Flyway 에러 발생

문제 원인

V1__init.sql 실행으로 문제 발생 -> 테이블이 이미 존재해서 에러

의문점 - 왜 V1__init.sql 이 계속 실행되는가?

Flyway는 각 마이그레이션 파일에 고유한 버전을 부여하며, 이 버전이 flyway_schema_history 테이블에 기록된 경우 동일한 버전의 스크립트는 재실행되지 않는다.
flyway_schema_history 확인하면 version 1 등록되어 있음

나만 그러면 도와줄 친구 찾음. applicaton-db.yml 파일을 dev 서버와 동일하게 변경해도 안됨

해결

@zeus6768 가 도와줌

1차 결론

결론 : applicatoin-db.yml 파일을 test 패키지에 구현하지 않았음.
줄줄 이야기 해봐요 :
test 패키지에 yml 파일이 하나라도 구현했다면 사용하는 모든 yml 파일을 새롭게 구현해야 했음.
하지만 그걸 모르고 applicatoin-db.yml 파일을 test 패키지에 구현하지 않았음.
application-db.yml 파일을 test 패키지에 추가하고 나니 해결

2차 결론

1차 결론 후 또 안됨, create-drop 이 아닌 validate인 경우 안됨
초켬제 머리 합친 결과 : 몰리가 구현해준 test annotaiton이랑 충돌하나로 결론

몰리가 구현해준 어노테이션 실행 시키면 모든 테이블 드랍하는 거와 flyway 충돌
테스트에서는 creat-drop 으로 하기로 함 ~

backend/src/main/java/codezap/member/repository/MemberJpaRepository.java

HoeSeong123

고생하셨습다 켬미 👍

HoeSeong123 · 2024-09-11T02:14:23Z

backend/src/test/java/codezap/member/repository/MemberJpaRepositoryTest.java

    @Autowired
-    MemberJpaRepositoryTest(MemberJpaRepository memberJpaRepository, TemplateJpaRepository templateJpaRepository,
-            CategoryJpaRepository categoryJpaRepository
+    MemberJpaRepositoryTest(
+            MemberJpaRepository memberRepository,
+            TemplateJpaRepository templateRepository,
+            CategoryJpaRepository categoryRepository
    ) {
-        this.memberJpaRepository = memberJpaRepository;
-        this.templateJpaRepository = templateJpaRepository;
-        this.categoryJpaRepository = categoryJpaRepository;
+        this.memberRepository = memberRepository;
+        this.templateRepository = templateRepository;
+        this.categoryRepository = categoryRepository;
    }


(질문) 오 이렇게 하는건 처음보네요??
필드들을 각각 @Autowired하는 것과 어떤 차이가 있나요??

저도 이전 미션 때 매번 @Autowired를 사용했습니다 ~!
하지만 이번에 수정 전 코드도 생성자 주입으로 되어 있고, 레벨 2 때 어디선가 테스트에서 Setter 주입보다 생성자 주입이 더 빠르다라는 이야기를 들었던 것 같아서 그대로 사용했습니다 !

하지만 초롱에 질문에 답변하기 위해 찾아보니 그런 아티클이나 글은 없더라고요 ㅎㅎ 그래서 ChatGPT한테 물어보니 성능 상은 거의 똑같을 거라고 하네요 !

그래도 혹시 몰라 테스트를 실행해 봤습니다.
결론적으로 전혀 상관 없네요 ㅎㅎ
가독성이 좋은 @Autowired 사용으로 변경하겠습니다.

찾아봐줘서 감사합니다!!

HoeSeong123 · 2024-09-11T02:20:13Z

backend/src/test/java/codezap/member/repository/MemberJpaRepositoryTest.java

+    }
+
+    @Nested
+    @DisplayName("아이디가 존재 여부")


정말 사소한 거지만 문장이 어색한 것 같아요.
아이디 존재 여부 등으로 수정하는 건 어떤가요?

HoeSeong123 · 2024-09-11T02:20:55Z

backend/src/test/java/codezap/member/repository/MemberJpaRepositoryTest.java

+        }
+
+        @Test
+        @DisplayName("성공 : 해당 아이디가 존재하면 false를 반환한다.")


⚡

Suggested change

@DisplayName("성공 : 해당 아이디가 존재하면 false를 반환한다.")

@DisplayName("성공 : 해당 아이디가 존재하지 않으면 false를 반환한다.")

zangsu

켬미~
코드를 너무 잘 짜 주셨네요!
컨벤션이나 코드 자체에서는 리뷰를 달 부분이 거의 없었던 것 같아요.
그래서 코드 스타일에 대한 코멘트를 조금 남겼으니 가볍게 확인 부탁드립니다!

backend/src/test/java/codezap/member/repository/MemberJpaRepositoryTest.java

zeus6768

켬미 고생했어요~

간단한 코멘트 남겼습니다.

팀원들이 해준 리뷰 반영되면 approve 하러 올게요!

backend/src/test/java/codezap/member/repository/MemberJpaRepositoryTest.java

jminkkk

안녕하세요 켬미
수고하셨습니다!
(아주) 사소한 리뷰 몇개 남겨보았으니 확인해주시면 좋을 것 같아요

backend/src/test/java/codezap/member/repository/MemberJpaRepositoryTest.java

HoeSeong123

고생하셨습니다

zeus6768

👍

zangsu

놓친 부분이 있었네요 ㅠ
아주아주 사소한 부분이니... 한번만 더 수정 부탁드려용

backend/src/test/java/codezap/member/repository/MemberJpaRepositoryTest.java

zangsu

요구사항 변경 너~~무 고생 많으셨어요!! ⚡

kyum-q added 2 commits September 9, 2024 21:52

test: MemberJpaRepository 테스트 구현

4e8ad69

test: TagJpaRepository 테스트 수정

c6e7b60

kyum-q added refactor 요구사항이 바뀌지 않은 변경사항 BE 백엔드 labels Sep 10, 2024

kyum-q added this to the 5차 스프린트🍗 milestone Sep 10, 2024

kyum-q requested review from zangsu, zeus6768, jminkkk and HoeSeong123 September 10, 2024 04:39

kyum-q self-assigned this Sep 10, 2024

kyum-q changed the title ~~Test/620 member jpa repository~~ MemberJpaRepository 테스트 구현 Sep 10, 2024

kyum-q commented Sep 10, 2024

View reviewed changes

backend/src/main/java/codezap/member/repository/MemberJpaRepository.java Show resolved Hide resolved

kyum-q mentioned this pull request Sep 10, 2024

TagJpaRepository 테스트 구현 #625

Merged

HoeSeong123 requested changes Sep 11, 2024

View reviewed changes

zangsu reviewed Sep 11, 2024

View reviewed changes

test: DisplayName 수정

06da3ef

zeus6768 reviewed Sep 11, 2024

View reviewed changes

kyum-q added 2 commits September 11, 2024 13:37

test: 변수명 수정 및 코드 개선

384ccbf

test: 의존성 주입 Setter 주입으로 변경

79f4eaf

jminkkk reviewed Sep 11, 2024

View reviewed changes

backend/src/test/java/codezap/member/repository/MemberJpaRepositoryTest.java Outdated Show resolved Hide resolved

backend/src/test/java/codezap/member/repository/MemberJpaRepositoryTest.java Outdated Show resolved Hide resolved

test: 코드 개선

fcba611

HoeSeong123 previously approved these changes Sep 11, 2024

View reviewed changes

jminkkk previously approved these changes Sep 11, 2024

View reviewed changes

zeus6768 previously approved these changes Sep 11, 2024

View reviewed changes

zangsu reviewed Sep 12, 2024

View reviewed changes

backend/src/test/java/codezap/member/repository/MemberJpaRepositoryTest.java Outdated Show resolved Hide resolved

kyum-q added 2 commits September 12, 2024 10:20

test: 변수명 수정

7e05adc

test: DisplayName 수정

5668339

kyum-q dismissed stale reviews from zeus6768, jminkkk, and HoeSeong123 via 5668339 September 12, 2024 01:21

jminkkk approved these changes Sep 12, 2024

View reviewed changes

zangsu approved these changes Sep 12, 2024

View reviewed changes

HoeSeong123 approved these changes Sep 12, 2024

View reviewed changes

zeus6768 approved these changes Sep 12, 2024

View reviewed changes

zeus6768 merged commit 1799fab into dev/be Sep 12, 2024
5 checks passed

zeus6768 deleted the test/620-MemberJpaRepository branch September 12, 2024 05:58

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

MemberJpaRepository 테스트 구현 #624

MemberJpaRepository 테스트 구현 #624

kyum-q commented Sep 10, 2024 •

edited

Loading

HoeSeong123 left a comment

HoeSeong123 Sep 11, 2024

kyum-q Sep 11, 2024

HoeSeong123 Sep 11, 2024

HoeSeong123 Sep 11, 2024

HoeSeong123 Sep 11, 2024

zangsu left a comment

zeus6768 left a comment

jminkkk left a comment

HoeSeong123 left a comment

zeus6768 left a comment

zangsu left a comment

zangsu left a comment

	@DisplayName("성공 : 해당 아이디가 존재하면 false를 반환한다.")
	@DisplayName("성공 : 해당 아이디가 존재하지 않으면 false를 반환한다.")

MemberJpaRepository 테스트 구현 #624

MemberJpaRepository 테스트 구현 #624

Conversation

kyum-q commented Sep 10, 2024 • edited Loading

⚡️ 관련 이슈

📍주요 변경 사항

🎸기타

문제 있었음 -> 결론 제우스가 도와줌 내 yml 파일 설정 문제 !

테스트 처음 실행 시 정상 실행 (DB에 테이블 없는 상태에서)

두번 째 실행부터 Flyway 에러 발생

문제 원인

의문점 - 왜 V1__init.sql 이 계속 실행되는가?

나만 그러면 도와줄 친구 찾음. applicaton-db.yml 파일을 dev 서버와 동일하게 변경해도 안됨

해결

1차 결론

2차 결론

HoeSeong123 left a comment

Choose a reason for hiding this comment

HoeSeong123 Sep 11, 2024

Choose a reason for hiding this comment

kyum-q Sep 11, 2024

Choose a reason for hiding this comment

HoeSeong123 Sep 11, 2024

Choose a reason for hiding this comment

HoeSeong123 Sep 11, 2024

Choose a reason for hiding this comment

HoeSeong123 Sep 11, 2024

Choose a reason for hiding this comment

zangsu left a comment

Choose a reason for hiding this comment

zeus6768 left a comment

Choose a reason for hiding this comment

jminkkk left a comment

Choose a reason for hiding this comment

HoeSeong123 left a comment

Choose a reason for hiding this comment

zeus6768 left a comment

Choose a reason for hiding this comment

zangsu left a comment

Choose a reason for hiding this comment

zangsu left a comment

Choose a reason for hiding this comment

kyum-q commented Sep 10, 2024 •

edited

Loading