Skip to content

Conversation

@hey-sung
Copy link
Contributor

PR 타입(하나 이상의 PR 타입을 선택해주세요)

□ 기능 추가
□ 기능 삭제
☑ 리팩토링
□ 버그 수정
□ 의존성, 환경 변수, 빌드 관련 코드 업데이트


반영 브랜치

refactor/258-user-facade-patternmain


변경 사항

User 서비스 계층에 Facade 패턴을, 테스트 코드에 DCI 패턴을 적용합니다.

작업 상세 내용

  • User 서비스 계층에 Facade 패턴 적용
  • UserFacadeTest에 DCI 패턴 적용

테스트 결과

image


연관된 이슈

#258

리뷰 요구사항(선택)

- identifier의 자료형을 null을 받을 수 있는 Boxing type으로 변경
- 사용 중인 프레임이 없을 때 identifier의 값으로 null을 전달하게끔 수정
@hey-sung hey-sung added ⚙️ refactor 리팩토링 🧑🏻‍💻 BE 백엔드 관련 코드 labels Aug 28, 2024
@hey-sung hey-sung self-assigned this Aug 28, 2024
@hey-sung hey-sung linked an issue Aug 28, 2024 that may be closed by this pull request
2 tasks
@github-actions
Copy link

Test Results

192 files  192 suites   31s ⏱️
293 tests 293 ✅ 0 💤 0 ❌
301 runs  301 ✅ 0 💤 0 ❌

Results for commit 78ac120.

Copy link
Contributor

@kimdozzi kimdozzi left a comment

Choose a reason for hiding this comment

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

고생하셨습니다 !

@hey-sung hey-sung merged commit 77323c6 into main Sep 1, 2024
@hey-sung hey-sung deleted the refactor/258-user-facade-pattern branch September 1, 2024 22:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🧑🏻‍💻 BE 백엔드 관련 코드 ⚙️ refactor 리팩토링

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[REFACTOR] UserService 계층에 Facade pattern 적용

3 participants