Skip to content

Conversation

@ivvve
Copy link

@ivvve ivvve commented Sep 29, 2025

안녕하세요 성현님!

빠르게 리뷰 요청드리고 싶었는데 전체 서비스를 파악하고 테스트를 구상하는 것도 쉽지 않더라구요 🥹
그리고 아직 claude code 같은 ai 툴을 써도 제가 보고 손으로 수정하는게 잦기도 하구요...
(미션 내용도 잘못 숙지했어서 시간이 더 걸렸습니다...)

결과물이 그렇게 퀄리티가 있어보이진 않는데 인수 테스트의 양을 더 늘리는게 목적은 아닌 것 같아
더 늦기전에 리뷰 요청드립니다 🙇

AI 가이드 문서

먼저 지침을 주는 가이드 문서를 무엇을 만들까 고민하였는데 아래와 같이 추가하였습니다.

  1. 기존에 테스트 코드를 참고하고 해당 테스트 스타일을 따를 수 있는 코딩 스타일 문서
  2. 테스트를 작성한 뒤에 테스트를 실행하기 위한 테스트 실행 가이드 문서
  3. 테스트 대상이 되는 API 서비스들이 어떤 식으로 구성되어 있는지(요청, 응답, 제약 사항 등) 확인할 수 있도록 API 분석 가이드 문서
  • claude code 쓰면서 처음 알았는데 git ignore 처리된 파일/디렉토리는 claude code가 접근하지 못하게 되있더라구요.
    그래서 꼼수로 확인하는 지침을 추가해두었습니다 👀
  1. 마지막으로 이 문서들을 어떻게 참고해야할지를 알려주는 상위 가이드 문서

인수 테스트

크게 전체 시스템을 아우르는 인수테스트를 작성하였습니다.
https://github.com/next-step/atdd-camping-tests/pull/27/files#diff-5167523b479c745e31586bb96d61012f1a92d1071e692529c4de6b661cc5b6eb

그리고 관련해서 테스트 코드는 따로 손대지 않고 프롬프트를 통해서만 수정하고 리팩토링 시켰는데
빨리빨리 뭔가 만들어지긴 하는데 그래도 퀄리티가 아쉽긴한 것 같습니다.
(그래도 가이드 문서의 디테일을 추가하고 프롬프트를 다듬으니 얼추 기존 테스트 코드와 비슷하게 따라가고 있습니다)


마지막 주차 미션을 통해서 얻어가는게 많은 것 같습니다.
MSA 환경에서 서비스 간에 인터랙션을 어떻게 확인하여야할지 고민이 많았는데 이번 미션을 통해서 그 방법을 하나 얻어갑니다.
덕분에 이번 우아콘 발표도 기다려지네요 ㅎㅎ
(Cucumber는 써보기 전에는 재미있어 보였는데 막상 써보니까 쉽지않네요... 🥲 )

강의 진행하시느라 고생 많으셨습니다 :)

(강의 리뷰는 내일 중으로 꼭 남기도록 하겠습니다 🙇 )

@ivvve ivvve marked this pull request as ready for review September 29, 2025 13:43
Copy link
Contributor

@boorownie boorownie left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

선택미션까지 잘 완료해주셨습니다 👍
이번 과정을 통해서 인수 테스트에 대한 사고가 충분히 확장되길 바라며
향후 적용에도 좋은 힌트와 경험이 될 수 있길 바랍니다!

@@ -0,0 +1,346 @@
# 인수 테스트 작성 가이드
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

인수 테스트 작성 가이드 👍
저도 가이드 문서의 크기를 줄이기 위해 노력을 하는데
예시 코드랑 이런것 저런 것 넣다보면 어쩔 수 없이 길어지는 거 같아요ㅠㅠ

@boorownie boorownie merged commit aca7345 into next-step:ivvve Oct 19, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants