src,test: disable per_context.js shim

The `Intl` usage in the the shim is causing issues in ChakraCore on
Linux and Mac. Since we don't necessarily need the `Atomics.notify`
shim code either the simplest solution is to remove the shim
completely for ChakraCore.

Refs: #565
Refs: #567
PR-URL: #566
Reviewed-By: Hitesh Kanwathirtha <hiteshk@microsoft.com>
Reviewed-By: Jimmy Thomson <jithomso@microsoft.com>

src/node.cc

@@ -3638,6 +3638,7 @@ Local<Context> NewContext(Isolate* isolate,
   context->SetEmbedderData(
       ContextEmbedderIndex::kAllowWasmCodeGeneration, True(isolate));
 
+#ifndef NODE_ENGINE_CHAKRACORE
   {
     // Run lib/internal/per_context.js
     Context::Scope context_scope(context);
@@ -3648,6 +3649,7 @@ Local<Context> NewContext(Isolate* isolate,
         &per_context_src).ToLocalChecked();
     s->Run(context).ToLocalChecked();
   }
+#endif
 
   return context;
 }

test/parallel/parallel.status

@@ -256,6 +256,10 @@ test-vm-createcacheddata : SKIP
 # Issue: https://github.com/nodejs/node-chakracore/issues/563
 test-zlib-unused-weak : SKIP
 
+# Removed the wake->notify shim code
+# Issue: https://github.com/nodejs/node-chakracore/issues/565
+test-atomics-notify : SKIP
+
 [$jsEngine==chakracore && $arch==x64]
 # These tests are failing for Node-Chakracore and should eventually be fixed
 test-buffer-includes : SKIP