Cleanups (#144)

+ Run lint first at npm publication CI
+ Import from cluster the minimum in test

Signed-off-by: Jérôme Benoit <jerome.benoit@sap.com>

diff --git a/.github/workflows/npmpublish.yml b/.github/workflows/npmpublish.yml
index 797287343717a5fd73800a965e9cd30f7595eac8..31b02ef4c6e2016e8e88b6daaf4d3e2292b84a42 100644 (file)
--- a/.github/workflows/npmpublish.yml
+++ b/.github/workflows/npmpublish.yml
@@ -17,8 +17,8 @@ jobs:
         with:
           node-version: 12
       - run: npm ci
-      - run: npm run test
       - run: npm run lint
+      - run: npm run test
       - run: npm run coverage
 
   publish-npm:

diff --git a/tests/worker/cluster/testWorker.js b/tests/worker/cluster/testWorker.js
index fd8c5603eaaf34cf85e9e3d25a08bdaa475620c2..a8a6bb9ee14a0e1e670bb12348dfc47a1590b1b3 100644 (file)
--- a/tests/worker/cluster/testWorker.js
+++ b/tests/worker/cluster/testWorker.js
@@ -1,6 +1,6 @@
 'use strict'
 const { ClusterWorker } = require('../../../lib/index')
-const cluster = require('cluster')
+const { isMaster } = require('cluster')
 
 function test (data) {
   for (let i = 0; i <= 50; i++) {
@@ -9,7 +9,7 @@ function test (data) {
     }
     JSON.stringify(o)
   }
-  return cluster.isMaster
+  return isMaster
 }
 
 module.exports = new ClusterWorker(test, { maxInactiveTime: 500 })