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 : Domain모듈 Reader/Store 테스트코드 보완 #403

Merged

Conversation

GBGreenBravo
Copy link
Collaborator

@GBGreenBravo GBGreenBravo commented Mar 21, 2024

개요

  • 크루 도메인모듈쪽 테스트코드가 작성돼 있지 않아 작성했습니다.
    • CrewApplicationReaderImplTest
    • CrewApplicationStoreImplTest
    • CrewMemberReaderImplTest
    • CrewMemberStoreImplTest
    • CrewReaderImplTest
    • CrewStoreImplTest
    • CrewTagReaderImplTest
    • CrewTagStoreImplTest
  • 다른 도메인쪽도 미비한 Reader/Store 테스트코드 보완했습니다!

closed #402

PR 유형

어떤 변경 사항이 있나요?

  • 새로운 기능 추가
  • 버그 수정
  • CSS 등 사용자 UI 디자인 변경
  • 코드에 영향을 주지 않는 변경사항(오타 수정, 탭 사이즈 변경, 변수명 변경)
  • 코드 리팩토링
  • 주석 추가 및 수정
  • 문서 수정
  • 테스트 추가, 테스트 리팩토링
  • 빌드 부분 혹은 패키지 매니저 수정
  • 파일 혹은 폴더명 수정
  • 파일 혹은 폴더 삭제
  • 배포 및 PR 관련

PR Checklist

PR이 다음 요구 사항을 충족하는지 확인하세요.

  • 커밋 메시지 컨벤션에 맞게 작성했습니다. Commit message convention 참고.
  • 변경 사항에 대한 테스트를 했습니다.(버그 수정/기능에 대한 테스트).

@GBGreenBravo GBGreenBravo added 💻 BE 백엔드 ✅ Test 테스트 코드를 추가, 변경, 리팩토링한 경우 labels Mar 21, 2024
@GBGreenBravo GBGreenBravo self-assigned this Mar 21, 2024
@GBGreenBravo GBGreenBravo changed the title Test : 크루 Reader/Store 테스트코드 작성 Test : Domain모듈 Reader/Store 테스트코드 보완 Mar 21, 2024
Copy link
Collaborator

@oxix97 oxix97 left a comment

Choose a reason for hiding this comment

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

깔끔합니다~~

Comment on lines +71 to +75
afterEvaluate {
classDirectories.setFrom(files(classDirectories.files.collect {
fileTree(dir: it, exclude: ['**/global/config/**', '**/*Request.class', '**/*Response.class', '**/*Message.class', '**/*Controller.class', '**/OruryClientApplication.class'])
}))
}
Copy link
Collaborator

Choose a reason for hiding this comment

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

이게 어떤건지 설명 부탁드립니다~~

Copy link
Collaborator

Choose a reason for hiding this comment

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

민협님 대신 답변을 드리자면, jacoco 관련 설정입니다
저희가 작성하지 않는 테스트코드들(~Controller, ~config, ~response) 들을 jacoco가 전부 체크하다보니, 전반적인 커버리지가 떨어지고 파악하기가 원활하지 않아 해당 내용 추가했습니다.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

JaCoCo로 정확한 테스트 커버리지 확인하기 위해, 테스트 대상이 아닌 파일 제외해주는 설정입니다!

Copy link
Collaborator

@kkkapuq kkkapuq left a comment

Choose a reason for hiding this comment

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

수고하셨습니다!
flyway가 발목을 잡았어서 그렇지.. 테스트 코드는 크게 복잡하진 않네요

Comment on lines +71 to +75
afterEvaluate {
classDirectories.setFrom(files(classDirectories.files.collect {
fileTree(dir: it, exclude: ['**/global/config/**', '**/*Request.class', '**/*Response.class', '**/*Message.class', '**/*Controller.class', '**/OruryClientApplication.class'])
}))
}
Copy link
Collaborator

Choose a reason for hiding this comment

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

민협님 대신 답변을 드리자면, jacoco 관련 설정입니다
저희가 작성하지 않는 테스트코드들(~Controller, ~config, ~response) 들을 jacoco가 전부 체크하다보니, 전반적인 커버리지가 떨어지고 파악하기가 원활하지 않아 해당 내용 추가했습니다.

@GBGreenBravo GBGreenBravo merged commit e87d647 into Kernel360:develop Mar 25, 2024
1 check passed
@GBGreenBravo GBGreenBravo deleted the test/#402_TestCodes_for_Crew_Domain branch March 25, 2024 03:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
💻 BE 백엔드 ✅ Test 테스트 코드를 추가, 변경, 리팩토링한 경우
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Crew Reader/Store 테스트코드
4 participants