[톰캣 구현하기 1, 2단계] 새양(양경호) 미션 제출합니다.

[geoje]

냥인🐈🧍‍♀️ 안녕하세요!
4단계까지 했지만 혼나고 일부 커밋을 드랍하고 다시 올립니다! 😢

우선 요구사항을 만족하여 최소한의 사항으로 미션을 진행해 보았어요!
잘 부탁 드립니다! 🤗

구현 사항

• / 접속 시 안녕 세상 텍스트 출력
• /index.html 접속 시 대시보드 화면 출력
• /login 접속 시 로그인 페이지 출력
• /login 에서 파라미터로 account 와 password 를 줄 경우 로그인 시도
  • 성공할 경우 쿠키와 함께 /index.html 으로 이동
  • 실패할 경우 /401.html 로 이동
  • 첫 로그인 이후 접속 시 세션을 통해 인증 후 위와 동일하게 행동
• /register 접속 시 회원 가입 페이지 출력

학습 테스트

• FileTest IOStreamTest GreetingControllerTest 을 학습하였습니다.
• GreetingControllerTest 에서 학습한 캐싱은 아직 적용하지 못하였습니다!

[cutehumanS2]

새양, 안녕하세요~ 냥인입니다. 😼 잘 지내시죠? ㅋㅋㅋ
단시간에 굉장히 많은 걸 구현하셨네요 !
배울 점이 많은 코드였어요. 고수신가요? ㅎ ㅎ

코멘트를 달다 보니 개수가 꽤 많아져서(죄송요 ^ ᵥ ^;; )
이번 단계에서는 여기서 끊고, 학습 테스트 포함 나머지 테스트 코드는 다음번에 같이 살펴볼게요.

모두 반영하고, 답변하실 필요는 없습니다.
새양이 보시기에 필요한 부분만 선택해서 진행해 주신 뒤 리뷰 요청 부탁드려요.

그리고 이미 4단계까지 구현을 끝내신 것 같아서 선택사항 두 가지 제안합니다.
1.이번 단계 PR에서 학습 테스트 포함 나머지 테스트 코드에 대한 리뷰 받기
2.다음 단계 PR에서 같이 받기
답변은 PR에 코멘트로 남겨주세요.

수고 많으셨어요. 즐(거운) 주(말) ~ ~ 👍

[geoje]

안녕하세요 냥인! 남겨주신 코멘트들 덕분에 생각과 코드의 개선이 엄청 많이 이루어졌어요~ 감사합니다! 😆
냥인이 남겨주신 아래 질문의 답은 아래와 같습니다!

Q. 그리고 이미 4단계까지 구현을 끝내신 것 같아서 선택사항 두 가지 제안합니다.
1.이번 단계 PR에서 학습 테스트 포함 나머지 테스트 코드에 대한 리뷰 받기
2.다음 단계 PR에서 같이 받기

A. 2번을 선택하겠습니다! 이번 PR 에는 마지막 단계 진행 사항은 커밋 드랍을 하여 반영을 취소했으며 3단계의 리팩토링도 조금 더 보수를 해야할 듯 합니다! (Enum 분리 등등)

감사합니다! 🤗

[cutehumanS2]

새양~ 굉장한 양의 코멘트를 모두 읽고 반영하셨네요! 😮
너어무 수고 많으셨어요. ㅜ.ㅜ

덕분에 저도 많이 배워갑니다. 진짜로요. ㅎ ㅎ
새양이 좀 더 보셨으면 하는 부분은 unresolve로 남겨두었으니 가볍게 읽어 보시길 ~ '◡'

이번 단계는 이만 머지합니다.
이번 주도 파이팅입니다.