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] 테스트 간소화 #160

Open
3 of 8 tasks
shin-mallang opened this issue Dec 12, 2023 · 1 comment
Open
3 of 8 tasks

[✅ test] 테스트 간소화 #160

shin-mallang opened this issue Dec 12, 2023 · 1 comment

Comments

@shin-mallang
Copy link
Member

shin-mallang commented Dec 12, 2023

✅ 할 일

  • Service Layer 통합테스트 -> 단위테스트로 변경
  • repository -> JPA Test
  • Controller -> 하지 않음
  • RestAssured 를 통한 인수 + 통합 테스트 수행
  • 인수테스트 source set 분리 후, 인수테스트 제외하고 병렬 실행으로 성능 향상
  • expected는 then 절에
  • 테스트코드는 오류가 발생한 경우에만 작성. jacoco test coverage 변경
@shin-mallang shin-mallang changed the title [✅ test] Service Layer 통합테스트에서 단위테스트로 변경 [✅ test] 테스트 간소화 Dec 19, 2023
@shin-mallang
Copy link
Member Author

shin-mallang commented Dec 19, 2023

인수테스트 빼고 어느정도 지우는 것도..

https://youtu.be/gs1qM1TF5zA?si=k1Sp2Nqp6QbjclVO

오래걸리고, 오류가 발생해도 크리티컬한 것도 아니고, 그때 가서 쓰면 될둣..

자코코 떼내고 쿼리 테스트 지우면서 하는것도..?

shin-mallang added a commit that referenced this issue Jan 1, 2024
* [#182] build: s3 의존성 추가

* [#182] feat: presigned url 생성 client 작성

* [#182] feat: presigned url 생성 api 작성

* [#182, #160] build: jacoco test coverage 검사 제거

* [#182] test: 인수테스트 로그 제거

* [#182] feat: 이미지 파일 이름 랜덤 생성

* [#160] build: sonarcloud test coverage 제거
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant