GV

GeezerVison

The World Wide Web, restored to 800 pixels of glory.

Home|About|Gaming cabinet|Archives|How it works|Starter sites

On this site

  • Sign in
  • ActionsCI/CD and automation
  • CodespacesCloud development environments
  • All Features
  • Documentation
  • Training
  • Certifications
  • Build with Copilot
  • Customize Copilot
  • Integrations
  • Trending repos
  • GitHub Sponsors
  • Developer articles
  • Enterprises

This view

  • Original page
  • New address

GeezerVison is showing github.com — fs: fix readdir and opendir recursive with unknown file types · nodejs/node@db3fc6d

File tree

Expand file treeCollapse file tree
Open diff view settings
Filter options
  • lib/internal/fs
    • dir.js
    • utils.js
  • test/sequential
    • test-fs-opendir-recursive.js
    • test-fs-readdir-recursive.js
Expand file treeCollapse file tree
Open diff view settings
Collapse file

‎lib/internal/fs/dir.js‎

Copy file name to clipboardExpand all lines: lib/internal/fs/dir.js
+5-4Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -152,7 +152,7 @@ class Dir {
152152
ArrayPrototypePush(
153153
this[kDirBufferedEntries],
154154
getDirent(
155-
pathModule.join(path, result[i]),
155+
path,
156156
result[i],
157157
result[i + 1],
158158
),
@@ -161,9 +161,10 @@ class Dir {
161161
}
162162

163163
readSyncRecursive(dirent) {
164-
const ctx = { path: dirent.path };
164+
const path = pathModule.join(dirent.path, dirent.name);
165+
const ctx = { path };
165166
const handle = dirBinding.opendir(
166-
pathModule.toNamespacedPath(dirent.path),
167+
pathModule.toNamespacedPath(path),
167168
this[kDirOptions].encoding,
168169
undefined,
169170
ctx,
@@ -177,7 +178,7 @@ class Dir {
177178
);
178179

179180
if (result) {
180-
this.processReadResult(dirent.path, result);
181+
this.processReadResult(path, result);
181182
}
182183

183184
handle.close(undefined, ctx);
Collapse file

‎lib/internal/fs/utils.js‎

Copy file name to clipboardExpand all lines: lib/internal/fs/utils.js
+1-1Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -234,7 +234,7 @@ function join(path, name) {
234234
}
235235

236236
if (typeof path === 'string' && typeof name === 'string') {
237-
return pathModule.basename(path) === name ? path : pathModule.join(path, name);
237+
return pathModule.join(path, name);
238238
}
239239

240240
if (isUint8Array(path) && isUint8Array(name)) {
Collapse file

‎test/sequential/test-fs-opendir-recursive.js‎

Copy file name to clipboardExpand all lines: test/sequential/test-fs-opendir-recursive.js
+10-7Lines changed: 10 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,7 @@ const tmpdir = require('../common/tmpdir');
1010
const testDir = tmpdir.path;
1111

1212
const fileStructure = [
13-
[ 'a', [ 'foo', 'bar' ] ],
13+
[ 'a', [ 'a', 'foo', 'bar' ] ],
1414
[ 'b', [ 'foo', 'bar' ] ],
1515
[ 'c', [ 'foo', 'bar' ] ],
1616
[ 'd', [ 'foo', 'bar' ] ],
@@ -91,7 +91,7 @@ fs.symlinkSync(symlinkTargetFile, pathModule.join(symlinksRootPath, 'symlink-src
9191
fs.symlinkSync(symlinkTargetDir, pathModule.join(symlinksRootPath, 'symlink-src-dir'));
9292

9393
const expected = [
94-
'a', 'a/bar', 'a/foo', 'aa', 'aa/bar', 'aa/foo',
94+
'a', 'a/a', 'a/bar', 'a/foo', 'aa', 'aa/bar', 'aa/foo',
9595
'abc', 'abc/def', 'abc/def/bar', 'abc/def/foo', 'abc/ghi', 'abc/ghi/bar', 'abc/ghi/foo',
9696
'b', 'b/bar', 'b/foo', 'bb', 'bb/bar', 'bb/foo',
9797
'c', 'c/bar', 'c/foo', 'cc', 'cc/bar', 'cc/foo',
@@ -128,15 +128,18 @@ for (let i = 0; i < expected.length; i++) {
128128
}
129129

130130
function getDirentPath(dirent) {
131-
return pathModule.relative(testDir, dirent.path);
131+
return pathModule.relative(testDir, pathModule.join(dirent.path, dirent.name));
132132
}
133133

134134
function assertDirents(dirents) {
135135
dirents.sort((a, b) => (getDirentPath(a) < getDirentPath(b) ? -1 : 1));
136-
for (const [i, dirent] of dirents.entries()) {
137-
assert(dirent instanceof fs.Dirent);
138-
assert.strictEqual(getDirentPath(dirent), expected[i]);
139-
}
136+
assert.deepStrictEqual(
137+
dirents.map((dirent) => {
138+
assert(dirent instanceof fs.Dirent);
139+
return getDirentPath(dirent);
140+
}),
141+
expected
142+
);
140143
}
141144

142145
function processDirSync(dir) {
Collapse file

‎test/sequential/test-fs-readdir-recursive.js‎

Copy file name to clipboardExpand all lines: test/sequential/test-fs-readdir-recursive.js
+10-7Lines changed: 10 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@ const tmpdir = require('../common/tmpdir');
99
const readdirDir = tmpdir.path;
1010

1111
const fileStructure = [
12-
[ 'a', [ 'foo', 'bar' ] ],
12+
[ 'a', [ 'a', 'foo', 'bar' ] ],
1313
[ 'b', [ 'foo', 'bar' ] ],
1414
[ 'c', [ 'foo', 'bar' ] ],
1515
[ 'd', [ 'foo', 'bar' ] ],
@@ -90,7 +90,7 @@ fs.symlinkSync(symlinkTargetFile, pathModule.join(symlinksRootPath, 'symlink-src
9090
fs.symlinkSync(symlinkTargetDir, pathModule.join(symlinksRootPath, 'symlink-src-dir'));
9191

9292
const expected = [
93-
'a', 'a/bar', 'a/foo', 'aa', 'aa/bar', 'aa/foo',
93+
'a', 'a/a', 'a/bar', 'a/foo', 'aa', 'aa/bar', 'aa/foo',
9494
'abc', 'abc/def', 'abc/def/bar', 'abc/def/foo', 'abc/ghi', 'abc/ghi/bar', 'abc/ghi/foo',
9595
'b', 'b/bar', 'b/foo', 'bb', 'bb/bar', 'bb/foo',
9696
'c', 'c/bar', 'c/foo', 'cc', 'cc/bar', 'cc/foo',
@@ -133,11 +133,14 @@ function getDirentPath(dirent) {
133133
function assertDirents(dirents) {
134134
assert.strictEqual(dirents.length, expected.length);
135135
dirents.sort((a, b) => (getDirentPath(a) < getDirentPath(b) ? -1 : 1));
136-
for (const [i, dirent] of dirents.entries()) {
137-
assert(dirent instanceof fs.Dirent);
138-
assert.notStrictEqual(dirent.name, undefined);
139-
assert.strictEqual(getDirentPath(dirent), expected[i]);
140-
}
136+
assert.deepStrictEqual(
137+
dirents.map((dirent) => {
138+
assert(dirent instanceof fs.Dirent);
139+
assert.notStrictEqual(dirent.name, undefined);
140+
return getDirentPath(dirent);
141+
}),
142+
expected
143+
);
141144
}
142145

143146
// readdirSync

0 commit comments

Comments
0 (0)
You are visitor 01402503 — GeezerVison does not store the pages you look up.
Best viewed with Netscape Navigator 4.7 or Internet Explorer 6 at 800×600. No cookies required.