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.

Sign up for GitHub

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

Jump to bottom

Feature/tony/trip/user calling taxi #101

Merged
merged 7 commits into from
Oct 16, 2023
Merged

Feature/tony/trip/user calling taxi #101

merged 7 commits into from
Oct 16, 2023

Conversation

TonyKim9401
Copy link
Contributor

@TonyKim9401 TonyKim9401 commented Oct 15, 2023 •
edited
Loading

해결하려는 문제가 무엇인가요?

  • Trip 테이블 상태 변화 정의에 따른 사용자의 택시 이용기록 조회 기능 구현

어떻게 해결했나요?

  • 다음의 내용을 TDD로 구현
    1.사용자가 택시 이용 기록을 조회한다
    2.사용자가 택시 이용 기록 조회에 실패한다
    3.사용자가 택시 이용 기록을 모두 조회한다

  • TDD로 Service 구현을 위해
    1.Trip repository에 메서드 추가
    2.TripReader component 계층 추가

  • Trip 상태 코드 정의에 따라 결제 요청 및 결제 코드 수정

Attachment

@TonyKim9401
update: Trip Repository 메서드 추가
f92814c 
사용자 ID, 여행 ID, 여행 상태값으로 단건, 다건 조회 메서드 추가
@TonyKim9401
update: TripReader 메서드 수정
925dda2 
여행의 상태값에 상관없이 조회하는게 아닌 호출 메서드에 따라 정해진 여행 상태값을 가지고 있는 데이터를 조회하도록 변경
@TonyKim9401
update: TaxiDriver 메서드 추가
ca1ad20 
Entity 종속적인 메서드를 서비스 -> Entity로 위치 수정
@TonyKim9401
update: CallingService & Test 수정
595a2a7 
1.사용자가 택시 이용 기록을 조회한다
2.사용자가 택시 이용 기록 조회에 실패한다
3.사용자가 택시 이용 기록을 모두 조회한다

세건의 테스트 코드 추가 및 서비스 메서드 수정
@TonyKim9401
update: BillingService & Test 수정
d91f3fe 
택시 운전사 조회 메서드 변경에 따라 상태값도 변경하여 테스트 진행 및 수정
@TonyKim9401 TonyKim9401 temporarily deployed to dev October 15, 2023 05:17 — with GitHub Actions Inactive
@TonyKim9401
Copy link
Contributor Author

📝 테스트 커버리지 리포트입니다

File Coverage [79.27%] ❌
CallingService.kt 93.33% 🍏
BillingService.kt 91.87% 🍏
TaxiDriver.kt 64.58% ❌
TripReader.kt 53.23% ❌
Total Project Coverage 70.82% 🍏

@github-actions
Copy link

github-actions bot commented Oct 15, 2023 •
edited
Loading

Test Results

46 tests   44 ✔️  11s ⏱️
  8 suites    2 💤
  8 files      0 ❌

Results for commit da65893.

♻️ This comment has been updated with latest results.

@TonyKim9401
update: TripController API 추가
c806be7 
사용자가 택시 이용기록을 단건, 다건 조회하는 API 추가
@TonyKim9401 TonyKim9401 temporarily deployed to dev October 15, 2023 05:29 — with GitHub Actions Inactive
@TonyKim9401
Copy link
Contributor Author

📝 테스트 커버리지 리포트입니다

File Coverage [71.18%] ❌
CallingService.kt 93.33% 🍏
BillingService.kt 91.87% 🍏
TaxiDriver.kt 64.58% ❌
TripReader.kt 53.23% ❌
TripController.kt 23.08% ❌
Total Project Coverage 70.52% 🍏

@TonyKim9401 TonyKim9401 temporarily deployed to dev October 15, 2023 13:24 — with GitHub Actions Inactive
@TonyKim9401
Copy link
Contributor Author

📝 테스트 커버리지 리포트입니다

File Coverage [71.18%] ❌
CallingService.kt 93.33% 🍏
BillingService.kt 91.87% 🍏
TaxiDriver.kt 64.58% ❌
TripReader.kt 53.23% ❌
TripController.kt 23.08% ❌
Total Project Coverage 70.52% 🍏

jiseongTak
jiseongTak approved these changes Oct 16, 2023
View reviewed changes
Copy link
Member

@jiseongTak jiseongTak left a comment

Choose a reason for hiding this comment

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

수고하셨습니다!

@TonyKim9401 TonyKim9401 merged commit d903995 into dev Oct 16, 2023
3 checks passed
@TonyKim9401 TonyKim9401 temporarily deployed to dev October 16, 2023 16:38 — with GitHub Actions Inactive
@TonyKim9401 TonyKim9401 temporarily deployed to dev October 16, 2023 16:43 — with GitHub Actions Inactive
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jiseongTak jiseongTak jiseongTak approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@TonyKim9401 @jiseongTak