From 43102e0ffccd9820461d24cc4797b44da49a10fe Mon Sep 17 00:00:00 2001 From: Henry Oswald Date: Thu, 25 Feb 2016 14:15:56 +0000 Subject: [PATCH] lowercase and trim add user to group email address --- .../Features/Subscription/SubscriptionGroupController.coffee | 2 +- .../Subscription/SubscriptionGroupControllerTests.coffee | 4 ++-- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/services/web/app/coffee/Features/Subscription/SubscriptionGroupController.coffee b/services/web/app/coffee/Features/Subscription/SubscriptionGroupController.coffee index e7fdc4f3e3..4f5d3cb3d0 100644 --- a/services/web/app/coffee/Features/Subscription/SubscriptionGroupController.coffee +++ b/services/web/app/coffee/Features/Subscription/SubscriptionGroupController.coffee @@ -12,7 +12,7 @@ module.exports = addUserToGroup: (req, res)-> adminUserId = req.session.user._id - newEmail = req.body.email + newEmail = req.body?.email?.toLowerCase()?.trim() logger.log adminUserId:adminUserId, newEmail:newEmail, "adding user to group subscription" SubscriptionGroupHandler.addUserToGroup adminUserId, newEmail, (err, user)-> if err? diff --git a/services/web/test/UnitTests/coffee/Subscription/SubscriptionGroupControllerTests.coffee b/services/web/test/UnitTests/coffee/Subscription/SubscriptionGroupControllerTests.coffee index dbaf751c70..455c441239 100644 --- a/services/web/test/UnitTests/coffee/Subscription/SubscriptionGroupControllerTests.coffee +++ b/services/web/test/UnitTests/coffee/Subscription/SubscriptionGroupControllerTests.coffee @@ -54,11 +54,11 @@ describe "SubscriptionGroupController", -> describe "addUserToGroup", -> it "should use the admin id for the logged in user and take the email address from the body", (done)-> - newEmail = "31231" + newEmail = " boB@gmaiL.com " @req.body = email: newEmail res = json : (data)=> - @GroupHandler.addUserToGroup.calledWith(@adminUserId, newEmail).should.equal true + @GroupHandler.addUserToGroup.calledWith(@adminUserId, "bob@gmail.com").should.equal true data.user.should.deep.equal @user done() @Controller.addUserToGroup @req, res