benchmark: fix api restriction for the permission category

give appropriate permissions to the following scripts: * permission-processhas-fs-read.js: 'ChildProcess' permission * permission-startup.js: 'FileSystemRead' permission > Paths delimited by comma (,) are no longer allowed. Refs: https://github.com/nodejs/node/blob/main/doc/api/cli.md#--allow-fs-read Signed-off-by: Ryan Qian <[email protected]> PR-URL: #51528 Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: James M Snell <[email protected]> Reviewed-By: Antoine du Hamel <[email protected]>
nodejs · May 18, 2024 · 8216cc9 · 8216cc9
1 parent 7380252
commit 8216cc9
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/benchmark/permission/permission-processhas-fs-read.js b/benchmark/permission/permission-processhas-fs-read.js
@@ -13,6 +13,7 @@ const options = {
   flags: [
     '--experimental-permission',
     `--allow-fs-read=${rootPath}`,
+    '--allow-child-process',
   ],
 };
 

diff --git a/benchmark/permission/permission-startup.js b/benchmark/permission/permission-startup.js
@@ -47,10 +47,10 @@ function spawnProcess(script, bench, state) {
 
 function main({ count, script, nFiles, prefixPath }) {
   script = path.resolve(__dirname, '../../', `${script}.js`);
-  const files = mockFiles(nFiles, prefixPath).join(',');
   const optionsWithScript = [
     '--experimental-permission',
-    `--allow-fs-read=${files},${script}`,
+    `--allow-fs-read=${script}`,
+    ...mockFiles(nFiles, prefixPath).map((file) => '--allow-fs-read=' + file),
     script,
   ];
   const warmup = 3;