test: do not run TSFN tests on NAPI_VERSION < 4

legendecas · web-flow · commit bc8fc23627df · 2019-10-30T00:16:31.000+08:00
PR-URL: #576 Reviewed-By: NickNaso <nicoladelgobbo@gmail.com> Reviewed-By: Gabriel Schulhof <gabriel.schulhof@intel.com>
diff --git a/test/index.js b/test/index.js
@@ -69,6 +69,7 @@ if ((process.env.npm_config_NAPI_VERSION !== undefined) &&
 if ((process.env.npm_config_NAPI_VERSION !== undefined) &&
     (process.env.npm_config_NAPI_VERSION < 4)) {
   testModules.splice(testModules.indexOf('threadsafe_function/threadsafe_function_ptr'), 1);
+  testModules.splice(testModules.indexOf('threadsafe_function/threadsafe_function_unref'), 1);
   testModules.splice(testModules.indexOf('threadsafe_function/threadsafe_function'), 1);
 }
 

Original file line number	Diff line number	Diff line change
`@@ -69,6 +69,7 @@ if ((process.env.npm_config_NAPI_VERSION !== undefined) &&`
`69`	`69`	`if ((process.env.npm_config_NAPI_VERSION !== undefined) &&`
`70`	`70`	`(process.env.npm_config_NAPI_VERSION < 4)) {`
`71`	`71`	`testModules.splice(testModules.indexOf('threadsafe_function/threadsafe_function_ptr'), 1);`
	`72`	`+ testModules.splice(testModules.indexOf('threadsafe_function/threadsafe_function_unref'), 1);`
`72`	`73`	`testModules.splice(testModules.indexOf('threadsafe_function/threadsafe_function'), 1);`
`73`	`74`	`}`
`74`	`75`