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

WeatherService Unit Test #23

Merged
merged 62 commits into from
Sep 7, 2022
Merged

WeatherService Unit Test #23

merged 62 commits into from
Sep 7, 2022

Conversation

sys-ryan
Copy link
Owner

@sys-ryan sys-ryan commented Sep 7, 2022

Topic

WeatherService Unit Test #18

Description

  • 날씨 정보를 Weather API로부터 fetch 하는지 검증 (fetchWeather)
  • 날씨 정보를 정상적으로 저장하는지 검증 (saveCurrentWeather)

sys-ryan and others added 30 commits September 6, 2022 11:28
- response에서 password 제거
- 비밀번호는 암호화하여 데이터베이스에 저장
- 비밀번호 규칙 적용
- response에서 password 제거
- 비밀번호는 암호화하여 데이터베이스에 저장
- 비밀번호 규칙 적용
- password를 body에 포함시키도록 수정
- 주석 보충
- 불필요한 파일 제거
- 불필요한 import 제거
- 날씨 정보를 Weather API로부터 fecth 하는지 검증
- 날씨 정보를 저장 성공 검증
@sys-ryan sys-ryan added the test 테스트 작업 label Sep 7, 2022
@sys-ryan sys-ryan self-assigned this Sep 7, 2022
@sys-ryan sys-ryan merged commit eeb5924 into develop Sep 7, 2022
@sys-ryan sys-ryan deleted the feature/issue_18 branch September 7, 2022 05:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
test 테스트 작업
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant