The test directory had linting for undefined variables disabled. It is enabled everywhere else in the code base. Let's disable the fule for individual lines in the handful of tests that use undefined variables. PR-URL: https://github.com/nodejs/node/pull/6255 Reviewed-By: Santiago Gimeno <santiago.gimeno@gmail.com> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Roman Reiss <me@silverwind.io>
28 lines
607 B
JavaScript
28 lines
607 B
JavaScript
/* eslint-disable strict */
|
|
require('../common');
|
|
var assert = require('assert');
|
|
|
|
/*
|
|
In Node.js 0.10, a bug existed that caused strict functions to not capture
|
|
their environment when evaluated. When run in 0.10 `test()` fails with a
|
|
`ReferenceError`. See https://github.com/nodejs/node/issues/2245 for details.
|
|
*/
|
|
|
|
function test() {
|
|
|
|
var code = [
|
|
'var foo = {m: 1};',
|
|
'',
|
|
'function bar() {',
|
|
'\'use strict\';',
|
|
'return foo; // foo isn\'t captured in 0.10',
|
|
'};'
|
|
].join('\n');
|
|
|
|
eval(code);
|
|
|
|
return bar(); // eslint-disable-line no-undef
|
|
|
|
}
|
|
|
|
assert.deepEqual(test(), {m: 1});
|