From 6c1629d8f8dd74665522c38562f9abaa88c21a7b Mon Sep 17 00:00:00 2001
From: Neeraj Yadav <neeraj.nyd@gmail.com>
Date: Wed, 5 Sep 2018 00:18:18 +0530
Subject: [PATCH] Large test case broken down into components

---
 .../java/com/baeldung/AppHttpUnitTest.java    | 51 +++++++++++--------
 1 file changed, 29 insertions(+), 22 deletions(-)

diff --git a/ratpack/src/test/java/com/baeldung/AppHttpUnitTest.java b/ratpack/src/test/java/com/baeldung/AppHttpUnitTest.java
index 6e090f35e9..a97516dd74 100644
--- a/ratpack/src/test/java/com/baeldung/AppHttpUnitTest.java
+++ b/ratpack/src/test/java/com/baeldung/AppHttpUnitTest.java
@@ -5,6 +5,7 @@ import org.junit.Test;
 import ratpack.exec.Promise;
 import ratpack.func.Action;
 import ratpack.handling.Chain;
+import ratpack.handling.Handler;
 import ratpack.registry.Registry;
 import ratpack.test.embed.EmbeddedApp;
 import ratpack.test.exec.ExecHarness;
@@ -15,37 +16,43 @@ public class AppHttpUnitTest {
 
     @Test
     public void givenAnyUri_GetEmployeeFromSameRegistry() throws Exception {
-        final Action<Chain> chainAction = chain -> chain.prefix("employee/:id", empChain -> empChain.all(ctx -> {
+        Handler allHandler = ctx -> {
             Long id = Long.valueOf(ctx.getPathTokens()
-                    .get("id"));
+                .get("id"));
             Employee employee = new Employee(id, "Mr", "NY");
             ctx.next(Registry.single(Employee.class, employee));
-        })
-                .get("name", ctx -> {
-                    Employee employee = ctx.get(Employee.class);
-                    ctx.getResponse()
-                            .send("Name of employee with ID " + employee.getId() + " is " + employee.getName());
-                })
-                .get("title", ctx -> {
-                    Employee employee = ctx.get(Employee.class);
-                    ctx.getResponse()
-                            .send("Title of employee with ID " + employee.getId() + " is " + employee.getTitle());
-                }));
+        };
+        Handler empNameHandler = ctx -> {
+            Employee employee = ctx.get(Employee.class);
+            ctx.getResponse()
+                .send("Name of employee with ID " + employee.getId() + " is " + employee.getName());
+        };
+        Handler empTitleHandler = ctx -> {
+            Employee employee = ctx.get(Employee.class);
+            ctx.getResponse()
+                .send("Title of employee with ID " + employee.getId() + " is " + employee.getTitle());
+        };
+
+        Action<Chain> chainAction = chain -> chain.prefix("employee/:id", empChain -> {
+            empChain.all(allHandler)
+                .get("name", empNameHandler)
+                .get("title", empTitleHandler);
+        });
         EmbeddedApp.fromHandlers(chainAction)
-                .test(testHttpClient -> {
-                    assertEquals("Name of employee with ID 1 is NY", testHttpClient.get("employee/1/name")
-                            .getBody()
-                            .getText());
-                    assertEquals("Title of employee with ID 1 is Mr", testHttpClient.get("employee/1/title")
-                            .getBody()
-                            .getText());
-                });
+            .test(testHttpClient -> {
+                assertEquals("Name of employee with ID 1 is NY", testHttpClient.get("employee/1/name")
+                    .getBody()
+                    .getText());
+                assertEquals("Title of employee with ID 1 is Mr", testHttpClient.get("employee/1/title")
+                    .getBody()
+                    .getText());
+            });
     }
 
     @Test
     public void givenSyncDataSource_GetDataFromPromise() throws Exception {
         String value = ExecHarness.yieldSingle(execution -> Promise.sync(() -> "Foo"))
-                .getValueOrThrow();
+            .getValueOrThrow();
         assertEquals("Foo", value);
     }
 }