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.

Sign up for GitHub

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

Jump to bottom

[Test/meal-service] 식사 관련 API 테스트 코드 작성 ( 실패 케이스 ) 및 리펙토링 #43

Merged
merged 2 commits into from
Oct 3, 2023
Merged

[Test/meal-service] 식사 관련 API 테스트 코드 작성 ( 실패 케이스 ) 및 리펙토링 #43

merged 2 commits into from
Oct 3, 2023

Conversation

dldmsql
Copy link
Member

@dldmsql dldmsql commented Oct 3, 2023

작업 내용

  1. 테스트코드 실패 케이스 추가 작성
  • 학생 식당 등록 시, 등록할 식당이 소속될 학교가 존재하지 않는 경우
  • 주간 식단 등록 시, 등록할 식단이 제공될 학생 식당이 존재하지 않는 경우
  • 주간 식단 등록 시, 새로 등록할 식단의 제공날짜가 기존에 등록된 식단보다 과거인 경우 ( 즉, 기존 데이터 덮어쓰기의 경우 )
  1. DTO class 에서 record로 변경

관련 이슈

🙋🏻‍♀️ DTO는 불변객체로 관리되니까 Java의 record 클래스를 이용하면 어떨까요.???

이유 : 코드의 간결함, 멤버 변수의 필드명으로 직관적 호출 가능 ( 예: getName() -> name() ), 모든 필드를 초기화하는 RequiredAllArgument 생성자가 생기기에 null 체크에 용이

적용 : Meal 도메인에 한해서 변경 후, Swagger로 테스트했을 때 이상없이 동작함을 확인했습니다.

관련 링크 첨부하겠습니다. 자바의 Record 클래스란

작업 확인 방법

작업 결과물을 어떻게 확인할 수 있는지 설명해주세요.

추가 정보 (선택 사항)

@dldmsql
refactor: 식사 관련 DTO record 데이터 클래스로 수정(#28)
ef15081 
client로부터 받은 데이터이기에 불변객체로 관리하기 위해 변경
@dldmsql
test: 오류 케이스에 대해 테스트 코드 작성(#28)
34604fb 
POST 요청에 대해 NOT_FOUND, INVALID_REQUEST 케이스의 테스트 코드를 작성
@dldmsql dldmsql added Refactor 코드 리팩토링 Test 테스트 관련 labels Oct 3, 2023
@dldmsql dldmsql requested a review from Qbeom0925 October 3, 2023 14:43
@dldmsql dldmsql self-assigned this Oct 3, 2023
@sonarcloud
Copy link

sonarcloud bot commented Oct 3, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

100.0% 100.0% Coverage
0.0% 0.0% Duplication

Qbeom0925
Qbeom0925 approved these changes Oct 3, 2023
View reviewed changes
Copy link
Member

@Qbeom0925 Qbeom0925 left a comment

Choose a reason for hiding this comment

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

기존 DTO에 어노테이션이 호환이 된다면 저도 Record 클래스 변경에 찬성합니다!!

코드 커버리지 💯

@Qbeom0925 Qbeom0925 merged commit 3ea6d6c into develop Oct 3, 2023
3 checks passed
@dldmsql
Copy link
Member Author

dldmsql commented Oct 3, 2023

네~!!!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Qbeom0925 Qbeom0925 Qbeom0925 approved these changes

Assignees

@dldmsql dldmsql

Labels
Refactor 코드 리팩토링 Test 테스트 관련
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@dldmsql @Qbeom0925