GeezerVison is showing github.com — test: convert readdir test to use test runner · nodejs/node@ff7b144
File tree
Expand file treeCollapse file tree
Open diff view settings
Expand file treeCollapse file tree
Open diff view settings
Collapse file
Copy file name to clipboard +14-10Lines changed: 14 additions & 10 deletions | Original file line number | Diff line number | Diff line change |
|---|
|
1 | 1 | 'use strict'; |
2 | | -const common = require('../common'); |
3 | | -const fs = require('fs'); |
4 | | -const net = require('net'); |
5 | 2 |
|
| 3 | +const { PIPE, mustCall } = require('../common'); |
6 | 4 | const tmpdir = require('../common/tmpdir'); |
7 | | -tmpdir.refresh(); |
| 5 | +const { test } = require('node:test'); |
| 6 | +const fs = require('node:fs'); |
| 7 | +const net = require('node:net'); |
8 | 8 |
|
9 | | -const server = net.createServer().listen(common.PIPE, common.mustCall(() => { |
10 | | - // The process should not crash |
11 | | - // See https://github.com/nodejs/node/issues/52159 |
12 | | - fs.readdirSync(tmpdir.path, { recursive: true }); |
13 | | - server.close(); |
14 | | -})); |
| 9 | +test('readdir should not recurse into Unix domain sockets', (t, done) => { |
| 10 | + tmpdir.refresh(); |
| 11 | + const server = net.createServer().listen(PIPE, mustCall(() => { |
| 12 | + // The process should not crash |
| 13 | + // See https://github.com/nodejs/node/issues/52159 |
| 14 | + fs.readdirSync(tmpdir.path, { recursive: true }); |
| 15 | + server.close(); |
| 16 | + done(); |
| 17 | + })); |
| 18 | +}); |