update MongoManager tests

services/track-changes/test/unit/coffee/MongoManager/MongoManagerTests.coffee

@@ -97,184 +97,6 @@ describe "MongoManager", ->
 				@callback.calledWith(null, null, @update.v_end).should.equal true
 
 
-	# describe "getDocUpdates", ->
-	# 	beforeEach ->
-	# 		@results = [
-	# 			{foo: "mock-update", v: 56, doc_id: 100, project_id: 1},
-	# 			{foo: "mock-update", v: 55, doc_id: 100, project_id: 1},
-	# 			{pack: [ {foo: "mock-update", v: 54, doc_id: 100, project_id: 1},
-	# 				{foo: "mock-update", v: 53, doc_id: 100, project_id: 1},
-	# 				{foo: "mock-update", v: 52, doc_id: 100, project_id: 1} ]
-	# 				, v: 52, doc_id: 100, project_id: 1},
-	# 			{foo: "mock-update", v: 42, doc_id: 100, project_id: 1},
-	# 			{foo: "mock-update", v: 41, doc_id: 100, project_id: 1}
-	# 		]
-	# 		@updates_between = [
-	# 			{foo: "mock-update", v: 55, doc_id: 100, project_id: 1},
-	# 			{foo: "mock-update", v: 54, doc_id: 100, project_id: 1},
-	# 			{foo: "mock-update", v: 53, doc_id: 100, project_id: 1},
-	# 			{foo: "mock-update", v: 52, doc_id: 100, project_id: 1},
-	# 			{foo: "mock-update", v: 42, doc_id: 100, project_id: 1}
-	# 		]
-	# 		@updates_after = [
-	# 			{foo: "mock-update", v: 56, doc_id: 100, project_id: 1},
-	# 			{foo: "mock-update", v: 55, doc_id: 100, project_id: 1},
-	# 			{foo: "mock-update", v: 54, doc_id: 100, project_id: 1},
-	# 			{foo: "mock-update", v: 53, doc_id: 100, project_id: 1},
-	# 			{foo: "mock-update", v: 52, doc_id: 100, project_id: 1},
-	# 			{foo: "mock-update", v: 42, doc_id: 100, project_id: 1}
-	# 		]
-	# 		@db.docHistory = {}
-	# 		@db.docHistory.find = sinon.stub().returns @db.docHistory
-	# 		@db.docHistory.sort = sinon.stub().returns @db.docHistory
-	# 		@db.docHistory.limit = sinon.stub().returns @db.docHistory
-	# 		@db.docHistory.toArray = sinon.stub().callsArgWith(0, null, @results)
-
-	# 		@from = 42
-	# 		@to   = 55
-
-	# 	describe "with a to version", ->
-	# 		beforeEach ->
-	# 			@MongoManager.getDocUpdates @doc_id, from: @from, to: @to, @callback
-
-	# 		it "should find the all updates between the to and from versions", ->
-	# 			@db.docHistory.find
-	# 				.calledWith({
-	# 					doc_id: ObjectId(@doc_id)
-	# 					v: { $gte: @from, $lte: @to }
-	# 				})
-	# 				.should.equal true
-
-	# 		it "should sort in descending version order", ->
-	# 			@db.docHistory.sort
-	# 				.calledWith("v": -1)
-	# 				.should.equal true
-
-	# 		#it "should not limit the results", ->
-	# 		#	@db.docHistory.limit
-	# 		#		.called.should.equal false
-
-	# 		it "should call the call back with the results", ->
-	# 			@callback.calledWith(null, @updates_between).should.equal true
-
-	# 	describe "without a to version", ->
-	# 		beforeEach ->
-	# 			@MongoManager.getDocUpdates @doc_id, from: @from, @callback
-
-	# 		it "should find the all updates after the from version", ->
-	# 			@db.docHistory.find
-	# 				.calledWith({
-	# 					doc_id: ObjectId(@doc_id)
-	# 					v: { $gte: @from }
-	# 				})
-	# 				.should.equal true
-
-	# 		it "should call the call back with the updates", ->
-	# 			@callback.calledWith(null, @updates_after).should.equal true
-
-	# 	describe "with a limit", ->
-	# 		beforeEach ->
-	# 			@MongoManager.getDocUpdates @doc_id, from: @from, limit: @limit = 10, @callback
-
-	# 		it "should limit the results", ->
-	# 			@db.docHistory.limit
-	# 				.calledWith(@limit)
-	# 				.should.equal true
-
-
-	# describe "getDocUpdates", ->
-	# 	beforeEach ->
-	# 		@results = [
-	# 			{foo: "mock-update", v: 56, meta: {end_ts: 110}, doc_id: 100, project_id: 1},
-	# 			{foo: "mock-update", v: 55, meta: {end_ts: 100}, doc_id: 100, project_id: 1},
-	# 			{pack: [
-	# 				{foo: "mock-update", v: 54, meta: {end_ts: 99}, doc_id: 300, project_id: 1},
-	# 				{foo: "mock-update", v: 53, meta: {end_ts: 98}, doc_id: 300, project_id: 1},
-	# 				{foo: "mock-update", v: 52, meta: {end_ts: 97}, doc_id: 300, project_id: 1}	]
-	# 				, v: 52, meta: {end_ts: 100}, doc_id: 300, project_id: 1},
-	# 			{pack: [
-	# 				{foo: "mock-update", v: 54, meta: {end_ts: 103}, doc_id: 200, project_id: 1},
-	# 				{foo: "mock-update", v: 53, meta: {end_ts: 101}, doc_id: 200, project_id: 1},
-	# 				{foo: "mock-update", v: 52, meta: {end_ts: 99}, doc_id: 200, project_id: 1}	]
-	# 				, v: 52, meta: {end_ts: 103}, doc_id: 200, project_id: 1},
-	# 			{foo: "mock-update", v: 42, meta:{end_ts: 90}, doc_id: 100, project_id: 1}
-	# 		]
-	# 		@updates_before = [
-	# 			{foo: "mock-update", v: 55, meta: {end_ts: 100}, doc_id: 100, project_id: 1},
-	# 			{foo: "mock-update", v: 52, meta: {end_ts: 99}, doc_id: 200, project_id: 1},
-	# 			{foo: "mock-update", v: 54, meta: {end_ts: 99}, doc_id: 300, project_id: 1},
-	# 			{foo: "mock-update", v: 53, meta: {end_ts: 98}, doc_id: 300, project_id: 1},
-	# 			{foo: "mock-update", v: 52, meta: {end_ts: 97}, doc_id: 300, project_id: 1},
-	# 			{foo: "mock-update", v: 42, meta: {end_ts: 90}, doc_id: 100, project_id: 1},
-	# 		]
-	# 		@updates_all = [
-	# 			{foo: "mock-update", v: 56, meta: {end_ts: 110}, doc_id: 100, project_id: 1},
-	# 			{foo: "mock-update", v: 54, meta: {end_ts: 103}, doc_id: 200, project_id: 1},
-	# 			{foo: "mock-update", v: 53, meta: {end_ts: 101}, doc_id: 200, project_id: 1},
-	# 			{foo: "mock-update", v: 55, meta: {end_ts: 100}, doc_id: 100, project_id: 1},
-	# 			{foo: "mock-update", v: 52, meta: {end_ts: 99}, doc_id: 200, project_id: 1},
-	# 			{foo: "mock-update", v: 54, meta: {end_ts: 99}, doc_id: 300, project_id: 1},
-	# 			{foo: "mock-update", v: 53, meta: {end_ts: 98}, doc_id: 300, project_id: 1},
-	# 			{foo: "mock-update", v: 52, meta: {end_ts: 97}, doc_id: 300, project_id: 1},
-	# 			{foo: "mock-update", v: 42, meta: {end_ts: 90}, doc_id: 100, project_id: 1}
-	# 		]
-
-
-	# 		@db.docHistory = {}
-	# 		@db.docHistory.find = sinon.stub().returns @db.docHistory
-	# 		@db.docHistory.sort = sinon.stub().returns @db.docHistory
-	# 		@db.docHistory.limit = sinon.stub().returns @db.docHistory
-	# 		@db.docHistory.toArray = sinon.stub().callsArgWith(0, null, @results)
-
-	# 		@before = 101
-
-	# 	describe "with a before timestamp", ->
-	# 		beforeEach ->
-	# 			@MongoManager.getProjectUpdates @project_id, before: @before, @callback
-
-	# 		it "should find the all updates before the timestamp", ->
-	# 			@db.docHistory.find
-	# 				.calledWith({
-	# 					project_id: ObjectId(@project_id)
-	# 					"meta.end_ts": { $lt: @before }
-	# 				})
-	# 				.should.equal true
-
-	# 		it "should sort in descending version order", ->
-	# 			@db.docHistory.sort
-	# 				.calledWith("meta.end_ts": -1)
-	# 				.should.equal true
-
-	# 		it "should not limit the results", ->
-	# 			@db.docHistory.limit
-	# 				.called.should.equal false
-
-	# 		it "should call the call back with the updates", ->
-	# 			@callback.calledWith(null, @updates_before).should.equal true
-
-	# 	describe "without a before timestamp", ->
-	# 		beforeEach ->
-	# 			@MongoManager.getProjectUpdates @project_id, {}, @callback
-
-	# 		it "should find the all updates", ->
-	# 			@db.docHistory.find
-	# 				.calledWith({
-	# 					project_id: ObjectId(@project_id)
-	# 				})
-	# 				.should.equal true
-
-	# 		it "should call the call back with the updates", ->
-	# 			@callback.calledWith(null, @updates_all).should.equal true
-
-	# 	describe "with a limit", ->
-	# 		beforeEach ->
-	# 			@MongoManager.getProjectUpdates @project_id, before: @before, limit: @limit = 10, @callback
-
-	# 		it "should limit the results", ->
-	# 			@db.docHistory.limit
-	# 				.calledWith(@limit)
-	# 				.should.equal true
-
 	describe "backportProjectId", ->
 		beforeEach ->
 			@db.docHistory =
@@ -332,83 +154,3 @@ describe "MongoManager", ->
 		it "should call the callback", ->
 			@callback.called.should.equal true
 
-	# describe "getDocChangesCount", ->
-	# 	beforeEach ->
-	# 		@db.docHistory =
-	# 			count: sinon.stub().callsArg(1)
-	# 		@MongoManager.getDocChangesCount @doc_id, @callback
-
-	# 	it "should return if there is any doc changes", ->
-	# 		@db.docHistory.count
-	# 			.calledWith({
-	# 				doc_id: ObjectId(@doc_id)
-	# 			})
-	# 			.should.equal true
-
-	# 	it "should call the callback", ->
-	# 		@callback.called.should.equal true
-
-	# describe "getArchivedDocStatus", ->
-	# 	beforeEach ->
-	# 		@db.docHistoryStats =
-	# 			findOne: sinon.stub().callsArg(2)
-	# 		@MongoManager.getArchivedDocStatus @doc_id, @callback
-
-	# 	it "should return if there is any archived doc changes", ->
-	# 		@db.docHistoryStats.findOne
-	# 			.calledWith({
-	# 				doc_id: ObjectId(@doc_id)
-	# 				inS3: {$exists: true}
-	# 			})
-	# 			.should.equal true
-
-	# 	it "should call the callback", ->
-	# 		@callback.called.should.equal true
-
-	# describe "markDocHistoryAsArchived", ->
-	# 	beforeEach ->
-	# 		@update = { _id: ObjectId(), op: "op", meta: "meta", v: "v"}
-	# 		@db.docHistoryStats =
-	# 			update: sinon.stub().callsArg(3)
-	# 		@db.docHistory =
-	# 			remove: sinon.stub().callsArg(1)
-	# 		@MongoManager.markDocHistoryAsArchived @doc_id, @update.v, @callback
-
-	# 	it "should update doc status with inS3 flag", ->
-	# 		@db.docHistoryStats.update
-	# 			.calledWith({
-	# 				doc_id: ObjectId(@doc_id)
-	# 			},{
-	# 				$set : { inS3 : true }
-	# 			})
-	# 			.should.equal true
-
-	# 	it "should remove any other doc changes before last update", ->
-	# 		@db.docHistory.remove
-	# 			.calledWith({
-	# 				doc_id: ObjectId(@doc_id)
-	# 				v: { $lte : @update.v }
-	# 				expiresAt: {$exists : false}
-	# 			})
-	# 			.should.equal true
-
-	# 	it "should call the callback", ->
-	# 		@callback.called.should.equal true
-
-	# describe "markDocHistoryAsUnarchived", ->
-	# 	beforeEach ->
-	# 		@db.docHistoryStats =
-	# 			update: sinon.stub().callsArg(2)
-	# 		@MongoManager.markDocHistoryAsUnarchived @doc_id, @callback
-
-	# 	it "should remove any doc changes inS3 flag", ->
-	# 		@db.docHistoryStats.update
-	# 			.calledWith({
-	# 				doc_id: ObjectId(@doc_id)
-	# 			},{
-	# 				$unset : { inS3 : true, lastVersion: true }
-	# 			})
-	# 			.should.equal true
-
-	# 	it "should call the callback", ->
-	# 		@callback.called.should.equal true