fix: pr comments

lib/HttpClient.js

@@ -370,7 +370,8 @@ function rawRequest(opts, cb) {
         clearTimeout(requestTimer);
         opts.client._decrementInflightRequests();
 
-        // in an error scenario, it's possible the connection died and we were
+        // in an error scenario, it's possible the connection died and the
+        // response's `end` event never fired. if that's the case, we were
         // never able to set timings/metrics methods on the req object.
         if (!req.getTimings || !req.getMetrics) {
             eventTimes.endAt = process.hrtime();
@@ -381,7 +382,7 @@ function rawRequest(opts, cb) {
             opts.client.emit('timings', timings);
 
             var metrics = {
-                statusCode: NaN,
+                statusCode: null,
                 method: req.method,
                 path: opts.path,
                 url: opts.href,

test/index.js

@@ -568,7 +568,7 @@ describe('restify-client tests', function () {
 
             METRICS_CLIENT_HOST.once('metrics', function (metrics) {
                 assert.isObject(metrics);
-                assert.strictEqual(metrics.statusCode.toString(), 'NaN');
+                assert.strictEqual(metrics.statusCode, null);
                 assert.strictEqual(metrics.method, 'GET');
                 assert.strictEqual(metrics.path, '/econnreset');
                 assert.include(metrics.url, 'http://localhost');