Browse Source

test: fix redeclared vars in test-url

PR-URL: https://github.com/nodejs/node/pull/4993
Reviewed-By: Michaël Zasso <mic.besace@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
v4.x
Rich Trott 9 years ago
committed by Myles Borins
parent
commit
5545dafdc3
  1. 26
      test/parallel/test-url.js

26
test/parallel/test-url.js

@ -856,10 +856,10 @@ var parseTests = {
}; };
for (var u in parseTests) { for (const u in parseTests) {
var actual = url.parse(u); let actual = url.parse(u);
var spaced = url.parse(' \t ' + u + '\n\t'); const spaced = url.parse(` \t ${u}\n\t`);
var expected = parseTests[u]; let expected = parseTests[u];
Object.keys(actual).forEach(function(i) { Object.keys(actual).forEach(function(i) {
if (expected[i] === undefined && actual[i] === null) { if (expected[i] === undefined && actual[i] === null) {
@ -928,10 +928,10 @@ var parseTestsWithQueryString = {
href: '/example?query=value' href: '/example?query=value'
} }
}; };
for (var u in parseTestsWithQueryString) { for (const u in parseTestsWithQueryString) {
var actual = url.parse(u, true); const actual = url.parse(u, true);
var expected = parseTestsWithQueryString[u]; const expected = parseTestsWithQueryString[u];
for (var i in actual) { for (const i in actual) {
if (actual[i] === null && expected[i] === undefined) { if (actual[i] === null && expected[i] === undefined) {
expected[i] = null; expected[i] = null;
} }
@ -1130,11 +1130,11 @@ var formatTests = {
pathname: '/fooA100%mBr', pathname: '/fooA100%mBr',
} }
}; };
for (var u in formatTests) { for (const u in formatTests) {
var expect = formatTests[u].href; const expect = formatTests[u].href;
delete formatTests[u].href; delete formatTests[u].href;
var actual = url.format(u); const actual = url.format(u);
var actualObj = url.format(formatTests[u]); const actualObj = url.format(formatTests[u]);
assert.equal(actual, expect, assert.equal(actual, expect,
'wonky format(' + u + ') == ' + expect + 'wonky format(' + u + ') == ' + expect +
'\nactual:' + actual); '\nactual:' + actual);
@ -1567,7 +1567,7 @@ var throws = [
0, 0,
function() {} function() {}
]; ];
for (var i = 0; i < throws.length; i++) { for (let i = 0; i < throws.length; i++) {
assert.throws(function() { url.format(throws[i]); }, TypeError); assert.throws(function() { url.format(throws[i]); }, TypeError);
} }
assert(url.format('') === ''); assert(url.format('') === '');

Loading…
Cancel
Save