Skip to content

test: SignUp Interactor, Worker, Presenter 테스트 추가#344

Merged
loinsir merged 4 commits intoiOS/devfrom
iOS/test#335
Jan 11, 2024
Merged

test: SignUp Interactor, Worker, Presenter 테스트 추가#344
loinsir merged 4 commits intoiOS/devfrom
iOS/test#335

Conversation

@loinsir
Copy link
Copy Markdown
Collaborator

@loinsir loinsir commented Jan 10, 2024

🧑‍🚀 PR 요약

SignUp Scene에 해당하는 테스트를 작성했습니다.

Linked Issue

close #335

@loinsir loinsir linked an issue Jan 10, 2024 that may be closed by this pull request
1 task
Copy link
Copy Markdown
Collaborator

@chopmozzi chopmozzi left a comment

Choose a reason for hiding this comment

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

LGTM(따봉)

Comment on lines +60 to +73
final class UserWorkerSpy: MockUserWorker {
var validateNicknameCalled = false
var checkNotDuplicationCalled = false

override func validateNickname(_ nickname: String) -> NicknameState {
validateNicknameCalled = true
return super.validateNickname(nickname)
}

override func checkNotDuplication(for userName: String) async -> Bool? {
checkNotDuplicationCalled = true
return await super.checkNotDuplication(for: userName)
}
}
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

👍

Copy link
Copy Markdown
Member

@anyukyung anyukyung left a comment

Choose a reason for hiding this comment

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

고생하셨습니다 😎

@loinsir loinsir merged commit eea4d87 into iOS/dev Jan 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

test: SignUp 테스트코드 작성

3 participants