mirror of
https://github.com/overleaf/overleaf.git
synced 2024-12-29 12:13:32 +00:00
f7532e5854
Use emails with tokens for team invites
216 lines
7.5 KiB
CoffeeScript
216 lines
7.5 KiB
CoffeeScript
SandboxedModule = require('sandboxed-module')
|
|
should = require('chai').should()
|
|
sinon = require 'sinon'
|
|
assert = require("chai").assert
|
|
modulePath = "../../../../app/js/Features/Subscription/SubscriptionGroupHandler"
|
|
|
|
|
|
describe "SubscriptionGroupHandler", ->
|
|
|
|
beforeEach ->
|
|
@adminUser_id = "12321"
|
|
@newEmail = "bob@smith.com"
|
|
@user_id = "3121321"
|
|
@email = "jim@example.com"
|
|
@user = {_id:@user_id, email:@newEmail}
|
|
@subscription_id = "31DSd1123D"
|
|
|
|
@subscription =
|
|
admin_id:@adminUser_id
|
|
_id:@subscription_id
|
|
|
|
@SubscriptionLocator =
|
|
getUsersSubscription: sinon.stub()
|
|
getSubscriptionByMemberIdAndId: sinon.stub()
|
|
getSubscription: sinon.stub()
|
|
|
|
@UserCreator =
|
|
getUserOrCreateHoldingAccount: sinon.stub().callsArgWith(1, null, @user)
|
|
|
|
@SubscriptionUpdater =
|
|
addUserToGroup: sinon.stub().callsArgWith(2)
|
|
removeUserFromGroup: sinon.stub().callsArgWith(2)
|
|
|
|
@TeamInvitesHandler =
|
|
createInvite: sinon.stub().callsArgWith(2)
|
|
|
|
@UserGetter =
|
|
getUser: sinon.stub()
|
|
getUserByAnyEmail: sinon.stub()
|
|
|
|
@LimitationsManager =
|
|
hasGroupMembersLimitReached: sinon.stub()
|
|
|
|
@OneTimeTokenHandler =
|
|
getValueFromTokenAndExpire:sinon.stub()
|
|
getNewToken:sinon.stub()
|
|
|
|
@EmailHandler =
|
|
sendEmail:sinon.stub()
|
|
|
|
@Subscription =
|
|
update: sinon.stub().yields()
|
|
|
|
@settings =
|
|
siteUrl:"http://www.sharelatex.com"
|
|
|
|
@readStub = sinon.stub()
|
|
@NotificationsBuilder =
|
|
groupPlan: sinon.stub().returns({read:@readStub})
|
|
|
|
@Handler = SandboxedModule.require modulePath, requires:
|
|
"logger-sharelatex": log:->
|
|
"../User/UserCreator": @UserCreator
|
|
"./SubscriptionUpdater": @SubscriptionUpdater
|
|
"./TeamInvitesHandler": @TeamInvitesHandler
|
|
"./SubscriptionLocator": @SubscriptionLocator
|
|
"../../models/Subscription": Subscription: @Subscription
|
|
"../User/UserGetter": @UserGetter
|
|
"./LimitationsManager": @LimitationsManager
|
|
"../Security/OneTimeTokenHandler":@OneTimeTokenHandler
|
|
"../Email/EmailHandler":@EmailHandler
|
|
"settings-sharelatex":@settings
|
|
"../Notifications/NotificationsBuilder": @NotificationsBuilder
|
|
"logger-sharelatex":
|
|
err:->
|
|
log:->
|
|
warn:->
|
|
|
|
|
|
describe "addUserToGroup", ->
|
|
beforeEach ->
|
|
@LimitationsManager.hasGroupMembersLimitReached.callsArgWith(1, null, false, @subscription)
|
|
@UserGetter.getUserByAnyEmail.callsArgWith(1, null, @user)
|
|
|
|
it "should find the user", (done)->
|
|
@Handler.addUserToGroup @adminUser_id, @newEmail, (err)=>
|
|
@UserGetter.getUserByAnyEmail.calledWith(@newEmail).should.equal true
|
|
done()
|
|
|
|
it "should add the user to the group", (done)->
|
|
@Handler.addUserToGroup @adminUser_id, @newEmail, (err)=>
|
|
@SubscriptionUpdater.addUserToGroup.calledWith(@adminUser_id, @user._id).should.equal true
|
|
done()
|
|
|
|
it "should not add the user to the group if the limit has been reached", (done)->
|
|
@LimitationsManager.hasGroupMembersLimitReached.callsArgWith(1, null, true, @subscription)
|
|
@Handler.addUserToGroup @adminUser_id, @newEmail, (err)=>
|
|
@SubscriptionUpdater.addUserToGroup.called.should.equal false
|
|
done()
|
|
|
|
it "should return error that limit has been reached", (done)->
|
|
@LimitationsManager.hasGroupMembersLimitReached.callsArgWith(1, null, true, @subscription)
|
|
@Handler.addUserToGroup @adminUser_id, @newEmail, (err)=>
|
|
err.limitReached.should.equal true
|
|
done()
|
|
|
|
it "should mark any notification as read if it is part of a licence", (done)->
|
|
@Handler.addUserToGroup @adminUser_id, @newEmail, (err)=>
|
|
@NotificationsBuilder.groupPlan.calledWith(@user, {subscription_id:@subscription._id}).should.equal true
|
|
@readStub.called.should.equal true
|
|
done()
|
|
|
|
it "should add an email invite if no user is found", (done) ->
|
|
@UserGetter.getUserByAnyEmail.callsArgWith(1, null, null)
|
|
@Handler.addUserToGroup @adminUser_id, @newEmail, (err)=>
|
|
@TeamInvitesHandler.createInvite.calledWith(@adminUser_id, @newEmail).should.equal true
|
|
done()
|
|
|
|
describe "removeUserFromGroup", ->
|
|
|
|
it "should call the subscription updater to remove the user", (done)->
|
|
@Handler.removeUserFromGroup @adminUser_id, @user._id, (err)=>
|
|
@SubscriptionUpdater.removeUserFromGroup.calledWith(@adminUser_id, @user._id).should.equal true
|
|
done()
|
|
|
|
describe "replaceUserReferencesInGroups", ->
|
|
beforeEach ->
|
|
@oldId = "ba5eba11"
|
|
@newId = "5ca1ab1e"
|
|
|
|
it "replaces the admin_id", (done) ->
|
|
@Handler.replaceUserReferencesInGroups @oldId, @newId, (err) =>
|
|
|
|
@Subscription.update.calledWith(
|
|
{ admin_id: @oldId },
|
|
{ admin_id: @newId }
|
|
).should.equal true
|
|
|
|
done()
|
|
|
|
it "replaces the member ids", (done) ->
|
|
@Handler.replaceUserReferencesInGroups @oldId, @newId, (err) =>
|
|
|
|
@Subscription.update.calledWith(
|
|
{ member_ids: @oldId },
|
|
{ $addToSet: { member_ids: @newId } }
|
|
).should.equal true
|
|
|
|
@Subscription.update.calledWith(
|
|
{ member_ids: @oldId },
|
|
{ $pull: { member_ids: @oldId } }
|
|
).should.equal true
|
|
|
|
done()
|
|
|
|
describe "getPopulatedListOfMembers", ->
|
|
beforeEach ->
|
|
@subscription = {}
|
|
@SubscriptionLocator.getUsersSubscription.callsArgWith(1, null, @subscription)
|
|
@UserGetter.getUser.callsArgWith(1, null, {_id:"31232"})
|
|
|
|
it "should locate the subscription", (done)->
|
|
@UserGetter.getUser.callsArgWith(1, null, {_id:"31232"})
|
|
@Handler.getPopulatedListOfMembers @adminUser_id, (err, users)=>
|
|
@SubscriptionLocator.getUsersSubscription.calledWith(@adminUser_id).should.equal true
|
|
done()
|
|
|
|
it "should get the users by id", (done)->
|
|
@UserGetter.getUser.callsArgWith(1, null, {_id:"31232"})
|
|
@subscription.member_ids = ["1234", "342432", "312312"]
|
|
@Handler.getPopulatedListOfMembers @adminUser_id, (err, users)=>
|
|
@UserGetter.getUser.calledWith(@subscription.member_ids[0]).should.equal true
|
|
@UserGetter.getUser.calledWith(@subscription.member_ids[1]).should.equal true
|
|
@UserGetter.getUser.calledWith(@subscription.member_ids[2]).should.equal true
|
|
users.length.should.equal @subscription.member_ids.length
|
|
done()
|
|
|
|
it "should just return the id if the user can not be found as they may have deleted their account", (done)->
|
|
@UserGetter.getUser.callsArgWith(1)
|
|
@subscription.member_ids = ["1234", "342432", "312312"]
|
|
@Handler.getPopulatedListOfMembers @adminUser_id, (err, users)=>
|
|
assert.deepEqual users[0], {_id:@subscription.member_ids[0]}
|
|
assert.deepEqual users[1], {_id:@subscription.member_ids[1]}
|
|
assert.deepEqual users[2], {_id:@subscription.member_ids[2]}
|
|
done()
|
|
|
|
it "should return any invited users", (done) ->
|
|
@subscription.invited_emails = [ "jo@example.com" ]
|
|
|
|
@subscription.teamInvites = [
|
|
{ email: "charlie@example.com" }
|
|
]
|
|
|
|
@Handler.getPopulatedListOfMembers @adminUser_id, (err, users)=>
|
|
users[0].email.should.equal "jo@example.com"
|
|
users[0].invite.should.equal true
|
|
users[1].email.should.equal "charlie@example.com"
|
|
users[1].invite.should.equal true
|
|
users.length.should.equal @subscription.teamInvites.length + @subscription.invited_emails.length
|
|
done()
|
|
|
|
describe "isUserPartOfGroup", ->
|
|
beforeEach ->
|
|
@subscription_id = "123ed13123"
|
|
|
|
it "should return true when user is part of subscription", (done)->
|
|
@SubscriptionLocator.getSubscriptionByMemberIdAndId.callsArgWith(2, null, {_id:@subscription_id})
|
|
@Handler.isUserPartOfGroup @user_id, @subscription_id, (err, partOfGroup)->
|
|
partOfGroup.should.equal true
|
|
done()
|
|
|
|
it "should return false when no subscription is found", (done)->
|
|
@SubscriptionLocator.getSubscriptionByMemberIdAndId.callsArgWith(2, null)
|
|
@Handler.isUserPartOfGroup @user_id, @subscription_id, (err, partOfGroup)->
|
|
partOfGroup.should.equal false
|
|
done()
|