fix: remove unnecessary console logs from async_progress_worker tests

Author: toyobayashi

packages/test/node-addon-api/async_progress_worker.cc

@@ -38,6 +38,7 @@ class TestWorker : public AsyncProgressWorker<ProgressData> {
     for (int32_t idx = 0; idx < _times; idx++) {
       data.progress = idx;
       progress.Send(&data, 1);
+
       {
         std::unique_lock<std::mutex> lk(_cvm);
         _cv.wait(lk, [this] { return dataSent; });

packages/test/node-addon-api/async_progress_worker.test.js

@@ -6,15 +6,10 @@ const assert = require('assert')
 module.exports = common.runTest(test)
 
 async function test ({ asyncprogressworker }) {
-  console.log('await success(asyncprogressworker)')
   await success(asyncprogressworker)
-  console.log('await fail(asyncprogressworker)')
   await fail(asyncprogressworker)
-  console.log('await signalTest(asyncprogressworker.doMalignTest)')
   await signalTest(asyncprogressworker.doMalignTest)
-  console.log('await signalTest(asyncprogressworker.doSignalAfterProgressTest)')
   await signalTest(asyncprogressworker.doSignalAfterProgressTest)
-  console.log('done')
 }
 
 function success (binding) {
@@ -23,13 +18,11 @@ function success (binding) {
     const actual = []
     binding.doWork(expected.length,
       common.mustCall((err) => {
-        console.log('success callback called', err)
         if (err) {
           reject(err)
         }
       }),
       common.mustCall((_progress) => {
-        console.log('success progress called', _progress)
         actual.push(_progress)
         if (actual.length === expected.length) {
           assert.deepEqual(actual, expected)