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

Better variable name in test

Browse Source 
It should be a subscription id, instead of an admin id. The test passes
anyway because the db calls are mocked, but the wrong variable name
can cause confusion.
This commit is contained in:
Alberto Fernández Capel
2018-07-16 14:40:53 +01:00
parent f08b64eeac
commit ae440f2ee6
1 changed files with 2 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
services/web/test/unit/coffee/Subscription/SubscriptionGroupHandlerTests.coffee
View File

@@ -88,8 +88,8 @@ describe "SubscriptionGroupHandler", ->
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
@Handler.addUserToGroup @subscription_id, @newEmail, (err)=>
@SubscriptionUpdater.addUserToGroup.calledWith(@subscription_id, @user._id).should.equal true
done()
it "should not add the user to the group if the limit has been reached", (done)->