From ae440f2ee6c8cf9bbbd2f8d03e3ba76f457245fb Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Alberto=20Fern=C3=A1ndez=20Capel?= Date: Mon, 16 Jul 2018 14:40:53 +0100 Subject: [PATCH] Better variable name in test 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. --- .../coffee/Subscription/SubscriptionGroupHandlerTests.coffee | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/services/web/test/unit/coffee/Subscription/SubscriptionGroupHandlerTests.coffee b/services/web/test/unit/coffee/Subscription/SubscriptionGroupHandlerTests.coffee index db8aac0375..87ab39a6b5 100644 --- a/services/web/test/unit/coffee/Subscription/SubscriptionGroupHandlerTests.coffee +++ b/services/web/test/unit/coffee/Subscription/SubscriptionGroupHandlerTests.coffee @@ -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)->