feat: 인터뷰 기능을 구현한다 #75

devholic22 · 2024-09-28T14:03:07Z

📄 Summary

인터뷰 기능을 구현했습니다.

어드민
- 인터뷰 등록
- 인터뷰 수정
- 인터뷰 목록 타입으로 조회 (기본값: '나')
회원
- 인터뷰 응시
- 인터뷰 목록 타입으로 조회 (기본값: '나', 응시했든 응시하지 않았든 해당 타입에 대한 모든 인터뷰 나오고 제출 여부만 달라짐)
- 인터뷰 단일 조회 (없었을 시 answer 빈 문자열)
- 인터뷰 답변 수정

🙋🏻 More

인수 테스트인 MemberInterviewsControllerAcceptanceTest에서 인터뷰 수정 테스트를 하던 중 response.as로 변환하는 부분이 계속 오류가 발생해서, 이 부분은 일단 주석처리했습니다. 원인은 text/plain으로 응답이 나와 json으로 매핑할 수 없는 것 같아 보이는데, 더 알아봐야 할 듯 합니다. 일단 이 부분 제외하고 리뷰 먼저 해주시는 게 좋을 것 같습니다.
- 원인은 인터뷰 응시를 하지 않았는데 인터뷰 수정을 시도함으로써 400 에러가 발생했었던 것이었습니다. 수정 완료하였습니다.
일정 때문에 계속 미루다가 오늘에서야 구현 및 테스트를 완료 (9시간 추가..)하게 되었습니다. 더 시간 내보도록 하겠습니다.

close #51

- 인터뷰 타입 등록 - 인터뷰 타입 조회 예외 처리 등록 - 인터뷰 타입 테스트 코드 작성

- memberinterview, interview 패키지 분리를 위해 기존 생성 파일 패키지 분리

- 인터뷰 모델 코드 정의 - 인터뷰 도메인 리포지터리 및 구현체 정의

- 인터뷰 등록 기능 개발 - 같은 질문 등록 불가능하도록 설정 - 컨트롤러, 서비스 연결

- 인터뷰 Fixture 리포지터리 작성 - 인터뷰 service, infrastructure 테스트 작성 - 인터뷰 fixture 등록

- 인터뷰 수정 기능 구현 - 관련 테스트 코드 작성 - 예외처리 추가

- 회원 인터뷰 응시 기능 구현 - 필요한 도메인 코드 기본 작성 - 필요한 서비스, 리포지터리 코드 기본 작성

- 인터뷰 타입 조회 기능 초안 구현 - 관련 QueryService, QueryRepository 구현

- 회원 인터뷰 목록 조회 기능 구현 - 관련 QueryRepository 테스트 작성 - 응시한 인터뷰는 true, 응시하지 않은 인터뷰는 false로 나오도록 구현

- MemberInterviewResponse 이름을 MemberInterviewSimpleResponse로 변경 - 패키지 역참조 수정하여 infrastructure에 들어가도록 수정

- 회원 인터뷰 상세 조회 기능 구현 - 응시한 인터뷰면 답변 나오고 응시하지 않은 인터뷰면 공백 나오도록 설정

- 회원 인터뷰 수정 기능 구현 - JPA만으로 구현하도록 설정

- 인터뷰 등록 인자 가독성 고려하여 위치 수정

- 인터뷰 단위테스트 추가 - 비교 판정 테스트 작성 - 필요한 fixture 등록

- 회원 인터뷰 단위테스트 추가

- 회원 인터뷰 목록 단위테스트 추가 - 필요없는 MemberInterviews 메서드 제거 - MemberInterview equals&hashcode 재정의

- InterviewFixture에 id가 고정으로 들어가던 문제 수정

- InterviewQueryService 테스트 작성 - 필요한 fixture 등록 - extracting을 이용하여 더 세밀하게 검증하도록 작성

- InterviewQueryRepository 테스트 작성 - InterviewResponse의 type 타입 변경

- InterviewController 통합 테스트 작성 - 인터뷰 문서화 작성

- InterviewController 인수 테스트 작성 - 필요한 fixture 등록

- MemberInterviewsService 테스트 작성 - 필요한 fake repository 작성 - 빌더 패턴 추가

- MemberInterviewsQueryService 테스트 작성 - MemberInterviewsFakeRepository 로직 수정

- MemberInterviewsQueryRepository 추가된 메서드 테스트 - MemberInterviewsQueryRepository 쿼리 수정

- MemberInterviewsJpaRepository 테스트 작성

- MemberInterviewsController 통합 테스트 작성 및 컨트롤러 이름 변경 - 문서화 작성

- interview.adoc 문서 보완

- Transactional 제거 및 save 전 미리 작업하도록 수정

- 테스트에 사용하는 fixture 패키지 분리

- MemberInterviewsController 인수 테스트 작성 - 필요한 fixture 등록 - 수정 API에서의 문제 보류

- MemberAuthConfig에 interview 관련 url 추가

eom-tae-in

고생하셨습니다.
테스트에서 전반적으로 중복되는 로직이 많아 보입니다. 이 부분 리팩터링을 좀 해야할 것 같네요
리뷰 확인해주시고 수정부탁드려요~

eom-tae-in · 2024-09-28T14:34:08Z

src/main/java/com/atwoz/interview/application/memberinterview/MemberInterviewsService.java

+    private final InterviewRepository interviewRepository;
+    private final MemberInterviewsRepository memberInterviewsRepository;
+
+    public void submitInterview(final Long interviewId, final Long memberId, final MemberInterviewSubmitRequest request) {


매개변수 3개 이상일 시 줄바꿈 추가해주세요~

갯수를 파악하지 못하고 길이로만 생각했던 것 같습니다. 나머지 부분들도 모두 반영하였습니다 :)

eom-tae-in · 2024-09-28T14:34:21Z

src/main/java/com/atwoz/interview/application/memberinterview/MemberInterviewsService.java

+        return memberInterviewsRepository.save(memberInterviews);
+    }
+
+    public MemberInterview updateMemberInterviewAnswer(final Long interviewId, final Long memberId, final MemberInterviewUpdateRequest request) {


여기도 줄바꿈 추가해주세요

eom-tae-in · 2024-09-28T14:35:46Z

src/main/java/com/atwoz/interview/domain/memberinterview/MemberInterviews.java

+    @Id
+    @GeneratedValue(strategy = GenerationType.IDENTITY)
+    private Long id;
+


해당 컬럼에 대한 제약조건이 없어도 되나요?

혹시 id에 대한 제약조건을 말씀하시는 걸까요? id에 대해 제약조건이 있을 필요가 있는지 궁금합니다.

아 아닙니다 memberId 컬럼에 대해 말씀드린겁니다

memberId에 대해 제약조건 추가하였습니다. (nullable false, unique true) 알려주셔서 감사합니다!

eom-tae-in · 2024-09-28T14:38:23Z

src/main/java/com/atwoz/interview/ui/interview/InterviewController.java

+
+    @GetMapping
+    public ResponseEntity<InterviewsResponse> findInterviewsByType(
+            @RequestParam(value = "type", defaultValue = "나") final String type) {


상수처리 해주세요

InterviewController랑 MemberInterviewsController 모두 상수 처리 완료했습니다

eom-tae-in · 2024-09-28T14:40:11Z

src/test/java/com/atwoz/interview/application/interview/InterviewQueryServiceTest.java

+        interviewRepository.save(인터뷰_타입_질문(InterviewType.ME, "내가 생각하는 내 장점과 단점은 이거다!"));
+        interviewRepository.save(인터뷰_타입_질문(InterviewType.ME, "내가 생각하는 나의 반전 매력은 이거야!"));
+        interviewRepository.save(인터뷰_타입_질문(InterviewType.ME, "10년 뒤 멋진 내 모습, 과연 어떤 모습일까?"));
+
+        interviewRepository.save(인터뷰_타입_질문(InterviewType.RELATIONSHIP, "관계 관련 질문 1"));
+        interviewRepository.save(인터뷰_타입_질문(InterviewType.RELATIONSHIP, "관계 관련 질문 2"));
+        interviewRepository.save(인터뷰_타입_질문(InterviewType.RELATIONSHIP, "관계 관련 질문 3"));
+
+        interviewRepository.save(인터뷰_타입_질문(InterviewType.COUPLE, "연인 관련 질문 1"));
+        interviewRepository.save(인터뷰_타입_질문(InterviewType.COUPLE, "연인 관련 질문 2"));
+        interviewRepository.save(인터뷰_타입_질문(InterviewType.COUPLE, "연인 관련 질문 3"));


save메서드를 자주 사용한 거 같은데 메서드로 분리하거나 fixture를 두고 사용하는게 좋아보입니다

메서드 분리로 적용하였습니다 :)

eom-tae-in · 2024-09-28T14:41:19Z

src/test/java/com/atwoz/interview/application/interview/InterviewQueryServiceTest.java

+        interviewQueryService = new InterviewQueryService(interviewRepository);
+    }
+
+    @Test


똑같은 기능에 대한 테스트면 @nested를 사용하는 건 어떤가요?

기능이 한 가지밖에 없어서 nested를 사용하지 않았었는데, 기능이 더 추가될 수 있기 때문에 미리 nested를 붙여놓는 게 좋다고 생각이 들었습니다. 말씀하신 대로 nested 적용하였습니다 :)

eom-tae-in · 2024-09-28T14:42:33Z

...t/java/com/atwoz/interview/application/memberinterview/MemberInterviewsQueryServiceTest.java

+            interviewRepository.save(인터뷰_타입_질문(InterviewType.ME, "일상에서 느끼는 나의 소소한 행복"));
+            interviewRepository.save(인터뷰_타입_질문(InterviewType.RELATIONSHIP, "남사친/여사친에 대한 나의 생각"));
+            interviewRepository.save(인터뷰_타입_질문(InterviewType.RELATIONSHIP, "나의 인간관계 스타일"));
+            interviewRepository.save(인터뷰_타입_질문(InterviewType.RELATIONSHIP, "호감을 느끼는 사람의 유형/타입"));
+            interviewRepository.save(인터뷰_타입_질문(InterviewType.COUPLE, "추구하는 만남 횟수와 연락 빈도"));
+            interviewRepository.save(인터뷰_타입_질문(InterviewType.COUPLE, "내가 연인을 사랑하는 방식"));
+            interviewRepository.save(인터뷰_타입_질문(InterviewType.COUPLE, "연인에게 바라는 점"));


이 부분도 중복 로직을 따로 분리하는게 좋을 거 같아요

메서드 분리 적용하였습니다 :)

eom-tae-in · 2024-09-28T14:43:49Z

src/test/java/com/atwoz/interview/fixture/interview/InterviewFixture.java

+                .build();
+    }
+
+    public static Interview 인터뷰_타입_질문_id(final InterviewType type, final String question, final Long id) {


매개변수 3개 이상일 시 줄바꿈하고 작성해주세요~

같은 리뷰 발생하지 않도록 하겠습니다!

- 인자 3개 이상 시 줄바꿈 추가되도록 수정

- 컨트롤러 인자 3개 이상 시 줄바꿈되도록 수정 - 괄호 위치 수정

- InterviewQueryServiceTest에 Nested 추가

- 반복적으로 인터뷰를 넣었던 코드를 saveOtherInterviews 메서드로 재사용

- 인터뷰 답변 수정 인수 테스트 수정 - 인터뷰 응시 과정 추가

eom-tae-in · 2024-09-28T15:31:02Z

src/main/java/com/atwoz/interview/domain/memberinterview/MemberInterviews.java

+    @Id
+    @GeneratedValue(strategy = GenerationType.IDENTITY)
+    private Long id;
+


아 아닙니다 memberId 컬럼에 대해 말씀드린겁니다

- memberId 제약조건 추가

devholic22 and others added 30 commits September 3, 2024 00:24

feat: 인터뷰 타입 등록

b0b48d6

- 인터뷰 타입 등록 - 인터뷰 타입 조회 예외 처리 등록 - 인터뷰 타입 테스트 코드 작성

refactor: interview 패키지 분리

3a2d82a

- memberinterview, interview 패키지 분리를 위해 기존 생성 파일 패키지 분리

feat: 인터뷰 모델 및 도메인 정의

f12b15f

- 인터뷰 모델 코드 정의 - 인터뷰 도메인 리포지터리 및 구현체 정의

feat: 인터뷰 등록 기능 개발

b3cd086

- 인터뷰 등록 기능 개발 - 같은 질문 등록 불가능하도록 설정 - 컨트롤러, 서비스 연결

test: 인터뷰 기본 테스트 코드 작성

709479f

- 인터뷰 Fixture 리포지터리 작성 - 인터뷰 service, infrastructure 테스트 작성 - 인터뷰 fixture 등록

feat: 인터뷰 수정 기능 추가

916ac67

- 인터뷰 수정 기능 구현 - 관련 테스트 코드 작성 - 예외처리 추가

feat: 회원 인터뷰 응시 기능 구현

046ec2e

- 회원 인터뷰 응시 기능 구현 - 필요한 도메인 코드 기본 작성 - 필요한 서비스, 리포지터리 코드 기본 작성

feat: 인터뷰 타입 조회 기능 구현

56d5f66

- 인터뷰 타입 조회 기능 초안 구현 - 관련 QueryService, QueryRepository 구현

feat: 회원 인터뷰 목록 조회 기능 구현

67fcd31

- 회원 인터뷰 목록 조회 기능 구현 - 관련 QueryRepository 테스트 작성 - 응시한 인터뷰는 true, 응시하지 않은 인터뷰는 false로 나오도록 구현

fix: MemberInterviewResponse 변경

f383fdd

- MemberInterviewResponse 이름을 MemberInterviewSimpleResponse로 변경 - 패키지 역참조 수정하여 infrastructure에 들어가도록 수정

feat: 회원 인터뷰 상세 조회 기능 구현

fa9f45c

- 회원 인터뷰 상세 조회 기능 구현 - 응시한 인터뷰면 답변 나오고 응시하지 않은 인터뷰면 공백 나오도록 설정

feat: 회원 인터뷰 수정 기능 구현

d86149c

- 회원 인터뷰 수정 기능 구현 - JPA만으로 구현하도록 설정

refactor: 인터뷰 등록 인자 수정

01567e8

- 인터뷰 등록 인자 가독성 고려하여 위치 수정

test: 인터뷰 단위테스트 추가

c237863

- 인터뷰 단위테스트 추가 - 비교 판정 테스트 작성 - 필요한 fixture 등록

test: 회원 인터뷰 단위테스트 추가

f7e0fb4

- 회원 인터뷰 단위테스트 추가

test: 회원 인터뷰 목록 단위테스트 추가

af1c56c

- 회원 인터뷰 목록 단위테스트 추가 - 필요없는 MemberInterviews 메서드 제거 - MemberInterview equals&hashcode 재정의

fix: InterviewFixture id 고정 수정

9e174a6

- InterviewFixture에 id가 고정으로 들어가던 문제 수정

test: InterviewQueryService 테스트 작성

92dce54

- InterviewQueryService 테스트 작성 - 필요한 fixture 등록 - extracting을 이용하여 더 세밀하게 검증하도록 작성

test: InterviewQueryRepository 테스트 작성

ef93d6c

- InterviewQueryRepository 테스트 작성 - InterviewResponse의 type 타입 변경

test: InterviewController 통합 테스트 작성

8413da2

- InterviewController 통합 테스트 작성 - 인터뷰 문서화 작성

test: InterviewController 인수 테스트 작성

b2e28b1

- InterviewController 인수 테스트 작성 - 필요한 fixture 등록

test: MemberInterviewsService 테스트 작성

286975d

- MemberInterviewsService 테스트 작성 - 필요한 fake repository 작성 - 빌더 패턴 추가

test: MemberInterviewsQueryService 테스트 작성

a1e4b69

- MemberInterviewsQueryService 테스트 작성 - MemberInterviewsFakeRepository 로직 수정

test: MemberInterviewsQueryRepository 테스트 보완

4e7412e

- MemberInterviewsQueryRepository 추가된 메서드 테스트 - MemberInterviewsQueryRepository 쿼리 수정

test: MemberInterviewsJpaRepository 테스트 작성

38bd964

- MemberInterviewsJpaRepository 테스트 작성

test: MemberInterviewsController 통합 테스트 작성

c643751

- MemberInterviewsController 통합 테스트 작성 및 컨트롤러 이름 변경 - 문서화 작성

docs: interview.adoc 문서 보완

9c0fe89

- interview.adoc 문서 보완

refactor: Transactional 제거

0d7a446

- Transactional 제거 및 save 전 미리 작업하도록 수정

refactor: fixture 분리

c5d4a3e

- 테스트에 사용하는 fixture 패키지 분리

test: MemberInterviewsControllerAcceptanceTest 작성

ed55f2f

- MemberInterviewsController 인수 테스트 작성 - 필요한 fixture 등록 - 수정 API에서의 문제 보류

fix: MemberAuthConfig url 추가

ee0dfdf

- MemberAuthConfig에 interview 관련 url 추가

devholic22 added the 기능 추가 label Sep 28, 2024

devholic22 self-assigned this Sep 28, 2024

devholic22 temporarily deployed to test September 28, 2024 14:03 — with GitHub Actions Inactive

eom-tae-in requested changes Sep 28, 2024

View reviewed changes

devholic22 added 2 commits September 28, 2024 23:54

refactor: 인자 3개 이상 시 줄바꿈 추가되도록 수정

97255dd

- 인자 3개 이상 시 줄바꿈 추가되도록 수정

refactor: 컨트롤러 가독성 수정

a091a2d

- 컨트롤러 인자 3개 이상 시 줄바꿈되도록 수정 - 괄호 위치 수정

devholic22 temporarily deployed to test September 28, 2024 14:56 — with GitHub Actions Inactive

devholic22 added 2 commits September 28, 2024 23:57

refactor: Nested 추가

5162759

- InterviewQueryServiceTest에 Nested 추가

refactor: saveOtherInterviews 메서드 추가

694e4cf

- 반복적으로 인터뷰를 넣었던 코드를 saveOtherInterviews 메서드로 재사용

devholic22 temporarily deployed to test September 28, 2024 15:09 — with GitHub Actions Inactive

fix: 인터뷰 답변 수정 인수 테스트 수정

94a1bad

- 인터뷰 답변 수정 인수 테스트 수정 - 인터뷰 응시 과정 추가

devholic22 temporarily deployed to test September 28, 2024 15:23 — with GitHub Actions Inactive

eom-tae-in approved these changes Sep 28, 2024

View reviewed changes

fix: memberId 제약조건 추가

7cb8fe9

- memberId 제약조건 추가

devholic22 temporarily deployed to test September 28, 2024 15:41 — with GitHub Actions Inactive

devholic22 merged commit 398168a into develop Sep 28, 2024
1 check passed

feat: 인터뷰 기능을 구현한다 #75

feat: 인터뷰 기능을 구현한다 #75

Uh oh!

Conversation

devholic22 commented Sep 28, 2024 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

📄 Summary

🙋🏻 More

Uh oh!

eom-tae-in left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

devholic22 commented Sep 28, 2024 •

edited

Loading