Jelajahi Sumber

Try to fix issue with acceptance tests timing out

James Allen 9 tahun lalu
induk
melakukan
5d377713d6

+ 8 - 2
services/real-time/test/acceptance/coffee/ApplyUpdateTests.coffee

@@ -27,9 +27,12 @@ describe "applyOtUpdate", ->
 				(cb) =>
 					FixturesManager.setUpDoc @project_id, {@lines, @version, @ops}, (e, {@doc_id}) =>
 						cb(e)
-						
+
 				(cb) =>
 					@client = RealTimeClient.connect()
+					@client.on "connectionAccepted", cb
+						
+				(cb) =>
 					@client.emit "joinProject", project_id: @project_id, cb
 				
 				(cb) =>
@@ -78,9 +81,12 @@ describe "applyOtUpdate", ->
 				(cb) =>
 					FixturesManager.setUpDoc @project_id, {@lines, @version, @ops}, (e, {@doc_id}) =>
 						cb(e)
-						
+
 				(cb) =>
 					@client = RealTimeClient.connect()
+					@client.on "connectionAccepted", cb
+						
+				(cb) =>
 					@client.emit "joinProject", project_id: @project_id, cb
 				
 				(cb) =>

+ 8 - 8
services/real-time/test/acceptance/coffee/JoinDocTests.coffee

@@ -26,10 +26,10 @@ describe "joinDoc", ->
 				(cb) =>
 					FixturesManager.setUpDoc @project_id, {@lines, @version, @ops}, (e, {@doc_id}) =>
 						cb(e)
-						
+
 				(cb) =>
 					@client = RealTimeClient.connect()
-					@client.on "connect", cb
+					@client.on "connectionAccepted", cb
 						
 				(cb) =>
 					@client.emit "joinProject", project_id: @project_id, cb
@@ -63,10 +63,10 @@ describe "joinDoc", ->
 				(cb) =>
 					FixturesManager.setUpDoc @project_id, {@lines, @version, @ops}, (e, {@doc_id}) =>
 						cb(e)
-						
+
 				(cb) =>
 					@client = RealTimeClient.connect()
-					@client.on "connect", cb
+					@client.on "connectionAccepted", cb
 						
 				(cb) =>
 					@client.emit "joinProject", project_id: @project_id, cb
@@ -100,10 +100,10 @@ describe "joinDoc", ->
 				(cb) =>
 					FixturesManager.setUpDoc @project_id, {@lines, @version, @ops}, (e, {@doc_id}) =>
 						cb(e)
-						
+
 				(cb) =>
 					@client = RealTimeClient.connect()
-					@client.on "connect", cb
+					@client.on "connectionAccepted", cb
 						
 				(cb) =>
 					@client.emit "joinProject", project_id: @project_id, cb
@@ -142,10 +142,10 @@ describe "joinDoc", ->
 				(cb) =>
 					FixturesManager.setUpDoc @project_id, {@lines, @version, @ops}, (e, {@doc_id}) =>
 						cb(e)
-						
+
 				(cb) =>
 					@client = RealTimeClient.connect()
-					@client.on "connect", cb
+					@client.on "connectionAccepted", cb
 						
 				(cb) =>
 					@client.emit "joinProject", project_id: @project_id, cb

+ 4 - 4
services/real-time/test/acceptance/coffee/JoinProjectTests.coffee

@@ -20,10 +20,10 @@ describe "joinProject", ->
 						}
 					}, (e, {@project_id, @user_id}) =>
 						cb(e)
-						
+
 				(cb) =>
 					@client = RealTimeClient.connect()
-					@client.on "connect", cb
+					@client.on "connectionAccepted", cb
 						
 				(cb) =>
 					@client.emit "joinProject", project_id: @project_id, (error, @project, @privilegeLevel, @protocolVersion) =>
@@ -72,10 +72,10 @@ describe "joinProject", ->
 						}
 					}, (e, {@project_id, @user_id}) =>
 						cb(e)
-						
+
 				(cb) =>
 					@client = RealTimeClient.connect()
-					@client.on "connect", cb
+					@client.on "connectionAccepted", cb
 						
 				(cb) =>
 					@client.emit "joinProject", project_id: @project_id, (@error, @project, @privilegeLevel, @protocolVersion) =>

+ 1 - 1
services/real-time/test/acceptance/coffee/LeaveDocTests.coffee

@@ -29,7 +29,7 @@ describe "leaveDoc", ->
 						
 				(cb) =>
 					@client = RealTimeClient.connect()
-					@client.on "connect", cb
+					@client.on "connectionAccepted", cb
 						
 				(cb) =>
 					@client.emit "joinProject", project_id: @project_id, cb

+ 4 - 4
services/real-time/test/acceptance/coffee/LeaveProjectTests.coffee

@@ -21,11 +21,11 @@ describe "leaveProject", ->
 					
 				(cb) =>
 					@clientA = RealTimeClient.connect()
-					@clientA.on "connect", cb
+					@clientA.on "connectionAccepted", cb
 					
 				(cb) =>
 					@clientB = RealTimeClient.connect()
-					@clientB.on "connect", cb
+					@clientB.on "connectionAccepted", cb
 					
 					@clientBDisconnectMessages = []
 					@clientB.on "clientTracking.clientDisconnected", (data) =>
@@ -46,7 +46,7 @@ describe "leaveProject", ->
 					
 				(cb) =>
 					# The API waits a little while before flushing changes
-					setTimeout done, require("../../../app/js/WebsocketController").FLUSH_IF_EMPTY_DELAY * 2
+					setTimeout done, 1000
 					
 			], done
 
@@ -91,7 +91,7 @@ describe "leaveProject", ->
 					
 				(cb) =>
 					# The API waits a little while before flushing changes
-					setTimeout done, require("../../../app/js/WebsocketController").FLUSH_IF_EMPTY_DELAY * 2
+					setTimeout done, 1000
 			], done
 
 		it "should flush the project to the document updater", ->

+ 2 - 2
services/real-time/test/acceptance/coffee/ReceiveUpdateTests.coffee

@@ -31,11 +31,11 @@ describe "receiveUpdate", ->
 			
 			(cb) =>
 				@clientA = RealTimeClient.connect()
-				@clientA.on "connect", cb
+				@clientA.on "connectionAccepted", cb
 				
 			(cb) =>
 				@clientB = RealTimeClient.connect()
-				@clientB.on "connect", cb
+				@clientB.on "connectionAccepted", cb
 				
 			(cb) =>
 				@clientA.emit "joinProject", {