Improved tests and correctness.

Author: Nyholm

spec/LoggerPluginSpec.php

@@ -45,8 +45,8 @@ function it_logs_request_and_response(
             "Received response:\n200 OK 1.1\n\nfor request:\nGET / 1.1",
             Argument::that(
                 function(array $context) use ($request, $response) {
-                    return $context['request'] === $request
-                        && $context['response'] === $response
+                    return $context['request'] === $request->getWrappedObject()
+                        && $context['response'] === $response->getWrappedObject()
                         && is_int($context['milliseconds'])
                     ;
                 }
@@ -71,7 +71,7 @@ function it_logs_exception(LoggerInterface $logger, Formatter $formatter, Reques
             "Error:\nCannot connect\nwhen sending request:\nGET / 1.1",
             Argument::that(
                 function(array $context) use ($request, $exception) {
-                    return $context['request'] === $request
+                    return $context['request'] === $request->getWrappedObject()
                         && $context['exception'] === $exception
                         && is_int($context['milliseconds'])
                     ;
@@ -102,8 +102,8 @@ function it_logs_response_within_exception(
             "Error:\nForbidden\nwith response:\n403 Forbidden 1.1\n\nwhen sending request:\nGET / 1.1",
             Argument::that(
                 function(array $context) use ($request, $response, $exception) {
-                    return $context['request'] === $request
-                        && $context['response'] === $response
+                    return $context['request'] === $request->getWrappedObject()
+                        && $context['response'] === $response->getWrappedObject()
                         && $context['exception'] === $exception
                         && is_int($context['milliseconds'])
                         ;

src/LoggerPlugin.php

@@ -32,11 +32,11 @@ public function __construct(LoggerInterface $logger, Formatter $formatter = null
      */
     public function handleRequest(RequestInterface $request, callable $next, callable $first)
     {
-        $start = microtime();
+        $start = microtime(true);
         $this->logger->info(sprintf("Sending request:\n%s", $this->formatter->formatRequest($request)), ['request' => $request]);
 
         return $next($request)->then(function (ResponseInterface $response) use ($request, $start) {
-            $milliseconds = round(microtime() - $start / 1000);
+            $milliseconds = (int) round((microtime(true) - $start) * 1000);
             $this->logger->info(
                 sprintf("Received response:\n%s\n\nfor request:\n%s", $this->formatter->formatResponse($response), $this->formatter->formatRequest($request)),
                 [
@@ -48,7 +48,7 @@ public function handleRequest(RequestInterface $request, callable $next, callabl
 
             return $response;
         }, function (Exception $exception) use ($request, $start) {
-            $milliseconds = round(microtime() - $start / 1000);
+            $milliseconds = (int) round((microtime(true) - $start) * 1000);
             if ($exception instanceof Exception\HttpException) {
                 $this->logger->error(
                     sprintf("Error:\n%s\nwith response:\n%s\n\nwhen sending request:\n%s", $exception->getMessage(), $this->formatter->formatResponse($exception->getResponse()), $this->formatter->formatRequest($request)),