feat: 로그인 상태에서 '/user/list' GET 요청 시 사용자 정보 조회하는 기능 구현

- UserListController가 '/user/list' 요청 시 GET 요청을 받는다. 이후 DB에서 값을 꺼내 템플릿화하여 반환한다.
- templateUtils를 통해 Handlebars를 만든다. list와 profile html 파일 역시 수정해 두었다.
- templateUtils와 UserListController에 대한 테스트 코드를 작성하였다.

src/main/java/controller/ControllerMapper.java

@@ -10,6 +10,7 @@ public class ControllerMapper {
 
     static {
         controllers.put("/user/create", new CreateUserController());
+        controllers.put("/user/list", new UserListController());
         controllers.put("/user/login", new LoginController());
         controllers.put("/", new IndexController());
     }

src/main/java/controller/UserListController.java

@@ -0,0 +1,39 @@
+package controller;
+
+import com.github.jknack.handlebars.Template;
+import db.DataBase;
+import exception.RequestParameterNotFoundException;
+import model.User;
+import utils.TemplateUtils;
+import web.request.HttpRequest;
+import web.response.HttpResponse;
+
+import java.io.IOException;
+import java.util.Collection;
+import java.util.Objects;
+
+public class UserListController extends AbstractController {
+    public static final String LOGIN_HTML_PATH = "/user/login.html";
+
+    @Override
+    protected void doGet(HttpRequest request, HttpResponse response) {
+        try {
+            validate(request);
+            String path = request.getTarget();
+            Template template = TemplateUtils.buildTemplate(path);
+            Collection<User> users = DataBase.findAll();
+            String result = template.apply(users);
+            response.ok(result);
+        } catch (IllegalAccessException | RequestParameterNotFoundException | IOException e) {
+            logger.error(e.getMessage());
+            response.found(LOGIN_HTML_PATH);
+        }
+    }
+
+    private void validate(HttpRequest request) throws IllegalAccessException {
+        String cookie = request.getCookieByKey("logined");
+        if (Objects.isNull(cookie) || cookie.isEmpty() || !Boolean.parseBoolean(cookie)) {
+            throw new IllegalAccessException("로그인하지 않은 사용자입니다.");
+        }
+    }
+}

src/main/java/web/HttpHeader.java

@@ -9,6 +9,7 @@ public class HttpHeader {
     public static final String CONTENT_TYPE = "Content-Type";
     public static final String LOCATION = "Location";
     public static final String SET_COOKIE = "Set-Cookie";
+    public static final String COOKIE = "Cookie";
     public static final String HEADER_DELIMITER = ": ";
 
     private final Map<String, String> headers;

src/main/java/web/request/Cookies.java

@@ -0,0 +1,32 @@
+package web.request;
+
+import exception.RequestParameterNotFoundException;
+
+import java.util.HashMap;
+import java.util.Map;
+import java.util.Objects;
+
+public class Cookies {
+    private static final String COOKIES_DELIMITER = "; ";
+    private static final String COOKIE_DELIMITER = "=";
+
+    private final Map<String, String> cookieMatcher = new HashMap<>();
+
+    public Cookies(String cookies) {
+        if (Objects.isNull(cookies) || cookies.isEmpty()) {
+            return;
+        }
+        String[] tokens = cookies.split(COOKIES_DELIMITER);
+        for (String token : tokens) {
+            String[] value = token.split(COOKIE_DELIMITER);
+            this.cookieMatcher.put(value[0], value[1]);
+        }
+    }
+
+    public String getCookieByKey(String key) {
+        if (cookieMatcher.containsKey(key)) {
+            return cookieMatcher.get(key);
+        }
+        throw new RequestParameterNotFoundException("[COOKIE]" + key);
+    }
+}

src/main/java/web/request/HttpRequest.java

@@ -21,13 +21,15 @@ public class HttpRequest {
     private final RequestLine requestLine;
     private final HttpHeader httpHeader;
     private final RequestBody requestBody;
+    private final Cookies cookies;
 
     public HttpRequest(InputStream inputStream) {
         try {
             BufferedReader request = new BufferedReader(new InputStreamReader(inputStream, StandardCharsets.UTF_8));
             requestLine = new RequestLine(request.readLine());
             httpHeader = new HttpHeader(mappingHeaders(request));
             requestBody = mappingBodies(request);
+            cookies = new Cookies(httpHeader.getHeaderByKey(HttpHeader.COOKIE));
         } catch (IndexOutOfBoundsException | NullPointerException | IOException e) {
             throw new InvalidHttpRequestException();
         }
@@ -91,4 +93,8 @@ public String getRequestHeaderByKey(String key) {
     public String getRequestBodyByKey(String key) {
         return requestBody.getParameterByKey(key);
     }
+
+    public String getCookieByKey(String key) {
+        return cookies.getCookieByKey(key);
+    }
 }

src/main/java/web/response/HttpResponse.java

@@ -39,6 +39,13 @@ public void ok(String path, String contentType) throws IOException, URISyntaxExc
         write();
     }
 
+    public void ok(String path) {
+        responseLine = new ResponseLine(ResponseStatus.OK, HTTP_VERSION);
+        addHeader(HttpHeader.CONTENT_TYPE, "text/html;charset=UTF-8");
+        responseBody = new ResponseBody(path.getBytes());
+        write();
+    }
+
     public void found(String location) {
         responseLine = new ResponseLine(ResponseStatus.FOUND, HTTP_VERSION);
         addHeader(HttpHeader.LOCATION, location);

src/main/resources/templates/user/profile.html

@@ -27,7 +27,8 @@
                 <div class="input-group" style="max-width:470px;">
                     <input type="text" class="form-control" placeholder="Search" name="srch-term" id="srch-term">
                     <div class="input-group-btn">
-                        <button class="btn btn-default btn-primary" type="submit"><i class="glyphicon glyphicon-search"></i></button>
+                        <button class="btn btn-default btn-primary" type="submit"><i
+                                class="glyphicon glyphicon-search"></i></button>
                     </div>
                 </div>
             </form>
@@ -47,23 +48,27 @@
 <div class="navbar navbar-default" id="subnav">
     <div class="col-md-12">
         <div class="navbar-header">
-            <a href="#" style="margin-left:15px;" class="navbar-btn btn btn-default btn-plus dropdown-toggle" data-toggle="dropdown"><i class="glyphicon glyphicon-home" style="color:#dd1111;"></i> Home <small><i class="glyphicon glyphicon-chevron-down"></i></small></a>
+            <a class="navbar-btn btn btn-default btn-plus dropdown-toggle" data-toggle="dropdown" href="#"
+               style="margin-left:15px;"><i class="glyphicon glyphicon-home" style="color:#dd1111;"></i> Home <small><i
+                    class="glyphicon glyphicon-chevron-down"></i></small></a>
             <ul class="nav dropdown-menu">
-                <li><a href="../user/profile.html"><i class="glyphicon glyphicon-user" style="color:#1111dd;"></i> Profile</a></li>
+                <li><a href="../user/profile.html"><i class="glyphicon glyphicon-user" style="color:#1111dd;"></i>
+                    Profile</a></li>
                 <li class="nav-divider"></li>
                 <li><a href="#"><i class="glyphicon glyphicon-cog" style="color:#dd1111;"></i> Settings</a></li>
             </ul>
-            
+
             <button type="button" class="navbar-toggle" data-toggle="collapse" data-target="#navbar-collapse2">
-            	<span class="sr-only">Toggle navigation</span>
-            	<span class="icon-bar"></span>
-            	<span class="icon-bar"></span>
-            	<span class="icon-bar"></span>
-            </button>            
+                <span class="sr-only">Toggle navigation</span>
+                <span class="icon-bar"></span>
+                <span class="icon-bar"></span>
+                <span class="icon-bar"></span>
+            </button>
         </div>
         <div class="collapse navbar-collapse" id="navbar-collapse2">
             <ul class="nav navbar-nav navbar-right">
-                <li class="active"><a href="../index.html">Posts</a></li>>
+                <li class="active"><a href="../index.html">Posts</a></li>
+                >
                 <li><a href="../user/login.html" role="button">로그인</a></li>
                 <li><a href="../user/form.html" role="button">회원가입</a></li>
                 <li><a href="#" role="button">로그아웃</a></li>
@@ -84,9 +89,10 @@
                             <img class="media-object" src="../images/80-text.png">
                         </a>
                         <div class="media-body">
-                            <h4 class="media-heading">자바지기</h4>
+                            <h4 class="media-heading">{{name}}</h4>
                             <p>
-                                <a href="#" class="btn btn-xs btn-default"><span class="glyphicon glyphicon-envelope"></span>&nbsp;javajigi@slipp.net</a>
+                                <a class="btn btn-xs btn-default" href="#"><span
+                                        class="glyphicon glyphicon-envelope"></span>&nbsp;{{email}}</a>
                             </p>
                         </div>
                     </div>
@@ -100,5 +106,5 @@ <h4 class="media-heading">자바지기</h4>
 <script src="../js/jquery-2.2.0.min.js"></script>
 <script src="../js/bootstrap.min.js"></script>
 <script src="../js/scripts.js"></script>
-	</body>
+</body>
 </html>

src/test/java/controller/UserListControllerTest.java

@@ -0,0 +1,77 @@
+package controller;
+
+import db.DataBase;
+import model.User;
+import org.junit.jupiter.api.DisplayName;
+import org.junit.jupiter.api.Test;
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
+import web.request.HttpRequest;
+import web.response.HttpResponse;
+
+import java.io.*;
+
+import static controller.AbstractControllerTest.TEST_DIRECTORY;
+import static org.assertj.core.api.Assertions.assertThat;
+import static org.junit.jupiter.api.Assertions.assertAll;
+
+public class UserListControllerTest {
+    private static final Logger logger = LoggerFactory.getLogger(UserListControllerTest.class);
+
+    @DisplayName("GET으로 '/user/list'에 접근할 시, 로그인이 되어있다면 목록을 표시한다.")
+    @Test
+    void doGetWithLoginTest() {
+        User kafka = new User("kafka123", "password", "kafka", "kafka@naver.com");
+        User pobi = new User("javajigi", "password", "자바지기", "javajigi@gmail.com");
+        DataBase.addUser(kafka);
+        DataBase.addUser(pobi);
+        Controller controller = new UserListController();
+        try {
+            InputStream inputStream = new FileInputStream(new File(TEST_DIRECTORY + "HTTP_USER_LIST_LOGIN.txt"));
+            HttpRequest request = new HttpRequest(inputStream);
+
+            ByteArrayOutputStream outputStream = new ByteArrayOutputStream();
+            HttpResponse response = new HttpResponse(outputStream);
+
+            controller.service(request, response);
+
+            String result = outputStream.toString();
+            assertAll(
+                    () -> assertThat(result).contains("HTTP/1.1 200 OK"),
+                    () -> assertThat(result).contains(kafka.getName()),
+                    () -> assertThat(result).contains(kafka.getEmail()),
+                    () -> assertThat(result).contains(kafka.getUserId()),
+                    () -> assertThat(result).contains(pobi.getName()),
+                    () -> assertThat(result).contains(pobi.getEmail()),
+                    () -> assertThat(result).contains(pobi.getUserId())
+            );
+        } catch (IOException e) {
+            logger.error(e.getMessage());
+            throw new AssertionError();
+        }
+    }
+
+    @DisplayName("GET으로 '/user/list'에 접근할 시, 로그인이 되어있지 않다면 로그인 페이지로 이동한다.")
+    @Test
+    void doGetWithLogoutTest() {
+        Controller controller = new UserListController();
+        try {
+            InputStream inputStream = new FileInputStream(new File(TEST_DIRECTORY + "HTTP_USER_LIST_LOGOUT.txt"));
+            HttpRequest request = new HttpRequest(inputStream);
+
+            ByteArrayOutputStream outputStream = new ByteArrayOutputStream();
+            HttpResponse response = new HttpResponse(outputStream);
+
+            controller.service(request, response);
+
+            String result = outputStream.toString();
+            assertAll(
+                    () -> assertThat(result).contains("HTTP/1.1 302 FOUND"),
+                    () -> assertThat(result).contains("Location: /user/login.html ")
+            );
+        } catch (IOException e) {
+            logger.error(e.getMessage());
+            throw new AssertionError();
+        }
+    }
+}

src/test/java/utils/HandlebarsTest.java

@@ -1,9 +1,6 @@
 package utils;
 
-import com.github.jknack.handlebars.Handlebars;
 import com.github.jknack.handlebars.Template;
-import com.github.jknack.handlebars.io.ClassPathTemplateLoader;
-import com.github.jknack.handlebars.io.TemplateLoader;
 import db.DataBase;
 import model.User;
 import org.junit.jupiter.api.Test;
@@ -12,36 +9,39 @@
 
 import java.util.Collection;
 
+import static org.assertj.core.api.Assertions.assertThat;
+import static org.junit.jupiter.api.Assertions.assertAll;
+
 public class HandlebarsTest {
     private static final Logger logger = LoggerFactory.getLogger(HandlebarsTest.class);
 
     @Test
     void profileTest() throws Exception {
-        TemplateLoader loader = new ClassPathTemplateLoader();
-        loader.setPrefix("/templates");
-        loader.setSuffix(".html");
-        Handlebars handlebars = new Handlebars(loader);
-
-        Template template = handlebars.compile("user/profile");
+        Template template = TemplateUtils.buildTemplate("user/profile");
 
         User user = new User("javajigi", "password", "자바지기", "javajigi@gmail.com");
         String profilePage = template.apply(user);
         logger.debug("ProfilePage : {}", profilePage);
+        assertAll(
+                () -> assertThat(profilePage).contains(user.getName()),
+                () -> assertThat(profilePage).contains(user.getEmail()),
+                () -> assertThat(profilePage).contains(user.getUserId())
+        );
     }
 
     @Test
     void listTest() throws Exception {
-        TemplateLoader loader = new ClassPathTemplateLoader();
-        loader.setPrefix("/templates");
-        loader.setSuffix(".html");
-        Handlebars handlebars = new Handlebars(loader);
-
-        Template template = handlebars.compile("user/list");
+        Template template = TemplateUtils.buildTemplate("user/list");
 
         User user = new User("javajigi", "password", "자바지기", "javajigi@gmail.com");
         DataBase.addUser(user);
         Collection<User> users = DataBase.findAll();
         String listPage = template.apply(users);
         logger.debug("ProfilePage : {}", listPage);
+        assertAll(
+                () -> assertThat(listPage).contains(user.getName()),
+                () -> assertThat(listPage).contains(user.getEmail()),
+                () -> assertThat(listPage).contains(user.getUserId())
+        );
     }
 }

src/test/resources/controller/HTTP_USER_LIST_LOGIN.txt

@@ -0,0 +1,5 @@
+GET /user/list HTTP/1.1
+Host: localhost:8080
+Connection: keep-alive
+Accept: */*
+Cookie: logined=true

src/test/resources/controller/HTTP_USER_LIST_LOGOUT.txt

@@ -0,0 +1,5 @@
+GET /user/list HTTP/1.1
+Host: localhost:8080
+Connection: keep-alive
+Accept: */*
+Cookie: logined=false