test: skip stringbytes-external-exceed-max on AIX

Add SKIP status for more tests in stringbytes-external-exceed-max that are failing on AIX. PR-URL: nodejs#28516 Backport-PR-URL: nodejs#29599 Reviewed-By: Rich Trott <[email protected]> Reviewed-By: Richard Lau <[email protected]> Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Michael Dawson <[email protected]>
andrewhughes101 · Sep 18, 2019 · 3ebdd70 · 3ebdd70
1 parent 84e4d05
commit 3ebdd70
Show file tree

Hide file tree

Showing 2 changed files with 19 additions and 5 deletions.
diff --git a/test/addons/addon.status b/test/addons/addon.status
@@ -0,0 +1,19 @@
+prefix addons
+
+[true] # This section applies to all platforms
+
+[$system==aix]
+# https://github.com/nodejs/build/issues/1820#issuecomment-505998851
+# https://github.com/nodejs/node/pull/28469
+# https://github.com/nodejs/node/pull/28516
+stringbytes-external-exceed-max/test-stringbytes-external-exceed-max.js: SKIP
+
+# https://github.com/nodejs/node/pull/28516
+stringbytes-external-exceed-max/test-stringbytes-external-at-max: SKIP
+stringbytes-external-exceed-max/test-stringbytes-external-exceed-max-by-1-ascii: SKIP
+stringbytes-external-exceed-max/test-stringbytes-external-exceed-max-by-1-base64: SKIP
+stringbytes-external-exceed-max/test-stringbytes-external-exceed-max-by-1-binary: SKIP
+stringbytes-external-exceed-max/test-stringbytes-external-exceed-max-by-1-hex: SKIP
+stringbytes-external-exceed-max/test-stringbytes-external-exceed-max-by-1-utf8: SKIP
+stringbytes-external-exceed-max/test-stringbytes-external-exceed-max-by-2: SKIP
+stringbytes-external-exceed-max/test-stringbytes-external-exceed-max: SKIP
diff --git a/test/addons/stringbytes-external-exceed-max/test-stringbytes-external-exceed-max.js b/test/addons/stringbytes-external-exceed-max/test-stringbytes-external-exceed-max.js
@@ -5,11 +5,6 @@ const skipMessage = 'intensive toString tests due to memory confinements';
 if (!common.enoughTestMem)
   common.skip(skipMessage);
 
-// See https://github.com/nodejs/build/issues/1820#issuecomment-505998851
-// See https://github.com/nodejs/node/pull/28469
-if (process.platform === 'aix')
-  common.skip('flaky on AIX');
-
 const binding = require(`./build/${common.buildType}/binding`);
 const assert = require('assert');