fix: Refined `ChatBackendControllerTest` and fixed a bug in `ChatRoom`

[demos/kafka/chat] / src / main / java / de / juplo / kafka / chat / backend / domain / ChatHome.java

diff --git a/src/main/java/de/juplo/kafka/chat/backend/domain/ChatHome.java b/src/main/java/de/juplo/kafka/chat/backend/domain/ChatHome.java
index 2b5152c..1450d3e 100644 (file)
--- a/src/main/java/de/juplo/kafka/chat/backend/domain/ChatHome.java
+++ b/src/main/java/de/juplo/kafka/chat/backend/domain/ChatHome.java
@@ -1,31 +1,33 @@
 package de.juplo.kafka.chat.backend.domain;
 
 import lombok.RequiredArgsConstructor;
+import lombok.extern.slf4j.Slf4j;
+import reactor.core.publisher.Flux;
+import reactor.core.publisher.Mono;
 
 import java.util.*;
 
 
+@Slf4j
 @RequiredArgsConstructor
 public class ChatHome
 {
-  private final Map<UUID, Chatroom> chatrooms;
-  private final ChatroomFactory factory;
+  private final ChatHomeService service;
 
-
-  public Chatroom createChatroom(String name)
+  public Mono<ChatRoom> createChatroom(String name)
   {
-    Chatroom chatroom = factory.createChatroom(UUID.randomUUID(), name);
-    chatrooms.put(chatroom.getId(), chatroom);
-    return chatroom;
+    return service.createChatRoom(UUID.randomUUID(), name);
   }
 
-  public Optional<Chatroom> getChatroom(UUID id)
+  public Mono<ChatRoom> getChatRoom(UUID id)
   {
-    return Optional.ofNullable(chatrooms.get(id));
+    return service
+        .getChatRoom(id)
+        .switchIfEmpty(Mono.error(() -> new UnknownChatroomException(id)));
   }
 
-  public Collection<Chatroom> list()
+  public Flux<ChatRoom> getChatRooms()
   {
-    return chatrooms.values();
+    return service.getChatRooms();
   }
 }