fixup! vfs: support renaming implicit ZIP directories

Author: trivikr

test/parallel/test-vfs-zip-provider.js

@@ -125,6 +125,33 @@ async function buildArchive(entries, comment) {
     assert.strictEqual(await archiveVfs.promises.readFile('/renamed-dir/b.txt', 'utf8'), 'nested');
   }
 
+  // Renaming an explicit directory also renames its children.
+  {
+    const archive = await buildArchive([
+      await zlib.ZipEntry.create('async-dir/', Buffer.alloc(0)),
+      await zlib.ZipEntry.create('async-dir/child.txt', Buffer.from('async')),
+      await zlib.ZipEntry.create('sync-dir/', Buffer.alloc(0)),
+      await zlib.ZipEntry.create('sync-dir/child.txt', Buffer.from('sync')),
+    ]);
+    const zip = new zlib.ZipBuffer(archive);
+    const archiveVfs = vfs.create(new vfs.ZipProvider(zip));
+
+    await archiveVfs.promises.rename('/async-dir', '/renamed-async-dir');
+    assert.strictEqual(zip.has('async-dir/'), false);
+    assert.strictEqual(zip.has('async-dir/child.txt'), false);
+    assert.strictEqual(zip.has('renamed-async-dir/'), true);
+    assert.strictEqual(
+      await archiveVfs.promises.readFile('/renamed-async-dir/child.txt', 'utf8'),
+      'async',
+    );
+
+    archiveVfs.renameSync('/sync-dir', '/renamed-sync-dir');
+    assert.strictEqual(zip.has('sync-dir/'), false);
+    assert.strictEqual(zip.has('sync-dir/child.txt'), false);
+    assert.strictEqual(zip.has('renamed-sync-dir/'), true);
+    assert.strictEqual(archiveVfs.readFileSync('/renamed-sync-dir/child.txt', 'utf8'), 'sync');
+  }
+
   // --- ZipFile-backed, read-only: writes rejected with EROFS ----------------
   {
     const archive = await buildArchive([await zlib.ZipEntry.create('a.txt', Buffer.from('x'))]);