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

[새기능] 최종 합격 자동화 및 점수 순 정렬 #112

Merged
merged 28 commits into from
Sep 3, 2024

Conversation

cabbage16
Copy link
Member

@cabbage16 cabbage16 commented Jul 30, 2024

🎫 관련 이슈

close #98


📄 개요

최종 합격을 최종 점수 순으로 자동으로 결정하는 기능을 만들었습니다.
원서를 전체 조회할 때, 최종 점수 순으로 정렬하는 기능을 만들었습니다.


🔨 작업 내용

  • 최종 합격 자동 결정 API를 만들었습니다.
  • 원서 전체 조회 시 정렬 파라미터를 추가했습니다.

🏁 확인 사항

  • 테스트를 완료했나요?
  • API 문서를 작성했나요?
  • 코드 컨벤션을 준수했나요?
  • 불필요한 로그, 주석, import 등을 삭제했나요?

🙋🏻 덧붙일 말

SelectFirstPassUseCaseTest처럼 만들고 싶은데 테스트 프로필에서 h2 db가 작동하지 않아서... 도움 부탁드립니다 ㅠㅠㅠ

- 어드민이 최종 점수 순으로 최종 합격 여부를 결정할 수 있습니다.
- 1차 합격 원서 중 최종 점수가 없는 원서가 있다면 예외가 발생합니다.
- 어드민이 최종 점수 순으로 최종 합격 여부를 결정할 수 있습니다.
- 1차 합격 원서 중 최종 점수가 없는 원서가 있다면 예외가 발생합니다.
- 잘못된 이름을 수정하고 좀 더 의미에 맞도록 수정했습니다
- 정원 외 전형은 다른 전형들과 함께 경쟁하지 않고 완전히 독립되어 경쟁해야하여서 수정했습니다.
- 자동으로 2차 합격 여부를 결정하는 테스트를 작성했어요.
- 2차 합격 여부를 결정할 때 최종 점수가 없는 원서가 존재하면 에러가 발생하는 테스트를 작성했어요.
- 2차 합격 여부를 자동으로 결정하는 API 를 추가했어요.
- 어드민이 최종 점수 순으로 최종 합격 여부를 결정할 수 있습니다.
- 1차 합격 원서 중 최종 점수가 없는 원서가 있다면 예외가 발생합니다.
- 잘못된 이름을 수정하고 좀 더 의미에 맞도록 수정했습니다
- 정원 외 전형은 다른 전형들과 함께 경쟁하지 않고 완전히 독립되어 경쟁해야하여서 수정했습니다.
- 자동으로 2차 합격 여부를 결정하는 테스트를 작성했어요.
- 2차 합격 여부를 결정할 때 최종 점수가 없는 원서가 존재하면 에러가 발생하는 테스트를 작성했어요.
- 2차 합격 여부를 자동으로 결정하는 API 를 추가했어요.
- 어드민이 최종 점수 순으로 최종 합격 여부를 결정할 수 있습니다.
- 1차 합격 원서 중 최종 점수가 없는 원서가 있다면 예외가 발생합니다.
- 정렬 기준을 추가해서 최종 점수 오름차순, 내림차순으로 정렬할 수 있습니다.
- 정렬 기준과 정렬 방식을 하나의 파라미터로 합쳤습니다.
- 최종 점수 내림차순 조회 테스트를 만들었어요.
- 최종 점수 오름차순 조회 테스트를 만들었어요.
- 테스트용으로 만든 1차합격 자동 메서드를 삭제했어요.
- 원서를 전체 조회할 때 최종 점수순으로 조회하는데 최종점수가 없는 원서가 있다면 오류를 발생하도록 했어요.
@cabbage16 cabbage16 requested a review from a team as a code owner July 30, 2024 20:39
@cabbage16 cabbage16 linked an issue Jul 30, 2024 that may be closed by this pull request
5 tasks
Copy link

이렇게 리뷰해주세요.

🖐️ 규칙

  1. 개선이 필요한지 충분히 설명해주세요.
  2. 답을 알려주기보다는 스스로 고민하고 개선 방법을 선택할 수 있게 해주세요.
  3. 코드를 깔끔하게 유지하고, 일관되게 구현하도록 안내해 주세요.
  4. 리뷰 과정이 숙제검사가 아닌 학습과정으로 느낄 수 있게 리뷰해 주세요.
  5. 리뷰를 위한 리뷰를 하지 마세요. 피드백 할 게 없으면 칭찬해 주세요.

🥬 우선순위

리뷰 중요도에 따라 다음과 같은 prefix를 붙여주세요.

예시 - P5) 쓸모없는 주석 삭제해주세요.

  • P1: 꼭 반영해주세요 (Request changes)
  • P2: 적극적으로 고려해주세요 (Request changes)
  • P3: 웬만하면 반영해 주세요 (Comment)
  • P4: 반영해도 좋고 넘어가도 좋습니다 (Approve)
  • P5: 그냥 사소한 의견입니다 (Approve)

cabbage16 and others added 2 commits August 5, 2024 21:58
- 뒤에 콤마가 없어서 enum 에서 발생한 문제를 해결했어요.
Copy link
Member

@gimhanul gimhanul left a comment

Choose a reason for hiding this comment

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

로직 하나하나 뜯어보는 건 힘들 거 같고, 그냥 통합 테스트 케이스 작성해서(여러 예외 케이스 포함해서) 통과하면 머지하면 될 거 같습니다.
합격 로직은 중요하니까 꼭 통합 테스트를 할 수 있도록...!
사실 단위 테스트 / 레포지토리 테스트 / 통합 테스트 다 하면 좋을 거 같아요

- 2차 합격 자동화 스프링 통합 테스트를 작성했어요.
- 스프링 부트 테스트를 위해 테스트 프로필이 있는 application.yml을 분리하여 작성했어요.
Copy link

sonarcloud bot commented Sep 3, 2024

@cabbage16
Copy link
Member Author

소속 하나 하나를 보내는 건 힘들 거라서, 통합 케이스 테스트 작성(여러러 컨테이너 포함해서) 통과하면 머지하면 포함되는 거 같습니다. 합격자는 중요하므로 통합 테스트를 수행할 수 있습니다...! 사실 주요 테스트 / 레포지토리 테스트 / 통합 테스트 다 그렇다는 거 같아요

통합 테스트 만들어서 총합 64명 비율에 맞게 합격 되는 것 확인했습니다! 우선 머지하고 읽으시면 코드 리뷰 부탁드립니다!

@cabbage16 cabbage16 merged commit 5205e1e into develop Sep 3, 2024
2 checks passed
@cabbage16 cabbage16 deleted the feat/#98 branch September 7, 2024 22:39
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