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

지원서 조회 / 게시글 댓글 테스트 코드 작성 #126

Merged
merged 7 commits into from
Oct 21, 2024

Conversation

kwonssshyeon
Copy link
Member

@kwonssshyeon kwonssshyeon commented Oct 20, 2024

🔎 작업 내용

  • 지원 현황 조회 테스트 코드 작성

  • 트랙별 지원 개수 테스트 코드 작성

    • 이 과정에서 처음 로그인한 사용자는 track 을 UNDEFINED로 두기로 수정했는데, 해당 필드가 응답에 포함되는 문제를 발견하고 getValidTrack을 정의하여 UNDEFINED는 제외하고 가져오도록 수정했습니다.
  • 임시저장된 지원서를 합/불 처리 하지 못하는지 테스트 코드 작성

    • 지원서 상태를 변경하는 메서드를 Application 클래스로 옮겨 객체지향적으로 수정
    • 'transaction template` 빈 등록하여 재사용하도록 수정
  • 댓글 등록 시 그룹이 올바르게 설정되는지 테스트 추가

  • 댓글 페이징 조회시 순서가 올바른지 조회 쿼리 테스트 추가

  • 기존 댓글 관련 테스트 코드 리팩토링

To Reviewers 📢

체크 리스트

  • 테스트를 작성했습니다.
  • 테스트를 통과했습니다.
  • API 변경사항이 존재합니다.
  • API 호출을 직접 실시하였고, 해당 데이터가 정상적으로 표시됩니다.
  • 기존 코드에 영향을 주는 작업 내용이 존재합니다.
  • 향후 추가적인 작업이 필요한 부분이 있습니다.

➕ 관련 이슈

Copy link

github-actions bot commented Oct 20, 2024

Unit Test Results

11 tests   11 ✔️  1s ⏱️
  5 suites    0 💤
  5 files      0

Results for commit 3b340b6.

♻️ This comment has been updated with latest results.

@kwonssshyeon kwonssshyeon linked an issue Oct 20, 2024 that may be closed by this pull request
2 tasks
Comment on lines 74 to 75
.studentNumber("202400"+id)
.phoneNumber("010-0000-"+id)
Copy link
Contributor

Choose a reason for hiding this comment

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

사소하긴 한데 그냥 id를 쓰는것 보단 String.format 함수 써서 각각 자릿수 양식에 맞게 만드는 게 조금 더 좋지 않을까 생각해봅니다.

@kwonssshyeon kwonssshyeon merged commit a93937d into dev Oct 21, 2024
3 checks passed
@kwonssshyeon kwonssshyeon deleted the Feat/issue-#88 branch October 21, 2024 17:31
@kwonssshyeon kwonssshyeon self-assigned this Oct 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

지원서류 조회 / 댓글 관련 테스트 코드 작성
2 participants