fix issues with workflow; remove unused function

Author: drinking-code

.github/workflows/tests.yml

@@ -22,23 +22,28 @@ jobs:
       - name: Build
         run: npm run build
 
-      - name: Cache lib
+      - name: Cache lib and node_modules
         uses: actions/cache@v3
         with:
-          path: lib
+          path: |
+            lib
+            node_modules
           key: ${{ runner.os }}-ipos-lib-${{ github.sha }}
 
   test:
+    needs: [ build ]
     name: Test
     runs-on: ubuntu-latest
     steps:
       # Checks-out your repository under $GITHUB_WORKSPACE, so your job can access it
       - uses: actions/checkout@v3
 
-      - name: Cache lib
+      - name: Cache lib and node_modules
         uses: actions/cache@v3
         with:
-          path: lib
+          path: |
+            lib
+            node_modules
           key: ${{ runner.os }}-ipos-lib-${{ github.sha }}
 
       - name: Run Tests

.gitignore

@@ -1,2 +1,3 @@
 lib
+coverage
 ipos-*.tgz

.npmignore

@@ -1,5 +1,6 @@
 .idea/
 .nvmrc
+coverage
 example
 src
 tsconfig.json

package.json

@@ -20,7 +20,7 @@
     "build": "rm -rf lib && tsc",
     "watch": "tsc --watch",
     "prepack": "npm run build",
-    "test": "node --experimental-vm-modules ./node_modules/.bin/jest --verbose"
+    "test": "node --experimental-vm-modules ./node_modules/.bin/jest --verbose --coverage"
   },
   "devDependencies": {
     "@types/jest": "^29.0.3",

src/__test__/createConnectedInstances.ts

@@ -1,11 +1,11 @@
 import subProcessIPCLoopback from './subProcessIPCLoopback'
 import IPOS from '../main'
-import {withoutProcessSendSync} from './withoutProcessSendSync'
+import {withoutProcessSend} from './withoutProcessSend'
 
 export default async function createConnectedInstances(): Promise<{ main_ipos: IPOS, sub_ipos: IPOS, sub_process: subProcessIPCLoopback }> {
     const sub_process = new subProcessIPCLoopback()
     let main_ipos: IPOS, sub_ipos: IPOS
-    await withoutProcessSendSync(() => {
+    await withoutProcessSend(() => {
         main_ipos = IPOS.new() as IPOS
     })
 

src/__test__/initialize.test.ts

@@ -1,12 +1,12 @@
 import IPOS from '../main'
 import subProcessIPCLoopback from './subProcessIPCLoopback'
-import {withoutProcessSendSync} from './withoutProcessSendSync'
+import {withoutProcessSend} from './withoutProcessSend'
 
 describe('Initialising IPOS', () => {
     it('Create new instance in a main process', () => {
         // this is a subprocess anyway
         // just simulate main process by setting process.send to undefined (jamming the subprocess detection)
-        withoutProcessSendSync(() => {
+        withoutProcessSend(() => {
             let ipos
             expect(() => ipos = IPOS.new()).not.toThrow()
             expect(ipos).toBeInstanceOf(IPOS)
@@ -33,7 +33,7 @@ describe('Initialising IPOS', () => {
 
         const sub_ipos: Promise<IPOS> = IPOS.new() as Promise<IPOS>
         let main_ipos: IPOS
-        withoutProcessSendSync(() => {
+        withoutProcessSend(() => {
             main_ipos = IPOS.new() as IPOS
         })
 
@@ -58,7 +58,7 @@ describe('Initialising IPOS', () => {
         const sub_process = new subProcessIPCLoopback()
 
         let main_ipos: IPOS
-        withoutProcessSendSync(() => {
+        withoutProcessSend(() => {
             main_ipos = IPOS.new() as IPOS
         })
 

src/__test__/synchronize.test.ts

@@ -1,15 +1,15 @@
 import IPOS from '../main'
 
 import subProcessIPCLoopback from './subProcessIPCLoopback'
-import {withoutProcessSendSync} from './withoutProcessSendSync'
+import {withoutProcessSend} from './withoutProcessSend'
 import createFieldsTest from './runCreateFieldsTest'
 
 describe('Synchronising fields', () =>
     createFieldsTest(
         async (setValue: (ipos_for_setting: IPOS) => void, probeValue: (ipos_for_probing: IPOS) => void) => {
             const sub_process = new subProcessIPCLoopback()
             let main_ipos: IPOS, sub_ipos: IPOS
-            await withoutProcessSendSync(() => {
+            await withoutProcessSend(() => {
                 main_ipos = IPOS.new() as IPOS
                 setValue(main_ipos)
             })

src/__test__/withoutProcessSend.ts

@@ -0,0 +1,6 @@
+export function withoutProcessSend(callback: () => void) {
+    const processSend = process.send
+    process.send = undefined
+    callback()
+    process.send = processSend
+}