@mojaloop/sdk-scheme-adapter 12.0.2 → 12.1.0

package/src/InboundServer/index.js

@@ -24,6 +24,7 @@ const Validate = require('../lib/validate');
 const router = require('../lib/router');
 const handlers = require('./handlers');
 const middlewares = require('./middlewares');
+const check = require('../lib/check');
 
 class InboundApi extends EventEmitter {
     constructor(conf, logger, cache, validator) {
@@ -43,7 +44,7 @@ class InboundApi extends EventEmitter {
         });
 
         if (conf.validateInboundJws) {
-            this._jwsVerificationKeys = InboundApi._GetJwsKeys(conf.jwsVerificationKeysDirectory);
+            this._jwsVerificationKeys = conf.pm4mlEnabled ? conf.peerJWSKeys :  InboundApi._GetJwsKeys(conf.jwsVerificationKeysDirectory);
         }
         this._api = InboundApi._SetupApi({
             conf,
@@ -187,6 +188,37 @@ class InboundServer extends EventEmitter {
         this._logger.log('inbound shut down complete');
     }
 
+    async reconfigure(conf, logger, cache) {
+        // It may be possible to extract the socket from an existing HTTP/HTTPS server and replace
+        // it in a new server of the other type, as Node's HTTP and HTTPS servers both eventually
+        // are subclasses of net.Server. This wasn't considered as a requirement at the time of
+        // writing.
+        assert(
+            this._conf.mutualTLS.inboundRequests.enabled === conf.mutualTLS.inboundRequests.enabled,
+            'Cannot live-restart an HTTPS server as HTTP or vice versa',
+        );
+        const newApi = new InboundApi(conf, logger, cache, this._validator);
+        await newApi.start();
+        return () => {
+            this._logger = logger;
+            this._cache = cache;
+            // TODO: .tls might be undefined, causing an.. err.. undefined dereference..
+            const tlsCredsChanged = check.notDeepEqual(
+                conf.inbound.tls.creds,
+                this._conf.inbound.tls.creds
+            );
+            if (this._conf.mutualTLS.inboundRequests.enabled && tlsCredsChanged) {
+                this._server.setSecureContext(conf.inbound.tls.creds);
+            }
+            this._server.removeAllListeners('request');
+            this._server.on('request', newApi.callback());
+            this._api.stop();
+            this._api = newApi;
+            this._conf = conf;
+            this._logger.log('restarted');
+        };
+    }
+
     _createServer(tlsEnabled, tlsCreds, handler) {
         if (!tlsEnabled) {
             return http.createServer(handler);

package/src/OutboundServer/index.js

@@ -132,6 +132,21 @@ class OutboundServer extends EventEmitter {
         }
         this._logger.log('Shut down complete');
     }
+
+    async reconfigure(conf, logger, cache, metricsClient) {
+        const newApi = new OutboundApi(conf, logger, cache, this._validator, metricsClient);
+        await newApi.start();
+        return () => {
+            this._logger = logger;
+            this._cache = cache;
+            this._server.removeAllListeners('request');
+            this._server.on('request', newApi.callback());
+            this._api.stop();
+            this._api = newApi;
+            this._conf = conf;
+            this._logger.log('restarted');
+        };
+    }
 }
 
 module.exports = OutboundServer;

package/src/TestServer/index.js

@@ -11,6 +11,7 @@
 const Koa = require('koa');
 const ws = require('ws');
 
+const assert = require('assert').strict;
 const http = require('http');
 const yaml = require('js-yaml');
 const fs = require('fs').promises;
@@ -210,6 +211,36 @@ class TestServer {
         }
         this._logger.log('Test server shutdown complete');
     }
+
+    async reconfigure({ port, logger, cache }) {
+        assert(port === this._port, 'Cannot reconfigure running port');
+        const newApi = new TestApi(logger, cache, this._validator);
+        const newWsApi = new WsServer(logger.push({ component: 'websocket-server' }), cache);
+        await newWsApi.start();
+
+        return () => {
+            const oldWsApi = this._wsapi;
+            this._logger = logger;
+            this._cache = cache;
+            this._wsapi = newWsApi;
+            this._api = newApi;
+            this._server.removeAllListeners('upgrade');
+            this._server.on('upgrade', (req, socket, head) => {
+                this._wsapi.handleUpgrade(req, socket, head, (ws) =>
+                    this._wsapi.emit('connection', ws, req));
+            });
+            this._server.removeAllListeners('request');
+            this._server.on('request', newApi.callback());
+            // TODO: we can't guarantee client implementations. Therefore we can't guarantee
+            // reconnect logic/behaviour. Therefore instead of closing all websocket client
+            // connections as we do below, we should replace handlers.
+            oldWsApi.stop().catch((err) => {
+                this._logger.push({ err }).log('Error stopping websocket server during reconfigure');
+            });
+
+            this._logger.log('restarted');
+        };
+    }
 }
 
 module.exports = TestServer;

package/src/config.js

@@ -58,6 +58,10 @@ const env = from(process.env, {
 
 module.exports = {
     __parseResourceVersion: parseResourceVersions,
+    control: {
+        mgmtAPIWsUrl: env.get('MGMT_API_WS_URL').required().asString(),
+        mgmtAPIWsPort: env.get('MGMT_API_WS_PORT').default('4005').asPortNumber()
+    },
     mutualTLS: {
         inboundRequests: {
             enabled: env.get('INBOUND_MUTUAL_TLS_ENABLED').default('false').asBool(),
@@ -160,4 +164,6 @@ module.exports = {
     // a transactionRequestId. this option decodes the ilp packet for
     // the `transactionId` to retrieve the quote from cache
     allowDifferentTransferTransactionId: env.get('ALLOW_DIFFERENT_TRANSFER_TRANSACTION_ID').default('false').asBool(),
+
+    pm4mlEnabled: env.get('PM4ML_ENABLED').default('false').asBool(),
 };

package/test/config/integration.env

@@ -131,3 +131,12 @@ TRANSFERS_ENDPOINT=ml-testing-toolkit:5000
 # The incoming transfer request should consists of an ILP packet and a matching condition in this case.
 # The fulfilment will be generated from the provided ILP packet, and must hash to the provided condition.
 ALLOW_TRANSFER_WITHOUT_QUOTE=false
+
+# Management API websocket connection settings.
+# The Management API uses this for exchanging connector management messages.
+MGMT_API_WS_URL=127.0.0.1
+MGMT_API_WS_PORT=4005
+
+# Set to true to enable the use of PM4ML-related services e.g MCM, Management API service
+# when running the scheme-adapter as a mojaloop connector component within Payment Manager for Mojaloop.
+PM4ML_ENABLED=false

package/test/unit/ControlClient.test.js

@@ -0,0 +1,113 @@
+
+const ControlAgent = require('~/ControlAgent');
+const TestControlServer = require('./ControlServer');
+const InboundServer = require('~/InboundServer');
+const OutboundServer = require('~/OutboundServer');
+const TestServer = require('~/TestServer');
+const defaultConfig = require('./data/defaultConfig.json');
+const { Logger } = require('@mojaloop/sdk-standard-components');
+
+jest.mock('~/lib/cache');
+const Cache = require('~/lib/cache');
+
+// TODO:
+// - diff against master to determine what else needs testing
+// - especially look for assertions in the code
+// - err.. grep the code for TODO
+
+describe('ControlAgent', () => {
+    it('exposes a valid message API', () => {
+        expect(Object.keys(ControlAgent.build).sort()).toEqual(
+            Object.keys(ControlAgent.MESSAGE).sort(),
+            'The API exposed by the builder object must contain as top-level keys all of the message types exposed in the MESSAGE constant. Check that ControlAgent.MESSAGE has the same keys as ControlAgent.build.'
+        );
+        Object.entries(ControlAgent.build).forEach(([messageType, builders]) => {
+            expect(Object.keys(ControlAgent.VERB)).toEqual(
+                expect.arrayContaining(Object.keys(builders)),
+                `For message type '${messageType}' every builder must correspond to a verb. Check that ControlAgent.build.${messageType} has the same keys as ControlAgent.VERB.`
+            );
+        });
+        expect(Object.keys(ControlAgent.build.ERROR.NOTIFY).sort()).toEqual(
+            Object.keys(ControlAgent.ERROR).sort(),
+            'ControlAgent.ERROR.NOTIFY should contain the same keys as ControlAgent.ERROR'
+        );
+    });
+
+    describe('API', () => {
+        let server, logger, client;
+        const appConfig = { control: { port: 4005 }, what: 'ever' };
+        const changedConfig = { ...appConfig, some: 'thing' };
+
+        beforeEach(async () => {
+            logger = new Logger.Logger({ stringify: () => '' });
+            server = new TestControlServer.Server({ logger, appConfig });
+            client = await ControlAgent.Client.Create({
+                address: 'localhost',
+                port: server.address().port,
+                logger,
+                appConfig
+            });
+        });
+
+        afterEach(async () => {
+            await client.stop();
+            await server.stop();
+        });
+
+        it('receives config when requested', async () => {
+            await client.send(ControlAgent.build.CONFIGURATION.READ());
+            const response = await client.receive();
+            expect(response).toEqual({
+                ...JSON.parse(ControlAgent.build.CONFIGURATION.NOTIFY(appConfig, response.id)),
+            });
+        });
+
+        it('emits new config when received', async () => {
+            const newConfigEvent = new Promise(
+                (resolve) => client.on(ControlAgent.EVENT.RECONFIGURE, resolve)
+            );
+            await server.broadcastConfigChange(changedConfig);
+            const newConfEventData = await newConfigEvent;
+            expect(newConfEventData).toEqual(changedConfig);
+        });
+    });
+});
+
+describe('Server reconfigure methods', () => {
+    let conf, logger, cache;
+
+    const isPromise = (o) => Promise.resolve(o) === o;
+
+    beforeEach(() => {
+        conf = JSON.parse(JSON.stringify(defaultConfig));
+        logger = new Logger.Logger({ stringify: () => '' });
+        cache = new Cache({ ...conf.cacheConfig, logger: logger.push({ component: 'cache' }) });
+    });
+
+    test('InboundServer reconfigure method returns sync function', async () => {
+        const server = new InboundServer(conf, logger, cache);
+        const res = await server.reconfigure(conf, logger, cache);
+        expect(isPromise(res)).toEqual(false);
+    });
+
+    test('OutboundServer reconfigure method returns sync function', async () => {
+        const server = new OutboundServer(conf, logger, cache);
+        const res = await server.reconfigure(conf, logger, cache);
+        expect(isPromise(res)).toEqual(false);
+    });
+
+    test('TestServer reconfigure method returns sync function', async () => {
+        const server = new TestServer({ logger, cache });
+        const res = await server.reconfigure({ logger, cache });
+        expect(isPromise(res)).toEqual(false);
+    });
+
+    test('ControlClient reconfigure method returns sync function', async () => {
+        const server = new TestControlServer.Server({ logger, appConfig: { ...conf, control: { port: 4005 }}});
+        const client = await ControlAgent.Client.Create({ port: 4005, logger, appConfig: {} });
+        const res = await client.reconfigure({ logger, port: 4005, appConfig: {} });
+        expect(isPromise(res)).toEqual(false);
+        await client.close();
+        await server.close();
+    });
+});

package/test/unit/ControlServer/events.js

@@ -0,0 +1,41 @@
+/**************************************************************************
+ *  (C) Copyright ModusBox Inc. 2020 - All rights reserved.               *
+ *                                                                        *
+ *  This file is made available under the terms of the license agreement  *
+ *  specified in the corresponding source code repository.                *
+ *                                                                        *
+ *  ORIGINAL AUTHOR:                                                      *
+ *      Steven Oderayi - steven.oderayi@modusbox.com                       *
+ **************************************************************************/
+
+const { EventEmitter } = require('events');
+
+/**************************************************************************
+ * Internal events received by the control server via the exposed internal 
+ * event emitter. 
+ *************************************************************************/
+const INTERNAL_EVENTS = {
+    SERVER: {
+        BROADCAST_CONFIG_CHANGE: 'BROADCAST_CONFIG_CHANGE',
+    }
+};
+const internalEventEmitter = new EventEmitter();
+
+/**************************************************************************
+ * getInternalEventEmitter
+ *
+ * Returns an EventEmmitter that can be used to exchange internal events with 
+ * either the control server or the client from other modules within this service. 
+ * This prevents the need to pass down references to either the server or the client 
+ * from one module to another in order to use their interfaces.
+ * 
+ * @returns {events.EventEmitter}
+ *************************************************************************/
+const getInternalEventEmitter = () => {
+    return internalEventEmitter;
+};
+
+module.exports = {
+    getInternalEventEmitter,
+    INTERNAL_EVENTS
+};

package/test/unit/ControlServer/index.js

@@ -0,0 +1,231 @@
+/**************************************************************************
+ *  (C) Copyright ModusBox Inc. 2020 - All rights reserved.               *
+ *                                                                        *
+ *  This file is made available under the terms of the license agreement  *
+ *  specified in the corresponding source code repository.                *
+ *                                                                        *
+ *  ORIGINAL AUTHOR:                                                      *
+ *       Matt Kingston - matt.kingston@modusbox.com                       *
+ **************************************************************************/
+'use strict';
+
+const ws = require('ws');
+const jsonPatch = require('fast-json-patch');
+const randomPhrase = require('~/lib/randomphrase');
+const { getInternalEventEmitter, INTERNAL_EVENTS } = require('./events');
+
+const ControlServerEventEmitter = getInternalEventEmitter();
+
+
+/**************************************************************************
+ * The message protocol messages, verbs, and errors
+ *************************************************************************/
+const MESSAGE = {
+    CONFIGURATION: 'CONFIGURATION',
+    ERROR: 'ERROR',
+};
+
+const VERB = {
+    READ: 'READ',
+    NOTIFY: 'NOTIFY',
+    PATCH: 'PATCH'
+};
+
+const ERROR = {
+    UNSUPPORTED_MESSAGE: 'UNSUPPORTED_MESSAGE',
+    UNSUPPORTED_VERB: 'UNSUPPORTED_VERB',
+    JSON_PARSE_ERROR: 'JSON_PARSE_ERROR',
+};
+
+/**************************************************************************
+ * Private convenience functions
+ *************************************************************************/
+const serialise = JSON.stringify;
+const deserialise = (msg) => {
+    //reviver function
+    return JSON.parse(msg.toString(), (k, v) => {
+        if (
+            v !== null &&
+            typeof v === 'object' &&
+            'type' in v &&
+            v.type === 'Buffer' &&
+            'data' in v &&
+            Array.isArray(v.data)) {
+            return new Buffer(v.data);
+        }
+        return v;
+    });
+};
+
+
+const buildMsg = (verb, msg, data, id = randomPhrase()) => serialise({
+    verb,
+    msg,
+    data,
+    id,
+});
+
+const buildPatchConfiguration = (oldConf, newConf, id) => {
+    const patches = jsonPatch.compare(oldConf, newConf);
+    return buildMsg(VERB.PATCH, MESSAGE.CONFIGURATION, patches, id);
+};
+
+/**************************************************************************
+ * build
+ *
+ * Public object exposing an API to build valid protocol messages.
+ * It is not the only way to build valid messages within the protocol.
+ *************************************************************************/
+const build = {
+    CONFIGURATION: {
+        PATCH: buildPatchConfiguration,
+        READ: (id) => buildMsg(VERB.READ, MESSAGE.CONFIGURATION, {}, id),
+        NOTIFY: (config, id) => buildMsg(VERB.NOTIFY, MESSAGE.CONFIGURATION, config, id),
+    },
+    ERROR: {
+        NOTIFY: {
+            UNSUPPORTED_MESSAGE: (id) => buildMsg(VERB.NOTIFY, MESSAGE.ERROR, ERROR.UNSUPPORTED_MESSAGE, id),
+            UNSUPPORTED_VERB: (id) => buildMsg(VERB.NOTIFY, MESSAGE.ERROR, ERROR.UNSUPPORTED_VERB, id),
+            JSON_PARSE_ERROR: (id) => buildMsg(VERB.NOTIFY, MESSAGE.ERROR, ERROR.JSON_PARSE_ERROR, id),
+        }
+    },
+};
+
+/**************************************************************************
+ * Server
+ *
+ * The Control Server. Exposes a websocket control API.
+ * Used to hot-restart the SDK.
+ *
+ * logger    - Logger- see SDK logger used elsewhere
+ * port      - HTTP port to host on
+ * appConfig - The configuration for the entire application- supplied here as this class uses it to
+ *             validate reconfiguration requests- it is not used for configuration here, however
+ * server    - optional HTTP/S server on which to serve the websocket
+ *************************************************************************/
+class Server extends ws.Server {
+    constructor({ logger, appConfig = {} }) {
+        super({ clientTracking: true, port: appConfig.control.port });
+
+        this._logger = logger;
+        this._port = appConfig.control.port;
+        this._appConfig = appConfig;
+        this._clientData = new Map();
+
+        this.on('error', err => {
+            this._logger.push({ err })
+                .log('Unhandled websocket error occurred. Shutting down.');
+            process.exit(1);
+        });
+
+        this.on('connection', (socket, req) => {
+            const logger = this._logger.push({
+                url: req.url,
+                ip: 'localhost',
+                remoteAddress: req.socket.remoteAddress,
+            });
+            logger.log('Websocket connection received');
+            this._clientData.set(socket, { ip: req.connection.remoteAddress, logger });
+
+            socket.on('close', (code, reason) => {
+                logger.push({ code, reason }).log('Websocket connection closed');
+                this._clientData.delete(socket);
+            });
+
+            socket.on('message', this._handle(socket, logger));
+        });
+        this._logger.push(this.address()).log('running on');
+    }
+
+    // Close the server then wait for all the client sockets to close
+    async stop() {
+        await new Promise(this.close.bind(this));
+        this._logger.log('Control server shutdown complete');
+    }
+
+    _handle(client, logger) {
+        return (data) => {
+            // TODO: json-schema validation of received message- should be pretty straight-forward
+            // and will allow better documentation of the API
+            let msg;
+            try {
+                msg = deserialise(data);
+            } catch (err) {
+                logger.push({ data }).log('Couldn\'t parse received message');
+                client.send(build.ERROR.NOTIFY.JSON_PARSE_ERROR());
+            }
+            logger.push({ msg }).log('Handling received message');
+            switch (msg.msg) {
+                case MESSAGE.CONFIGURATION:
+                    switch (msg.verb) {
+                        case VERB.READ:
+                            (async () => {
+                                const jwsCerts = await this.populateConfig();
+                                client.send(build.CONFIGURATION.NOTIFY(jwsCerts, msg.id));
+                            })();
+                            break;
+                        default:
+                            client.send(build.ERROR.NOTIFY.UNSUPPORTED_VERB(msg.id));
+                            break;
+                    }
+                    break;
+                default:
+                    client.send(build.ERROR.NOTIFY.UNSUPPORTED_MESSAGE(msg.id));
+                    break;
+            }
+        };
+    }
+
+    async populateConfig(){
+        return this._appConfig;
+    }
+
+
+    /**
+     * Register this server instance to receive internal server messages
+     * from other modules.
+     */
+    registerInternalEvents() {
+        ControlServerEventEmitter.on(INTERNAL_EVENTS.SERVER.BROADCAST_CONFIG_CHANGE, (params) => {
+            this.broadcastConfigChange(params);
+        });
+    }
+
+    /**
+     * Broadcast configuration change to all connected clients.
+     *
+     * @param {object} params Updated configuration
+     */
+    async broadcastConfigChange(updatedConfig) {
+        const updateConfMsg = build.CONFIGURATION.PATCH({}, updatedConfig, randomPhrase());
+        const errorLogger = (socket, message) => (err) =>
+            this._logger
+                .push({ message, ip: this._clientData.get(socket).ip, err })
+                .log('Error sending JWS keys notification to client');
+        return await this.broadcast(updateConfMsg, errorLogger);
+    }
+
+    /**
+    * Broadcasts a protocol message to all connected clients.
+    *
+    * @param {string} msg
+    * @param {object} errorLogger
+    */
+    async broadcast(msg, errorLogger) {
+        const sendToAllClients = (msg, errorLogger) => Promise.all(
+            [...this.clients.values()].map((socket) =>
+                (new Promise((resolve) => socket.send(msg, resolve))).catch(errorLogger(socket, msg))
+            )
+        );
+        return await sendToAllClients(msg, errorLogger);
+    }
+}
+
+
+module.exports = {
+    Server,
+    build,
+    MESSAGE,
+    VERB,
+    ERROR,
+};

package/test/unit/ControlServer.test.js

@@ -0,0 +1,126 @@
+
+const ControlServer = require('~/ControlServer');
+const InboundServer = require('~/InboundServer');
+const OutboundServer = require('~/OutboundServer');
+const TestServer = require('~/TestServer');
+const defaultConfig = require('./data/defaultConfig.json');
+const { Logger } = require('@mojaloop/sdk-standard-components');
+
+jest.mock('~/lib/cache');
+const Cache = require('~/lib/cache');
+
+// TODO:
+// - diff against master to determine what else needs testing
+// - especially look for assertions in the code
+// - err.. grep the code for TODO
+
+describe('ControlServer', () => {
+    it('exposes a valid message API', () => {
+        expect(Object.keys(ControlServer.build).sort()).toEqual(
+            Object.keys(ControlServer.MESSAGE).sort(),
+            'The API exposed by the builder object must contain as top-level keys all of the message types exposed in the MESSAGE constant. Check that ControlServer.MESSAGE has the same keys as ControlServer.build.'
+        );
+        Object.entries(ControlServer.build).forEach(([messageType, builders]) => {
+            expect(Object.keys(ControlServer.VERB)).toEqual(
+                expect.arrayContaining(Object.keys(builders)),
+                `For message type '${messageType}' every builder must correspond to a verb. Check that ControlServer.build.${messageType} has the same keys as ControlServer.VERB.`
+            );
+        });
+        expect(Object.keys(ControlServer.build.ERROR.NOTIFY).sort()).toEqual(
+            Object.keys(ControlServer.ERROR).sort(),
+            'ControlServer.ERROR.NOTIFY should contain the same keys as ControlServer.ERROR'
+        );
+    });
+
+    describe('API', () => {
+        let server, logger, client;
+        const appConfig = { what: 'ever' };
+        const changedConfig = { ...appConfig, some: 'thing' };
+
+        beforeEach(async () => {
+            logger = new Logger.Logger({ stringify: () => '' });
+            server = new ControlServer.Server({ logger, appConfig });
+            client = await ControlServer.Client.Create({
+                address: 'localhost',
+                port: server.address().port,
+                logger
+            });
+        });
+
+        afterEach(async () => {
+            await server.stop();
+        });
+
+        it('supplies config when requested', async () => {
+            await client.send(ControlServer.build.CONFIGURATION.READ());
+            const response = await client.receive();
+            expect(response).toEqual({
+                ...JSON.parse(ControlServer.build.CONFIGURATION.NOTIFY(appConfig, response.id)),
+            });
+        });
+
+        it('emits new config when received', async () => {
+            const newConfigEvent = new Promise(
+                (resolve) => server.on(ControlServer.EVENT.RECONFIGURE, resolve)
+            );
+            await client.send(ControlServer.build.CONFIGURATION.PATCH(appConfig, changedConfig));
+            const newConfEventData = await newConfigEvent;
+            expect(newConfEventData).toEqual(changedConfig);
+        });
+
+        it('sends new config to clients when instructed', async () => {
+            const client2 = await ControlServer.Client.Create({
+                address: 'localhost',
+                port: server.address().port,
+                logger
+            });
+            const changedConfig = { ...appConfig, some: 'thing' };
+            await client.send(ControlServer.build.CONFIGURATION.PATCH(appConfig, changedConfig));
+            const restart = server.reconfigure({ appConfig: changedConfig });
+            restart();
+            await server.notifyClientsOfCurrentConfig();
+            const [notification, notification2] =
+                await Promise.all([client.receive(), client2.receive()]);
+            const expected = ControlServer.build.CONFIGURATION.NOTIFY(changedConfig, notification.id);
+            expect(JSON.stringify(notification)).toEqual(expected);
+            expect(JSON.stringify(notification2)).toEqual(expected);
+        });
+    });
+});
+
+describe('Server reconfigure methods', () => {
+    let conf, logger, cache;
+
+    const isPromise = (o) => Promise.resolve(o) === o;
+
+    beforeEach(() => {
+        conf = JSON.parse(JSON.stringify(defaultConfig));
+        logger = new Logger.Logger({ stringify: () => '' });
+        cache = new Cache({ ...conf.cacheConfig, logger: logger.push({ component: 'cache' }) });
+    });
+
+    test('InboundServer reconfigure method returns sync function', async () => {
+        const server = new InboundServer(conf, logger, cache);
+        const res = await server.reconfigure(conf, logger, cache);
+        expect(isPromise(res)).toEqual(false);
+    });
+
+    test('OutboundServer reconfigure method returns sync function', async () => {
+        const server = new OutboundServer(conf, logger, cache);
+        const res = await server.reconfigure(conf, logger, cache);
+        expect(isPromise(res)).toEqual(false);
+    });
+
+    test('TestServer reconfigure method returns sync function', async () => {
+        const server = new TestServer({ logger, cache });
+        const res = await server.reconfigure({ logger, cache });
+        expect(isPromise(res)).toEqual(false);
+    });
+
+    test('ControlServer reconfigure method returns sync function', async () => {
+        const server = new ControlServer.Server({ logger, appConfig: {} });
+        const res = await server.reconfigure({ logger, appConfig: {} });
+        expect(isPromise(res)).toEqual(false);
+        await server.close();
+    });
+});

package/test/unit/config.test.js

@@ -28,6 +28,7 @@ describe('config', () => {
         process.env.BACKEND_ENDPOINT = '172.17.0.5:4000';
         process.env.CACHE_HOST = '172.17.0.2';
         process.env.CACHE_PORT = '6379';
+        process.env.MGMT_API_WS_URL = '0.0.0.0';
         certDir = fs.mkdtempSync(path.join(os.tmpdir(), 'jest-'));
     });
 

package/test/unit/index.test.js

@@ -21,6 +21,7 @@ process.env.PEER_ENDPOINT = '172.17.0.3:4000';
 process.env.BACKEND_ENDPOINT = '172.17.0.5:4000';
 process.env.CACHE_HOST = '172.17.0.2';
 process.env.CACHE_PORT = '6379';
+process.env.MGMT_API_WS_URL = '0.0.0.0';
 
 const index = require('~/index.js');