[test] Mark skipped tests as pending

websockets · Dec 1, 2017 · 009d05c · 009d05c
1 parent 4e7d48a
commit 009d05c
Show file tree

Hide file tree

Showing 2 changed files with 17 additions and 7 deletions.
diff --git a/test/WebSocket.test.js b/test/WebSocket.test.js
@@ -71,9 +71,14 @@ describe('WebSocket', function () {
         });
 
         ws.on('error', (err) => {
-          // Skip this test on machines where 127.0.0.2 is disabled.
-          if (err.code === 'EADDRNOTAVAIL') err = undefined;
-          wss.close(() => done(err));
+          wss.close(() => {
+            //
+            // Skip this test on machines where 127.0.0.2 is disabled.
+            //
+            if (err.code === 'EADDRNOTAVAIL') return this.skip();
+
+            done(err);
+          });
         });
       });
 
@@ -101,9 +106,14 @@ describe('WebSocket', function () {
       });
 
       wss.on('error', (err) => {
-        // Skip this test on machines where IPv6 is not supported.
-        if (err.code === 'EADDRNOTAVAIL') err = undefined;
-        wss.close(() => done(err));
+        wss.close(() => {
+          //
+          // Skip this test on machines where IPv6 is not supported.
+          //
+          if (err.code === 'EADDRNOTAVAIL') return this.skip();
+
+          done(err);
+        });
       });
 
       wss.on('connection', (ws, req) => {

diff --git a/test/WebSocketServer.test.js b/test/WebSocketServer.test.js
@@ -83,7 +83,7 @@ describe('WebSocketServer', function () {
       //
       // Skip this test on Windows as it throws errors for obvious reasons.
       //
-      if (process.platform === 'win32') return done();
+      if (process.platform === 'win32') return this.skip();
 
       const server = http.createServer();
       const sockPath = `/tmp/ws.${crypto.randomBytes(16).toString('hex')}.socket`;