test: fix inconsistency write size in test-fs-readfile-tostring-fail

Jungku Lee · richardlau · commit cab7737f7ee8 · 2024-03-25T18:15:14.000Z
PR-URL: #51141 Refs: #51133 Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: Deokjin Kim <deokjin81.kim@gmail.com>
diff --git a/test/pummel/test-fs-readfile-tostring-fail.js b/test/pummel/test-fs-readfile-tostring-fail.js
@@ -9,13 +9,15 @@ const assert = require('assert');
 const fs = require('fs');
 const cp = require('child_process');
 const kStringMaxLength = require('buffer').constants.MAX_STRING_LENGTH;
+const size = Math.floor(kStringMaxLength / 200);
+
 if (common.isAIX && (Number(cp.execSync('ulimit -f')) * 512) < kStringMaxLength)
 common.skip('intensive toString tests due to file size confinements');
 
 const tmpdir = require('../common/tmpdir');
 tmpdir.refresh();
 
-if (!tmpdir.hasEnoughSpace(kStringMaxLength)){
+if (!tmpdir.hasEnoughSpace(kStringMaxLength + size)){
 common.skip(`Not enough space in ${tmpdir.path}`);
 }
 
@@ -26,7 +28,6 @@ const stream = fs.createWriteStream(file,{
 
 stream.on('error', (err) =>{throw err});
 
-const size = kStringMaxLength / 200;
 const a = Buffer.alloc(size, 'a');
 let expectedSize = 0;
 

-Original file line number
+Diff line change
 constfs=require('fs');
 constcp=require('child_process');
 constkStringMaxLength=require('buffer').constants.MAX_STRING_LENGTH;
 +constsize=Math.floor(kStringMaxLength/200);
++
 if(common.isAIX&&(Number(cp.execSync('ulimit -f'))*512)<kStringMaxLength)
 common.skip('intensive toString tests due to file size confinements');
 consttmpdir=require('../common/tmpdir');
 tmpdir.refresh();
 -if(!tmpdir.hasEnoughSpace(kStringMaxLength)){
 +if(!tmpdir.hasEnoughSpace(kStringMaxLength+size)){
 common.skip(`Not enough space in ${tmpdir.path}`);
+}
 stream.on('error',(err)=>{throwerr;});
 -constsize=kStringMaxLength/200;
 consta=Buffer.alloc(size,'a');
 letexpectedSize=0;