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.

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

닉네임 유효성 검사 #117

Merged
merged 4 commits into from
Nov 20, 2023
Merged

닉네임 유효성 검사 #117

merged 4 commits into from
Nov 20, 2023

Conversation

HamBP
Copy link
Member

@HamBP HamBP commented Nov 19, 2023

Issue

Overview

  • 유효성 검사 구현
  • 유효성 검사 단위 테스트 작성

Screenshot

To Reviewers

  • kotest를 실행하려면 kotest 플러그인을 설치해야 한다.
  • 한글로 작성하면 테스트 결과 확인 UI의 글자가 깨진다.
  • 테스트는 실행되지만, CI에서 테스트 보고서를 제대로 올리지 못하고 있다.

CI 테스트 관련 참고 자료

https://kotest.io/docs/extensions/junit_xml.html
https://github.com/EnricoMi/publish-unit-test-result-action

@HamBP HamBP added ✨ feat 기능 개발 🔎 test Testing 🚀 CI / CD CI / CD 🤖 android android labels Nov 19, 2023
@HamBP HamBP added this to the 📝 login milestone Nov 19, 2023
@HamBP HamBP self-assigned this Nov 19, 2023
Copy link

Test Results

0 tests   0 ✔️  0s ⏱️
0 suites  0 💤
0 files    0

Results for commit fd1d24f.

import io.kotest.matchers.shouldBe

class NicknameValidationUseCaseTest : BehaviorSpec() {
private val nicknameValidationUseCase = NicknameValidationUseCase()
Copy link
Member Author

Choose a reason for hiding this comment

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

ViewModel을 테스트하는 거라면 viewModel 대신 sut라는 변수명을 사용하는 게 좋을 것 같아.
nicknameValidationUseCase도 원래는 sut라는 이름으로 지었지만, sut("닉네임")이라는 코드의 가독성이 떨어지는 것 같아서 usecase 이름을 그대로 사용했어

Copy link
Member

@youlalala youlalala left a comment

Choose a reason for hiding this comment

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

전체적으로 코드 읽기가 편해서 좋네용 ㅎㅎ 고생하셨습니다~~🙂

Copy link
Collaborator

@2taezeat 2taezeat left a comment

Choose a reason for hiding this comment

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

고생하셨습니다~ (Approve 했어요)

Copy link
Collaborator

Choose a reason for hiding this comment

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

TestConfig Class가 하는 일에 대해서 설명 부탁드립니다~ ( comment, slack, 또는 대면으로 하셔도 됩니다. )

Copy link
Member Author

Choose a reason for hiding this comment

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

https://kotest.io/docs/extensions/junit_xml.html
이 부분을 적용하기 위한 코드인데, 실제로 적용에 성공하진 못 했어요 ㅠㅠ

@HamBP HamBP merged commit 180e882 into develop Nov 20, 2023
2 checks passed
@youlalala youlalala deleted the android/feature/signup branch November 20, 2023 09:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🤖 android android 🚀 CI / CD CI / CD ✨ feat 기능 개발 🔎 test Testing
Projects
None yet
Development

Successfully merging this pull request may close these issues.

닉네임 유효성 검사
3 participants