diff options
| author | Daijiro Wachi <daijiro.wachi@gmail.com> | 2017-02-13 21:51:19 +0100 |
|---|---|---|
| committer | James M Snell <jasnell@gmail.com> | 2017-02-16 14:49:59 -0800 |
| commit | eec216fb27aabbdeb27c73b1ddad6c16b551f4f1 (patch) | |
| tree | 7133f6057f8c98cabc2c81c99aa8142c4cc19aa0 /test | |
| parent | 5ddf72290b3035832fcdb4bbbc8f4b10931ebecf (diff) | |
| download | android-node-v8-eec216fb27aabbdeb27c73b1ddad6c16b551f4f1.tar.gz android-node-v8-eec216fb27aabbdeb27c73b1ddad6c16b551f4f1.tar.bz2 android-node-v8-eec216fb27aabbdeb27c73b1ddad6c16b551f4f1.zip | |
test: cases to querystring related to empty string
+ Add cases to `qs.stringify` that return empty string
+ Add cases to `qs.parse` when `sep` or `eq` is empty
PR-URL: https://github.com/nodejs/node/pull/11329
Reviewed-By: Yuta Hiroto <hello@about-hiroppy.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Diffstat (limited to 'test')
| -rw-r--r-- | test/parallel/test-querystring.js | 12 |
1 files changed, 12 insertions, 0 deletions
diff --git a/test/parallel/test-querystring.js b/test/parallel/test-querystring.js index 3f397577df..3b42cff718 100644 --- a/test/parallel/test-querystring.js +++ b/test/parallel/test-querystring.js @@ -257,8 +257,20 @@ assert.doesNotThrow(function() { assert.strictEqual(f, 'a:b;q:x%3Ay%3By%3Az'); } +// empty string +assert.strictEqual(qs.stringify(), ''); +assert.strictEqual(qs.stringify(0), ''); +assert.strictEqual(qs.stringify([]), ''); +assert.strictEqual(qs.stringify(null), ''); +assert.strictEqual(qs.stringify(true), ''); + check(qs.parse(), {}); +// empty sep +check(qs.parse('a', []), { a: '' }); + +// empty eq +check(qs.parse('a', null, []), { '': 'a' }); // Test limiting assert.strictEqual( |