sync-rpc 1.3.2 → 1.3.6

package/lib/__tests__/index.test.js

@@ -7,6 +7,8 @@ test('it should work', () => {
   }
 });
 
+let nativeNCFails = false;
+
 rpc.FUNCTION_PRIORITY.forEach((fn, i) => {
   test('profile ' + fn.name, () => {
     try {
@@ -18,7 +20,56 @@ rpc.FUNCTION_PRIORITY.forEach((fn, i) => {
       const end = Date.now();
       console.log(fn.name + ': ' + (end - start));
     } catch (ex) {
-      console.log(fn.name + ' fails');
+      if (fn.name === 'nativeNC') {
+        console.log(fn.name + ' fails');
+        nativeNCFails = true;
+        return;
+      }
+      throw ex;
+    }
+  });
+});
+
+rpc.FUNCTION_PRIORITY.forEach((fn, i) => {
+  test('test 30MB ' + fn.name, () => {
+    let result;
+    try {
+      rpc.configuration.fastestFunction = fn;
+      result = client('big');
+    } catch (ex) {
+      if (fn.name === 'nativeNC' && nativeNCFails) {
+        console.log(fn.name + ' fails');
+        return;
+      }
+      throw ex;
+    }
+    expect(result.length).toBe(30 * 1024 * 1024, 42);
+    // for (let i = 0; i < 30 * 1024 * 1024, 42; i++) {
+    //   expect(result[i]).toBe(42);
+    // }
+  });
+});
+
+rpc.FUNCTION_PRIORITY.forEach((fn, i) => {
+  let longMessage = '';
+  for (let i = 0; i < 100000; i++) {
+    longMessage += 'My Long Message Content';
+  }
+  test('profile large ' + fn.name, () => {
+    try {
+      rpc.configuration.fastestFunction = fn;
+      const start = Date.now();
+      for (let i = 0; i < 10; i++) {
+        expect(client(longMessage)).toBe(`sent ${longMessage} to My Server`);
+      }
+      const end = Date.now();
+      console.log('large ' + fn.name + ': ' + (end - start));
+    } catch (ex) {
+      if (fn.name === 'nativeNC' && nativeNCFails) {
+        console.log('large ' + fn.name + ' fails');
+        return;
+      }
+      throw ex;
     }
   });
 });

package/lib/index.js

@@ -32,6 +32,7 @@ function start() {
   const port = findPort();
   const p = spawn(process.execPath, [require.resolve('./worker'), port], {
     stdio: 'inherit',
+    windowsHide: true,
   });
   p.unref();
   process.on('exit', () => {
@@ -45,21 +46,25 @@ function start() {
 }
 
 function findPort() {
-  const findPortResult = spawnSync(process.execPath, [
-    require.resolve('./find-port'),
-  ]);
-  if (findPortResult.status !== 0) {
-    throw new Error(
-      findPortResult.stderr.toString() ||
-        'find port exited with code ' + findPortResult.status
-    );
-  }
+  const findPortResult = spawnSync(
+    process.execPath,
+    [require.resolve('./find-port')],
+    {
+      windowsHide: true,
+    }
+  );
   if (findPortResult.error) {
     if (typeof findPortResult.error === 'string') {
       throw new Error(findPortResult.error);
     }
     throw findPortResult.error;
   }
+  if (findPortResult.status !== 0) {
+    throw new Error(
+      findPortResult.stderr.toString() ||
+        'find port exited with code ' + findPortResult.status
+    );
+  }
   const portString = findPortResult.stdout.toString('utf8').trim();
   if (!/^[0-9]+$/.test(portString)) {
     throw new Error('Invalid port number string returned: ' + portString);
@@ -87,11 +92,27 @@ function waitForAlive(port) {
 }
 
 function nativeNC(port, input) {
-  return spawnSync('nc', [host, port], {input: input});
+  return spawnSync('nc', [host, port], {
+    input: input,
+    windowsHide: true,
+    maxBuffer: Infinity,
+  });
 }
 
 function nodeNC(port, input) {
-  return spawnSync(process.execPath, ['-e', nodeNetCatSrc(port, input)]);
+  const src = nodeNetCatSrc(port, input);
+  if (src.length < 1000) {
+    return spawnSync(process.execPath, ['-e', src], {
+      windowsHide: true,
+      maxBuffer: Infinity,
+    });
+  } else {
+    return spawnSync(process.execPath, [], {
+      input: src,
+      windowsHide: true,
+      maxBuffer: Infinity,
+    });
+  }
 }
 
 function test(fn, port) {
@@ -117,6 +138,10 @@ function sendMessage(input) {
   try {
     return JSON.parse(res.stdout.toString('utf8'));
   } catch (ex) {
+    if (res.error) {
+      if (typeof res.error === 'string') res.error = new Error(res.error);
+      throw res.error;
+    }
     if (res.status !== 0) {
       throw new Error(
         configuration.fastestFunction.name +
@@ -126,10 +151,6 @@ function sendMessage(input) {
           (res.stderr && res.stderr.toString())
       );
     }
-    if (res.error) {
-      if (typeof res.error === 'string') res.error = new Error(res.error);
-      throw res.error;
-    }
     throw new Error(
       configuration.fastestFunction.name +
         ' failed:\n' +

package/lib/test-worker.js

@@ -1,5 +1,8 @@
 function init(connection) {
   return function(message) {
+    if (message === 'big') {
+      return Promise.resolve(Buffer.alloc(30 * 1024 * 1024, 42));
+    }
     return Promise.resolve('sent ' + message + ' to ' + connection);
   };
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "sync-rpc",
-  "version": "1.3.2",
+  "version": "1.3.6",
   "main": "lib/index.js",
   "description": "Run asynchronous commands synchronously by putting them in a separate process",
   "keywords": [],