Przeglądaj źródła

Add more assertions about project access

Shane Kilkelly 10 lat temu
rodzic
commit
9787edd716

+ 8 - 2
services/web/test/acceptance/coffee/ProjectInviteTests.coffee

@@ -222,6 +222,7 @@ describe "ProjectInviteTests", ->
 					Async.series [
 					Async.series [
 						(cb) => expectInvitePage @user, @link, cb
 						(cb) => expectInvitePage @user, @link, cb
 						(cb) => expectAcceptInviteAndRedirect @user, @invite, cb
 						(cb) => expectAcceptInviteAndRedirect @user, @invite, cb
+						(cb) => expectProjectAccess @user, @invite.projectId, cb
 					], done
 					], done
 
 
 				describe 'when user clicks on the invite a second time', ->
 				describe 'when user clicks on the invite a second time', ->
@@ -230,6 +231,7 @@ describe "ProjectInviteTests", ->
 						Async.series [
 						Async.series [
 							(cb) => expectProjectAccess @user, @invite.projectId, cb
 							(cb) => expectProjectAccess @user, @invite.projectId, cb
 							(cb) => expectInviteRedirectToProject @user, @link, @invite, cb
 							(cb) => expectInviteRedirectToProject @user, @link, @invite, cb
+							(cb) => expectProjectAccess @user, @invite.projectId, cb
 						], done
 						], done
 
 
 					describe 'when the user recieves another invite to the same project', ->
 					describe 'when the user recieves another invite to the same project', ->
@@ -244,6 +246,7 @@ describe "ProjectInviteTests", ->
 										@secondLink = CollaboratorsEmailHandler._buildInviteUrl(@fakeProject, invite)
 										@secondLink = CollaboratorsEmailHandler._buildInviteUrl(@fakeProject, invite)
 										cb()
 										cb()
 								(cb) => expectInviteRedirectToProject @user, @secondLink, @secondInvite, cb
 								(cb) => expectInviteRedirectToProject @user, @secondLink, @secondInvite, cb
+								(cb) => expectProjectAccess @user, @invite.projectId, cb
 								(cb) => revokeInvite @sendingUser, @projectId, @secondInvite._id, cb
 								(cb) => revokeInvite @sendingUser, @projectId, @secondInvite._id, cb
 							], done
 							], done
 
 
@@ -264,8 +267,8 @@ describe "ProjectInviteTests", ->
 							(cb) =>
 							(cb) =>
 								link = @link.replace(@invite.token, 'not_a_real_token')
 								link = @link.replace(@invite.token, 'not_a_real_token')
 								expectInvalidInvitePage @user, link, cb
 								expectInvalidInvitePage @user, link, cb
-							(cb) =>
-								expectNoProjectAccess @user, @invite.projectId, cb
+							(cb) => expectNoProjectAccess @user, @invite.projectId, cb
+							(cb) => expectNoProjectAccess @user, @invite.projectId, cb
 						], done
 						], done
 					)
 					)
 
 
@@ -328,6 +331,7 @@ describe "ProjectInviteTests", ->
 				it 'should redirect to the register page', (done) ->
 				it 'should redirect to the register page', (done) ->
 					Async.series [
 					Async.series [
 						(cb) => expectInviteRedirectToRegister(@user, @link, cb)
 						(cb) => expectInviteRedirectToRegister(@user, @link, cb)
+						(cb) => expectNoProjectAccess @user, @invite.projectId, cb
 					], done
 					], done
 
 
 				it 'should display invalid-invite if the user registers a new account', (done) ->
 				it 'should display invalid-invite if the user registers a new account', (done) ->
@@ -350,6 +354,7 @@ describe "ProjectInviteTests", ->
 				it 'should redirect to the register page', (done) ->
 				it 'should redirect to the register page', (done) ->
 					Async.series [
 					Async.series [
 						(cb) => expectInviteRedirectToRegister(@user, @link, cb)
 						(cb) => expectInviteRedirectToRegister(@user, @link, cb)
+						(cb) => expectNoProjectAccess @user, @invite.projectId, cb
 					], done
 					], done
 
 
 				it 'should allow the user to login to view the invite', (done) ->
 				it 'should allow the user to login to view the invite', (done) ->
@@ -383,6 +388,7 @@ describe "ProjectInviteTests", ->
 				it 'should redirect to the register page', (done) ->
 				it 'should redirect to the register page', (done) ->
 					Async.series [
 					Async.series [
 						(cb) => expectInviteRedirectToRegister(@user, @link, cb)
 						(cb) => expectInviteRedirectToRegister(@user, @link, cb)
+						(cb) => expectNoProjectAccess @user, @invite.projectId, cb
 					], done
 					], done
 
 
 				it 'should show the invalid-invite page once the user has logged in', (done) ->
 				it 'should show the invalid-invite page once the user has logged in', (done) ->