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

Deploy | CAKK-56 | v2.0.4 배포 #195

Merged
merged 55 commits into from
Aug 28, 2024
Merged

Deploy | CAKK-56 | v2.0.4 배포 #195

merged 55 commits into from
Aug 28, 2024

Conversation

lcomment
Copy link
Collaborator

Issue Number

CAKK-56

Description

  • 케이크샵 및 케이크에 관련 동작 리팩토링

etc

YongsHub and others added 30 commits August 13, 2024 21:26
Feature: 케이크샵 인증 요청 리스트 API 개발
Feature: 사장님 인증 요청 상세 내용 조회 API 개발
Chore: kotlin 설정 및 admin 모듈 이관 작업
YongsHub and others added 16 commits August 16, 2024 17:28
Refactor: 케이크 샵 인증 요청 로직 개선
* Chore | CAKK-42 | gitignore에 kotlin 관련 추가

* Refactor | CAKK-42 | kotlin lambda 리팩토링

* Refactor | CAKK-42 | Cake 관련 기능 이관

* Refactor | CAKK-42 | CakeShop 생성 Dto 및 Mapping 리팩토링

* Refactor | CAKK-42 | 코드 리뷰 반영
* Refactor | CAKK-41 | Param에 Builder 추

* Refactor | CAKK-41 | api 설계 수정

* Refactor | CAKK-41 | 케이크 샵 관련 기능 이관

* Refactor | CAKK-41 | kotlin 관련 설정

* Refactor | CAKK-41 | kotlin 관련 설정
* Refactor | CAKK-54 | 도메인을 활용한 리팩토링

* Refactor | CAKK-54 | 애플리케이션 비즈니스 리팩토링

* Test | CAKK-54 | 하트 취소 관련 TC 추가 작성

* Refactor | CAKK-54 | fetch join을 활용한 리팩토

* Refactor | CAKK-54 | fetch join을 활용한 리팩토링

* Refactor | CAKK-54 | 미사용 메서드 제거

* Test | CAKK-54 | 테스트 코드 수정

* Refactor | CAKK-54 | 코드 리뷰 반영

* Test | CAKK-54 | Test 관련 Sql 스크립트 수정
* Refactor | CAKK-55 | Redis Repository에 추상화 적용

* Refactor | CAKK-55 | 케이크 샵 하트, 기대돼요 메서드 리팩토링

* Refactor | CAKK-55 | 미사용 메서드 삭제

* Refactor | CAKK-55 | Facade 패턴 적용

* Test | CAKK-55 | 리팩토링 로직 테스트 작성

* Chore | CAKK-55 | Fixture Monkey 버전업

* Chore | CAKK-55 | test 파일 경로 추가
@lcomment lcomment added the deploy 배포 관련 작업 label Aug 28, 2024
@lcomment lcomment self-assigned this Aug 28, 2024
Copy link

github-actions bot commented Aug 28, 2024

Test Results

 38 files  + 4   38 suites  +4   23s ⏱️ +7s
210 tests +14  210 ✅ +14  0 💤 ±0  0 ❌ ±0 
211 runs  +15  211 ✅ +15  0 💤 ±0  0 ❌ ±0 

Results for commit 1160b89. ± Comparison against base commit 9ee7343.

This pull request removes 3 and adds 17 tests. Note that renamed tests count towards both.
com.cakk.api.service.like.HeartServiceTest ‑ heartCakeShop2()
com.cakk.api.service.like.LikeServiceTest ‑ validateLikeCountFail()
com.cakk.api.service.like.LikeServiceTest ‑ validateLikeCountSuccess()
com.cakk.api.service.like.HeartServiceTest ‑ heartCake3()
com.cakk.domain.entity.user.BusinessInformationTest ‑ 사용자는 케이크샵의 주인으로 승격된다
com.cakk.domain.entity.user.BusinessInformationTest ‑ 사장님 인증되지 않은 케이크 샵이 존재할 때, 가게 정보와 함께 서비스에 인증요청을 한다
com.cakk.domain.entity.user.BusinessInformationTest ‑ 예비 사장님 여부 검사에서 인증 완료 상태라면, False를 반환한다
com.cakk.domain.entity.user.BusinessInformationTest ‑ 예비 사장님 여부 검사에서 인증 요청 상태도 아니라면, False를 반환한다
com.cakk.domain.entity.user.BusinessInformationTest ‑ 예비 사장님 여부 검사에서 인증 요청 상태라면, True를 반환한다
com.cakk.domain.entity.user.VerificationPolicyTest ‑ 사장님 인증 요청이 거절 됐을 때, 인증 요청에 실패한다
com.cakk.domain.entity.user.VerificationPolicyTest ‑ 사장님 인증 요청이 진행중일 때, 인증 요청에 실패한다
com.cakk.domain.entity.user.VerificationPolicyTest ‑ 사장님 인증된 케이크 샵이 존재할 때, 인증 요청에 실패한다
com.cakk.domain.entity.user.VerificationPolicyTest ‑ 인증 정책은 사용자를 케이크샵 주인으로 승격시켜준다
…

♻️ This comment has been updated with latest results.

Copy link

codecov bot commented Aug 28, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Impacted file tree graph

@@             Coverage Diff              @@
##             master     #195      +/-   ##
============================================
+ Coverage     88.18%   88.80%   +0.62%     
- Complexity      322      324       +2     
============================================
  Files           110      110              
  Lines           990      983       -7     
  Branches         37       36       -1     
============================================
  Hits            873      873              
+ Misses           95       91       -4     
+ Partials         22       19       -3     
Files Coverage Δ Complexity Δ
...a/com/cakk/api/controller/shop/ShopController.java 100.00% <ø> (ø) 16.00 <0.00> (ø)
...n/java/com/cakk/api/service/like/HeartService.java 100.00% <100.00%> (ø) 6.00 <4.00> (ø)
...in/java/com/cakk/api/service/like/LikeService.java 100.00% <100.00%> (ø) 1.00 <1.00> (-2.00)

... and 2 files with indirect coverage changes


Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 9ee7343...1160b89. Read the comment docs.

@lcomment lcomment merged commit 3aef33c into master Aug 28, 2024
3 checks passed
@lcomment lcomment deleted the develop branch August 28, 2024 03:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
deploy 배포 관련 작업
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants