UserRegistrationHandlerTests.coffee 6.8 KB

123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143144145146147148149150151152153154155156157158159160161162163164165166167168169170171172173174175176177178179180181182183184185186187188189190191192193
  1. should = require('chai').should()
  2. SandboxedModule = require('sandboxed-module')
  3. assert = require('assert')
  4. path = require('path')
  5. modulePath = path.join __dirname, '../../../../app/js/Features/User/UserRegistrationHandler'
  6. sinon = require("sinon")
  7. expect = require("chai").expect
  8. describe "UserRegistrationHandler", ->
  9. beforeEach ->
  10. @user =
  11. _id: @user_id = "31j2lk21kjl"
  12. @User =
  13. findOne:sinon.stub()
  14. update: sinon.stub().callsArgWith(2)
  15. @UserCreator =
  16. createNewUser:sinon.stub().callsArgWith(1, null, @user)
  17. @AuthenticationManager =
  18. setUserPassword: sinon.stub().callsArgWith(2)
  19. @NewsLetterManager =
  20. subscribe: sinon.stub().callsArgWith(1)
  21. @EmailHandler =
  22. sendEmail:sinon.stub().callsArgWith(2)
  23. @OneTimeTokenHandler =
  24. getNewToken: sinon.stub()
  25. @handler = SandboxedModule.require modulePath, requires:
  26. "../../models/User": {User:@User}
  27. "./UserCreator": @UserCreator
  28. "../Authentication/AuthenticationManager":@AuthenticationManager
  29. "../Newsletter/NewsletterManager":@NewsLetterManager
  30. "logger-sharelatex": @logger = { log: sinon.stub() }
  31. "crypto": @crypto = {}
  32. "../Email/EmailHandler": @EmailHandler
  33. "../Security/OneTimeTokenHandler": @OneTimeTokenHandler
  34. "../Analytics/AnalyticsManager": @AnalyticsManager = { recordEvent: sinon.stub() }
  35. "settings-sharelatex": @settings = {siteUrl: "http://sl.example.com"}
  36. @passingRequest = {email:"something@email.com", password:"123"}
  37. describe 'validate Register Request', ->
  38. it 'allow working account through', ->
  39. result = @handler._registrationRequestIsValid @passingRequest
  40. result.should.equal true
  41. it 'not allow not valid email through ', ()->
  42. @passingRequest.email = "notemail"
  43. result = @handler._registrationRequestIsValid @passingRequest
  44. result.should.equal false
  45. it 'not allow no email through ', ->
  46. @passingRequest.email = ""
  47. result = @handler._registrationRequestIsValid @passingRequest
  48. result.should.equal false
  49. it 'not allow no password through ', ()->
  50. @passingRequest.password= ""
  51. result = @handler._registrationRequestIsValid @passingRequest
  52. result.should.equal false
  53. describe "registerNewUser", ->
  54. describe "holdingAccount", (done)->
  55. beforeEach ->
  56. @user.holdingAccount = true
  57. @handler._registrationRequestIsValid = sinon.stub().returns true
  58. @User.findOne.callsArgWith(1, null, @user)
  59. it "should not create a new user if there is a holding account there", (done)->
  60. @handler.registerNewUser @passingRequest, (err)=>
  61. @UserCreator.createNewUser.called.should.equal false
  62. done()
  63. it "should set holding account to false", (done)->
  64. @handler.registerNewUser @passingRequest, (err)=>
  65. update = @User.update.args[0]
  66. assert.deepEqual update[0], {_id:@user._id}
  67. assert.deepEqual update[1], {"$set":{holdingAccount:false}}
  68. done()
  69. describe "invalidRequest", ->
  70. it "should not create a new user if the the request is not valid", (done)->
  71. @handler._registrationRequestIsValid = sinon.stub().returns false
  72. @handler.registerNewUser @passingRequest, (err)=>
  73. expect(err).to.exist
  74. @UserCreator.createNewUser.called.should.equal false
  75. done()
  76. it "should return email registered in the error if there is a non holdingAccount there", (done)->
  77. @User.findOne.callsArgWith(1, null, @user = {holdingAccount:false})
  78. @handler.registerNewUser @passingRequest, (err, user)=>
  79. err.should.deep.equal new Error("EmailAlreadyRegistered")
  80. user.should.deep.equal @user
  81. done()
  82. describe "validRequest", ->
  83. beforeEach ->
  84. @handler._registrationRequestIsValid = sinon.stub().returns true
  85. @User.findOne.callsArgWith 1
  86. it "should create a new user", (done)->
  87. @handler.registerNewUser @passingRequest, (err)=>
  88. @UserCreator.createNewUser.calledWith({email:@passingRequest.email, holdingAccount:false, first_name:@passingRequest.first_name, last_name:@passingRequest.last_name}).should.equal true
  89. done()
  90. it 'lower case email', (done)->
  91. @passingRequest.email = "soMe@eMail.cOm"
  92. @handler.registerNewUser @passingRequest, (err)=>
  93. @UserCreator.createNewUser.args[0][0].email.should.equal "some@email.com"
  94. done()
  95. it 'trim white space from email', (done)->
  96. @passingRequest.email = " some@email.com "
  97. @handler.registerNewUser @passingRequest, (err)=>
  98. @UserCreator.createNewUser.args[0][0].email.should.equal "some@email.com"
  99. done()
  100. it "should set the password", (done)->
  101. @handler.registerNewUser @passingRequest, (err)=>
  102. @AuthenticationManager.setUserPassword.calledWith(@user._id, @passingRequest.password).should.equal true
  103. done()
  104. it "should add the user to the news letter manager", (done)->
  105. @handler.registerNewUser @passingRequest, (err)=>
  106. @NewsLetterManager.subscribe.calledWith(@user).should.equal true
  107. done()
  108. it "should track the registration event", (done)->
  109. @handler.registerNewUser @passingRequest, (err)=>
  110. @AnalyticsManager.recordEvent
  111. .calledWith(@user._id, "user-registered")
  112. .should.equal true
  113. done()
  114. it "should call the ReferalAllocator", (done)->
  115. done()
  116. describe "registerNewUserAndSendActivationEmail", ->
  117. beforeEach ->
  118. @email = "email@example.com"
  119. @crypto.randomBytes = sinon.stub().returns({toString: () => @password = "mock-password"})
  120. @OneTimeTokenHandler.getNewToken.callsArgWith(2, null, @token = "mock-token")
  121. @handler.registerNewUser = sinon.stub()
  122. @callback = sinon.stub()
  123. describe "with a new user", ->
  124. beforeEach ->
  125. @handler.registerNewUser.callsArgWith(1, null, @user)
  126. @handler.registerNewUserAndSendActivationEmail @email, @callback
  127. it "should ask the UserRegistrationHandler to register user", ->
  128. @handler.registerNewUser
  129. .calledWith({
  130. email: @email
  131. password: @password
  132. }).should.equal true
  133. it "should generate a new password reset token", ->
  134. @OneTimeTokenHandler.getNewToken
  135. .calledWith(@user_id, expiresIn: 7 * 24 * 60 * 60)
  136. .should.equal true
  137. it "should send a registered email", ->
  138. @EmailHandler.sendEmail
  139. .calledWith("registered", {
  140. to: @user.email
  141. setNewPasswordUrl: "#{@settings.siteUrl}/user/activate?token=#{@token}&user_id=#{@user_id}"
  142. })
  143. .should.equal true
  144. it "should return the user", ->
  145. @callback
  146. .calledWith(null, @user, "#{@settings.siteUrl}/user/activate?token=#{@token}&user_id=#{@user_id}")
  147. .should.equal true
  148. describe "with a user that already exists", ->
  149. beforeEach ->
  150. @handler.registerNewUser.callsArgWith(1, new Error("EmailAlreadyRegistered"), @user)
  151. @handler.registerNewUserAndSendActivationEmail @email, @callback
  152. it "should still generate a new password token and email", ->
  153. @OneTimeTokenHandler.getNewToken.called.should.equal true
  154. @EmailHandler.sendEmail.called.should.equal true