Skip to content
This repository has been archived by the owner on Aug 31, 2018. It is now read-only.

Commit

Permalink
test: skip test due to file size limit
Browse files Browse the repository at this point in the history
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: nodejs/node#16273
Fixes: nodejs/node#16319
Reviewed-By: Anna Henningsen <[email protected]>
Reviewed-By: MichaëZasso <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Myles Borins <[email protected]>
Reviewed-By: Yuta Hiroto <[email protected]>
Reviewed-By: Gibson Fahnestock <[email protected]>
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Michael Dawson <[email protected]>
  • Loading branch information
jBarz authored and addaleax committed Dec 7, 2017
1 parent 20985a3 commit 3158787
Showing 1 changed file with 3 additions and 0 deletions.
3 changes: 3 additions & 0 deletions test/sequential/test-fs-readfile-tostring-fail.js
Original file line number Diff line number Diff line change
Expand Up @@ -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();

Expand Down

0 comments on commit 3158787

Please sign in to comment.