Skip to content

Commit

Permalink
test: convert readdir test to use test runner
Browse files Browse the repository at this point in the history
Signed-off-by: tchetwin <[email protected]>
  • Loading branch information
tchetwin committed Nov 6, 2024
1 parent 6f12f1e commit 6a7fa7b
Showing 1 changed file with 14 additions and 10 deletions.
24 changes: 14 additions & 10 deletions test/parallel/test-fs-readdir-recursive.js
Original file line number Diff line number Diff line change
@@ -1,14 +1,18 @@
'use strict';

Check failure on line 1 in test/parallel/test-fs-readdir-recursive.js

View workflow job for this annotation

GitHub Actions / lint-js-and-md

Mandatory module "common" must be loaded before any other modules
const common = require('../common');
const fs = require('fs');
const net = require('net');

const { test } = require('node:test');
const fs = require('node:fs');
const net = require('node:net');
const common = require('../common');
const tmpdir = require('../common/tmpdir');
tmpdir.refresh();

const server = net.createServer().listen(common.PIPE, common.mustCall(() => {
// The process should not crash
// See https://github.com/nodejs/node/issues/52159
fs.readdirSync(tmpdir.path, { recursive: true });
server.close();
}));
test('readdir should not recurse into Unix domain sockets', (t, done) => {
tmpdir.refresh();
const server = net.createServer().listen(common.PIPE, common.mustCall(() => {
// The process should not crash
// See https://github.com/nodejs/node/issues/52159
fs.readdirSync(tmpdir.path, { recursive: true });
server.close();
done();
}));
});

0 comments on commit 6a7fa7b

Please sign in to comment.