url: use "empty" object for empty query strings

This makes things consistent with the way that the querystring
module creates parsed results.

PR-URL: https://github.com/nodejs/node/pull/6289
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Michaël Zasso <mic.besace@gmail.com>
Reviewed-By: Сковорода Никита Андреевич <chalkerx@gmail.com>
Reviewed-By: Minwoo Jung <jmwsoft@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>

1 files changed, 3 insertions, 1 deletions

diff --git a/test/parallel/test-url.js b/test/parallel/test-url.js
index 760303e98b..1499fb8733 100644
--- a/test/parallel/test-url.js
+++ b/test/parallel/test-url.js
@@ -927,7 +927,7 @@ var parseTestsWithQueryString = {
     path: '/example',
     href: '/example'
   },
-  '/example?query=value':{
+  '/example?query=value': {
     protocol: null,
     slashes: null,
     auth: null,
@@ -951,6 +951,8 @@ for (const u in parseTestsWithQueryString) {
     }
   }
 
+  assert.notStrictEqual(Object.getPrototypeOf(actual.query), Object.prototype);
+
   assert.deepEqual(actual, expected);
 }