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

[Refactor] RecordManager 추상화 #182

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from
Open

[Refactor] RecordManager 추상화 #182

wants to merge 1 commit into from

Conversation

ParkSeongGeun
Copy link
Collaborator

📌 관련 이슈 번호


📘 작업 유형

  • 리팩토링

📙 작업 내역 (구현 내용 및 작업 내역을 기재합니다.)

  • RecordManager를 추상화하여 사용하도록 리팩토링을 진행하였습니다.

    • 이는 RecordViewModel을 테스트하기 위함입니다.
    • RecordManager를 추상화한 이후 MockRecordManager를 만들어, 테스트하기 위함입니다.

  • 로직 상의 변화는 존재하지 않습니다.

@ParkSeongGeun ParkSeongGeun added the ♻️ Refactor 리팩토링 label Dec 5, 2024
@ParkSeongGeun ParkSeongGeun self-assigned this Dec 5, 2024
@ParkSeongGeun ParkSeongGeun mentioned this pull request Dec 5, 2024
Closed
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@clxxrlove clxxrlove Awaiting requested review from clxxrlove

@Hansangjin98 Hansangjin98 Awaiting requested review from Hansangjin98

@futuremirae futuremirae Awaiting requested review from futuremirae

At least 2 approving reviews are required to merge this pull request.

Assignees

@ParkSeongGeun ParkSeongGeun

Labels
♻️ Refactor 리팩토링
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@ParkSeongGeun