fix capturing stdout and stderr when running tests (#454)

* fix capturing stdout and stderr when running tests

* use node 18.x for tests

* use node 18.x in tests ci

.github/workflows/tests.yml

@@ -13,7 +13,7 @@ jobs:
 
     strategy:
       matrix:
-        node-version: [16.x]
+        node-version: [18.x]
         # See supported Node.js release schedule at https://nodejs.org/en/about/releases/
 
     steps:

build/pipeline.yml

@@ -34,13 +34,13 @@ extends:
         testPlatforms:
           - name: Linux
             nodeVersions:
-              - 16.x
+              - 18.x
           - name: MacOS
             nodeVersions:
-              - 16.x
+              - 18.x
           - name: Windows
             nodeVersions:
-              - 16.x
+              - 18.x
 
         testSteps:
           - script: npm i

test/test-integration.mjs

@@ -119,12 +119,19 @@ describe('integration tests', function () {
 
 async function doSpawn(execName, allArguments, options) {
     return new Promise((resolve, reject) => {
-        const child = cp.execFile(execName, allArguments, { stdio: 'inherit', ...options });
+        const child = cp.execFile(execName, allArguments, { stdio: 'pipe', ...options });
+        let stdout = [], stderr = [];
+        child.stdout.on('data', (data) => {
+            stdout.push(data.toString());
+        });
+        child.stderr.on('data', (data) => {
+            stderr.push(data.toString());
+        });
         child.on('error', (err) => {
             reject(err);
         });
         child.on('exit', (exitCode) => {
-            resolve({ exitCode });
+            resolve({ exitCode, stdout: stdout.join(''), stderr: stderr.join('')});
         });
     });
 }