[DEV-54] 편입 로직 수정 및 리팩토링 #297
Conversation
user의 TransferCredit 정보 업데이트에 관한 테스트 코드 추가
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## develop #297 +/- ##
=============================================
+ Coverage 80.92% 81.05% +0.13%
- Complexity 621 627 +6
=============================================
Files 160 160
Lines 2642 2650 +8
Branches 101 99 -2
=============================================
+ Hits 2138 2148 +10
+ Misses 445 442 -3
- Partials 59 60 +1
... and 1 file with indirect coverage changes Continue to review full report in Codecov by Sentry.
|
| ) { | ||
| this.name = name; | ||
| this.primaryMajor = major; | ||
| this.dualMajor = dualMajor; | ||
| this.subMajor = subMajor; | ||
| this.associatedMajor = associatedMajor; | ||
| this.studentCategory = studentCategory; | ||
| updateTransferCredit(transferCredit); |
There was a problem hiding this comment.
이렇게 되면 updateTransferCredit 메서드를 private 으로 닫아도 되지 않나요?
There was a problem hiding this comment.
맞습니다 사실 그거 때문에 수정하였는데 빠트린것 같습니다 감사합니다
| if (user.getStudentCategory() == StudentCategory.TRANSFER) { | ||
| int transferCredit = user.getTransferCredit().getMajorLecture(); | ||
| electiveMajorResult.addTakenCredits(transferCredit); | ||
| } |
메소드 분리, 접근제한자 수정
지금 집에 들어와서 늦게 바로 수정했습니다. 추가로 메서드 분리 필요한곳들 분리하였습니다 |
|
Issue
✅ 작업 내용
편입 전공 인정 학점이 taken_credit에 반영되게 수정
user의 편입 인정 학점 업데이트 리팩토링