🚀 1단계 - 토큰 기반 로그인 구현 #180

loop-study · 2022-02-12T10:54:53Z

안녕하세요 리뷰어님!
3주차 미션을 진행하게된 박현철이라고 합니다.

이전까지는 기능마다 인수테스트를 작성하다가, 하나로 묶어 진행되었는데, 중간에 한번 에러가 발생하니 독립적인 것보다 에러를 확인하는게 더 어려웠네요. 각각 분리된 인수테스트와 하나로 통합된 인수테스트 중에서 무엇이 더 선호하시는지 궁금합니다.
TokenAuthenticationInterceptorTest에서는 Mock으로 가짜객체를 만들어서 진행하다보니, preHandle() 테스트에서 성공한다는 가정하에 쉽게 통과되었습니다. 하지만 실객체 테스트인 AuthAcceptanceTest.myInfoWithBearerAuth() 토큰 발행에 실패하는 것을 보고 뒤늦게 잘못된 로직을 발견해서 수정하게 되었는데요.
실제 코드에서는 에러가 발생하는데 가짜객체테스트에서 통과를 한다면 Mock 객체의 단점인지 아니면, Mock을 활용한 테스트를 잘못 작성한건지 잘 모르겠습니다.
이에 대해 어떻게 생각하시는지 궁금합니다.
스프링 시큐리티, JWT에 대한 개념이 없어서 3주차 미션을 진행하는데 살짝 늦은 감이 있습니다. 잘못 구현한게 있으면 확인 부탁드리겠습니다.

많은 지도 부탁드리곘습니다!📖
감사합니다.🙇🏻‍♀️

2022. 02. 12 로그인 인증 프로세스 실습 정리 Resolver : loop-study See also : heesunjeong

2022. 02. 12 MemberController, PathController, StationController - 공통 경로, 메서드에서 클래스단위로 변경 Resolver : loop-study See also : heesunjeong

2022. 02. 12 MemberAcceptanceTest.java - 회원 생성, 조회, 수정, 삭제를 하나로 통합 Resolver : loop-study See also : heesunjeong

2022. 02. 12 MemberController - AuthAcceptanceTest 테스트 통과를 위한 기능 구현 Resolver : loop-study See also : heesunjeong

2022. 02. 12 READMD.md - step1 요구사항 정리 Resolver : loop-study See also : heesunjeong

2022. 02. 12 TokenAuthenticationInterceptor - authenticate 메서드, AuthenticationToken에서 AuthenticationToken 객체 생성하기 Resolver : loop-study See also : heesunjeong

2022. 02. 12 TokenAuthenticationInterceptorTest - preHandle 테스트 통과 Resolver : loop-study See also : heesunjeong

2022. 02. 12 TokenAuthenticationInterceptorTest - 검증 코드 수정 TokenAuthenticationInterceptor - 오류 코드 수정 Resolver : loop-study See also : heesunjeong

2022. 02. 12 MemberController - 토큰 기반 메서드 @AuthenticationPrincipal 이용 MemberSteps - 토큰 기반 인수테스트 도우미 메서드 추가 MemberAcceptanceTest - 토큰 기반 인수테스트 추가 - 요청 확인 검증 로직을 메서드로 추출 Resolver : loop-study See also : heesunjeong

2022. 02. 12 README.md - 요구사항 체크리스트 수정 Resolver : loop-study See also : heesunjeong

heesunjeong

안녕하세요 현철님, 3주차 미션 리뷰를 맡게된 정희선입니다 :) 잘 부탁드려요 😆
1단계 미션 잘 구현해주신 것 같아요 👍 질문주신 부분은 코멘트를 추가했으니 확인 부탁드려요~!
이어서 다음 단계 진행해주시면 됩니다 🔥 🔥

heesunjeong · 2022-02-12T23:45:33Z

README.md

+  - [x] AuthenticationPrincipalArgumentResolver를 참고하여 ArgumentResolver 기능을 사용하기
+
+- [x] MemberAcceptanceTest의 manageMyInfo 성공 시키기
+  - [x] @AuthenticationPrincipal을 활용하여 로그인 정보 받아오기


질문주신 부분은 여기에 코멘트 추가할게요 :)

이전까지는 기능마다 인수테스트를 작성하다가, 하나로 묶어 진행되었는데, 중간에 한번 에러가 발생하니 독립적인 것보다 에러를 확인하는게 더 어려웠네요. 각각 분리된 인수테스트와 하나로 통합된 인수테스트 중에서 무엇이 더 선호하시는지 궁금합니다.

통합 인수테스를 작성하시면서 어려움을 겪으셨군요 😢
처음부터 통합인수테스트를 작성하지 않고 기존처럼 기능별로 인수테스트를 작성하고, 기능을 구현한 후에 인수테스트를 통합해보시면 더욱 쉽게 작성해보실수 있으실 것 같아요! 저도 분리된 인수 테스트의 작성이 훨씬 쉽기때문에 분리해서 작성하고 성공한 이후에 하나로 통합하고 있어요 :)

TokenAuthenticationInterceptorTest에서는 Mock으로 가짜객체를 만들어서 진행하다보니, preHandle() 테스트에서 성공한다는 가정하에 쉽게 통과되었습니다. 하지만 실객체 테스트인 AuthAcceptanceTest.myInfoWithBearerAuth() 토큰 발행에 실패하는 것을 보고 뒤늦게 잘못된 로직을 발견해서 수정하게 되었는데요.
실제 코드에서는 에러가 발생하는데 가짜객체테스트에서 통과를 한다면 Mock 객체의 단점인지 아니면, Mock을 활용한 테스트를 잘못 작성한건지 잘 모르겠습니다.
이에 대해 어떻게 생각하시는지 궁금합니다.

네 맞습니다 mock 객체를 활용한 테스트는 현철님이 경험하신 것처럼 실제는 실패하는 케이스이지만 테스트는 통과하는 경우가 생기게 되는 것 같아요. 그래서 적절하게 사용하고 가능하다면 실제객체를 통해서 테스트를 작성해주시면 좋을 것 같아요. :)

heesunjeong · 2022-02-12T23:46:39Z

src/main/java/nextstep/auth/authentication/TokenAuthenticationInterceptor.java

@@ -27,8 +29,9 @@ public boolean preHandle(HttpServletRequest request, HttpServletResponse respons
        AuthenticationToken authenticationToken = convert(request);
        Authentication authentication = authenticate(authenticationToken);

-        // TODO: authentication으로 TokenResponse 추출하기
-        TokenResponse tokenResponse = null;
+        String payload = new ObjectMapper().writeValueAsString(authentication.getPrincipal());


메소드 호출시마다 ObjectMapper를 생성하지 않고 bean을 활용해보시면 어떨까요?

하나로 통합하겠습니다!

src/main/java/nextstep/member/ui/MemberController.java

heesunjeong · 2022-02-12T23:49:46Z

src/test/java/nextstep/subway/acceptance/MemberSteps.java

    public static void 회원_정보_조회됨(ExtractableResponse<Response> response, String email, int age) {
        assertThat(response.jsonPath().getString("id")).isNotNull();
        assertThat(response.jsonPath().getString("email")).isEqualTo(email);
        assertThat(response.jsonPath().getInt("age")).isEqualTo(age);
    }
+
+    public static void 회원_생성_요청_확인(ExtractableResponse<Response> response) {


메소드가 비어있네요 👀

헛. 지우는걸 반영안했네요. 🙈

src/test/java/nextstep/subway/unit/TokenAuthenticationInterceptorTest.java

src/test/java/nextstep/subway/acceptance/MemberAcceptanceTest.java

* 4th init * Step1 : 기능 구현 (next-step#175) * 3rd init * 4th init * feat : 프론트가 변경되었네요 (next-step#139) * step1 : 지하철 노선 기능 구현 (next-step#171) * feat : front end 코드 삭제 * feat : 노선 생성 * feat : 지하철 노선 목록 조회 * feat : 노선 단건 조회 * feat : 노선 업데이트 * feat : 노선 삭제 기능 추가 * fix : test 수정 * feat : responseEntity 수정 * Step2 : 리팩토링 (next-step#180) * feat : front end 코드 삭제 * feat : 노선 생성 * feat : 지하철 노선 목록 조회 * feat : 노선 단건 조회 * feat : 노선 업데이트 * feat : 리베이스 * feat : 충돌 해결 * style : 주석과 본문을 일치 시키기 * refactor : 생성시 반환되는 location 재사용 * style : 주석 수정 * feat : 중복된 지하철 역 생성 불가 * feat : 노선이름 중복은 생성 실패 기능구현 * feat : http request 테스트를 위한 유틸 생성 * refactor : 지하철 역 생성 리팩토링 - 중복된 코드는 httpRequestTestUtil을 사용해서 변경 * chore : HttpRequestTestUtil 패키지 이동 - 유틸 패키지로 이동 * refactor : 지하철 역 test 리팩토링 - 유틸을 이용해 리팩토링 * style : 코드 스타일 수정 - 한 줄에 점 하나 * refactor : 엔티티는 DTO를 모르게 변경 - 고수준에서는 접근은 어렵게 * Step3 : 기능구현 (next-step#260) * refactor : lineTest 메소드 분리 * refactor : stationTest 메소드 분리 * refactor : 메소드 명 한들로 변경 * feat : 노선 생성시 필드 추가 * feat : Section 도메인 추가 - line에도 추가 * feat : 찾는 개체가 없을 때 발생하는 예외 선언 * feat : 노선 생성시 section을 함께 등록한다 - station이 있는지 먼저 검사를 진행한다 * feat : 구간 추가하기 - 구간을 추가한다 * feat : 구간 등록시 규칙이 존재 - 신규 구간의 상행은 기존 구간의 하행 종점이어야한다 * feat : 등록되지 않은 하행 구간을 등록한다 * refactor : step으로 메소드 분리 - 각 클래스마다 선행작업을 분리 * refactor : 예외 계층 수정 * fix : 더티체킹 위해 transaction 선언 * feat : 마지막 구간만 삭제 가능 * feat : 마지막 역은 삭제가 불가하다 * feat : 마지막 구간은 삭제된다 * feat : 조회 로직 변경 * refactor : 메소드 이름 변경 * refactor : 중복 제거 * refactor : 파라미터 테스트로 변경 * refactor ; 지하철 삭제 메소드 추가 * refactor : 노선 삭제 메소드 생성, 구간생성시 static string 사용 * refactor : 고정적인 값 보다 유연한 값으로 변경 * refactor : 디스플레이 네임 변경 * refactor : 구간 등록 검증 로직 추가 * refactor : 일급 컬렉션으로 변경, 부정형 메소드 추가 * refactor : response의 동등성 비교을 위해 추가 * refactor : 생성자 숨기기 * style : 줄바꿈 추가 * style ; 줄바꿈 추가 * style : 오타 수정 * refactor : 기본 생성자 접근 제한자 변경 * refactor : 파라미터의 키를 상수로 변환 * refactor : 값 검사도 메소드로 분리 * refactor : transaction의 위치 변경 - class 단이 아닌 메소드 단으로 변경 * refactor : 피드백 반영 * feat : 중간 구간에 구간을 생성할 수 있다 * refactor : 구간 등록 수정 * refactor : 노선 조회 시 순서대로 조회 * test : 나머지 인수 테스트 추가 * feat : 모든 역이 노선에 등록되어 있을 경우 추가 불가 * feat : 상행역과 하행역 둘 중 하나라도 노선에 포한된 역이어야 한다 * fix : 조건 수정 - 상행역, 하행역만 뒤지는 것이 아닌 모두 검사해야한다 * feat : 충돌 해결 * feat : 기존 코드 삭제 * feat : given ,when ,then 추가 * feat : 구간 삭제 단위 테스트 추가 * feat : sections에 대한 단위 테스트 작성 * feat : 구간 등록 단위 테스트 작성 * refactor : 리스폰스 한글로 정의 * refactor : 프론트 충돌 해결 * refactor : stream 으로 변경 * feat : 서비스 단위 테스트 Co-authored-by: boorownie <probitanima11@gmail.com> * Step2 : 기능 구현 (next-step#191) * feat : 역 삭제 시 봉합이 가능하다 * feat : 노선에 없는 역은 삭제가 불가하다 * feat : fake 객체 사용으로 test * feat : fake 객체를 이용한 service test 코드 작성 * Step3 : 기능 구현 (next-step#219) * refactor : 리뷰 반영 * refactor : fake 객체 리뷰 반영 - fake 객체 반영 * feat : path finder 도메인 단위 테스트 * feat : 인수 테스트 * feat : 예외 테스트 추가 * feat : path finder 단위 테스트 * feat : service 테스트 추가 * refactor : 기본 생성자 잠금 - 유틸로 사용 * feat : 기존 파일 삭제 * feat : 이전 코드 옮기기 * feat : test 코드 리팩토링 * feat : 인수 테스트 성공 시키기 * test : 인터셉터 단위 test * feat : authenticate test * feat : preHandle test * feat : bearer test 수정 * feat : 리뷰반양 Co-authored-by: boorownie <probitanima11@gmail.com>

* 3rd init * 4th init * feat : 프론트가 변경되었네요 (next-step#139) * step1 : 지하철 노선 기능 구현 (next-step#171) * feat : front end 코드 삭제 * feat : 노선 생성 * feat : 지하철 노선 목록 조회 * feat : 노선 단건 조회 * feat : 노선 업데이트 * feat : 노선 삭제 기능 추가 * fix : test 수정 * feat : responseEntity 수정 * Step2 : 리팩토링 (next-step#180) * feat : front end 코드 삭제 * feat : 노선 생성 * feat : 지하철 노선 목록 조회 * feat : 노선 단건 조회 * feat : 노선 업데이트 * feat : 리베이스 * feat : 충돌 해결 * style : 주석과 본문을 일치 시키기 * refactor : 생성시 반환되는 location 재사용 * style : 주석 수정 * feat : 중복된 지하철 역 생성 불가 * feat : 노선이름 중복은 생성 실패 기능구현 * feat : http request 테스트를 위한 유틸 생성 * refactor : 지하철 역 생성 리팩토링 - 중복된 코드는 httpRequestTestUtil을 사용해서 변경 * chore : HttpRequestTestUtil 패키지 이동 - 유틸 패키지로 이동 * refactor : 지하철 역 test 리팩토링 - 유틸을 이용해 리팩토링 * style : 코드 스타일 수정 - 한 줄에 점 하나 * refactor : 엔티티는 DTO를 모르게 변경 - 고수준에서는 접근은 어렵게 * Step3 : 기능구현 (next-step#260) * refactor : lineTest 메소드 분리 * refactor : stationTest 메소드 분리 * refactor : 메소드 명 한들로 변경 * feat : 노선 생성시 필드 추가 * feat : Section 도메인 추가 - line에도 추가 * feat : 찾는 개체가 없을 때 발생하는 예외 선언 * feat : 노선 생성시 section을 함께 등록한다 - station이 있는지 먼저 검사를 진행한다 * feat : 구간 추가하기 - 구간을 추가한다 * feat : 구간 등록시 규칙이 존재 - 신규 구간의 상행은 기존 구간의 하행 종점이어야한다 * feat : 등록되지 않은 하행 구간을 등록한다 * refactor : step으로 메소드 분리 - 각 클래스마다 선행작업을 분리 * refactor : 예외 계층 수정 * fix : 더티체킹 위해 transaction 선언 * feat : 마지막 구간만 삭제 가능 * feat : 마지막 역은 삭제가 불가하다 * feat : 마지막 구간은 삭제된다 * feat : 조회 로직 변경 * refactor : 메소드 이름 변경 * refactor : 중복 제거 * refactor : 파라미터 테스트로 변경 * refactor ; 지하철 삭제 메소드 추가 * refactor : 노선 삭제 메소드 생성, 구간생성시 static string 사용 * refactor : 고정적인 값 보다 유연한 값으로 변경 * refactor : 디스플레이 네임 변경 * refactor : 구간 등록 검증 로직 추가 * refactor : 일급 컬렉션으로 변경, 부정형 메소드 추가 * refactor : response의 동등성 비교을 위해 추가 * refactor : 생성자 숨기기 * style : 줄바꿈 추가 * style ; 줄바꿈 추가 * style : 오타 수정 * refactor : 기본 생성자 접근 제한자 변경 * refactor : 파라미터의 키를 상수로 변환 * refactor : 값 검사도 메소드로 분리 * refactor : transaction의 위치 변경 - class 단이 아닌 메소드 단으로 변경 * refactor : 피드백 반영 * feat : 중간 구간에 구간을 생성할 수 있다 * refactor : 구간 등록 수정 * refactor : 노선 조회 시 순서대로 조회 * test : 나머지 인수 테스트 추가 * feat : 모든 역이 노선에 등록되어 있을 경우 추가 불가 * feat : 상행역과 하행역 둘 중 하나라도 노선에 포한된 역이어야 한다 * fix : 조건 수정 - 상행역, 하행역만 뒤지는 것이 아닌 모두 검사해야한다 * feat : 충돌 해결 * feat : 기존 코드 삭제 * feat : given ,when ,then 추가 * feat : 구간 삭제 단위 테스트 추가 * feat : sections에 대한 단위 테스트 작성 * feat : 구간 등록 단위 테스트 작성 * refactor : 리스폰스 한글로 정의 * refactor : 프론트 충돌 해결 * refactor : stream 으로 변경 * feat : 서비스 단위 테스트 Co-authored-by: boorownie <probitanima11@gmail.com>

loop-study added 10 commits February 12, 2022 14:41

docs : 요구사항 정리

d50baac

2022. 02. 12 로그인 인증 프로세스 실습 정리 Resolver : loop-study See also : heesunjeong

refactor : 경로 위치 수정

96af7f9

2022. 02. 12 MemberController, PathController, StationController - 공통 경로, 메서드에서 클래스단위로 변경 Resolver : loop-study See also : heesunjeong

refactor : 인수테스트 통합

678f67c

2022. 02. 12 MemberAcceptanceTest.java - 회원 생성, 조회, 수정, 삭제를 하나로 통합 Resolver : loop-study See also : heesunjeong

feat : findMemberOfMine 구현

0540c08

2022. 02. 12 MemberController - AuthAcceptanceTest 테스트 통과를 위한 기능 구현 Resolver : loop-study See also : heesunjeong

docs : step1 요구사항 정리

40e1f41

2022. 02. 12 READMD.md - step1 요구사항 정리 Resolver : loop-study See also : heesunjeong

feat : authenticate 구현

3ece592

2022. 02. 12 TokenAuthenticationInterceptor - authenticate 메서드, AuthenticationToken에서 AuthenticationToken 객체 생성하기 Resolver : loop-study See also : heesunjeong

test : preHandle 테스트

952fe95

2022. 02. 12 TokenAuthenticationInterceptorTest - preHandle 테스트 통과 Resolver : loop-study See also : heesunjeong

refactor : 코드 수정

eac9759

2022. 02. 12 TokenAuthenticationInterceptorTest - 검증 코드 수정 TokenAuthenticationInterceptor - 오류 코드 수정 Resolver : loop-study See also : heesunjeong

docs : 체크리스트 수정

0321744

2022. 02. 12 README.md - 요구사항 체크리스트 수정 Resolver : loop-study See also : heesunjeong

heesunjeong approved these changes Feb 13, 2022

View reviewed changes

heesunjeong merged commit ab0d4bc into next-step:loop-study Feb 13, 2022

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

🚀 1단계 - 토큰 기반 로그인 구현 #180

🚀 1단계 - 토큰 기반 로그인 구현 #180

loop-study commented Feb 12, 2022

heesunjeong left a comment

heesunjeong Feb 12, 2022

heesunjeong Feb 12, 2022

loop-study Feb 13, 2022

heesunjeong Feb 12, 2022

loop-study Feb 13, 2022

🚀 1단계 - 토큰 기반 로그인 구현 #180

🚀 1단계 - 토큰 기반 로그인 구현 #180

Conversation

loop-study commented Feb 12, 2022

heesunjeong left a comment

Choose a reason for hiding this comment

heesunjeong Feb 12, 2022

Choose a reason for hiding this comment

heesunjeong Feb 12, 2022

Choose a reason for hiding this comment

loop-study Feb 13, 2022

Choose a reason for hiding this comment

heesunjeong Feb 12, 2022

Choose a reason for hiding this comment

loop-study Feb 13, 2022

Choose a reason for hiding this comment