From 248546b34771dbe4c847cbc10ca8e819544b980a Mon Sep 17 00:00:00 2001
From: Sebastian <sebastian.mihalache@thinslices.com>
Date: Wed, 4 Apr 2018 15:46:44 +0300
Subject: [PATCH] feat(component-invite): fix invitation setup
---
.../src/routes/collectionsInvitations/post.js | 2 +-
.../src/tests/collectionsInvitations/post.test.js | 4 ++++
2 files changed, 5 insertions(+), 1 deletion(-)
diff --git a/packages/component-invite/src/routes/collectionsInvitations/post.js b/packages/component-invite/src/routes/collectionsInvitations/post.js
index 0ebecc69a..d00b18ab5 100644
--- a/packages/component-invite/src/routes/collectionsInvitations/post.js
+++ b/packages/component-invite/src/routes/collectionsInvitations/post.js
@@ -46,7 +46,7 @@ module.exports = models => async (req, res) => {
await teamHelper.setupManuscriptTeam(models, user, collectionId, role)
// find if there already is a matching invitation in the collection
- await collectionHelper.addInvitation(collection)
+ await collectionHelper.addInvitation(collection, user.id, role)
try {
await mailService.setupAssignEmail(
diff --git a/packages/component-invite/src/tests/collectionsInvitations/post.test.js b/packages/component-invite/src/tests/collectionsInvitations/post.test.js
index cf7cce909..cebe37f0a 100644
--- a/packages/component-invite/src/tests/collectionsInvitations/post.test.js
+++ b/packages/component-invite/src/tests/collectionsInvitations/post.test.js
@@ -70,6 +70,10 @@ describe('Post collections invitations route handler', () => {
expect(standardCollection.invitations.length).toBeGreaterThan(initialSize)
expect(heTeam.members).toContain(author.id)
expect(author.teams).toContain(heTeam.id)
+ const matchingInvitation = standardCollection.invitations.find(
+ inv => inv.userId === author.id && inv.role === body.role,
+ )
+ expect(matchingInvitation).toBeDefined()
})
it('should return an error when inviting his self', async () => {
body.role = roles[random(0, roles.length - 1)]
--
GitLab