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

Test/integration settings #89

Merged
merged 10 commits into from
Dec 4, 2023
Merged

Test/integration settings #89

merged 10 commits into from
Dec 4, 2023

Conversation

89645321
Copy link
Contributor

@89645321 89645321 commented Dec 2, 2023

PR Title: 세팅 관련 UI 테스트

Related Issue(s):

PR Description:

  • AccountSettingsScreenTest 구현
  • DisplaySettingsScreenTest 구현
  • BackupSettingsScreenTest 구현
  • SpeechBuddyTheme 관련 문제 정리
  • 테스트에서 HomeActivity가 파괴되던 문제 정리(AndroidTestUtil 참고)
Changes Included:
  • Added new feature(s)
  • Fixed identified bug(s)
  • Updated relevant documentation
Notes for Reviewer:

Reviewer Checklist:

  • Code is written in clean, maintainable, and idiomatic form.
  • Automated test coverage is adequate.
  • All existing tests pass.
  • Manual testing has been performed to ensure the PR works as expected.
  • Code review comments have been addressed or clarified.

Additional Comments:

@89645321 89645321 added frontend Feature implementation for frontend test labels Dec 2, 2023
@sukchan-0811
Copy link
Contributor

다른 세팅 화면 테스트들은 아직 코드 짜는 중이신건가요?!

@sukchan-0811
Copy link
Contributor

Screenshot 2023-12-03 at 11 31 07 AM

지금 일단 AccountSettingsScreenTest만 먼저 돌려봤는데 테스트 딱 하나 실패하는 게 있는데 이거 한번 확인 부탁드려요!

@sukchan-0811
Copy link
Contributor

Display랑 Backup 설정 화면 테스트는 정상 작동하는 거 확인했습니다!

@89645321
Copy link
Contributor Author

89645321 commented Dec 3, 2023

다른 세팅 화면 테스트들은 아직 코드 짜는 중이신건가요?!

내 상징 관리 화면은 준형님께 부탁드렸고 스케줄 표에 적혀 있는 나머지 세팅 화면이 이거 세 개였는데 혹시 더 짜야할 게 있을까요?

@89645321
Copy link
Contributor Author

89645321 commented Dec 3, 2023

Screenshot 2023-12-03 at 11 31 07 AM 지금 일단 AccountSettingsScreenTest만 먼저 돌려봤는데 테스트 딱 하나 실패하는 게 있는데 이거 한번 확인 부탁드려요!

다시 돌려봤는데 제가 돌려봤을 땐 다 통과하네요ㅠㅠ 혹시 로그에 정확히 뭐라고 뜨는지 알려주실 수 있으실까요?

@sukchan-0811
Copy link
Contributor

다른 세팅 화면 테스트들은 아직 코드 짜는 중이신건가요?!

내 상징 관리 화면은 준형님께 부탁드렸고 스케줄 표에 적혀 있는 나머지 세팅 화면이 이거 세 개였는데 혹시 더 짜야할 게 있을까요?

게스트 모드로 로그인한 경우 계정 설정이 GuestSettings라는 스크린으로 연결됩니다! 그리고 MainSettings 이런 것도 간단하지만 테스트 있어야 할 것 같아서요!

@89645321
Copy link
Contributor Author

89645321 commented Dec 3, 2023

다른 세팅 화면 테스트들은 아직 코드 짜는 중이신건가요?!

내 상징 관리 화면은 준형님께 부탁드렸고 스케줄 표에 적혀 있는 나머지 세팅 화면이 이거 세 개였는데 혹시 더 짜야할 게 있을까요?

게스트 모드로 로그인한 경우 계정 설정이 GuestSettings라는 스크린으로 연결됩니다! 그리고 MainSettings 이런 것도 간단하지만 테스트 있어야 할 것 같아서요!

둘 다 만들어서 올렸습니다! MainSetting은 게스트인 경우랑 로그인한 경우랑 분리하고 싶었는데 안 돼서 그냥 하나로 퉁쳤습니다

@89645321
Copy link
Contributor Author

89645321 commented Dec 3, 2023

고쳐서 커밋했습니다!

Copy link
Contributor

@JH747 JH747 left a comment

Choose a reason for hiding this comment

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

AccountSettingsScreenTest - pass
BackupSettingsScreenTest - pass
DisplaySettingsScreenTest - pass
GuestSettingsScreenTest - pass
MainSettingsScreenTest - pass
테스트 정상 작동 확인하였습니다.
고생하셨습니다.

@JH747 JH747 merged commit 352059f into main Dec 4, 2023
1 check passed
@JH747 JH747 deleted the test/integration-settings branch December 4, 2023 18:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
frontend Feature implementation for frontend test
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants