agw/overleaf-cep
1
0
Fork 0
mirror of https://github.com/yu-i-i/overleaf-cep.git synced 2026-05-28 19:41:33 +02:00
Code Issues Packages Projects Releases Wiki Activity

Use id to find subscription

Browse Source 
It's the new method signature.
This commit is contained in:
Alberto Fernández Capel
2018-07-16 14:42:37 +01:00
parent ae440f2ee6
commit 66b492ba5f
2 changed files with 3 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
services/web/app/coffee/Features/Subscription/TeamInvitesHandler.coffee
View File

@@ -76,7 +76,7 @@ module.exports = TeamInvitesHandler =
return callback(err) if err?
return callback(new Errors.NotFoundError('invite not found')) unless invite?
SubscriptionUpdater.addUserToGroup subscription.admin_id, userId, (err) ->
SubscriptionUpdater.addUserToGroup subscription._id, userId, (err) ->
return callback(err) if err?
removeInviteFromTeam(subscription.id, invite.email, callback)

3
services/web/test/unit/coffee/Subscription/TeamInvitesHandlerTests.coffee
View File

@@ -26,6 +26,7 @@ describe "TeamInvitesHandler", ->
@subscription = {
id: "55153a8014829a865bbf700d",
_id: new ObjectId("55153a8014829a865bbf700d"),
admin_id: @manager.id,
groupPlan: true,
member_ids: [],
@@ -210,7 +211,7 @@ describe "TeamInvitesHandler", ->
it "adds the user to the team", (done) ->
@TeamInvitesHandler.acceptInvite "dddddddd", @user.id, =>
@SubscriptionUpdater.addUserToGroup.calledWith(@manager.id, @user.id).should.eq true
@SubscriptionUpdater.addUserToGroup.calledWith(@subscription._id, @user.id).should.eq true
done()
it "removes the invite from the subscription", (done) ->