fix: initialize eventHandlers only once during test

Author: imcaizheng

package.json

@@ -20,13 +20,13 @@
     "data:import": "node scripts/data/importData.js",
     "migrate": "npx sequelize db:migrate",
     "migrate:undo": "npx sequelize db:migrate:undo",
-    "test": "mocha test/unit/*.test.js --timeout 30000 --exit",
+    "test": "mocha test/unit/*.test.js --timeout 30000 --require test/prepare.js --exit",
     "services:up": "docker-compose -f ./local/docker-compose.yml up -d",
     "services:down": "docker-compose -f ./local/docker-compose.yml down",
     "services:logs": "docker-compose -f ./local/docker-compose.yml logs",
     "local:init": "npm run local:reset && npm run data:import -- --force",
     "local:reset": "npm run delete-index -- --force || true && npm run create-index -- --force && npm run init-db force",
-    "cov": "nyc --reporter=html --reporter=text mocha test/unit/*.test.js --timeout 30000 --exit",
+    "cov": "nyc --reporter=html --reporter=text npm run test",
     "demo-payment": "node scripts/demo-payment"
   },
   "keywords": [],

test/prepare.js

@@ -0,0 +1,7 @@
+/*
+ * Prepare for tests.
+ */
+
+process.env.NODE_ENV = 'test'
+require('../src/bootstrap')
+require('../src/eventHandlers').init()

test/unit/ResourceBookingService.test.js

@@ -1,6 +1,4 @@
 /* eslint-disable no-unused-expressions */
-process.env.NODE_ENV = 'test'
-require('../../src/bootstrap')
 
 // const _ = require('lodash')
 const expect = require('chai').expect
@@ -10,12 +8,10 @@ const service = require('../../src/services/ResourceBookingService')
 const workPeriodService = require('../../src/services/WorkPeriodService')
 const testData = require('./common/testData')
 const helper = require('../../src/common/helper')
-const eventHandlers = require('../../src/eventHandlers')
 // const esClient = helper.getESClient()
 const busApiClient = helper.getBusApiClient()
 const ResourceBooking = models.ResourceBooking
 const WorkPeriod = models.WorkPeriod
-eventHandlers.init()
 describe('resourceBooking service test', () => {
   let stubEnsureJobById
   let stubEnsureUserById

test/unit/WorkPeriodPaymentService.test.js

@@ -1,6 +1,4 @@
 /* eslint-disable no-unused-expressions */
-process.env.NODE_ENV = 'test'
-require('../../src/bootstrap')
 
 // const _ = require('lodash')
 const expect = require('chai').expect
@@ -10,10 +8,8 @@ const service = require('../../src/services/WorkPeriodPaymentService')
 const paymentService = require('../../src/services/PaymentService')
 const testData = require('./common/testData')
 const helper = require('../../src/common/helper')
-const eventHandlers = require('../../src/eventHandlers')
 // const esClient = helper.getESClient()
 const busApiClient = helper.getBusApiClient()
-eventHandlers.init()
 describe('workPeriod service test', () => {
   beforeEach(() => {
     sinon.stub(busApiClient, 'postEvent').callsFake(async () => {})