test: fixed the execution for the threadsfafe function test.

[NickNaso]

Fixes: #806

[mhdawson]

Can you add a bit more detail on what you saw in terms of it not running?

When I run the tests before apply the patch I see:

Running test 'threadsafe_function/threadsafe_function_ctx'
Running test 'threadsafe_function/threadsafe_function_existing_tsfn'
Running test 'threadsafe_function/threadsafe_function_ptr'
Running test 'threadsafe_function/threadsafe_function_sum'
Running test 'threadsafe_function/threadsafe_function_unref'
Running test 'threadsafe_function/threadsafe_function'

[NickNaso]

Hi @mhdawson,
what you see printed on the console is in https://github.com/nodejs/node-addon-api/blob/master/test/index.js#L106 but this does not mean that the corresponding test is executed. In the test for threadsafe function https://github.com/nodejs/node-addon-api/blob/master/test/threadsafe_function/threadsafe_function.js#L7 we are exporting only the function instead we should export a a function invocation. Trying to insert a console.log('...') like reported below:

module.exports = async function() {
  console.log('Execution of test for threadsafe function');
  await test(require(`../build/${buildType}/binding.node`));
  await test(require(`../build/${buildType}/binding_noexcept.node`));
};

you could see that the test will not be executed. This happened in the PR #787.

[mhdawson]

@NickNaso thanks for the clarification.

[mhdawson]

LGTM

[mhdawson]

Landed as a13b36c