Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

test: add hasCrypto check to test-debug-usage #12357

Closed
wants to merge 1 commit into from

Commits on Apr 12, 2017

  1. test: add hasCrypto check to test-debug-usage

    When configuring --without-ssl test-debug-usage.js fails with the
    following error:
    
    assert.js:82
      throw new assert.AssertionError({
      ^
    AssertionError: false == true
        at ChildProcess.<anonymous>
    (/Users/danielbevenius/work/nodejs/node/test/parallel/test-debug-usage.js:24:5)
        at ChildProcess.<anonymous>
    (/Users/danielbevenius/work/nodejs/node/test/common.js:461:15)
        at emitTwo (events.js:125:13)
        at ChildProcess.emit (events.js:213:7)
        at Process.ChildProcess._handle.onexit
    (internal/child_process.js:208:12)
    
    But the underlying error is the following:
    
    (node:55665) [DEP0068] DeprecationWarning: `node debug` is deprecated.
    Please use `node inspect` instead.
    internal/util.js:83
        throw new Error('Node.js is not compiled with openssl crypto
    support');
        ^
    
    Error: Node.js is not compiled with openssl crypto support
        at Object.exports.assertCrypto (internal/util.js:83:11)
        at crypto.js:28:14
        at NativeModule.compile (bootstrap_node.js:559:7)
        at NativeModule.require (bootstrap_node.js:500:18)
        at node-inspect/lib/internal/inspect_client.js:24:16
        at NativeModule.compile (bootstrap_node.js:559:7)
        at NativeModule.require (bootstrap_node.js:500:18)
        at node-inspect/lib/_inspect.js:39:5
        at NativeModule.compile (bootstrap_node.js:559:7)
        at Function.NativeModule.require (bootstrap_node.js:500:18)
    
    assert.js:82
      throw new assert.AssertionError({
      ^
    AssertionError: false == true
        at ChildProcess.<anonymous>
    (/Users/danielbevenius/work/nodejs/node/test/parallel/test-debug-usage.js:25:5)
        at ChildProcess.<anonymous>
    (/Users/danielbevenius/work/nodejs/node/test/common.js:461:15)
        at emitTwo (events.js:125:13)
        at ChildProcess.emit (events.js:213:7)
        at Process.ChildProcess._handle.onexit
    (internal/child_process.js:208:12)
    
    This commit adds a check for crypto like some of the other tests do.
    danbev committed Apr 12, 2017
    Configuration menu
    Copy the full SHA
    fd9d9b1 View commit details
    Browse the repository at this point in the history