Commit 5367336
test: simplify test-http-client-unescaped-path
PR-URL: #9649
Reviewed-By: Trevor Norris <trev.norris@gmail.com>
Reviewed-By: Santiago Gimeno <santiago.gimeno@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Anna Henningsen <anna@addaleax.net>1 parent 15231aa commit 5367336
1 file changed
Lines changed: 2 additions & 6 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -3,12 +3,8 @@ const common = require('../common'); | |||
| 3 | 3 | const assert = require('assert'); | |
| 4 | 4 | const http = require('http'); | |
| 5 | 5 | ||
| 6 | - function* bad() { | ||
| 7 | - for (let i = 0; i <= 32; i += 1) | ||
| 8 | - yield 'bad' + String.fromCharCode(i) + 'path'; | ||
| 9 | - } | ||
| 10 | - | ||
| 11 | - for (const path of bad()) { | ||
| 6 | + for (let i = 0; i <= 32; i += 1) { | ||
| 7 | + const path = 'bad' + String.fromCharCode(i) + 'path'; | ||
| 12 | 8 | assert.throws(() => http.get({ path }, common.fail), | |
| 13 | 9 | /contains unescaped characters/); | |
| 14 | 10 | } | |
0 commit comments