agw/overleaf-cep
1
0
Fork 0
mirror of https://github.com/yu-i-i/overleaf-cep.git synced 2026-05-23 17:19:37 +02:00
Code Issues Packages Projects Releases Wiki Activity

Don't choke on null projects

Browse Source
This commit is contained in:
James Allen
2016-06-30 14:30:04 +01:00
parent b76cc7e314
commit 882e28a811
2 changed files with 4 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

1
services/web/app/coffee/Features/Collaborators/CollaboratorsHandler.coffee
View File

@@ -95,6 +95,7 @@ module.exports = CollaboratorsHandler =
for project in allProjects
do (project) ->
jobs.push (cb) ->
return cb() if !project?
CollaboratorsHandler.removeUserFromProject project._id, user_id, cb
async.series jobs, callback

6
services/web/test/UnitTests/coffee/Collaborators/CollaboratorsHandlerTests.coffee
View File

@@ -276,13 +276,13 @@ describe "CollaboratorsHandler", ->
it "should not add any users to the proejct", ->
@CollaboratorHandler.addUserIdToProject.called.should.equal false
describe "removeUserFromAllProjets", ->
describe "removeUserFromAllProjects", ->
beforeEach (done) ->
@CollaboratorHandler.getProjectsUserIsCollaboratorOf = sinon.stub()
@CollaboratorHandler.getProjectsUserIsCollaboratorOf.withArgs(@user_id, { _id: 1 }).yields(
null,
[ { _id: "read-and-write-0" }, { _id: "read-and-write-1" } ],
[ { _id: "read-only-0" }, { _id: "read-only-1" } ]
[ { _id: "read-and-write-0" }, { _id: "read-and-write-1" }, null ],
[ { _id: "read-only-0" }, { _id: "read-only-1" }, null ]
)
@CollaboratorHandler.removeUserFromProject = sinon.stub().yields()
@CollaboratorHandler.removeUserFromAllProjets @user_id, done