test: check symbols in shared lib

When building the node with `--shared` option, we need to verify the symbols in shared lib instead of executable. Refs: nodejs#18535 Signed-off-by: Yihong Wang <[email protected]> PR-URL: nodejs#18806 Refs: nodejs#18535 Reviewed-By: Richard Lau <[email protected]> Reviewed-By: Ruben Bridgewater <[email protected]> Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Gibson Fahnestock <[email protected]> Reviewed-By: Yuta Hiroto <[email protected]> Reviewed-By: Anna Henningsen <[email protected]>
MayaLekova · May 8, 2018 · 17a1691 · 17a1691
1 parent 6b5a569
commit 17a1691
Show file tree

Hide file tree

Showing 2 changed files with 22 additions and 2 deletions.
diff --git a/test/common/shared-lib-util.js b/test/common/shared-lib-util.js
@@ -1,5 +1,5 @@
-/* eslint-disable node-core/required-modules */
 'use strict';
+const common = require('../common');
 const path = require('path');
 
 // If node executable is linked to shared lib, need to take care about the
@@ -27,3 +27,17 @@ exports.addLibraryPath = function(env) {
     (env.PATH ? env.PATH + path.delimiter : '') +
     path.dirname(process.execPath);
 };
+
+// Get the full path of shared lib
+exports.getSharedLibPath = function() {
+  if (common.isWindows) {
+    return path.join(path.dirname(process.execPath), 'node.dll');
+  } else if (common.isOSX) {
+    return path.join(path.dirname(process.execPath),
+                     `libnode.${process.config.variables.shlib_suffix}`);
+  } else {
+    return path.join(path.dirname(process.execPath),
+                     'lib.target',
+                     `libnode.${process.config.variables.shlib_suffix}`);
+  }
+};
diff --git a/test/parallel/test-postmortem-metadata.js b/test/parallel/test-postmortem-metadata.js
@@ -7,7 +7,13 @@
 const common = require('../common');
 const assert = require('assert');
 const { spawnSync } = require('child_process');
-const args = [process.execPath];
+const { getSharedLibPath } = require('../common/shared-lib-util.js');
+
+// For shared lib case, check shared lib instead
+const args = [
+  process.config.variables.node_shared ?
+    getSharedLibPath() : process.execPath
+];
 
 if (common.isAIX)
   args.unshift('-Xany', '-B');