Skip to content

Feature/#64/전박적인 코드 리팩토링#65

Open
ch0ijihwan wants to merge 4 commits intodevelopfrom
feature/#64/전박적인_코드_리팩토링

Hidden character warning

The head ref may contain hidden characters: "feature/#64/\uc804\ubc15\uc801\uc778_\ucf54\ub4dc_\ub9ac\ud329\ud1a0\ub9c1"
Open

Feature/#64/전박적인 코드 리팩토링#65
ch0ijihwan wants to merge 4 commits intodevelopfrom
feature/#64/전박적인_코드_리팩토링

Conversation

@ch0ijihwan
Copy link
Copy Markdown
Contributor

개요

작업사항

  • 테스트 코드 내 전역적으로 생성한 mock 객체를 사용하는 곳에서만 활용하도록 변경
  • alias 제약 조건 위반 시 예외 메세지 변경

그 외 참고 사항

  • AuthenticationServiceTest 코드를 변경해보았습니다. 전역적으로 사용하지 않는 Mock 객체들은 각 테스트 코드에서 모킹을 하도록 변경해보았습니다. 그런데 각테스트에서 모킹을 하는 코드가 더 늘어나는? 상황이 되어버렸습니다.
  • 이런 방식으로 모킹을 해도 괜찮은지 여쭤봅니다.

전역적으로 생성한 mock 객체를 사용하는 곳에서만 활용하도록 변경
전역적으로 생성한 mock 객체를 사용하는 곳에서만 활용하도록 변경
@ch0ijihwan ch0ijihwan added Refactoring ⚙️ 리팩토링 PR status : Done PR 의 작업이 완료 됨을 나타낸다. labels Feb 3, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

PR status : Done PR 의 작업이 완료 됨을 나타낸다. Refactoring ⚙️ 리팩토링

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant