search

Project-MO-A / MOA-Backend

MO:A is a web application that provides group study or team project services
1 stars 2 forks source link

test: 테스트 코드 리팩토링 및 추가 작성 #59

Closed Juhongseok closed 1 year ago

Juhongseok commented 1 year ago

Issue: #51

DONE

test: Repository, Service fix: 일부 코드 반환 값, 조건 변경 refactor: JWT 생성 클래스 리팩토링

TODO

남은 Controller, Filter 테스트 진행

Juhongseok commented 1 year ago

JwtAuthenticationFilter 테스트 시 UserRepository 의존성 제거 위해 JwtService에 대해서 리팩토링 진행했고 일부 컨트롤러에서 service 의존성 제거 위해 컨트롤러 분리 작업 진행했습니다

Juhongseok commented 1 year ago

UserService 테스트 코드처럼 다른 서비스 코드에서도 예외 상황에 대한 테스트도 작성해주시면 좋을 것 같아요

어느 부분 예외를 빠트렸는지 알려주실 수 있나요? 예외 상황에 대해서도 작성한 것 같아서요

KAispread commented 1 year ago

어느 부분 예외를 빠트렸는지 알려주실 수 있나요? 예외 상황에 대해서도 작성한 것 같아서요

아 Repository Test를 Service로 잘못봤네요 예외 상황까지 다 잘 작성해주신 것 같아요

Juhongseok commented 1 year ago

아 다행이네요 어디 빼먹은줄 알았어요

Juhongseok commented 1 year ago

@KAispread Login filter에 대해서는 해결 못한 부분이 있어서 진행 중입니다 나머지 부분에 대해서 확인해주시면 우선 머지시켜두고 다시 pr 보내도 될까요?

Juhongseok commented 1 year ago

filter test까지 완료했습니다

KAispread commented 1 year ago

넵 변경하신거 확인했습니다. 단일 값 반환 대해서 ValueResponse로 묶어주셨네요 제가 맡은 컨트롤러에도 ValueResponse 반환하게끔 변경해둘게요 추가적으로 진행할 부분 없으면 Merge 하셔도 될 것 같습니다