node/test/parallel/test-fs-sync-fd-leak.js
Roman Reiss f29762f4dd test: enable linting for tests
Enable linting for the test directory. A number of changes was made so
all tests conform the current rules used by lib and src directories. The
only exception for tests is that unreachable (dead) code is allowed.

test-fs-non-number-arguments-throw had to be excluded from the changes
because of a weird issue on Windows CI.

PR-URL: https://github.com/nodejs/io.js/pull/1721
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
2015-05-19 21:21:27 +02:00

50 lines
933 B
JavaScript

'use strict';
var common = require('../common');
var assert = require('assert');
var fs = require('fs');
// ensure that (read|write|append)FileSync() closes the file descriptor
fs.openSync = function() {
return 42;
};
fs.closeSync = function(fd) {
assert.equal(fd, 42);
close_called++;
};
fs.readSync = function() {
throw new Error('BAM');
};
fs.writeSync = function() {
throw new Error('BAM');
};
fs.fstatSync = function() {
throw new Error('BAM');
};
ensureThrows(function() {
fs.readFileSync('dummy');
});
ensureThrows(function() {
fs.writeFileSync('dummy', 'xxx');
});
ensureThrows(function() {
fs.appendFileSync('dummy', 'xxx');
});
var close_called = 0;
function ensureThrows(cb) {
var got_exception = false;
close_called = 0;
try {
cb();
} catch (e) {
assert.equal(e.message, 'BAM');
got_exception = true;
}
assert.equal(close_called, 1);
assert.equal(got_exception, true);
}