Browse Source

test: module loading error fix solaris #3798

- refactor test to accept multiple error messages per platform
- add new message to be found in Solaris 11.3 as per #3798

PR-URL: https://github.com/nodejs/node/pull/3855
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Evan Lucas <evanlucas@me.com>
process-exit-stdio-flushing
fansworld-claudio 9 years ago
committed by Evan Lucas
parent
commit
9a628e2dac
  1. 16
      test/parallel/test-module-loading-error.js

16
test/parallel/test-module-loading-error.js

@ -5,12 +5,10 @@ var assert = require('assert');
console.error('load test-module-loading-error.js'); console.error('load test-module-loading-error.js');
var error_desc = { var error_desc = {
win32: '%1 is not a valid Win32 application', win32: ['%1 is not a valid Win32 application'],
linux: 'file too short', linux: ['file too short', 'Exec format error'],
sunos: 'unknown file type' sunos: ['unknown file type', 'not an ELF file']
}; };
var musl_errno_enoexec = 'Exec format error';
var dlerror_msg = error_desc[process.platform]; var dlerror_msg = error_desc[process.platform];
if (!dlerror_msg) { if (!dlerror_msg) {
@ -21,11 +19,9 @@ if (!dlerror_msg) {
try { try {
require('../fixtures/module-loading-error.node'); require('../fixtures/module-loading-error.node');
} catch (e) { } catch (e) {
if (process.platform === 'linux' && assert.strictEqual(dlerror_msg.some((errMsgCase) => {
e.toString().indexOf(musl_errno_enoexec) !== -1) { return e.toString().indexOf(errMsgCase) !== -1;
dlerror_msg = musl_errno_enoexec; }), true);
}
assert.notEqual(e.toString().indexOf(dlerror_msg), -1);
} }
try { try {

Loading…
Cancel
Save