From 240fbe5b18a7c2711dab79f312648d3f07f043b6 Mon Sep 17 00:00:00 2001 From: jBarz Date: Tue, 17 Oct 2017 20:01:31 -0400 Subject: [PATCH] test: skip test due to file size limit MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit The test requires a file size limit that is greater than the string size limit. Some aix machines might not meet this criteria so in which case we should skip the test. PR-URL: https://github.com/nodejs/node/pull/16273 Fixes: https://github.com/nodejs/node/issues/16319 Reviewed-By: Anna Henningsen Reviewed-By: MichaƫZasso Reviewed-By: Colin Ihrig Reviewed-By: Myles Borins Reviewed-By: Yuta Hiroto Reviewed-By: Gibson Fahnestock Reviewed-By: James M Snell Reviewed-By: Michael Dawson --- test/sequential/test-fs-readfile-tostring-fail.js | 3 +++ 1 file changed, 3 insertions(+) diff --git a/test/sequential/test-fs-readfile-tostring-fail.js b/test/sequential/test-fs-readfile-tostring-fail.js index 62c855de86..c79e13daeb 100644 --- a/test/sequential/test-fs-readfile-tostring-fail.js +++ b/test/sequential/test-fs-readfile-tostring-fail.js @@ -8,7 +8,10 @@ if (!common.enoughTestMem) const assert = require('assert'); const fs = require('fs'); const path = require('path'); +const cp = require('child_process'); const kStringMaxLength = process.binding('buffer').kStringMaxLength; +if (common.isAIX && (Number(cp.execSync('ulimit -f')) * 512) < kStringMaxLength) + common.skip('intensive toString tests due to file size confinements'); common.refreshTmpDir();