Test on windows in addition to ubuntu (#267)

* Test on windows in addition to ubuntu

* Skip cluster test on windows

* Skip ECMAScript modules on windows for now

Author: bjornstar

.github/workflows/nodejs.yml

@@ -7,7 +7,7 @@ on:
     branches: [ master ]
 
 jobs:
-  build:
+  build_ubuntu:
 
     runs-on: ubuntu-latest
 
@@ -23,3 +23,19 @@ jobs:
         node-version: ${{ matrix.node-version }}
     - run: npm i
     - run: npm test
+
+  build_windows:
+    runs-on: windows-latest
+
+    strategy:
+      matrix:
+        node-version: [12.x, 14.x, 16.x]
+
+    steps:
+    - 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 test

test/spawn/cluster.js

@@ -3,6 +3,11 @@ const tap = require('tap');
 const { spawn, touchFile } = require('../utils');
 
 tap.test('Restart the cluster', t => {
+  if (process.platform === 'win32') {
+    t.pass('Restart the cluster', { skip: 'Signals are not supported on Windows' });
+    return t.end();
+  }
+
   spawn('cluster.js', out => {
     if (out.match(/touch message\.js/m)) {
       touchFile('message.js');

test/spawn/esmodule.js

@@ -5,6 +5,7 @@ const { spawn, touchFile } = require('../utils');
 
 tap.test('Supports ECMAScript modules with experimental-specifier-resolution', t => {
   if (semver.satisfies(process.version, '<12.17')) return t.skip();
+  if (process.platform === 'win32') return t.skip('ESM support on windows is broken');
 
   spawn('--experimental-specifier-resolution=node resolution.mjs', out => {
     if (out.match(/touch message.js/)) {
@@ -20,6 +21,8 @@ tap.test('Supports ECMAScript modules with experimental-specifier-resolution', t
 });
 
 tap.test('Supports ECMAScript modules', t => {
+  if (process.platform === 'win32') return t.skip('ESM support on windows is broken');
+
   spawn('ecma-script-modules.mjs', out => {
     if (out.match(/touch message.mjs/)) {
       touchFile('message.mjs');