Ensure we have fully decoded the request in the no mock early exit case.

lib/unexpectedMitm.js

@@ -800,6 +800,33 @@ module.exports = {
                                 unchunkedBody: Buffer.concat(result.body)
                             }, metadata);
 
+                            if (!hasRequestDescription) {
+                                // there was no mock so arrange "<no response>"
+                                assertMockResponse(null);
+
+                                /*
+                                 * We wish to cause the generation of a diff
+                                 * so we arrange to enter our 'success' path
+                                 * but ensure the delegated assertion fully
+                                 * completes by signalling an error condition.
+                                 * Given we later fail on "to satisfy" we pass
+                                 * a null fulfilment value.
+                                 *
+                                 * Note the use of the single reject/resolve
+                                 * behaviour of promises: while the delegated
+                                 * assertion is reject()ed, we have already
+                                 * resolve()d thus the rejection of the former
+                                 * is effectively ignored and we proceed with
+                                 * our output.
+                                 */
+
+                                // cancel the delegated assertion
+                                lastHijackedSocket.emit('error', new Error('unexpected-mitm: Saw unexpected requests.'));
+                                // continue with current assertion
+                                resolve([null, httpConversation, httpConversationSatisfySpec]);
+                                return [null, null];
+                            }
+
                             if (typeof responseProperties === 'function') {
                                 // reset the readable req stream state
                                 stream.Readable.call(req);
@@ -916,32 +943,6 @@ module.exports = {
                             });
                         }
 
-                        if (!hasRequestDescription) {
-                            // there was no mock so arrange "<no response>"
-                            assertMockResponse(null);
-
-                            /*
-                             * We wish to cause the generation of a diff
-                             * so we arrange to enter our 'success' path
-                             * but ensure the delegated assertion fully
-                             * completes by signalling an error condition.
-                             * Given we later fail on "to satisfy" we pass
-                             * a null fulfilment value.
-                             *
-                             * Note the use of the single reject/resolve
-                             * behaviour of promises: while the delegated
-                             * assertion is reject()ed, we have already
-                             * resolve()d thus the rejection of the former
-                             * is effectively ignored and we proceed with
-                             * our output.
-                             */
-
-                            // cancel the delegated assertion
-                            lastHijackedSocket.emit('error', new Error('unexpected-mitm: Saw unexpected requests.'));
-                            // continue with current assertion
-                            resolve([null, httpConversation, httpConversationSatisfySpec]);
-                        }
-
                         /*
                          * hook final write callback to record raw data and immediately
                          * assert request so it occurs prior to request completion