🚀 3단계 - 테스트를 통한 코드 보호 #825

sumiini · 2025-02-03T14:01:11Z

안녕하세요. 😀

서비스별로 정상 케이스, 예외 검증 케이스 구분하였습니다.
하면서 어려웠던 부분은 테스트 코드의 함수 명을 짓는 것이었습니다.
이 부분에서 많은 시간을 할애하였습니다.

이번을 계기로 SpringBootTest 를 많이 연습하였습니다.

이후 시간이 된다면, @ExtendWith(MockitoExtension.class) 단위 테스트도 추가로 구현하여
많이 연습하려합니다 !

sah3122

테스트 코드 작성 너무 잘해주셨습니다 👍
몇가지 생각거리를 남겨두었는데 확인 부탁드립니다.
질문이 있다면 편하게 DM 주세요 😄

sah3122 · 2025-02-04T13:06:31Z

src/test/java/kitchenpos/application/MenuGroupServiceTest.java

+    @Test
+    @Transactional
+    void create() {
+        //given


given / when / then 을 주석으로 표기하기 보단 공백 라인을 통해 충분히 표현할수 있을것 같습니다 😄

sah3122 · 2025-02-04T13:08:17Z

src/test/java/kitchenpos/application/MenuGroupServiceTest.java

+
+    @DisplayName(value = "메뉴 그룹 생성 시 그룹 명이 null일 경우 IllegalArgumentException 예외 처리를 한다.")
+    @Test
+    @Transactional


@transactional 어노테이션이 어떤 역할을 하고 있을까요 😄

모두 정상적으로 실행되면 정상적으로 실행되면 커밋되고, 중간에 예외가 발생하면 롤백해주는 역할을 합니다.

위는 한 create라는 메서드를 트랜잭션으로 명시한 것입니다.
메뉴 그룹 생성시 예외가 발생하면, 메뉴 그룹을 DB에 저장하지 않도록 하는 역할을 합니다.

@SpringBootTest + @transactional 어노테이션의 역할은 테스트 코드가 정상적으로 실행되어 커밋이 되었을때 롤백을 하는 역할을 합니다.
현재 작성해주신 테스트 코드는 실패를 검증하는 테스트 코드이기 때문에 명시하지 않아도 될것 같아요 !

sah3122 · 2025-02-04T13:09:45Z

src/test/java/kitchenpos/application/MenuGroupServiceTest.java

+        String name = "한마리메뉴";
+        MenuGroup request = new MenuGroup();
+        request.setName(name);


자주 사용하는 테스트 객체를 TestFixture로 정의하여 가독성과 재사용성을 높일수 있습니다 😄
이번 미션을 진행하며 TestFixture에 대해 경험해보는것을 권장 드려요 !
https://velog.io/@langoustine/Test-Fixture

sah3122 · 2025-02-04T13:12:08Z

src/test/java/kitchenpos/application/MenuGroupServiceTest.java

+                .isThrownBy(() -> menuGroupService.create(request));
+    }
+
+    @DisplayName(value = "메뉴 그룹 생성 시 그룹 명이 빈 값일 경우 IllegalArgumentException 예외 처리를 한다.")


Junit은 테스트를 작성하기 위한 다양한 기능을 제공합니다.
@NullAndEmptySource와 같은 ParameterizedTest에 대해 학습해보세요 😄
https://gmlwjd9405.github.io/2019/11/27/junit5-guide-parameterized-test.html

sah3122 · 2025-02-04T13:14:25Z

src/test/java/kitchenpos/application/MenuServiceTest.java

+import java.util.NoSuchElementException;
+import java.util.UUID;
+
+@SpringBootTest


SpringBootTest를 활용한 통합 테스트를 작성하는것도 좋지만 Mocking을 활용한 단위 테스트 작성법에 대해서도 경험해보는것을 권장 드립니다 😄

sah3122 · 2025-02-04T13:18:22Z

src/test/java/kitchenpos/application/MenuServiceTest.java

+
+    @DisplayName("메뉴 생성 시 메뉴 가격이 null 이면 IllegalArgumentException 예외 처리를 한다.")
+    @Test
+    void canNotCreateMenuIfMenuPriceIsNull() {


테스트 코드의함수명을 짓는것에 대해 어려움을 겪으셨다고 남겨주셨는데 개인적으로 테스트 코드를 작성할때 가독성을 위해 함수명을 한글로 짓기도 합니다 😄

Suggested change

void canNotCreateMenuIfMenuPriceIsNull() {

void `메뉴생성_실패_가격_Null`() {

sah3122 · 2025-02-04T13:22:02Z

src/test/java/kitchenpos/application/MenuServiceTest.java

+        Assertions.assertThat(result).extracting(Menu::getId).containsExactly(menu.getId());
+
+    }
+}


파일의 마지막은 new line 으로 끝내는것을 권장 드립니다 😄
https://stackoverflow.com/questions/729692/why-should-text-files-end-with-a-newline
intellij를 사용하고 계시다면 Setting -> Editor -> General �에서 Ensure every saved file ends with a line break 을 켜주시면 됩니다 😄

sah3122 · 2025-02-04T13:24:04Z

src/test/java/kitchenpos/application/OrderServiceTest.java

+
+@SpringBootTest
+@Transactional
+class OrderServiceTest {


꼼꼼한 테스트 코드 작성 👍
주문 유형에 따라 달라지는 테스트 케이스가 있습니다.
@nested 를 활용하여 주문 유형에 따른 케스트 케이스를 분리해보는것도 좋을것 같습니다 😄
https://www.baeldung.com/junit-5-nested-test-classes

주문과 관련된 테스트 코드가 부족한것 같습니다.
배달 주문 / 포장 주문 / 매장 식사 주문 과 같이 유형별로 정의해주신 요구사항을 기반으로 조금더 상세하게 테스트 코드를 작성해보세요 😄
작성해주신 요구사항 문서가 모두 테스트 코드에 녹아져 새로운 개발자가 프로젝트에 참여시 테스트 코드만으로 요구사항을 분석할수 있을 정도가 되면 좋을것 같아요 😄

…dTest 활용)

sumiini · 2025-02-10T14:22:38Z

리뷰 내용을 반영하여 전체 테스트 코드를 수정하였습니다.

@nested, Test Fixture, @ParameterizedTest를 적극 활용하여
이전 코드보다 가독성, 유지보수성, 재사용성이 크게 향상 되는 것을 느꼈습니다 🙂

sah3122

피드백 반영 잘해주셨습니다 👍
주문 서비스에 대한 테스트 코드 작성이 부족한것 같아 보완 요청 드릴게요 🙏
추후 진행될 미션에서 기존에 작성된 테스트 코드를 활용할수 있기 때문에 조금만 더 힘내주시길 바랍니다 !

sah3122 · 2025-02-11T08:12:11Z

src/test/java/kitchenpos/application/OrderServiceTest.java

+
+@SpringBootTest
+@Transactional
+class OrderServiceTest {


주문과 관련된 테스트 코드가 부족한것 같습니다.
배달 주문 / 포장 주문 / 매장 식사 주문 과 같이 유형별로 정의해주신 요구사항을 기반으로 조금더 상세하게 테스트 코드를 작성해보세요 😄
작성해주신 요구사항 문서가 모두 테스트 코드에 녹아져 새로운 개발자가 프로젝트에 참여시 테스트 코드만으로 요구사항을 분석할수 있을 정도가 되면 좋을것 같아요 😄

sah3122 · 2025-02-11T08:14:21Z

src/test/java/kitchenpos/application/MenuGroupServiceTest.java

+
+    @DisplayName(value = "메뉴 그룹 생성 시 그룹 명이 null일 경우 IllegalArgumentException 예외 처리를 한다.")
+    @Test
+    @Transactional


@SpringBootTest + @transactional 어노테이션의 역할은 테스트 코드가 정상적으로 실행되어 커밋이 되었을때 롤백을 하는 역할을 합니다.
현재 작성해주신 테스트 코드는 실패를 검증하는 테스트 코드이기 때문에 명시하지 않아도 될것 같아요 !

hongsumin added 19 commits January 21, 2025 21:59

feat(racingcar):Car 생성

7a209bb

feat(racingcar): move 기능 추가

66e2d15

feat(racingcar): 인터페이스 생성

958d190

docs : 요구사항 정의

d21602b

feat : 문자열 계산기에 숫자 이외의 값 또는 음수를 전달하는 경우 RuntimeException 예외 생성

6b98e7b

docs : 요구사항 수정

03f9aa1

refactor : StringCalculator add 함수 변경

d8ed15c

feat : 빈 문자열 또는 null을 입력할 경우 0을 반환 & 숫자 하나를 문자열로 입력할 경우 해당 숫자를 반환

80c2853

feat : 숫자 두개를 쉼표(,) ,(:} 구분자로 입력할 경우 두 숫자의 합을 반환

16da34d

feat : //와 \\n 문자 사이에 커스텀 구분자 추가

7a3eaa9

feat : 문자열 파싱 함수화

6112401

refactor : step1 피드백 반영 (클래스 세분화,양수 보장 클래스 생성)

620e32e

docs : 키친 포스 서비스의 요구사항 정의

7234252

docs(kitchenpos) : 피드백 반영 (용어 수정, 주문 타입 별 기능 세분화)

dc44547

feat(kitchenpos) : MenuGroupService 테스트 코드 작성

4ba5b69

feat(kitchenpos) : MenuService 테스트 코드 작성

9641b17

feat(kitchenpos) : ProductService 테스트 코드 작성

e814da7

feat(kitchenpos) : OrderTableService 테스트 코드 작성

791e5a3

feat(kitchenpos) : OrderService 테스트 코드 작성

893bdfe

sah3122 requested changes Feb 4, 2025

View reviewed changes

feat(kitchenpos) : 리뷰 기반 테스트 코드 수정 (Nested, TestFixture, Parameterize…

6f7f196

…dTest 활용)

sah3122 requested changes Feb 11, 2025

View reviewed changes

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

🚀 3단계 - 테스트를 통한 코드 보호 #825

🚀 3단계 - 테스트를 통한 코드 보호 #825

sumiini commented Feb 3, 2025

sah3122 left a comment

sah3122 Feb 4, 2025

sah3122 Feb 4, 2025

sumiini Feb 5, 2025

sah3122 Feb 11, 2025

sah3122 Feb 4, 2025

sah3122 Feb 4, 2025

sah3122 Feb 4, 2025

sah3122 Feb 4, 2025

sah3122 Feb 4, 2025

sah3122 Feb 4, 2025

sah3122 Feb 11, 2025

sumiini commented Feb 10, 2025

sah3122 left a comment

sah3122 Feb 11, 2025

sah3122 Feb 11, 2025

	void canNotCreateMenuIfMenuPriceIsNull() {
	void `메뉴생성_실패_가격_Null`() {

🚀 3단계 - 테스트를 통한 코드 보호 #825

Are you sure you want to change the base?

🚀 3단계 - 테스트를 통한 코드 보호 #825

Conversation

sumiini commented Feb 3, 2025

sah3122 left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

sumiini commented Feb 10, 2025

sah3122 left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment