Refactor handling queryString in RequestLogger

Author: DrorBuhnik

jooby/src/main/java/org/jooby/RequestLogger.java

@@ -250,9 +250,8 @@ public void handle(final Request req, final Response rsp) throws Throwable {
       sb.append(Q).append(req.method());
       sb.append(SP);
       sb.append(req.path());
-      if (queryString && req.queryString().isPresent()) {
-        sb.append(QUERY);
-        sb.append(req.queryString().get());
+      if (queryString) {
+        req.queryString().ifPresent(s -> sb.append(QUERY).append(s));
       }
       sb.append(SP);
       sb.append(req.protocol());

jooby/src/test/java/org/jooby/RequestLoggerTest.java

@@ -172,7 +172,7 @@ private Block path(final String path) {
   private Block query(final String query) {
     return unit -> {
       Request req = unit.get(Request.class);
-      expect(req.queryString()).andReturn(Optional.of(query)).atLeastOnce();
+      expect(req.queryString()).andReturn(Optional.of(query));
     };
   }