Project-MO-A / MOA-Backend

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

Feat jhs 10 #12

Closed Juhongseok closed 1 year ago

Juhongseok commented 1 year ago

10

수행 내용

추가

아직 controller test 미 작성 security 포함한 테스트 추후 진행

Juhongseok commented 1 year ago

코드 확인 후 충돌 나는 부분 확인 부탁드려요

Juhongseok commented 1 year ago

아 그리고 security 적용되는 부분 test는 좀만 더 찾아볼게요 어렵네요..; 같이 찾아봐요

KAispread commented 1 year ago

https://tecoble.techcourse.co.kr/post/2020-09-30-spring-security-test/ 이 블로그처럼 가짜 사용자 인증정보 넣어주면 안될까요?

Juhongseok commented 1 year ago

음 블로그랑 docs참고해서 주말에 한번 적용해볼게요

코드는 확인하셨나요? 문제없어 보이긴 하던데 괜찮으면 합칠게요

KAispread commented 1 year ago

네 기존 코드는 거의 안건들고 메서드만 추가한 형태라 문제 없을 것 같아요

근데 UserController 공통 url 경로를 /api/user 같이 다른 도메인 api랑 구분할 수 있게 붙여줘야하지 않나요? 이부분은 어떻게 정할까요?

Juhongseok commented 1 year ago

/api는 어차피 view를 보여주는 controller가 없으니 딱히 필요진 않을 거 같아요 앞에 /user 처럼 도메인 구분만 해도 좋을 것 같아요