Skip GC tests in Node 13 due to bug NAPI.

Author: rolftimmermans

.travis.yml

@@ -48,11 +48,13 @@ jobs:
   # Test recent Node versions.
   - os: linux
     node_js: "12"
-    env: ZMQ_DRAFT=true INCLUDE_COMPAT_TESTS=true
+    # Skip GC tests due to https://github.com/node-ffi-napi/weak-napi/issues/16
+    env: ZMQ_DRAFT=true SKIP_GC_TESTS=true INCLUDE_COMPAT_TESTS=true
 
   - os: linux
     node_js: "13"
-    env: ZMQ_DRAFT=true
+    # Skip GC tests due to https://github.com/node-ffi-napi/weak-napi/issues/16
+    env: ZMQ_DRAFT=true SKIP_GC_TESTS=true
 
   ## PREBUILD STAGE
 

test/unit/socket-close-test.ts

@@ -97,6 +97,7 @@ for (const proto of testProtos("tcp", "ipc", "inproc")) {
       })
 
       it("should release reference to context", async function() {
+        if (process.env.SKIP_GC_TESTS) this.skip()
         this.slow(200)
 
         const weak = require("weak-napi")
@@ -124,6 +125,7 @@ for (const proto of testProtos("tcp", "ipc", "inproc")) {
 
     describe("in gc finalizer", function() {
       it("should release reference to context", async function() {
+        if (process.env.SKIP_GC_TESTS) this.skip()
         this.slow(200)
 
         const weak = require("weak-napi")

test/unit/socket-send-receive-test.ts

@@ -78,6 +78,7 @@ for (const proto of testProtos("tcp", "ipc", "inproc")) {
       })
 
       it("should copy and release small buffers", async function() {
+        if (process.env.SKIP_GC_TESTS) this.skip()
         const weak = require("weak-napi")
 
         let released = false
@@ -95,6 +96,7 @@ for (const proto of testProtos("tcp", "ipc", "inproc")) {
       })
 
       it("should retain large buffers", async function() {
+        if (process.env.SKIP_GC_TESTS) this.skip()
         const weak = require("weak-napi")
 
         let released = false
@@ -285,6 +287,7 @@ for (const proto of testProtos("tcp", "ipc", "inproc")) {
       })
 
       it("should release buffers", async function() {
+        if (process.env.SKIP_GC_TESTS) this.skip()
         const weak = require("weak-napi")
 
         const n = 10
@@ -320,6 +323,7 @@ for (const proto of testProtos("tcp", "ipc", "inproc")) {
       })
 
       it("should release buffers after echo", async function() {
+        if (process.env.SKIP_GC_TESTS) this.skip()
         const weak = require("weak-napi")
 
         const n = 10