test: add more tsd test cases (#12)

run test on node >= 8

Author: fengmk2

.github/workflows/nodejs.yml

@@ -8,24 +8,35 @@ on:
     branches: [ master ]
   pull_request:
     branches: [ master ]
+  schedule:
+    - cron: '0 2 * * *'
 
 jobs:
   build:
-
     runs-on: ${{ matrix.os }}
 
     strategy:
+      fail-fast: false
       matrix:
-        node-version: [4.x, 6.x, 8.x, 10.x, 12.x]
+        node-version: [8, 10, 12, 14, 16]
         os: [ubuntu-latest, windows-latest, macos-latest]
 
     steps:
-    - uses: actions/checkout@v2
+    - name: Checkout Git Source
+      uses: actions/checkout@v2
+
     - name: Use Node.js ${{ matrix.node-version }}
       uses: actions/setup-node@v1
       with:
         node-version: ${{ matrix.node-version }}
-    - run: npm i
-    - run: npm run ci
-      env:
-        CI: true
+
+    - name: Install Dependencies
+      run: npm i
+
+    - name: Continuous Integration
+      run: npm run ci
+
+    - name: Code Coverage
+      uses: codecov/codecov-action@v1
+      with:
+        token: ${{ secrets.CODECOV_TOKEN }}

.gitignore

@@ -14,3 +14,4 @@ node_modules
 npm-debug.log
 coverage/
 test/fixtures/ts/**/*.js
+.DS_Store

package.json

@@ -45,7 +45,7 @@
     "node": ">=4.2.3"
   },
   "ci": {
-    "version": "4, 6, 8, 10, 12",
+    "version": "8, 10, 12, 14, 16",
     "type": "github",
     "npminstall": false
   },

test/fixtures/timeout-and-exit.js

@@ -2,10 +2,10 @@
 
 console.error('timer start');
 setInterval(() => {
-  console.error('echo every 500ms');
-}, 500);
+  console.error('echo every 600ms');
+}, 600);
 
 setTimeout(() => {
   console.error('exit');
   process.exit(0);
-}, 1100);
+}, 1500);

test/fixtures/timeout-stderr.js

@@ -2,5 +2,5 @@
 
 console.error('timer start');
 setInterval(() => {
-  console.error('echo every 500ms');
-}, 500);
+  console.error('echo every 600ms');
+}, 600);

test/fixtures/ts/checkExtraOptions.ts

@@ -0,0 +1,19 @@
+import * as runscript from '../../../';
+import assert = require('assert');
+
+runscript('node -v', { stdio: 'pipe' }, { timeout: 30000 })
+  .then(result => {
+    assert(!result.stderr);
+    assert(!!result.stdout!.toString());
+    console.info(result.stdout!.toString());
+
+    return runscript('node -h', {
+      stdio: 'pipe',
+      stdout: process.stdout,
+      stderr: process.stderr,
+    }, {});
+  })
+  .catch((err: runscript.StdError) => {
+    assert(err.stdio);
+    throw new Error('should not throw');
+  });

test/index.d.ts.test.js

@@ -0,0 +1,45 @@
+'use strict';
+
+const path = require('path');
+const assert = require('assert');
+const runScript = require('..');
+
+describe('index.d.ts.test.js', () => {
+  before(async () => {
+    try {
+      const stdio = await runScript('tsc -p ./ts/tsconfig.json', {
+        stdio: 'pipe',
+        cwd: path.join(__dirname, 'fixtures'),
+        debug: true,
+      });
+      assert(!stdio.stderr);
+    } catch (err) {
+      console.error('should not throw error:', err.stdio.stdout.toString());
+      throw err;
+    }
+  });
+
+  it('should compile ts without error', async () => {
+    const stdio = await runScript('node ./ts/check.js', {
+      stdio: 'pipe',
+      cwd: path.join(__dirname, 'fixtures'),
+    });
+    assert(!stdio.stderr);
+    const stdout = stdio.stdout.toString();
+    assert(stdout);
+    assert(stdout.match(/v\d+\.\d+\.\d+/));
+    assert(stdout.match(/Options:/));
+  });
+
+  it('should tsd support ExtraOptions', async () => {
+    const stdio = await runScript('node ./ts/checkExtraOptions.js', {
+      stdio: 'pipe',
+      cwd: path.join(__dirname, 'fixtures'),
+    });
+    assert(!stdio.stderr);
+    const stdout = stdio.stdout.toString();
+    assert(stdout);
+    assert(stdout.match(/v\d+\.\d+\.\d+/));
+    assert(stdout.match(/Options:/));
+  });
+});

test/runscript.test.js

@@ -5,7 +5,7 @@ const path = require('path');
 const assert = require('assert');
 const runScript = require('..');
 
-describe('test/runscript.test.js', () => {
+describe('runscript.test.js', () => {
   it('should run `$ node -v`', () => {
     return runScript('node -v');
   });
@@ -62,20 +62,20 @@ describe('test/runscript.test.js', () => {
   it('should reject on timeout (stderr)', () => {
     return runScript(`node ${path.join(__dirname, 'fixtures/timeout-stderr.js')}`, {
       stdio: 'pipe',
-    }, { timeout: 1200 })
+    }, { timeout: 1500 })
       .catch(err => {
         console.log(err);
         assert(err.name === 'RunScriptTimeoutError');
-        assert(err.stdio.stderr.toString() === 'timer start\necho every 500ms\necho every 500ms\n');
+        assert(err.stdio.stderr.toString() === 'timer start\necho every 600ms\necho every 600ms\n');
       });
   });
 
   it('should normal exit before timeout', () => {
     return runScript(`node ${path.join(__dirname, 'fixtures/timeout-and-exit.js')}`, {
       stdio: 'pipe',
-    }, { timeout: 1300 })
+    }, { timeout: 1800 })
       .then(stdio => {
-        assert(stdio.stderr.toString() === 'timer start\necho every 500ms\necho every 500ms\nexit\n');
+        assert(stdio.stderr.toString() === 'timer start\necho every 600ms\necho every 600ms\nexit\n');
       });
   });
 
@@ -170,26 +170,6 @@ describe('test/runscript.test.js', () => {
     });
   });
 
-  it('should compile ts without error', () => {
-    return runScript('tsc -p ./ts/tsconfig.json', {
-      stdio: 'pipe',
-      cwd: path.join(__dirname, 'fixtures'),
-    }).then(stdio => {
-      assert(!stdio.stderr);
-
-      return runScript('node ./ts/check.js', {
-        stdio: 'pipe',
-        cwd: path.join(__dirname, 'fixtures'),
-      }).then(stdio => {
-        assert(!stdio.stderr);
-        const stdout = stdio.stdout.toString();
-        assert(stdout);
-        assert(stdout.match(/v\d+\.\d+\.\d+/));
-        assert(stdout.match(/Options:/));
-      });
-    });
-  });
-
   if (process.platform === 'win32') {
     it('should run relative path .\\node_modules\\.bin\\autod', () => {
       return runScript('.\\node_modules\\.bin\\autod -V', {