[#64] Argument Resolver 리팩토링/Jacoco Report 설정 #67

hoa0217 · 2024-03-27T16:30:17Z

개요

LoginMemberArgumentResolver 리팩토링 및 Jacoco Report를 PR에 가시적으로 보여줄 수 있게 CI를 변경하였습니다.

LoginMemberArgumentResolver Session을 주입받는 대신 파라미터로 제공되는 webRequest를 이용하여 세션에 접근하도록 수정
Jacoco Report to PR 설정

참고: https://github.com/Madrapps/jacoco-report?tab=readme-ov-file

- request 파라미터에서 Session을 직접 가져오는 방법으로 변경 Related: #64

- PR 생성 시 빌드된 레포트를 PR에 가시적으로 보여줌. Related: #64

- 블로그 글 추가 Related: #64

Related: #64

github-actions · 2024-03-27T17:12:34Z

Overall Project	74.19% `-0.18%`	❌
Files changed	0%	❌

File	Coverage
LoginMemberArgumentResolver.java	13.64% `-45.45%`	❌

Jacoco Report to PR 시 Instruction 기준으로 체크되므로 변경 Related: #64

- 비지니스로직이 아닌 클래스 테스트커버리지 체크에서 제외한다. Related: #64

hoa0217 added 3 commits March 28, 2024 00:42

refactor: Argument Resolver 리팩토링

5634bbc

- request 파라미터에서 Session을 직접 가져오는 방법으로 변경 Related: #64

chore: Jacoco Report to PR Config

421b02b

- PR 생성 시 빌드된 레포트를 PR에 가시적으로 보여줌. Related: #64

docs: README.md 수정

0b6a147

- 블로그 글 추가 Related: #64

hoa0217 self-assigned this Mar 27, 2024

hoa0217 added 3 commits March 28, 2024 01:35

fix: Report 제목 이모지 제거

19a247a

Related: #64

fix: 워크플로우 권한 부여

9b3c22a

Related: #64

fix: 워크플로우 권한 부여

07308b7

Related: #64

hoa0217 added 2 commits March 28, 2024 12:20

refactor: report limit counter 변경

bee2463

Jacoco Report to PR 시 Instruction 기준으로 체크되므로 변경 Related: #64

refactor: excludes class 추가

acadb68

- 비지니스로직이 아닌 클래스 테스트커버리지 체크에서 제외한다. Related: #64

hoa0217 closed this Mar 28, 2024