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

README.md

@@ -45,13 +45,15 @@ docker compose -p kitchenpos up -d
 
 ### 주문
 
-- 주문 상태는 `대기중`, `접수됨`, `전달됨`, `배달중`, `배달됨`, `주문완료` 로 나타낼 수 있다.
+- 주문 방식은 `배달주문`, `포장주문`, `매장주문` 중 하나를 선택해야 한다.
+- 주문 상태는 주문 방식에 따라 다음과 같은 순서로 진행된다.
+  - `배달주문` 의 경우 `대기중`, `접수됨`, `전달됨`, `배달중`, `배달됨`, `주문완료` 순서로 진행된다.
+  - `포장주문`, `매장주문` 의 경우 `대기중`, `접수됨`, `전달됨`, `주문완료` 순서로 진행된다.
 - 주문을 생성할 수 있다.
-  - 주문 방식은 `배달주문`, `포장주문`, `매장식사` 중 하나를 선택해야 한다.
   - 주문은 최소 하나 이상의 주문 상품이 필요하다.
   - 등록된 메뉴에 없는 주문 상품은 주문할 수 없다.
   - 주문 상품의 수량이 0 미만이 될 수 없다.
-    - 단, `매장식사` 인 경우에는 주문 상품 수량을 0 미만으로 설정할 수 있다.
+    - 단, `매장주문` 인 경우에는 주문 상품 수량을 0 미만으로 설정할 수 있다.
   - 비노출 처리된 메뉴는 주문할 수 없다.
   - 주문 상품의 가격은 메뉴의 가격과 같아야 한다.
   - `배달주문` 인 경우에는 배달 주소가 있어야 한다.
@@ -74,8 +76,8 @@ docker compose -p kitchenpos up -d
   - 주문 배달 완료 처리가 되면 주문 상태는 `배달됨` 이 되어야 한다.
 - 주문 완료 처리를 할 수 있다.
   - `배달주문` 인 경우 `배달됨` 상태여야 주문 완료 처리를 할 수 있다.
-  - `포장주문`, `매장식사` 인 경우 `전달됨` 상태여야 주문 완료 처리를 할 수 있다.
-  - `매장식사` 인 경우 완료된 주문의 테이블은 공석 처리가 되어야 한다.
+  - `포장주문`, `매장주문` 인 경우 `전달됨` 상태여야 주문 완료 처리를 할 수 있다.
+  - `매장주문` 인 경우 완료된 주문의 테이블은 공석 처리가 되어야 한다.
   - 주문 완료 처리가 되면 주문 상태는 `완료됨` 이 되어야 한다.
 - 주문 목록을 조회할 수 있다.
 

build.gradle.kts

@@ -33,6 +33,7 @@ dependencies {
     runtimeOnly("com.mysql:mysql-connector-j")
     testImplementation("org.springframework.boot:spring-boot-starter-test")
     testImplementation("io.kotest:kotest-runner-junit5:5.8.1")
+    testImplementation("io.mockk:mockk:1.13.10")
 }
 
 tasks.withType<KotlinCompile> {

src/main/java/kitchenpos/application/OrderService.java

@@ -50,6 +50,7 @@ public Order create(final Order request) {
         if (Objects.isNull(orderLineItemRequests) || orderLineItemRequests.isEmpty()) {
             throw new IllegalArgumentException();
         }
+
         final List<Menu> menus = menuRepository.findAllByIdIn(
             orderLineItemRequests.stream()
                 .map(OrderLineItem::getMenuId)

src/main/java/kitchenpos/domain/JpaMenuGroupRepository.java

@@ -0,0 +1,8 @@
+package kitchenpos.domain;
+
+import org.springframework.data.jpa.repository.JpaRepository;
+
+import java.util.UUID;
+
+public interface JpaMenuGroupRepository extends MenuGroupRepository, JpaRepository<MenuGroup, UUID> {
+}

src/main/java/kitchenpos/domain/JpaMenuRepository.java

@@ -0,0 +1,19 @@
+package kitchenpos.domain;
+
+import jakarta.annotation.Nonnull;
+import java.util.List;
+import java.util.UUID;
+import org.springframework.data.jpa.repository.JpaRepository;
+import org.springframework.data.jpa.repository.Query;
+import org.springframework.data.repository.query.Param;
+
+public interface JpaMenuRepository extends MenuRepository, JpaRepository<Menu, UUID> {
+    @Nonnull
+    @Override
+    List<Menu> findAllByIdIn(@Nonnull List<UUID> ids);
+
+    @Nonnull
+    @Override
+    @Query("select m from Menu m join m.menuProducts mp where mp.product.id = :productId")
+    List<Menu> findAllByProductId(@Nonnull @Param("productId") UUID productId);
+}

src/main/java/kitchenpos/domain/JpaOrderRepository.java

@@ -0,0 +1,11 @@
+package kitchenpos.domain;
+
+import jakarta.annotation.Nonnull;
+import org.springframework.data.jpa.repository.JpaRepository;
+
+import java.util.UUID;
+
+public interface JpaOrderRepository extends OrderRepository, JpaRepository<Order, UUID> {
+    @Override
+    boolean existsByOrderTableAndStatusNot(@Nonnull OrderTable orderTable, @Nonnull OrderStatus status);
+}

src/main/java/kitchenpos/domain/JpaOrderTableRepository.java

@@ -0,0 +1,8 @@
+package kitchenpos.domain;
+
+import org.springframework.data.jpa.repository.JpaRepository;
+
+import java.util.UUID;
+
+public interface JpaOrderTableRepository extends OrderTableRepository, JpaRepository<OrderTable, UUID> {
+}

src/main/java/kitchenpos/domain/JpaProductRepository.java

@@ -0,0 +1,12 @@
+package kitchenpos.domain;
+
+import jakarta.annotation.Nonnull;
+import org.springframework.data.jpa.repository.JpaRepository;
+
+import java.util.List;
+import java.util.UUID;
+
+public interface JpaProductRepository extends ProductRepository, JpaRepository<Product, UUID> {
+    @Nonnull
+    List<Product> findAllByIdIn(@Nonnull List<UUID> ids);
+}

src/main/java/kitchenpos/domain/MenuGroupRepository.java

@@ -1,8 +1,17 @@
 package kitchenpos.domain;
 
-import org.springframework.data.jpa.repository.JpaRepository;
-
+import jakarta.annotation.Nonnull;
+import java.util.List;
+import java.util.Optional;
 import java.util.UUID;
 
-public interface MenuGroupRepository extends JpaRepository<MenuGroup, UUID> {
+public interface MenuGroupRepository {
+    @Nonnull
+    MenuGroup save(@Nonnull MenuGroup menuGroup);
+
+    @Nonnull
+    List<MenuGroup> findAll();
+
+    @Nonnull
+    Optional<MenuGroup> findById(@Nonnull UUID menuGroupId);
 }

src/main/java/kitchenpos/domain/MenuRepository.java

@@ -1,15 +1,23 @@
 package kitchenpos.domain;
 
-import org.springframework.data.jpa.repository.JpaRepository;
-import org.springframework.data.jpa.repository.Query;
-import org.springframework.data.repository.query.Param;
-
+import jakarta.annotation.Nonnull;
 import java.util.List;
+import java.util.Optional;
 import java.util.UUID;
 
-public interface MenuRepository extends JpaRepository<Menu, UUID> {
-    List<Menu> findAllByIdIn(List<UUID> ids);
+public interface MenuRepository {
+    @Nonnull
+    Menu save(@Nonnull Menu menu);
+
+    @Nonnull
+    List<Menu> findAll();
+
+    @Nonnull
+    List<Menu> findAllByIdIn(@Nonnull List<UUID> list);
+
+    @Nonnull
+    List<Menu> findAllByProductId(@Nonnull UUID productId);
 
-    @Query("select m from Menu m join m.menuProducts mp where mp.product.id = :productId")
-    List<Menu> findAllByProductId(@Param("productId") UUID productId);
+    @Nonnull
+    Optional<Menu> findById(@Nonnull UUID menuId);
 }

src/main/java/kitchenpos/domain/OrderRepository.java

@@ -1,9 +1,19 @@
 package kitchenpos.domain;
 
-import org.springframework.data.jpa.repository.JpaRepository;
-
+import jakarta.annotation.Nonnull;
+import java.util.List;
+import java.util.Optional;
 import java.util.UUID;
 
-public interface OrderRepository extends JpaRepository<Order, UUID> {
-    boolean existsByOrderTableAndStatusNot(OrderTable orderTable, OrderStatus status);
+public interface OrderRepository {
+    @Nonnull
+    Order save(@Nonnull Order order);
+
+    @Nonnull
+    Optional<Order> findById(@Nonnull UUID orderId);
+
+    boolean existsByOrderTableAndStatusNot(@Nonnull OrderTable orderTable, @Nonnull OrderStatus orderStatus);
+
+    @Nonnull
+    List<Order> findAll();
 }

src/main/java/kitchenpos/domain/OrderTableRepository.java

@@ -1,8 +1,17 @@
 package kitchenpos.domain;
 
-import org.springframework.data.jpa.repository.JpaRepository;
-
+import jakarta.annotation.Nonnull;
+import java.util.List;
+import java.util.Optional;
 import java.util.UUID;
 
-public interface OrderTableRepository extends JpaRepository<OrderTable, UUID> {
+public interface OrderTableRepository {
+    @Nonnull
+    Optional<OrderTable> findById(@Nonnull UUID orderTableId);
+
+    @Nonnull
+    OrderTable save(@Nonnull OrderTable orderTable);
+
+    @Nonnull
+    List<OrderTable> findAll();
 }

src/main/java/kitchenpos/domain/ProductRepository.java

@@ -1,10 +1,20 @@
 package kitchenpos.domain;
 
-import org.springframework.data.jpa.repository.JpaRepository;
-
+import jakarta.annotation.Nonnull;
 import java.util.List;
+import java.util.Optional;
 import java.util.UUID;
 
-public interface ProductRepository extends JpaRepository<Product, UUID> {
-    List<Product> findAllByIdIn(List<UUID> ids);
+public interface ProductRepository {
+    @Nonnull
+    List<Product> findAllByIdIn(@Nonnull List<UUID> list);
+
+    @Nonnull
+    Optional<Product> findById(@Nonnull UUID productId);
+
+    @Nonnull
+    List<Product> findAll();
+
+    @Nonnull
+    Product save(@Nonnull Product product);
 }

src/test/java/kitchenpos/application/MenuGroupServiceCreateTest.kt

@@ -0,0 +1,79 @@
+package kitchenpos.application
+
+import io.kotest.assertions.throwables.shouldThrowAny
+import io.kotest.core.spec.style.ShouldSpec
+import io.kotest.matchers.shouldBe
+import io.kotest.matchers.types.shouldBeTypeOf
+import java.util.UUID
+import kitchenpos.domain.MenuGroup
+import kitchenpos.testsupport.FakeMenuGroupRepository
+
+class MenuGroupServiceCreateTest : ShouldSpec({
+    lateinit var service: MenuGroupService
+
+    beforeTest {
+        service = MenuGroupService(
+            FakeMenuGroupRepository()
+        )
+    }
+
+    context("메뉴 그룹 생성") {
+        should("성공") {
+            // given
+            val request = createRequest(
+                id = null
+            )
+
+            // when
+            val result = service.create(request)
+
+            // then
+            result.name shouldBe request.name
+        }
+
+        should("실패 - 메뉴 그룹명이 입력되지 않은 경우") {
+            // given
+            val request = createRequest(
+                id = null,
+                name = null
+            )
+
+            // when
+            val result = shouldThrowAny {
+                service.create(request)
+            }
+
+            // then
+            result.shouldBeTypeOf<IllegalArgumentException>()
+        }
+
+        should("실패 - 메뉴 그룹명이 비어있는 경우") {
+            // given
+            val request = createRequest(
+                id = null,
+                name = ""
+            )
+
+            // when
+            val result = shouldThrowAny {
+                service.create(request)
+            }
+
+            // then
+            result.shouldBeTypeOf<IllegalArgumentException>()
+        }
+    }
+}) {
+    companion object {
+        private fun createRequest(
+            id: UUID? = UUID.randomUUID(),
+            name: String? = "test-menu-group-name"
+        ): MenuGroup {
+            return MenuGroup().apply {
+                this.id = id
+                this.name = name
+
+            }
+        }
+    }
+}

src/test/java/kitchenpos/application/MenuGroupServiceFindAllTest.kt

@@ -0,0 +1,41 @@
+package kitchenpos.application
+
+import io.kotest.core.spec.style.ShouldSpec
+import io.kotest.matchers.collections.shouldNotBeEmpty
+import io.kotest.matchers.should
+import io.kotest.matchers.shouldBe
+import kitchenpos.domain.MenuGroupRepository
+import kitchenpos.testsupport.FakeMenuGroupRepository
+import kitchenpos.testsupport.MenuGroupFixtures.createMenuGroup
+
+class MenuGroupServiceFindAllTest : ShouldSpec({
+    lateinit var menuGroupRepository: MenuGroupRepository
+    lateinit var service: MenuGroupService
+
+    beforeTest {
+        menuGroupRepository = FakeMenuGroupRepository()
+
+        service = MenuGroupService(
+            menuGroupRepository
+        )
+    }
+
+    context("메뉴 그룹 목록 조회") {
+        should("성공") {
+            // given
+            val savedMenuGroup = menuGroupRepository.save(
+                createMenuGroup()
+            )
+
+            // when
+            val result = service.findAll()
+
+            // then
+            result.shouldNotBeEmpty()
+            result.first() should { menuGroup ->
+                menuGroup.id shouldBe savedMenuGroup.id
+                menuGroup.name shouldBe savedMenuGroup.name
+            }
+        }
+    }
+})