Skip to content

Commit d409ac4

Browse files
mhdawsonjasnell
authored andcommitted
test: disable test-tick-processor - aix and be ppc
This test is already partially disabled for several platforms with the comment that the required info is not provided at the C++ level. I'm adding AIX as and PPC BE linux as they currently fall into the same category. We are working to see if we can change that in v8 but it will be non-trivial if is possible at all so I don't want to leave the CI with failing tests until that point. PR-URL: #3491 Reviewed-By: James M Snell <[email protected]> Reviewed-By: Rich Trott <[email protected]>
1 parent c162303 commit d409ac4

File tree

2 files changed

+10
-3
lines changed

2 files changed

+10
-3
lines changed

test/common.js

+5
Original file line numberDiff line numberDiff line change
@@ -13,6 +13,11 @@ exports.tmpDirName = 'tmp';
1313
exports.PORT = +process.env.NODE_COMMON_PORT || 12346;
1414
exports.isWindows = process.platform === 'win32';
1515
exports.isAix = process.platform === 'aix';
16+
exports.isLinuxPPCBE = (process.platform === 'linux') &&
17+
(process.arch === 'ppc64') &&
18+
(os.endianness() === 'BE');
19+
exports.isSunOS = process.platform === 'sunos';
20+
exports.isFreeBSD = process.platform === 'freebsd';
1621

1722
function rimrafSync(p) {
1823
try {

test/parallel/test-tick-processor.js

+5-3
Original file line numberDiff line numberDiff line change
@@ -20,9 +20,11 @@ runTest(/LazyCompile.*\[eval\]:1|.*% UNKNOWN/,
2020
};
2121
setTimeout(function() { process.exit(0); }, 2000);
2222
f();`);
23-
if (process.platform === 'win32' ||
24-
process.platform === 'sunos' ||
25-
process.platform === 'freebsd') {
23+
if (common.isWindows ||
24+
common.isSunOS ||
25+
common.isAix ||
26+
common.isLinuxPPCBE ||
27+
common.isFreeBSD) {
2628
console.log('1..0 # Skipped: C++ symbols are not mapped for this os.');
2729
return;
2830
}

0 commit comments

Comments
 (0)