2014-05-16 12:45:48 -04:00
|
|
|
should = require('chai').should()
|
|
|
|
SandboxedModule = require('sandboxed-module')
|
|
|
|
assert = require('assert')
|
|
|
|
path = require('path')
|
|
|
|
sinon = require('sinon')
|
|
|
|
modulePath = path.join __dirname, "../../../../app/js/Features/User/UserUpdater"
|
|
|
|
expect = require("chai").expect
|
2018-06-19 08:55:34 -04:00
|
|
|
tk = require('timekeeper')
|
2014-05-16 12:45:48 -04:00
|
|
|
|
|
|
|
describe "UserUpdater", ->
|
|
|
|
|
|
|
|
beforeEach ->
|
2018-06-19 08:55:34 -04:00
|
|
|
tk.freeze(Date.now())
|
2014-05-16 12:45:48 -04:00
|
|
|
@mongojs =
|
|
|
|
db:{}
|
|
|
|
ObjectId:(id)-> return id
|
2018-05-23 10:12:23 -04:00
|
|
|
@UserGetter =
|
2018-05-25 07:04:09 -04:00
|
|
|
getUserEmail: sinon.stub()
|
|
|
|
getUserByAnyEmail: sinon.stub()
|
2018-05-28 10:08:37 -04:00
|
|
|
ensureUniqueEmailAddress: sinon.stub()
|
2018-05-25 07:04:09 -04:00
|
|
|
@logger = err: sinon.stub(), log: ->
|
2018-06-21 07:31:55 -04:00
|
|
|
settings = apis: { v1: { url: 'v1.url', user: '', pass: '' } }
|
2018-06-19 07:46:15 -04:00
|
|
|
@request = sinon.stub()
|
2014-05-16 12:45:48 -04:00
|
|
|
@UserUpdater = SandboxedModule.require modulePath, requires:
|
2018-05-25 07:04:09 -04:00
|
|
|
"logger-sharelatex": @logger
|
2018-05-23 10:12:23 -04:00
|
|
|
"./UserGetter": @UserGetter
|
2014-05-16 12:45:48 -04:00
|
|
|
"../../infrastructure/mongojs":@mongojs
|
2017-04-03 11:18:30 -04:00
|
|
|
"metrics-sharelatex": timeAsyncMethod: sinon.stub()
|
2018-06-19 07:46:15 -04:00
|
|
|
'settings-sharelatex': settings
|
|
|
|
'request': @request
|
2014-05-16 12:45:48 -04:00
|
|
|
|
|
|
|
@stubbedUser =
|
2018-05-25 07:04:09 -04:00
|
|
|
_id: "3131231"
|
2014-05-16 12:45:48 -04:00
|
|
|
name:"bob"
|
|
|
|
email:"hello@world.com"
|
|
|
|
@newEmail = "bob@bob.com"
|
|
|
|
|
2018-06-19 08:55:34 -04:00
|
|
|
afterEach ->
|
|
|
|
tk.reset()
|
|
|
|
|
2018-05-25 07:04:09 -04:00
|
|
|
describe 'changeEmailAddress', ->
|
2014-05-16 12:45:48 -04:00
|
|
|
beforeEach ->
|
2018-05-25 07:04:09 -04:00
|
|
|
@UserGetter.getUserEmail.callsArgWith(1, null, @stubbedUser.email)
|
|
|
|
@UserUpdater.addEmailAddress = sinon.stub().callsArgWith(2)
|
|
|
|
@UserUpdater.setDefaultEmailAddress = sinon.stub().callsArgWith(2)
|
|
|
|
@UserUpdater.removeEmailAddress = sinon.stub().callsArgWith(2)
|
2014-05-16 12:45:48 -04:00
|
|
|
|
2018-05-25 07:04:09 -04:00
|
|
|
it 'change email', (done)->
|
|
|
|
@UserUpdater.changeEmailAddress @stubbedUser._id, @newEmail, (err)=>
|
|
|
|
should.not.exist(err)
|
|
|
|
@UserUpdater.addEmailAddress.calledWith(
|
|
|
|
@stubbedUser._id, @newEmail
|
|
|
|
).should.equal true
|
|
|
|
@UserUpdater.setDefaultEmailAddress.calledWith(
|
|
|
|
@stubbedUser._id, @newEmail
|
|
|
|
).should.equal true
|
|
|
|
@UserUpdater.removeEmailAddress.calledWith(
|
|
|
|
@stubbedUser._id, @stubbedUser.email
|
|
|
|
).should.equal true
|
|
|
|
done()
|
|
|
|
|
|
|
|
it 'handle error', (done)->
|
|
|
|
@UserUpdater.removeEmailAddress.callsArgWith(2, new Error('nope'))
|
|
|
|
@UserUpdater.changeEmailAddress @stubbedUser._id, @newEmail, (err)=>
|
|
|
|
should.exist(err)
|
|
|
|
done()
|
|
|
|
|
|
|
|
describe 'addEmailAddress', ->
|
|
|
|
beforeEach ->
|
2018-05-28 10:08:37 -04:00
|
|
|
@UserGetter.ensureUniqueEmailAddress = sinon.stub().callsArgWith(1)
|
2018-05-25 07:04:09 -04:00
|
|
|
@UserUpdater.updateUser = sinon.stub().callsArgWith(2, null)
|
2018-06-19 07:46:15 -04:00
|
|
|
@request.callsArgWith(1, null, { statusCode: 201 })
|
2018-05-25 07:04:09 -04:00
|
|
|
|
2018-06-19 07:46:15 -04:00
|
|
|
it 'add email', (done)->
|
2018-05-25 07:04:09 -04:00
|
|
|
@UserUpdater.addEmailAddress @stubbedUser._id, @newEmail, (err)=>
|
2018-05-28 10:08:37 -04:00
|
|
|
@UserGetter.ensureUniqueEmailAddress.called.should.equal true
|
2018-05-25 07:04:09 -04:00
|
|
|
should.not.exist(err)
|
|
|
|
@UserUpdater.updateUser.calledWith(
|
|
|
|
@stubbedUser._id,
|
|
|
|
$push: { emails: { email: @newEmail, createdAt: sinon.match.date } }
|
|
|
|
).should.equal true
|
|
|
|
done()
|
|
|
|
|
2018-06-19 07:46:15 -04:00
|
|
|
it 'add affiliation', (done)->
|
|
|
|
affiliationOptions =
|
|
|
|
university: { id: 1 }
|
|
|
|
role: 'Prof'
|
|
|
|
department: 'Math'
|
|
|
|
@UserUpdater.addEmailAddress @stubbedUser._id, @newEmail, affiliationOptions, (err)=>
|
|
|
|
should.not.exist(err)
|
|
|
|
@request.calledOnce.should.equal true
|
|
|
|
requestOptions = @request.lastCall.args[0]
|
2018-06-21 07:31:55 -04:00
|
|
|
expectedUrl = "v1.url/api/v2/users/#{@stubbedUser._id}/affiliations"
|
2018-06-19 07:46:15 -04:00
|
|
|
requestOptions.url.should.equal expectedUrl
|
|
|
|
requestOptions.method.should.equal 'POST'
|
|
|
|
|
|
|
|
body = requestOptions.body
|
|
|
|
Object.keys(body).length.should.equal 4
|
|
|
|
body.email.should.equal @newEmail
|
|
|
|
body.university.should.equal affiliationOptions.university
|
|
|
|
body.department.should.equal affiliationOptions.department
|
|
|
|
body.role.should.equal affiliationOptions.role
|
|
|
|
done()
|
|
|
|
|
2018-05-25 07:04:09 -04:00
|
|
|
it 'handle error', (done)->
|
|
|
|
@UserUpdater.updateUser = sinon.stub().callsArgWith(2, new Error('nope'))
|
|
|
|
|
|
|
|
@UserUpdater.addEmailAddress @stubbedUser._id, @newEmail, (err)=>
|
|
|
|
@logger.err.called.should.equal true
|
2014-05-16 12:45:48 -04:00
|
|
|
should.exist(err)
|
|
|
|
done()
|
|
|
|
|
2018-06-19 07:46:15 -04:00
|
|
|
it 'handle affiliation error', (done)->
|
|
|
|
body = errors: 'affiliation error message'
|
|
|
|
@request.callsArgWith(1, null, { statusCode: 422 }, body)
|
|
|
|
@UserUpdater.addEmailAddress @stubbedUser._id, @newEmail, (err)=>
|
|
|
|
err.message.should.have.string 422
|
|
|
|
err.message.should.have.string body.errors
|
|
|
|
@UserUpdater.updateUser.called.should.equal false
|
|
|
|
done()
|
|
|
|
|
2018-05-25 07:04:09 -04:00
|
|
|
describe 'removeEmailAddress', ->
|
2018-06-19 07:46:15 -04:00
|
|
|
beforeEach ->
|
2018-05-25 07:04:09 -04:00
|
|
|
@UserUpdater.updateUser = sinon.stub().callsArgWith(2, null, nMatched: 1)
|
2018-06-19 07:46:15 -04:00
|
|
|
@request.callsArgWith(1, null, { statusCode: 404 })
|
2014-05-16 12:45:48 -04:00
|
|
|
|
2018-06-19 07:46:15 -04:00
|
|
|
it 'remove email', (done)->
|
2018-05-25 07:04:09 -04:00
|
|
|
@UserUpdater.removeEmailAddress @stubbedUser._id, @newEmail, (err)=>
|
|
|
|
should.not.exist(err)
|
|
|
|
@UserUpdater.updateUser.calledWith(
|
|
|
|
{ _id: @stubbedUser._id, email: { $ne: @newEmail } },
|
|
|
|
$pull: { emails: { email: @newEmail } }
|
|
|
|
).should.equal true
|
2014-05-16 12:45:48 -04:00
|
|
|
done()
|
2014-06-11 07:46:28 -04:00
|
|
|
|
2018-06-19 07:46:15 -04:00
|
|
|
it 'remove affiliation', (done)->
|
|
|
|
@UserUpdater.removeEmailAddress @stubbedUser._id, @newEmail, (err)=>
|
|
|
|
should.not.exist(err)
|
|
|
|
@request.calledOnce.should.equal true
|
|
|
|
requestOptions = @request.lastCall.args[0]
|
2018-06-21 07:31:55 -04:00
|
|
|
expectedUrl = "v1.url/api/v2/users/#{@stubbedUser._id}/affiliations/"
|
2018-06-19 07:46:15 -04:00
|
|
|
expectedUrl += encodeURIComponent(@newEmail)
|
|
|
|
requestOptions.url.should.equal expectedUrl
|
|
|
|
requestOptions.method.should.equal 'DELETE'
|
|
|
|
done()
|
|
|
|
|
2018-05-25 07:04:09 -04:00
|
|
|
it 'handle error', (done)->
|
|
|
|
@UserUpdater.updateUser = sinon.stub().callsArgWith(2, new Error('nope'))
|
|
|
|
|
|
|
|
@UserUpdater.removeEmailAddress @stubbedUser._id, @newEmail, (err)=>
|
|
|
|
should.exist(err)
|
|
|
|
done()
|
|
|
|
|
|
|
|
it 'handle missed update', (done)->
|
2018-06-19 08:55:34 -04:00
|
|
|
@UserUpdater.updateUser = sinon.stub().callsArgWith(2, null, n: 0)
|
2018-05-25 07:04:09 -04:00
|
|
|
|
|
|
|
@UserUpdater.removeEmailAddress @stubbedUser._id, @newEmail, (err)=>
|
|
|
|
should.exist(err)
|
|
|
|
done()
|
|
|
|
|
2018-06-19 07:46:15 -04:00
|
|
|
it 'handle affiliation error', (done)->
|
|
|
|
@request.callsArgWith(1, null, { statusCode: 500 })
|
|
|
|
@UserUpdater.removeEmailAddress @stubbedUser._id, @newEmail, (err)=>
|
|
|
|
err.message.should.exist
|
|
|
|
@UserUpdater.updateUser.called.should.equal false
|
|
|
|
done()
|
|
|
|
|
2018-05-25 07:04:09 -04:00
|
|
|
describe 'setDefaultEmailAddress', ->
|
|
|
|
it 'set default', (done)->
|
2018-06-19 08:55:34 -04:00
|
|
|
@UserUpdater.updateUser = sinon.stub().callsArgWith(2, null, n: 1)
|
2018-05-25 07:04:09 -04:00
|
|
|
|
|
|
|
@UserUpdater.setDefaultEmailAddress @stubbedUser._id, @newEmail, (err)=>
|
|
|
|
should.not.exist(err)
|
|
|
|
@UserUpdater.updateUser.calledWith(
|
|
|
|
{ _id: @stubbedUser._id, 'emails.email': @newEmail },
|
|
|
|
$set: { email: @newEmail }
|
|
|
|
).should.equal true
|
|
|
|
done()
|
|
|
|
|
|
|
|
it 'handle error', (done)->
|
|
|
|
@UserUpdater.updateUser = sinon.stub().callsArgWith(2, new Error('nope'))
|
|
|
|
|
|
|
|
@UserUpdater.setDefaultEmailAddress @stubbedUser._id, @newEmail, (err)=>
|
|
|
|
should.exist(err)
|
|
|
|
done()
|
|
|
|
|
|
|
|
it 'handle missed update', (done)->
|
2018-06-19 08:55:34 -04:00
|
|
|
@UserUpdater.updateUser = sinon.stub().callsArgWith(2, null, n: 0)
|
2018-05-25 07:04:09 -04:00
|
|
|
|
|
|
|
@UserUpdater.setDefaultEmailAddress @stubbedUser._id, @newEmail, (err)=>
|
|
|
|
should.exist(err)
|
|
|
|
done()
|
|
|
|
|
2018-06-19 08:55:34 -04:00
|
|
|
describe 'confirmEmail', ->
|
|
|
|
it 'should update the email record', (done)->
|
|
|
|
@UserUpdater.updateUser = sinon.stub().callsArgWith(2, null, n: 1)
|
|
|
|
|
|
|
|
@UserUpdater.confirmEmail @stubbedUser._id, @newEmail, (err)=>
|
|
|
|
should.not.exist(err)
|
|
|
|
@UserUpdater.updateUser.calledWith(
|
|
|
|
{ _id: @stubbedUser._id, 'emails.email': @newEmail },
|
|
|
|
$set: { 'emails.$.confirmedAt': new Date() }
|
|
|
|
).should.equal true
|
|
|
|
done()
|
|
|
|
|
|
|
|
it 'handle error', (done)->
|
|
|
|
@UserUpdater.updateUser = sinon.stub().callsArgWith(2, new Error('nope'))
|
|
|
|
|
|
|
|
@UserUpdater.confirmEmail @stubbedUser._id, @newEmail, (err)=>
|
|
|
|
should.exist(err)
|
|
|
|
done()
|
|
|
|
|
|
|
|
it 'handle missed update', (done)->
|
|
|
|
@UserUpdater.updateUser = sinon.stub().callsArgWith(2, null, n: 0)
|
|
|
|
|
|
|
|
@UserUpdater.confirmEmail @stubbedUser._id, @newEmail, (err)=>
|
|
|
|
should.exist(err)
|
|
|
|
done()
|
|
|
|
|
|
|
|
|
2018-05-25 07:04:09 -04:00
|
|
|
|