test: complete coverage of querystring

PR-URL: nodejs#38520 Refs: https://coverage.nodejs.org/coverage-68e6673224365120/lib/querystring.js.html#L179 Reviewed-By: Rich Trott <[email protected]> Reviewed-By: James M Snell <[email protected]> Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Zijian Liu <[email protected]> Reviewed-By: Darshan Sen <[email protected]>
Ayase-252 · May 5, 2021 · efd70f4 · efd70f4
1 parent 7d74487
commit efd70f4
Showing 1 changed file with 9 additions and 0 deletions.
diff --git a/test/parallel/test-querystring.js b/test/parallel/test-querystring.js
@@ -307,6 +307,7 @@ assert.strictEqual(qs.stringify({ foo: -0 }), 'foo=0');
 assert.strictEqual(qs.stringify({ foo: 3 }), 'foo=3');
 assert.strictEqual(qs.stringify({ foo: -72.42 }), 'foo=-72.42');
 assert.strictEqual(qs.stringify({ foo: NaN }), 'foo=');
+assert.strictEqual(qs.stringify({ foo: 1e21 }), 'foo=1e%2B21');
 assert.strictEqual(qs.stringify({ foo: Infinity }), 'foo=');
 
 // nested
@@ -450,6 +451,14 @@ check(qs.parse('%\u0100=%\u0101'), { '%Ā': '%ā' });
     'a=a&b=b&c=c');
 }
 
+// Test custom encode for different types
+{
+  const obj = { number: 1, bigint: 2n, true: true, false: false, object: {} };
+  assert.strictEqual(
+    qs.stringify(obj, null, null, { encodeURIComponent: (v) => v }),
+    'number=1&bigint=2&true=true&false=false&object=');
+}
+
 // Test QueryString.unescapeBuffer
 qsUnescapeTestCases.forEach((testCase) => {
   assert.strictEqual(qs.unescape(testCase[0]), testCase[1]);