build,test: Skipped napi test, fixed build issue

nodejs · May 17, 2017 · ed9ec1f · ed9ec1f
1 parent 1caa2c7
commit ed9ec1f
Show file tree

Hide file tree

Showing 6 changed files with 21 additions and 4 deletions.
diff --git a/deps/chakrashim/include/v8.h b/deps/chakrashim/include/v8.h
@@ -994,8 +994,7 @@ class V8_EXPORT StackFrame {
 
 enum class PromiseHookType { kInit, kResolve, kBefore, kAfter };
 
-typedef void(*PromiseHook)(PromiseHookType type, Local<Promise> promise,
-  Local<Value> parent);
+typedef void(*PromiseHook)(PromiseHookType type, Local<Promise> promise, Local<Value> parent);
 
 
 class V8_EXPORT Value : public Data {

diff --git a/test/addons-napi/test_async/test.js b/test/addons-napi/test_async/test.js
@@ -2,6 +2,13 @@
 const common = require('../../common');
 const assert = require('assert');
 const child_process = require('child_process');
+
+if (common.isChakraEngine) {
+  common.skip('Skipped for node-chakracore till #246 is fixed.');
+  return;
+}
+
+
 const test_async = require(`./build/${common.buildType}/test_async`);
 
 const testException = 'test_async_cb_exception';

diff --git a/test/addons-napi/test_make_callback_recurse/test.js b/test/addons-napi/test_make_callback_recurse/test.js
@@ -6,6 +6,11 @@ const domain = require('domain');
 const binding = require(`./build/${common.buildType}/binding`);
 const makeCallback = binding.makeCallback;
 
+if (common.isChakraEngine) {
+  common.skip('Skipped for node-chakracore till #246 is fixed.');
+  return;
+}
+
 // Make sure this is run in the future.
 const mustCallCheckDomains = common.mustCall(checkDomains);
 

diff --git a/test/addons-napi/test_reference/test.js b/test/addons-napi/test_reference/test.js
@@ -4,6 +4,12 @@
 const common = require('../../common');
 const assert = require('assert');
 
+if (common.isChakraEngine) {
+  common.skip('Skipped for node-chakracore till #246 is fixed.');
+  return;
+}
+
+
 const test_reference = require(`./build/${common.buildType}/test_reference`);
 
 // This test script uses external values with finalizer callbacks

diff --git a/test/parallel/test-error-reporting.js b/test/parallel/test-error-reporting.js
@@ -38,7 +38,7 @@ function errExec(script, callback) {
 
     if (!common.isChakraEngine) { // chakra does not output script
       // Assert the script is mentioned in error output.
-       assert.ok(stderr.includes(script));
+      assert.ok(stderr.includes(script));
     }
 
     // Proxy the args for more tests.

diff --git a/test/parallel/test-repl.js b/test/parallel/test-repl.js
@@ -279,7 +279,7 @@ function error_test() {
       expect: prompt_multiline,
       chakracore: 'skip' },
     { client: client_unix, send: ')',
-      expect: `undefined\n${prompt_unix},
+      expect: `undefined\n${prompt_unix}`,
       chakracore: 'skip' },
     // npm prompt error message
     { client: client_unix, send: 'npm install foobar',