test: simplify test-http-client-unescaped-path

rvagg · MylesBorins · commit ada7166dfd3d · 2017-03-08T17:10:15.000-08:00
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>
diff --git a/test/parallel/test-http-client-unescaped-path.js b/test/parallel/test-http-client-unescaped-path.js
@@ -3,12 +3,8 @@ const common = require('../common');
 const assert = require('assert');
 const http = require('http');
 
-function* bad() {
- for (let i = 0; i <= 32; i += 1)
- yield 'bad' + String.fromCharCode(i) + 'path';
-}
-
-for (const path of bad()) {
+for (let i = 0; i <= 32; i += 1) {
+ const path = 'bad' + String.fromCharCode(i) + 'path';
 assert.throws(() => http.get({ path }, common.fail),
 /contains unescaped characters/);
 }

-Original file line number
+Diff line change
 constassert=require('assert');
 consthttp=require('http');
 -function*bad(){
 -for(leti=0;i<=32;i+=1)
 -yield'bad'+String.fromCharCode(i)+'path';
 -}
+-
 -for(constpathofbad()){
 +for(leti=0;i<=32;i+=1){
 +constpath='bad'+String.fromCharCode(i)+'path';
 assert.throws(()=>http.get({ path },common.fail),
 /containsunescapedcharacters/);
+}