Browse Source

test: refresh temp directory when using pipe

common.PIPE resides in the temp directory (except on Windows). Insure
that the temp directory is refreshed in tests that use common.PIPE.

PR-URL: https://github.com/nodejs/node/pull/3231
Fixes: https://github.com/nodejs/node/issues/3227
Reviewed-By: Johan Bergström <bugs@bergstroem.nu>
v5.x
Rich Trott 9 years ago
parent
commit
a1040f2065
  1. 4
      test/parallel/test-async-wrap-check-providers.js
  2. 1
      test/parallel/test-cluster-eaccess.js
  3. 2
      test/parallel/test-http-client-pipe-end.js
  4. 2
      test/parallel/test-http-client-response-domain.js
  5. 2
      test/parallel/test-http-unix-socket.js
  6. 2
      test/parallel/test-net-pingpong.js
  7. 1
      test/parallel/test-net-pipe-connect-errors.js
  8. 1
      test/parallel/test-repl.js
  9. 2
      test/parallel/test-tls-connect-pipe.js
  10. 2
      test/sequential/test-pipe-address.js
  11. 2
      test/sequential/test-pipe-stream.js
  12. 2
      test/sequential/test-pipe-unref.js

4
test/parallel/test-async-wrap-check-providers.js

@ -47,9 +47,7 @@ new (process.binding('tty_wrap').TTY)();
crypto.randomBytes(1, noop);
try {
fs.unlinkSync(common.PIPE);
} catch(e) { }
common.refreshTmpDir();
net.createServer(function(c) {
c.end();

1
test/parallel/test-cluster-eaccess.js

@ -36,6 +36,7 @@ if (cluster.isMaster) {
});
} else {
common.refreshTmpDir();
var cp = fork(common.fixturesDir + '/listen-on-socket-and-exit.js',
{ stdio: 'inherit' });

2
test/parallel/test-http-client-pipe-end.js

@ -14,6 +14,8 @@ var server = http.createServer(function(req, res) {
});
});
common.refreshTmpDir();
server.listen(common.PIPE, function() {
var req = http.request({
socketPath: common.PIPE,

2
test/parallel/test-http-client-response-domain.js

@ -11,6 +11,8 @@ process.on('exit', function() {
assert(gotDomainError);
});
common.refreshTmpDir();
// first fire up a simple HTTP server
var server = http.createServer(function(req, res) {
res.writeHead(200);

2
test/parallel/test-http-unix-socket.js

@ -18,6 +18,8 @@ var server = http.createServer(function(req, res) {
res.end();
});
common.refreshTmpDir();
server.listen(common.PIPE, function() {
var options = {

2
test/parallel/test-net-pingpong.js

@ -106,7 +106,7 @@ function pingPongTest(port, host) {
}
/* All are run at once, so run on different ports */
console.log(common.PIPE);
common.refreshTmpDir();
pingPongTest(common.PIPE);
pingPongTest(common.PORT);
pingPongTest(common.PORT + 1, 'localhost');

1
test/parallel/test-net-pipe-connect-errors.js

@ -21,6 +21,7 @@ if (common.isWindows) {
} else {
// use common.PIPE to ensure we stay within POSIX socket path length
// restrictions, even on CI
common.refreshTmpDir();
emptyTxt = common.PIPE + '.txt';
function cleanup() {

1
test/parallel/test-repl.js

@ -3,6 +3,7 @@ var common = require('../common');
var assert = require('assert');
common.globalCheck = false;
common.refreshTmpDir();
var net = require('net'),
repl = require('repl'),

2
test/parallel/test-tls-connect-pipe.js

@ -18,6 +18,8 @@ var options = {
cert: fs.readFileSync(common.fixturesDir + '/keys/agent1-cert.pem')
};
common.refreshTmpDir();
var server = tls.Server(options, function(socket) {
++serverConnected;
server.close();

2
test/sequential/test-pipe-address.js

@ -9,6 +9,8 @@ var server = net.createServer(function() {
assert(false); // should not be called
});
common.refreshTmpDir();
server.listen(common.PIPE, function() {
address = server.address();
server.close();

2
test/sequential/test-pipe-stream.js

@ -3,6 +3,8 @@ var common = require('../common');
var assert = require('assert');
var net = require('net');
common.refreshTmpDir();
function test(clazz, cb) {
var have_ping = false;
var have_pong = false;

2
test/sequential/test-pipe-unref.js

@ -5,6 +5,8 @@ var assert = require('assert');
var net = require('net');
var closed = false;
common.refreshTmpDir();
var s = net.Server();
s.listen(common.PIPE);
s.unref();

Loading…
Cancel
Save