test: add test for urlstrings usage in node:fs

LiviaMedeiros · ruyadorno · commit 34c17417927b · 2023-09-27T22:20:15.000-04:00
PR-URL: #49471 Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
diff --git a/test/parallel/test-fs-whatwg-url.js b/test/parallel/test-fs-whatwg-url.js
@@ -4,6 +4,8 @@ const common = require('../common');
 const fixtures = require('../common/fixtures');
 const assert = require('assert');
 const fs = require('fs');
+const tmpdir = require('../common/tmpdir');
+tmpdir.refresh();
 
 const url = fixtures.fileURL('a.js');
 
@@ -80,3 +82,25 @@ if (common.isWindows) {
     }
   );
 }
+
+// Test that strings are interpreted as paths and not as URL
+// Can't use process.chdir in Workers
+// Please avoid testing fs.rmdir('file:') or using it as cleanup
+if (common.isMainThread && !common.isWindows) {
+  const oldCwd = process.cwd();
+  process.chdir(tmpdir.path);
+
+  for (let slashCount = 0; slashCount < 9; slashCount++) {
+    const slashes = '/'.repeat(slashCount);
+
+    const dirname = `file:${slashes}thisDirectoryWasMadeByFailingNodeJSTestSorry/subdir`;
+    fs.mkdirSync(dirname, { recursive: true });
+    fs.writeFileSync(`${dirname}/file`, `test failed with ${slashCount} slashes`);
+
+    const expected = fs.readFileSync(tmpdir.resolve(dirname, 'file'));
+    const actual = fs.readFileSync(`${dirname}/file`);
+    assert.deepStrictEqual(actual, expected);
+  }
+
+  process.chdir(oldCwd);
+}
