Просмотр исходного кода

set pathname in acceptance tests

Hayden Faulds 9 лет назад
Родитель
Сommit
04ecd1e7ee

+ 8 - 8
services/document-updater/test/acceptance/coffee/ApplyingUpdatesToADocTests.coffee

@@ -136,7 +136,7 @@ describe "Applying updates to a doc", ->
 					rclient_history.sismember HistoryKeys.docsWithHistoryOps({@project_id}), @doc_id, (error, result) =>
 						result.should.equal 1
 						done()
-			
+
 			it "should store the doc ops in the correct order", (done) ->
 				rclient_du.lrange Keys.docOps({doc_id: @doc_id}), 0, -1, (error, updates) =>
 					updates = (JSON.parse(u) for u in updates)
@@ -181,7 +181,7 @@ describe "Applying updates to a doc", ->
 			[@project_id, @doc_id] = [DocUpdaterClient.randomId(), DocUpdaterClient.randomId()]
 			@broken_update = { doc_id: @doc_id, v: @version, op: [d: "not the correct content", p: 0 ] }
 			MockWebApi.insertDoc @project_id, @doc_id, {lines: @lines, version: @version}
-	
+
 			DocUpdaterClient.subscribeToAppliedOps @messageCallback = sinon.stub()
 
 			DocUpdaterClient.sendUpdate @project_id, @doc_id, @broken_update, (error) ->
@@ -192,14 +192,14 @@ describe "Applying updates to a doc", ->
 			DocUpdaterClient.getDoc @project_id, @doc_id, (error, res, doc) =>
 				doc.lines.should.deep.equal @lines
 				done()
-		
+
 		it "should send a message with an error", ->
 			@messageCallback.called.should.equal true
 			[channel, message] = @messageCallback.args[0]
 			channel.should.equal "applied-ops"
 			JSON.parse(message).should.deep.equal {
 				project_id: @project_id,
-				doc_id: @doc_id, 
+				doc_id: @doc_id,
 				error:'Delete component \'not the correct content\' does not match deleted text \'one\ntwo\nthree\''
 			}
 
@@ -240,7 +240,7 @@ describe "Applying updates to a doc", ->
 				lines: @lines
 			}
 
-			update = 
+			update =
 				doc: @doc_id
 				op: @update.op
 				v: 0
@@ -252,12 +252,12 @@ describe "Applying updates to a doc", ->
 			DocUpdaterClient.getDoc @project_id, @doc_id, (error, res, doc) =>
 				doc.lines.should.deep.equal @result
 				done()
-	
+
 	describe "when the sending duplicate ops", ->
 		before (done) ->
 			[@project_id, @doc_id] = [DocUpdaterClient.randomId(), DocUpdaterClient.randomId()]
 			MockWebApi.insertDoc @project_id, @doc_id, {lines: @lines, version: @version}
-	
+
 			DocUpdaterClient.subscribeToAppliedOps @messageCallback = sinon.stub()
 
 			# One user delete 'one', the next turns it into 'once'. The second becomes a NOP.
@@ -292,7 +292,7 @@ describe "Applying updates to a doc", ->
 			DocUpdaterClient.getDoc @project_id, @doc_id, (error, res, doc) =>
 				doc.lines.should.deep.equal @result
 				done()
-		
+
 		it "should return a message about duplicate ops", ->
 			@messageCallback.calledTwice.should.equal true
 			@messageCallback.args[0][0].should.equal "applied-ops"

+ 2 - 0
services/document-updater/test/acceptance/coffee/helpers/MockWebApi.coffee

@@ -9,6 +9,7 @@ module.exports = MockWebApi =
 	insertDoc: (project_id, doc_id, doc) ->
 		doc.version ?= 0
 		doc.lines ?= []
+		doc.pathname = '/a/b/c.tex'
 		@docs["#{project_id}:#{doc_id}"] = doc
 
 	setDocument: (project_id, doc_id, lines, version, ranges, callback = (error) ->) ->
@@ -16,6 +17,7 @@ module.exports = MockWebApi =
 		doc.lines = lines
 		doc.version = version
 		doc.ranges = ranges
+		doc.pathname = '/a/b/c.tex'
 		callback null
 
 	getDocument: (project_id, doc_id, callback = (error, doc) ->) ->