| 123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143144145146147148149150151152153154155156157158159160161162163164165166167168169170171172173174175176177178179180181182183184185186187188189190191192193194195196197198199200201202203204205206207208209210211212213214215216217218219220221222223224225226227228229230231232233234235236237238239240241242243244245246247248249250251252253254255256257258259260261262263264265266267268269270271272273274275276277278279280281282283284285286287288289290291292293294295296297298299300301302303304305306307308309310311312313314315316317318319320321322323324325326327328329330331332333334335336337338339340341342343344345346347348349350351352353354355356357358359360361362363364365366367368369370371372373374375376377378379380 |
- sinon = require('sinon')
- chai = require('chai')
- should = chai.should()
- expect = chai.expect
- modulePath = "../../../../app/js/Features/User/UserController.js"
- SandboxedModule = require('sandboxed-module')
- events = require "events"
- MockResponse = require "../helpers/MockResponse"
- MockRequest = require "../helpers/MockRequest"
- ObjectId = require("mongojs").ObjectId
- assert = require("assert")
- describe "UserController", ->
- beforeEach ->
- @user_id = "323123"
- @user =
- _id:@user_id
- save: sinon.stub().callsArgWith(0)
- ace:{}
- @req =
- user: {}
- session:
- destroy:->
- user :
- _id : @user_id
- email:"old@something.com"
- body:{}
- @UserDeleter =
- deleteUser: sinon.stub().callsArgWith(1)
- @UserLocator =
- findById: sinon.stub().callsArgWith(1, null, @user)
- @User =
- findById: sinon.stub().callsArgWith(1, null, @user)
- @NewsLetterManager =
- unsubscribe: sinon.stub().callsArgWith(1)
- @UserRegistrationHandler =
- registerNewUser: sinon.stub()
- @AuthenticationController =
- establishUserSession: sinon.stub().callsArg(2)
- getLoggedInUserId: sinon.stub().returns(@user._id)
- getSessionUser: sinon.stub().returns(@req.session.user)
- setInSessionUser: sinon.stub()
- @AuthenticationManager =
- authenticate: sinon.stub()
- setUserPassword: sinon.stub()
- @ReferalAllocator =
- allocate:sinon.stub()
- @SubscriptionDomainHandler =
- autoAllocate:sinon.stub()
- @UserUpdater =
- changeEmailAddress:sinon.stub()
- @settings =
- siteUrl: "sharelatex.example.com"
- @UserHandler =
- populateGroupLicenceInvite:sinon.stub().callsArgWith(1)
- @UserSessionsManager =
- trackSession: sinon.stub()
- untrackSession: sinon.stub()
- revokeAllUserSessions: sinon.stub().callsArgWith(2, null)
- @UserController = SandboxedModule.require modulePath, requires:
- "./UserLocator": @UserLocator
- "./UserDeleter": @UserDeleter
- "./UserUpdater":@UserUpdater
- "../../models/User": User:@User
- '../Newsletter/NewsletterManager':@NewsLetterManager
- "./UserRegistrationHandler":@UserRegistrationHandler
- "../Authentication/AuthenticationController": @AuthenticationController
- "../Authentication/AuthenticationManager": @AuthenticationManager
- "../Referal/ReferalAllocator":@ReferalAllocator
- "../Subscription/SubscriptionDomainHandler":@SubscriptionDomainHandler
- "./UserHandler":@UserHandler
- "./UserSessionsManager": @UserSessionsManager
- "settings-sharelatex": @settings
- "logger-sharelatex":
- log:->
- err:->
- "../../infrastructure/Metrics": inc:->
- @res =
- send: sinon.stub()
- sendStatus: sinon.stub()
- json: sinon.stub()
- @next = sinon.stub()
- describe 'tryDeleteUser', ->
- beforeEach ->
- @req.body.password = 'wat'
- @req.logout = sinon.stub()
- @req.session.destroy = sinon.stub().callsArgWith(0, null)
- @AuthenticationController.getLoggedInUserId = sinon.stub().returns(@user._id)
- @AuthenticationManager.authenticate = sinon.stub().callsArgWith(2, null, @user)
- @UserDeleter.deleteUser = sinon.stub().callsArgWith(1, null)
- it 'should send 200', (done) ->
- @res.sendStatus = (code) =>
- code.should.equal 200
- done()
- @UserController.tryDeleteUser @req, @res, @next
- it 'should try to authenticate user', (done) ->
- @res.sendStatus = (code) =>
- @AuthenticationManager.authenticate.callCount.should.equal 1
- @AuthenticationManager.authenticate.calledWith({_id: @user._id}, @req.body.password).should.equal true
- done()
- @UserController.tryDeleteUser @req, @res, @next
- it 'should delete the user', (done) ->
- @res.sendStatus = (code) =>
- @UserDeleter.deleteUser.callCount.should.equal 1
- @UserDeleter.deleteUser.calledWith(@user._id).should.equal true
- done()
- @UserController.tryDeleteUser @req, @res, @next
- describe 'when no password is supplied', ->
- beforeEach ->
- @req.body.password = ''
- it 'should return 403', (done) ->
- @res.sendStatus = (code) =>
- code.should.equal 403
- done()
- @UserController.tryDeleteUser @req, @res, @next
- describe 'when authenticate produces an error', ->
- beforeEach ->
- @AuthenticationManager.authenticate = sinon.stub().callsArgWith(2, new Error('woops'))
- it 'should call next with an error', (done) ->
- @next = (err) =>
- expect(err).to.not.equal null
- expect(err).to.be.instanceof Error
- done()
- @UserController.tryDeleteUser @req, @res, @next
- describe 'when authenticate does not produce a user', ->
- beforeEach ->
- @AuthenticationManager.authenticate = sinon.stub().callsArgWith(2, null, null)
- it 'should return 403', (done) ->
- @res.sendStatus = (code) =>
- code.should.equal 403
- done()
- @UserController.tryDeleteUser @req, @res, @next
- describe 'when deleteUser produces an error', ->
- beforeEach ->
- @UserDeleter.deleteUser = sinon.stub().callsArgWith(1, new Error('woops'))
- it 'should call next with an error', (done) ->
- @next = (err) =>
- expect(err).to.not.equal null
- expect(err).to.be.instanceof Error
- done()
- @UserController.tryDeleteUser @req, @res, @next
- describe 'when session.destroy produces an error', ->
- beforeEach ->
- @req.session.destroy = sinon.stub().callsArgWith(0, new Error('woops'))
- it 'should call next with an error', (done) ->
- @next = (err) =>
- expect(err).to.not.equal null
- expect(err).to.be.instanceof Error
- done()
- @UserController.tryDeleteUser @req, @res, @next
- describe "unsubscribe", ->
- it "should send the user to unsubscribe", (done)->
- @res.send = (code)=>
- @NewsLetterManager.unsubscribe.calledWith(@user).should.equal true
- done()
- @UserController.unsubscribe @req, @res
- describe "updateUserSettings", ->
- beforeEach ->
- @newEmail = "hello@world.com"
- it "should call save", (done)->
- @req.body = {}
- @res.sendStatus = (code)=>
- @user.save.called.should.equal true
- done()
- @UserController.updateUserSettings @req, @res
- it "should set the first name", (done)->
- @req.body =
- first_name: "bobby "
- @res.sendStatus = (code)=>
- @user.first_name.should.equal "bobby"
- done()
- @UserController.updateUserSettings @req, @res
- it "should set the role", (done)->
- @req.body =
- role: "student"
- @res.sendStatus = (code)=>
- @user.role.should.equal "student"
- done()
- @UserController.updateUserSettings @req, @res
- it "should set the institution", (done)->
- @req.body =
- institution: "MIT"
- @res.sendStatus = (code)=>
- @user.institution.should.equal "MIT"
- done()
- @UserController.updateUserSettings @req, @res
- it "should set some props on ace", (done)->
- @req.body =
- theme: "something"
- @res.sendStatus = (code)=>
- @user.ace.theme.should.equal "something"
- done()
- @UserController.updateUserSettings @req, @res
- it "should send an error if the email is 0 len", (done)->
- @req.body.email = ""
- @res.sendStatus = (code)->
- code.should.equal 400
- done()
- @UserController.updateUserSettings @req, @res
- it "should send an error if the email does not contain an @", (done)->
- @req.body.email = "bob at something dot com"
- @res.sendStatus = (code)->
- code.should.equal 400
- done()
- @UserController.updateUserSettings @req, @res
- it "should call the user updater with the new email and user _id", (done)->
- @req.body.email = @newEmail.toUpperCase()
- @UserUpdater.changeEmailAddress.callsArgWith(2)
- @res.sendStatus = (code)=>
- code.should.equal 200
- @UserUpdater.changeEmailAddress.calledWith(@user_id, @newEmail).should.equal true
- done()
- @UserController.updateUserSettings @req, @res
- it "should update the email on the session", (done)->
- @req.body.email = @newEmail.toUpperCase()
- @UserUpdater.changeEmailAddress.callsArgWith(2)
- callcount = 0
- @User.findById = (id, cb)=>
- if ++callcount == 2
- @user.email = @newEmail
- cb(null, @user)
- @res.sendStatus = (code)=>
- code.should.equal 200
- @AuthenticationController.setInSessionUser.calledWith(
- @req, {email: @newEmail, first_name: undefined, last_name: undefined}
- ).should.equal true
- done()
- @UserController.updateUserSettings @req, @res
- it "should call populateGroupLicenceInvite", (done)->
- @req.body.email = @newEmail.toUpperCase()
- @UserUpdater.changeEmailAddress.callsArgWith(2)
- @res.sendStatus = (code)=>
- code.should.equal 200
- @UserHandler.populateGroupLicenceInvite.calledWith(@user).should.equal true
- done()
- @UserController.updateUserSettings @req, @res
- describe 'when using an external auth source', ->
- beforeEach ->
- @UserUpdater.changeEmailAddress.callsArgWith(2)
- @newEmail = 'someone23@example.com'
- @settings.ldap = {active: true}
- afterEach ->
- delete @settings.ldap
- it 'should not set a new email', (done) ->
- @req.body.email = @newEmail
- @res.sendStatus = (code)=>
- code.should.equal 200
- @UserUpdater.changeEmailAddress.calledWith(@user_id, @newEmail).should.equal false
- done()
- @UserController.updateUserSettings @req, @res
- describe "logout", ->
- it "should destroy the session", (done)->
- @req.session.destroy = sinon.stub().callsArgWith(0)
- @res.redirect = (url)=>
- url.should.equal "/login"
- @req.session.destroy.called.should.equal true
- done()
- @UserController.logout @req, @res
- describe "register", ->
- beforeEach ->
- @UserRegistrationHandler.registerNewUserAndSendActivationEmail = sinon.stub().callsArgWith(1, null, @user, @url = "mock/url")
- @req.body.email = @user.email = @email = "email@example.com"
- @UserController.register @req, @res
- it "should register the user and send them an email", ->
- @UserRegistrationHandler.registerNewUserAndSendActivationEmail
- .calledWith(@email)
- .should.equal true
- it "should return the user and activation url", ->
- @res.json
- .calledWith({
- email: @email,
- setNewPasswordUrl: @url
- })
- .should.equal true
- describe 'clearSessions', ->
- it 'should call revokeAllUserSessions', (done) ->
- @UserController.clearSessions @req, @res
- @UserSessionsManager.revokeAllUserSessions.callCount.should.equal 1
- done()
- it 'send a 201 response', (done) ->
- @res.sendStatus = (status) =>
- status.should.equal 201
- done()
- @UserController.clearSessions @req, @res
- describe 'when revokeAllUserSessions produces an error', ->
- it 'should call next with an error', (done) ->
- @UserSessionsManager.revokeAllUserSessions.callsArgWith(2, new Error('woops'))
- next = (err) =>
- expect(err).to.not.equal null
- expect(err).to.be.instanceof Error
- done()
- @UserController.clearSessions @req, @res, next
- describe "changePassword", ->
- it "should check the old password is the current one at the moment", (done)->
- @AuthenticationManager.authenticate.callsArgWith(2)
- @req.body =
- currentPassword: "oldpasshere"
- @res.send = =>
- @AuthenticationManager.authenticate.calledWith(_id:@user._id, "oldpasshere").should.equal true
- @AuthenticationManager.setUserPassword.called.should.equal false
- done()
- @UserController.changePassword @req, @res
- it "it should not set the new password if they do not match", (done)->
- @AuthenticationManager.authenticate.callsArgWith(2, null, {})
- @req.body =
- newPassword1: "1"
- newPassword2: "2"
- @res.send = =>
- @AuthenticationManager.setUserPassword.called.should.equal false
- done()
- @UserController.changePassword @req, @res
- it "should set the new password if they do match", (done)->
- @AuthenticationManager.authenticate.callsArgWith(2, null, @user)
- @AuthenticationManager.setUserPassword.callsArgWith(2)
- @req.body =
- newPassword1: "newpass"
- newPassword2: "newpass"
- @res.send = =>
- @AuthenticationManager.setUserPassword.calledWith(@user._id, "newpass").should.equal true
- done()
- @UserController.changePassword @req, @res
|