Fixes issue 29, according comments of issue.

Author: narekcat

src/services/ProcessorServiceProjectMemberInvite.js

@@ -57,10 +57,9 @@ create.schema = {
 const updateInvitesPromise = message => async (doc) => {
   // now merge the updated changes and re-index the document
   const invites = _.isArray(doc._source.invites) ? doc._source.invites : []
-  const removedInvites = _.remove(invites, invite => (!!message.email && invite.email === message.email) ||
-  (!!message.userId && invite.userId === message.userId))
-  if (invites.length && !removedInvites.length) {
-    throw new Error(`Invite with email "${message.email}" and userId "${message.userId}" is not found and not removed.`)
+  const removedInvites = _.remove(invites, invite => message.id === invite.id)
+  if (!removedInvites.length) {
+    throw new Error(`Invite with id "${message.id}" is not found and not removed.`)
   }
   return _.assign(doc._source, { invites })
 }

test/e2e/processor.project.index.test.js

@@ -887,15 +887,6 @@ describe('TC Project Member Invite Topic Tests', () => {
   it('update project member invite message - not found', async () => {
     const message = _.cloneDeep(projectMemberInviteUpdatedMessage)
     message.payload.id = notFoundId
-    await ProcessorService.update(message)
-    const data = await testHelper.getProjectESData(projectId)
-    expect(_.find(data.invites, { id: notFoundId })).to.be.an('undefined')
-  })
-
-  it('update project member invite message - invalid parameters, not existing member', async () => {
-    const message = _.cloneDeep(projectMemberInviteUpdatedMessage)
-    message.payload.userId = '11111111'
-    message.payload.email = 'wrong@mail.com'
     try {
       await ProcessorService.create(projectMemberInviteCreatedMessage)
       await ProcessorService.update(message)
@@ -923,15 +914,6 @@ describe('TC Project Member Invite Topic Tests', () => {
   it('delete project member invite message - not found', async () => {
     const message = _.cloneDeep(projectMemberInviteDeletedMessage)
     message.payload.id = notFoundId
-    await ProcessorService.deleteMessage(message)
-    const data = await testHelper.getProjectESData(projectId)
-    expect(_.find(data.invites, { id: notFoundId })).to.be.an('undefined')
-  })
-
-  it('delete project member invite message - invalid parameters, not existing member', async () => {
-    const message = _.cloneDeep(projectMemberInviteDeletedMessage)
-    message.payload.userId = '11111111'
-    message.payload.email = 'wrong@mail.com'
     try {
       await ProcessorService.create(projectMemberInviteCreatedMessage)
       await ProcessorService.deleteMessage(message)