[3주차] 🚀 2단계 - 다중 서비스 테스트 환경 by JoyDaheeCha · Pull Request #23 · next-step/atdd-camping-tests

JoyDaheeCha · 2025-09-26T13:43:47Z

브라운님 안녕하세요!
이번 리뷰도 잘 부탁드립니다~

질문 1

Hooks에서 baseUrl을 계속 추가하는데 이 경우, 테스트 하는 서비스가 많아질때마다 Hook 에서 초기화할 대상이 많겠다는 생각이 들었어요. 현재 레벨에서는 3개 서비스 뿐이라 하나의 Hook으로 두었는데, 별개의 Hook으로 두는게 나을지 의견 여쭤봅니다~

질문 2

docker 구성이 익숙하지 않아 cursor 와 작업하며 여러번 시행착오를 겪었습니다 ㅎㅎ
cursor 가 작업한 내용에 대해 기술 지식이 충분하지 않아, 코드를 봤을때 좋은 코드인지 애매할 경우, "이정도면 되었다" 결정하시는 기준을 가지고 계신지 궁금합니다!

업무할때는 cursor한테 한번 작업을 시키고, Junie로 리뷰를 꼼꼼히 하는 방식으로 일을 했는데, Junie도 자주 틀리더라구요
(프롬프트 탓이겠지만...)

감사합니다!

This reverts commit e8f203b.

- 모든 서비스(kiosk, admin, reservation)를 공용 네트워크 atdd-net에 연결 - admin과 reservation 서비스에 공용 DB(atdd-db) 접속 정보 환경변수 추가 - MySQL 8.0 드라이버 및 방언 설정 - DDL 모드를 validate로 변경하여 기존 스키마 사용 - kiosk가 admin에 의존하도록 depends_on 설정

- 인프라 DB(atdd-db) 자동 시작 및 초기화 기능 추가 - start_infra_db() 함수로 DB 헬스체크 및 초기화 자동화 - stop_infra_db() 함수로 인프라 DB 정리 기능 추가 - 서비스 시작 시 인프라 DB를 먼저 시작하도록 순서 조정 - 모든 서비스 중지 시 인프라 DB도 함께 중지하도록 수정 - 도움말에 공용 DB 관련 정보 추가

- customers 테이블 추가 (Admin 서비스에서 필요) - JPA DDL 모드를 validate에서 none으로 변경하여 스키마 검증 비활성화 - 공용 DB와 애플리케이션 엔티티 간 호환성 개선

- atdd-tests 내 init.sql과 충돌하여 제거

boorownie

안녕하세요 다희님!
2단계도 고생 많으셨습니다!
앞 단계 피드백도 많이 반영해주셨네요 👍

2단계 요구사항 중 몇 가지 요구사항에 대해서 코멘트를 남겨두었는데 확인 부탁드리고요!
혹시 제가 잘못이해한 부분이 있다면 코멘트 남겨주시고 수정할 부분이 없으면 바로 리뷰요청 주셔도 됩니다!

질문1번은 코드에 코멘트로 남겨두었고, 2번은 아래 남겨두겠습니다!
참고해서 확인부탁드립니다 :)

===

cursor 가 작업한 내용에 대해 기술 지식이 충분하지 않아, 코드를 봤을때 좋은 코드인지 애매할 경우, "이정도면 되었다" 결정하시는 기준을 가지고 계신지 궁금합니다!

우선 저는 cursor로 구현을 할 때도 TDD 할 때 처럼 동작하는 코드가 되는지를 우선 확인합니다. 제대로 동작하는 것을 확인한 후 리팩터링을 이어서 진행해주는데요, 아마 리팩터링을 언제까지 하는지에 대한 질문을 해주신 것으로 이해했어요.

사실 AI 도구를 사용하기 전과 사용할 때 리팩터링 수행 기준이 다르지 않습니다. 추가로 더 개선할 부분이 없어보일 때 까지 진행합니다. 단, 수정을 했을 때 큰 코드 변경이 생길 것 같다 느껴지면 지금 할 지 다음에 할 지를 판단하고 대부분 그만두는 편이에요!

boorownie · 2025-09-27T06:05:32Z

+    private static void initializeRestAssured() {
+        kioskBaseUrl = KIOSK_URL;
+        adminBaseUrl = ADMIN_URL;
+        reservationBaseUrl = RESERVATION_URL;
        RestAssured.enableLoggingOfRequestAndResponseIfValidationFails();
    }


Hooks에서 baseUrl을 계속 추가하는데 이 경우, 테스트 하는 서비스가 많아질때마다 Hook 에서 초기화할 대상이 많겠다는 생각이 들었어요. 현재 레벨에서는 3개 서비스 뿐이라 하나의 Hook으로 두었는데, 별개의 Hook으로 두는게 나을지 의견 여쭤봅니다~

개인적으로는 Context에 kioskBaseUrl 처럼 관리해주는 것도 좋지만 바로 BaseUrl에 정의한 URL을 쓰도록하면
매번 재정의해줄 필요없이 BaseUrl에 정의한 부분으로 사용해줄 수 있을 것 같은데 어떨까요?

boorownie · 2025-09-27T06:10:08Z

+      - SPRING_PROFILES_ACTIVE=docker
+      # Database configuration for shared DB
+      - SPRING_DATASOURCE_URL=jdbc:mysql://atdd-db:3306/atdd?useSSL=false&allowPublicKeyRetrieval=true&serverTimezone=Asia/Seoul
+      - SPRING_DATASOURCE_USERNAME=root
+      - SPRING_DATASOURCE_PASSWORD=secret
+      - SPRING_DATASOURCE_DRIVER_CLASS_NAME=com.mysql.cj.jdbc.Driver
+      - SPRING_JPA_DATABASE_PLATFORM=org.hibernate.dialect.MySQL8Dialect
+      - SPRING_JPA_HIBERNATE_DDL_AUTO=none


공통적으로 사용할 수 있는 부분을 묶어서 재사용해볼 수 있는데 참고해보세요 :)

https://docs.docker.com/reference/compose-file/fragments/

boorownie · 2025-09-27T06:11:35Z

@@ -0,0 +1,53 @@
+# Multi-stage build for Spring Boot application


도커파일 잘 만들어주셨네요 👍
다른 도커파일과 크게 다른 부분이 없다면 공통으로 사용해볼 수 있는 방법도 고려해볼만할 것 같아요!

boorownie · 2025-09-27T11:27:43Z

-            "https://github.com/next-step/atdd-camping-kiosk.git",
-            "repos/atdd-camping-kiosk"
+        // 클론할 저장소 목록
+        val repositories = listOf(


가져 올 브랜치 명도 각 서비스별로 따로 관리해줘도 좋을 것 같네요 :)

boorownie · 2025-09-27T11:29:09Z

+./manage-services.sh status  # 상태 확인
+./manage-services.sh restart # 재시작


해당 파일이 보이지 않네요!
이 파일들의 작업도 clone과 마찬가지로 gradle task로 옮겨보는건 어떨까요?

boorownie · 2025-09-27T11:30:01Z

@@ -1,4 +1,12 @@
 -- Create tables if not exist (so we can seed before apps start)
+CREATE TABLE IF NOT EXISTS customers (


서비스에 맞게 필요한 테이블 추가 👍

boorownie · 2025-09-27T11:35:18Z

+
+    @When("키오스크 상품 목록을 조회한다")
+    public void 키오스크상품목록을조회한다() {
+        lastResponse = getWithAuth(kioskBaseUrl, "/api/products", authToken);


키오스크 상품 목록 조회 API에서 인증이 필요한가요?
혹시 admin아 아닌가요?

JoyDaheeCha added 22 commits September 25, 2025 22:49

adjust: 테스트와 중복되는 전제를 제거하라

b3e7831

adjust: 헬스 체크 제거 (smoke 테스트에서 확인되는 내용)

992f060

adjust: 공용 클래스에서 Kiosk 네이밍 제거

d1139ca

config: cloneRepository 에서 reservation과 admin도 클론하도록 설정 추가

3a5dd1a

config: Dockerfile-admin, Dockerfile-reservation 추가하라

55fee32

config: docker-compose 구성

3e656a2

test: admin, reservation, kiosk 각각에 대해 smoke test 작성

e8f203b

Revert "test: admin, reservation, kiosk 각각에 대해 smoke test 작성"

adda73e

This reverts commit e8f203b.

test: admin, reservation, kiosk 각각에 대해 smoke test 작성

6617f9f

config: 애플리케이션 실행 방법 및 readme 수정

1c9ebd4

fix: 관리자 애플리케이션은 스모크테스트시 401 응답을 검증하라

6c52cff

test: kiosk의 상품 조회 e2e 테스트 추가

95373f0

refactor: e2e 테스트에서 검증 불필요한 부분 제거

bbcf680

fix: e2e 테스트시 토큰 가져오는 로직, admin 접근 안되던 현상 수정

32d7728

refactor: 로깅 추가

7891c12

refactor: 불필요 로직 제거

d1f57e8

DB 스키마 수정 및 DDL 모드 조정

188e5ef

- customers 테이블 추가 (Admin 서비스에서 필요) - JPA DDL 모드를 validate에서 none으로 변경하여 스키마 검증 비활성화 - 공용 DB와 애플리케이션 엔티티 간 호환성 개선

fix: admin 서비스의 data.sql을 사용하지 않도록 한다.

5de93b2

- atdd-tests 내 init.sql과 충돌하여 제거

fix: reservation 서비스의 data.sql을 사용하지 않도록 한다.

2e745bf

- atdd-tests 내 init.sql과 충돌하여 제거

refactor: 클래스명은 카멜케이스로 하라

7e9f737

boorownie requested changes Sep 27, 2025

View reviewed changes

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[3주차] 🚀 2단계 - 다중 서비스 테스트 환경#23

[3주차] 🚀 2단계 - 다중 서비스 테스트 환경#23
JoyDaheeCha wants to merge 22 commits into
next-step:joydaheechafrom
JoyDaheeCha:step2

JoyDaheeCha commented Sep 26, 2025 •

edited

Loading

Uh oh!

boorownie left a comment

Uh oh!

boorownie Sep 27, 2025

Uh oh!

boorownie Sep 27, 2025

Uh oh!

boorownie Sep 27, 2025

Uh oh!

boorownie Sep 27, 2025

Uh oh!

boorownie Sep 27, 2025

Uh oh!

boorownie Sep 27, 2025

Uh oh!

boorownie Sep 27, 2025

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

		@@ -0,0 +1,53 @@
		# Multi-stage build for Spring Boot application

		./manage-services.sh status # 상태 확인
		./manage-services.sh restart # 재시작

		@@ -1,4 +1,12 @@
		-- Create tables if not exist (so we can seed before apps start)
		CREATE TABLE IF NOT EXISTS customers (

Conversation

JoyDaheeCha commented Sep 26, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

질문 1

질문 2

Uh oh!

boorownie left a comment

Choose a reason for hiding this comment

Uh oh!

boorownie Sep 27, 2025

Choose a reason for hiding this comment

Uh oh!

boorownie Sep 27, 2025

Choose a reason for hiding this comment

Uh oh!

boorownie Sep 27, 2025

Choose a reason for hiding this comment

Uh oh!

boorownie Sep 27, 2025

Choose a reason for hiding this comment

Uh oh!

boorownie Sep 27, 2025

Choose a reason for hiding this comment

Uh oh!

boorownie Sep 27, 2025

Choose a reason for hiding this comment

Uh oh!

boorownie Sep 27, 2025

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

JoyDaheeCha commented Sep 26, 2025 •

edited

Loading