Remove fs/path globals from test suite
It is way too easy to not notice bugs due to those modules being exported as globals while running the test suite. See: http://github.com/ry/node/issues#issue/75
This commit is contained in:
parent
e898c1f2e3
commit
44dde5f24c
@ -11,6 +11,4 @@ var assert = require('assert');
|
|||||||
var sys = require("sys");
|
var sys = require("sys");
|
||||||
|
|
||||||
process.mixin(exports, sys);
|
process.mixin(exports, sys);
|
||||||
exports.assert = require('assert');
|
exports.assert = require('assert');
|
||||||
exports.fs = require("fs");
|
|
||||||
exports.path = path;
|
|
@ -1,9 +1,7 @@
|
|||||||
process.mixin(require("../common"));
|
process.mixin(require("../common"));
|
||||||
|
|
||||||
var dirname = path.dirname(__filename);
|
assert.equal(true, process.cwd() !== __dirname);
|
||||||
|
|
||||||
assert.equal(true, process.cwd() !== dirname);
|
process.chdir(__dirname);
|
||||||
|
|
||||||
process.chdir(dirname);
|
assert.equal(true, process.cwd() === __dirname);
|
||||||
|
|
||||||
assert.equal(true, process.cwd() === dirname);
|
|
||||||
|
@ -1,4 +1,5 @@
|
|||||||
process.mixin(require("../common"));
|
process.mixin(require("../common"));
|
||||||
|
var path = require('path');
|
||||||
var testTxt = path.join(fixturesDir, "x.txt");
|
var testTxt = path.join(fixturesDir, "x.txt");
|
||||||
var fs = require('fs');
|
var fs = require('fs');
|
||||||
|
|
||||||
|
@ -1,5 +1,5 @@
|
|||||||
process.mixin(require("../common"));
|
process.mixin(require("../common"));
|
||||||
|
var fs = require('fs');
|
||||||
var N = 100;
|
var N = 100;
|
||||||
var j = 0;
|
var j = 0;
|
||||||
|
|
||||||
|
@ -1,4 +1,6 @@
|
|||||||
process.mixin(require("../common"));
|
process.mixin(require("../common"));
|
||||||
|
var path = require('path');
|
||||||
|
var fs = require('fs');
|
||||||
var got_error = false;
|
var got_error = false;
|
||||||
|
|
||||||
var filename = path.join(fixturesDir, "does_not_exist.txt");
|
var filename = path.join(fixturesDir, "does_not_exist.txt");
|
||||||
|
@ -1,6 +1,8 @@
|
|||||||
process.mixin(require('../common'));
|
process.mixin(require('../common'));
|
||||||
|
|
||||||
var
|
var
|
||||||
|
path = require('path'),
|
||||||
|
fs = require('fs'),
|
||||||
fn = path.join(fixturesDir, 'multipart.js'),
|
fn = path.join(fixturesDir, 'multipart.js'),
|
||||||
file = fs.createReadStream(fn),
|
file = fs.createReadStream(fn),
|
||||||
|
|
||||||
|
@ -1,6 +1,8 @@
|
|||||||
process.mixin(require('../common'));
|
process.mixin(require('../common'));
|
||||||
|
|
||||||
var
|
var
|
||||||
|
path = require('path'),
|
||||||
|
fs = require('fs'),
|
||||||
fn = path.join(fixturesDir, "write.txt"),
|
fn = path.join(fixturesDir, "write.txt"),
|
||||||
file = fs.createWriteStream(fn),
|
file = fs.createWriteStream(fn),
|
||||||
|
|
||||||
|
@ -1,5 +1,6 @@
|
|||||||
process.mixin(require("../common"));
|
process.mixin(require("../common"));
|
||||||
|
var path = require('path');
|
||||||
|
var fs = require('fs');
|
||||||
var got_error = false;
|
var got_error = false;
|
||||||
var success_count = 0;
|
var success_count = 0;
|
||||||
|
|
||||||
|
@ -1,5 +1,6 @@
|
|||||||
process.mixin(require("../common"));
|
process.mixin(require("../common"));
|
||||||
|
var fs = require('fs');
|
||||||
|
var path = require('path');
|
||||||
var async_completed = 0, async_expected = 0, unlink = [];
|
var async_completed = 0, async_expected = 0, unlink = [];
|
||||||
|
|
||||||
function asynctest(testBlock, args, callback, assertBlock) {
|
function asynctest(testBlock, args, callback, assertBlock) {
|
||||||
|
@ -1,5 +1,5 @@
|
|||||||
process.mixin(require("../common"));
|
process.mixin(require("../common"));
|
||||||
|
var fs = require('fs');
|
||||||
var got_error = false;
|
var got_error = false;
|
||||||
var success_count = 0;
|
var success_count = 0;
|
||||||
|
|
||||||
|
@ -1,5 +1,6 @@
|
|||||||
process.mixin(require("../common"));
|
process.mixin(require("../common"));
|
||||||
|
var path = require('path');
|
||||||
|
var fs = require('fs');
|
||||||
var completed = 0;
|
var completed = 0;
|
||||||
|
|
||||||
// test creating and reading symbolic link
|
// test creating and reading symbolic link
|
||||||
|
@ -1,5 +1,6 @@
|
|||||||
process.mixin(require("../common"));
|
process.mixin(require("../common"));
|
||||||
|
var path = require('path');
|
||||||
|
var fs = require('fs');
|
||||||
var fn = path.join(fixturesDir, "write.txt");
|
var fn = path.join(fixturesDir, "write.txt");
|
||||||
var expected = "hello";
|
var expected = "hello";
|
||||||
var found;
|
var found;
|
||||||
|
@ -1,5 +1,6 @@
|
|||||||
process.mixin(require("../common"));
|
process.mixin(require("../common"));
|
||||||
require("fs");
|
var path = require('path');
|
||||||
|
var fs = require("fs");
|
||||||
parse = require("ini").parse;
|
parse = require("ini").parse;
|
||||||
|
|
||||||
debug("load fixtures/fixture.ini");
|
debug("load fixtures/fixture.ini");
|
||||||
|
@ -1,4 +1,6 @@
|
|||||||
process.mixin(require("../common"));
|
process.mixin(require("../common"));
|
||||||
|
var path = require('path');
|
||||||
|
var fs = require('fs');
|
||||||
|
|
||||||
var dirname = path.dirname(__filename);
|
var dirname = path.dirname(__filename);
|
||||||
var fixtures = path.join(dirname, "../fixtures");
|
var fixtures = path.join(dirname, "../fixtures");
|
||||||
|
@ -1,4 +1,5 @@
|
|||||||
process.mixin(require("../common"));
|
process.mixin(require("../common"));
|
||||||
|
var path = require('path');
|
||||||
|
|
||||||
debug("load test-module-loading.js");
|
debug("load test-module-loading.js");
|
||||||
|
|
||||||
|
@ -1,4 +1,6 @@
|
|||||||
process.mixin(require("../common"));
|
process.mixin(require("../common"));
|
||||||
|
var path = require('path');
|
||||||
|
var fs = require('fs');
|
||||||
|
|
||||||
var got_error = false,
|
var got_error = false,
|
||||||
readdirDir = path.join(fixturesDir, "readdir")
|
readdirDir = path.join(fixturesDir, "readdir")
|
||||||
|
@ -1,4 +1,5 @@
|
|||||||
process.mixin(require("../common"));
|
process.mixin(require("../common"));
|
||||||
|
var path = require('path');
|
||||||
|
|
||||||
var sub = path.join(fixturesDir, 'echo.js');
|
var sub = path.join(fixturesDir, 'echo.js');
|
||||||
|
|
||||||
|
@ -1,4 +1,5 @@
|
|||||||
process.mixin(require("../common"));
|
process.mixin(require("../common"));
|
||||||
|
var path = require('path');
|
||||||
|
|
||||||
var sub = path.join(fixturesDir, 'print-chars.js');
|
var sub = path.join(fixturesDir, 'print-chars.js');
|
||||||
|
|
||||||
|
@ -1,4 +1,6 @@
|
|||||||
process.mixin(require('../common'));
|
process.mixin(require('../common'));
|
||||||
|
var path = require('path');
|
||||||
|
var fs = require('fs');
|
||||||
|
|
||||||
var fixture = path.join(__dirname, "../fixtures/x.txt");
|
var fixture = path.join(__dirname, "../fixtures/x.txt");
|
||||||
|
|
||||||
|
Loading…
x
Reference in New Issue
Block a user