SYS-197 sync up with other changes and add unit tests

.eslintrc.json

@@ -12,7 +12,7 @@
     "plugin:security/recommended",
     "prettier"
   ],
-  "ignorePatterns": ["index.ts", "types.ts", "src/util/*.ts", "test/*"],
+  "ignorePatterns": ["index.ts", "src/util/*.ts", "test/*"],
   "rules": {
     "no-empty": [
       1,

.github/workflows/ci.yml

@@ -0,0 +1,27 @@
+name: Node CI Workflow
+
+on:
+  push:
+    branches:
+      - dev
+  pull_request:
+    branches:
+      - dev
+  workflow_dispatch:
+    inputs:
+      workflowBranch:
+        description: 'Select the branch to run the workflow on'
+        required: true
+        default: 'main'
+        type: choice
+        options:
+          - dev
+          - main
+jobs:
+  ci-dev:
+    if: ${{ github.event.inputs.workflowBranch == 'dev' }}
+    uses: shardeum/github-automation/.github/workflows/node-ci-shared.yml@dev
+
+  ci-main:
+    if: ${{ github.event.inputs.workflowBranch == 'main' || !github.event.inputs.workflowBranch }}
+    uses: shardeum/github-automation/.github/workflows/node-ci-shared.yml@main

package.json

@@ -24,18 +24,18 @@
     "release": "np --no-cleanup --no-yarn --any-branch",
     "prepack": "npm run build-node",
     "postinstall": "npm run build-rust",
-    "test": "cargo test",
+    "test": "npm run test:cargo",
     "test2": "ts-node test/test_lru.ts -p 44001 -c 2",
+    "test:cargo": "cargo test",
     "build:rust": "cargo build --workspace && npm run postbuild-cargo",
     "postbuild-cargo": "node scripts/copy-rename.js",
-    "lint": "eslint ./src/**/*.ts --quiet",
-    "lint-windows": "eslint ./src/**/*.ts --quiet",
-    "format-check": "prettier --check './src/**/*.ts'",
+    "lint": "eslint \"./src/**/*.ts\"",
+    "format-check": "prettier --check \"./src/**/*.ts\"",
     "clean": "cargo clean"
   },
   "repository": {
     "type": "git",
-    "url": "https://gitlab.com/Shardus/shardus-net.git"
+    "url": "https://github.com/shardeum/lib-net.git"
   },
   "publishConfig": {
     "access": "public"

src/index.ts

@@ -182,7 +182,7 @@ export const Sn = (opts: SnOpts) => {
               sendCallback
             )
           }
-        } else{
+        } else {
           /* prettier-ignore */ if(logFlags.net_verbose) console.log('sending without header')
           _net.send(port, address, stringifiedData, sendCallback)
         }

src/types.ts

@@ -66,7 +66,7 @@ export type SnOpts = {
  * @param opts - The options object to validate.
  * @throws An error if the options object is not valid.
  */
-export const validateSnOpts = (opts: SnOpts) => {
+export const validateSnOpts = (opts: SnOpts): void => {
   if (!opts) throw new Error('snq: must supply options')
 
   if (!opts.port || typeof opts.port !== 'number') throw new Error('snq: must supply port')