fixing event handler error

Author: Parth Shah

src/events/analytics.js

@@ -1,10 +1,7 @@
 import Analytics from 'analytics-node';
 import _ from 'lodash';
 import 'config';
-import {
-  EVENT,
-  PROJECT_STATUS,
-} from '../constants';
+import { EVENT, PROJECT_STATUS } from '../constants';
 
 const PROJECT_CREATED = 'Project Created';
 const PROJECT_SUBMITTED = 'Project Submitted';

src/events/index.js

@@ -1,21 +1,10 @@
 
-import {
-  EVENT,
-} from '../constants';
-import {
-  projectCreatedHandler,
-} from './projects';
-import {
-  projectMemberAddedHandler,
-  projectMemberRemovedHandler,
-} from './projectMembers';
+import { EVENT } from '../constants';
+import { projectCreatedHandler } from './projects';
+import { projectMemberAddedHandler, projectMemberRemovedHandler } from './projectMembers';
 
-const handlers = {
+export default {
   [EVENT.ROUTING_KEY.PROJECT_DRAFT_CREATED]: projectCreatedHandler,
   [EVENT.ROUTING_KEY.PROJECT_MEMBER_ADDED]: projectMemberAddedHandler,
   [EVENT.ROUTING_KEY.PROJECT_MEMBER_REMOVED]: projectMemberRemovedHandler,
 };
-
-export default {
-  handlers,
-};

src/services/rabbitmq.js

@@ -1,19 +1,16 @@
 /* globals Promise */
 import _ from 'lodash';
 import amqplib from 'amqplib';
-import { EventEmitter } from 'events';
-import { handlers as msgHandlers } from '../events';
+import handlers from '../events';
 
-module.exports = class RabbitMQService extends EventEmitter {
+module.exports = class RabbitMQService {
 
   /**
    * constructor
    * @param {Object} app express app Object
    * @param {Object} logger logger object
    */
   constructor(app, logger) {
-    super();
-    EventEmitter.call(this);
     this.app = app;
     this.logger = logger;
     this.subscriberCxn = null;
@@ -87,7 +84,7 @@ module.exports = class RabbitMQService extends EventEmitter {
         self.subscriberQ = qok.queue;
         // bindings for the queue
         // all these keys/bindings should be routed to the same queue
-        const bindings = _.keys(msgHandlers);
+        const bindings = _.keys(handlers);
         self.logger.debug('Adding bindings: ', bindings);
         const bindingPromises = _.map(bindings, rk =>
           channel.bindQueue(self.subscriberQ, self.exchangeName, rk));
@@ -101,7 +98,7 @@ module.exports = class RabbitMQService extends EventEmitter {
             requestId: msg.properties.correlationId,
           });
           cLogger.debug('Received Message', key, msg.fields);
-          const handler = msgHandlers[key];
+          const handler = handlers[key];
           if (!_.isFunction(handler)) {
             cLogger.error(`Unknown message type: ${key}, NACKing... `);
             // channel.nack(msg, false, false)