2016-06-01 07:22:47 -04:00
|
|
|
sinon = require('sinon')
|
|
|
|
chai = require('chai')
|
|
|
|
should = chai.should()
|
|
|
|
modulePath = "../../../../app/js/RedisManager.js"
|
|
|
|
SandboxedModule = require('sandboxed-module')
|
|
|
|
Errors = require "../../../../app/js/Errors"
|
|
|
|
|
|
|
|
describe "RedisManager", ->
|
|
|
|
beforeEach ->
|
|
|
|
@rclient =
|
|
|
|
auth: () ->
|
|
|
|
exec: sinon.stub()
|
|
|
|
@rclient.multi = () => @rclient
|
|
|
|
@RedisManager = SandboxedModule.require modulePath, requires:
|
2016-06-23 10:38:51 -04:00
|
|
|
"./RedisBackend":
|
|
|
|
createClient: () => @rclient
|
2016-06-07 12:58:18 -04:00
|
|
|
"./RedisKeyBuilder":
|
|
|
|
blockingKey: ({doc_id}) -> "Blocking:#{doc_id}"
|
|
|
|
docLines: ({doc_id}) -> "doclines:#{doc_id}"
|
|
|
|
docOps: ({doc_id}) -> "DocOps:#{doc_id}"
|
|
|
|
docVersion: ({doc_id}) -> "DocVersion:#{doc_id}"
|
|
|
|
projectKey: ({doc_id}) -> "ProjectId:#{doc_id}"
|
|
|
|
pendingUpdates: ({doc_id}) -> "PendingUpdates:#{doc_id}"
|
|
|
|
docsInProject: ({project_id}) -> "DocsIn:#{project_id}"
|
2016-06-01 07:22:47 -04:00
|
|
|
"logger-sharelatex": @logger = { error: sinon.stub(), log: sinon.stub(), warn: sinon.stub() }
|
|
|
|
"./Metrics": @metrics =
|
|
|
|
inc: sinon.stub()
|
|
|
|
Timer: class Timer
|
|
|
|
done: () ->
|
|
|
|
"./Errors": Errors
|
|
|
|
@doc_id = "doc-id-123"
|
|
|
|
@project_id = "project-id-123"
|
|
|
|
@callback = sinon.stub()
|
|
|
|
|
|
|
|
describe "getDoc", ->
|
|
|
|
beforeEach ->
|
|
|
|
@lines = ["one", "two", "three"]
|
|
|
|
@jsonlines = JSON.stringify @lines
|
|
|
|
@version = 42
|
|
|
|
@rclient.get = sinon.stub()
|
|
|
|
@rclient.exec = sinon.stub().callsArgWith(0, null, [@jsonlines, @version])
|
|
|
|
@RedisManager.getDoc @doc_id, @callback
|
|
|
|
|
|
|
|
it "should get the lines from redis", ->
|
|
|
|
@rclient.get
|
|
|
|
.calledWith("doclines:#{@doc_id}")
|
|
|
|
.should.equal true
|
|
|
|
|
|
|
|
it "should get the version from", ->
|
|
|
|
@rclient.get
|
|
|
|
.calledWith("DocVersion:#{@doc_id}")
|
|
|
|
.should.equal true
|
|
|
|
|
|
|
|
it 'should return the document', ->
|
|
|
|
@callback
|
|
|
|
.calledWith(null, @lines, @version)
|
|
|
|
.should.equal true
|
|
|
|
|
|
|
|
describe "getPreviousDocOpsTests", ->
|
|
|
|
describe "with a start and an end value", ->
|
|
|
|
beforeEach ->
|
|
|
|
@first_version_in_redis = 30
|
|
|
|
@version = 70
|
|
|
|
@length = @version - @first_version_in_redis
|
|
|
|
@start = 50
|
|
|
|
@end = 60
|
|
|
|
@ops = [
|
|
|
|
{ "mock": "op-1" },
|
|
|
|
{ "mock": "op-2" }
|
|
|
|
]
|
|
|
|
@jsonOps = @ops.map (op) -> JSON.stringify op
|
|
|
|
@rclient.llen = sinon.stub().callsArgWith(1, null, @length)
|
|
|
|
@rclient.get = sinon.stub().callsArgWith(1, null, @version.toString())
|
|
|
|
@rclient.lrange = sinon.stub().callsArgWith(3, null, @jsonOps)
|
|
|
|
@RedisManager.getPreviousDocOps(@doc_id, @start, @end, @callback)
|
|
|
|
|
|
|
|
it "should get the length of the existing doc ops", ->
|
|
|
|
@rclient.llen
|
|
|
|
.calledWith("DocOps:#{@doc_id}")
|
|
|
|
.should.equal true
|
|
|
|
|
|
|
|
it "should get the current version of the doc", ->
|
|
|
|
@rclient.get
|
|
|
|
.calledWith("DocVersion:#{@doc_id}")
|
|
|
|
.should.equal true
|
|
|
|
|
|
|
|
it "should get the appropriate docs ops", ->
|
|
|
|
@rclient.lrange
|
|
|
|
.calledWith("DocOps:#{@doc_id}", @start - @first_version_in_redis, @end - @first_version_in_redis)
|
|
|
|
.should.equal true
|
|
|
|
|
|
|
|
it "should return the docs with the doc ops deserialized", ->
|
|
|
|
@callback.calledWith(null, @ops).should.equal true
|
|
|
|
|
|
|
|
describe "with an end value of -1", ->
|
|
|
|
beforeEach ->
|
|
|
|
@first_version_in_redis = 30
|
|
|
|
@version = 70
|
|
|
|
@length = @version - @first_version_in_redis
|
|
|
|
@start = 50
|
|
|
|
@end = -1
|
|
|
|
@ops = [
|
|
|
|
{ "mock": "op-1" },
|
|
|
|
{ "mock": "op-2" }
|
|
|
|
]
|
|
|
|
@jsonOps = @ops.map (op) -> JSON.stringify op
|
|
|
|
@rclient.llen = sinon.stub().callsArgWith(1, null, @length)
|
|
|
|
@rclient.get = sinon.stub().callsArgWith(1, null, @version.toString())
|
|
|
|
@rclient.lrange = sinon.stub().callsArgWith(3, null, @jsonOps)
|
|
|
|
@RedisManager.getPreviousDocOps(@doc_id, @start, @end, @callback)
|
|
|
|
|
|
|
|
it "should get the appropriate docs ops to the end of list", ->
|
|
|
|
@rclient.lrange
|
|
|
|
.calledWith("DocOps:#{@doc_id}", @start - @first_version_in_redis, -1)
|
|
|
|
.should.equal true
|
|
|
|
|
|
|
|
it "should return the docs with the doc ops deserialized", ->
|
|
|
|
@callback.calledWith(null, @ops).should.equal true
|
|
|
|
|
|
|
|
describe "when the requested range is not in Redis", ->
|
|
|
|
beforeEach ->
|
|
|
|
@first_version_in_redis = 30
|
|
|
|
@version = 70
|
|
|
|
@length = @version - @first_version_in_redis
|
|
|
|
@start = 20
|
|
|
|
@end = -1
|
|
|
|
@ops = [
|
|
|
|
{ "mock": "op-1" },
|
|
|
|
{ "mock": "op-2" }
|
|
|
|
]
|
|
|
|
@jsonOps = @ops.map (op) -> JSON.stringify op
|
|
|
|
@rclient.llen = sinon.stub().callsArgWith(1, null, @length)
|
|
|
|
@rclient.get = sinon.stub().callsArgWith(1, null, @version.toString())
|
|
|
|
@rclient.lrange = sinon.stub().callsArgWith(3, null, @jsonOps)
|
|
|
|
@RedisManager.getPreviousDocOps(@doc_id, @start, @end, @callback)
|
|
|
|
|
|
|
|
it "should return an error", ->
|
|
|
|
@callback.calledWith(new Errors.OpRangeNotAvailableError("doc ops range is not loaded in redis")).should.equal true
|
|
|
|
|
|
|
|
it "should log out the problem", ->
|
|
|
|
@logger.warn.called.should.equal true
|
|
|
|
|
|
|
|
describe "pushDocOp", ->
|
|
|
|
beforeEach ->
|
|
|
|
@rclient.rpush = sinon.stub()
|
|
|
|
@rclient.expire = sinon.stub()
|
|
|
|
@rclient.incr = sinon.stub()
|
|
|
|
@rclient.ltrim = sinon.stub()
|
|
|
|
@op = { op: [{ i: "foo", p: 4 }] }
|
|
|
|
@version = 42
|
|
|
|
_ = null
|
|
|
|
@rclient.exec = sinon.stub().callsArgWith(0, null, [_, _, _, @version])
|
|
|
|
@RedisManager.pushDocOp @doc_id, @op, @callback
|
|
|
|
|
|
|
|
it "should push the doc op into the doc ops list", ->
|
|
|
|
@rclient.rpush
|
|
|
|
.calledWith("DocOps:#{@doc_id}", JSON.stringify(@op))
|
|
|
|
.should.equal true
|
|
|
|
|
|
|
|
it "should renew the expiry ttl on the doc ops array", ->
|
|
|
|
@rclient.expire
|
|
|
|
.calledWith("DocOps:#{@doc_id}", @RedisManager.DOC_OPS_TTL)
|
|
|
|
.should.equal true
|
|
|
|
|
|
|
|
it "should truncate the list to 100 members", ->
|
|
|
|
@rclient.ltrim
|
|
|
|
.calledWith("DocOps:#{@doc_id}", -@RedisManager.DOC_OPS_MAX_LENGTH, -1)
|
|
|
|
.should.equal true
|
|
|
|
|
|
|
|
it "should increment the version number", ->
|
|
|
|
@rclient.incr
|
|
|
|
.calledWith("DocVersion:#{@doc_id}")
|
|
|
|
.should.equal true
|
|
|
|
|
|
|
|
it "should call the callback with the version number", ->
|
|
|
|
@callback.calledWith(null, parseInt(@version, 10)).should.equal true
|
|
|
|
|
|
|
|
describe "putDocInMemory", ->
|
|
|
|
beforeEach (done) ->
|
|
|
|
@rclient.set = sinon.stub()
|
|
|
|
@rclient.sadd = sinon.stub().yields()
|
|
|
|
@rclient.exec.yields()
|
|
|
|
@lines = ["one", "two", "three"]
|
|
|
|
@version = 42
|
|
|
|
@RedisManager.putDocInMemory @project_id, @doc_id, @lines, @version, done
|
|
|
|
|
|
|
|
it "should set the lines", ->
|
|
|
|
@rclient.set
|
|
|
|
.calledWith("doclines:#{@doc_id}", JSON.stringify @lines)
|
|
|
|
.should.equal true
|
|
|
|
|
|
|
|
it "should set the version", ->
|
|
|
|
@rclient.set
|
|
|
|
.calledWith("DocVersion:#{@doc_id}", @version)
|
|
|
|
.should.equal true
|
|
|
|
|
|
|
|
it "should set the project_id for the doc", ->
|
|
|
|
@rclient.set
|
|
|
|
.calledWith("ProjectId:#{@doc_id}", @project_id)
|
|
|
|
.should.equal true
|
|
|
|
|
|
|
|
it "should add the doc_id to the project set", ->
|
|
|
|
@rclient.sadd
|
|
|
|
.calledWith("DocsIn:#{@project_id}", @doc_id)
|
|
|
|
.should.equal true
|
|
|
|
|
|
|
|
describe "removeDocFromMemory", ->
|
|
|
|
beforeEach (done) ->
|
|
|
|
@rclient.del = sinon.stub()
|
|
|
|
@rclient.srem = sinon.stub().yields()
|
|
|
|
@rclient.exec.yields()
|
|
|
|
@RedisManager.removeDocFromMemory @project_id, @doc_id, done
|
|
|
|
|
|
|
|
it "should delete the lines", ->
|
|
|
|
@rclient.del
|
|
|
|
.calledWith("doclines:#{@doc_id}")
|
|
|
|
.should.equal true
|
|
|
|
|
|
|
|
it "should delete the version", ->
|
|
|
|
@rclient.del
|
|
|
|
.calledWith("DocVersion:#{@doc_id}")
|
|
|
|
.should.equal true
|
|
|
|
|
|
|
|
it "should delete the project_id for the doc", ->
|
|
|
|
@rclient.del
|
|
|
|
.calledWith("ProjectId:#{@doc_id}")
|
|
|
|
.should.equal true
|
|
|
|
|
|
|
|
it "should remove the doc_id from the project set", ->
|
|
|
|
@rclient.srem
|
|
|
|
.calledWith("DocsIn:#{@project_id}", @doc_id)
|
2016-06-29 07:57:56 -04:00
|
|
|
.should.equal true
|
|
|
|
|
|
|
|
describe "getAndSetDoc", ->
|
|
|
|
beforeEach ->
|
|
|
|
@rclient.get = sinon.stub()
|
|
|
|
@rclient.lrange = sinon.stub()
|
|
|
|
@rclient.del = sinon.stub()
|
|
|
|
@rclient.set = sinon.stub()
|
|
|
|
@rclient.rpush = sinon.stub()
|
|
|
|
@rclient.exec = sinon.stub()
|
|
|
|
@rclient.exec.yields(null, [
|
|
|
|
@lines = '["mock","lines"]',
|
|
|
|
@version = 42,
|
|
|
|
@doc_ops = ["mock", "doc", "ops"],
|
|
|
|
@project_id = "mock-project-id"
|
|
|
|
])
|
|
|
|
@RedisManager.getAndSetDoc @doc_id, @callback
|
|
|
|
|
|
|
|
it "should get the original values", ->
|
|
|
|
@rclient.get
|
|
|
|
.calledWith("doclines:#{@doc_id}")
|
|
|
|
.should.equal true
|
|
|
|
@rclient.get
|
|
|
|
.calledWith("DocVersion:#{@doc_id}")
|
|
|
|
.should.equal true
|
|
|
|
@rclient.get
|
|
|
|
.calledWith("ProjectId:#{@doc_id}")
|
|
|
|
.should.equal true
|
|
|
|
@rclient.lrange
|
|
|
|
.calledWith("DocOps:#{@doc_id}", 0, -1)
|
|
|
|
.should.equal true
|
|
|
|
|
|
|
|
it "should set the doclines again", ->
|
|
|
|
@rclient.set
|
|
|
|
.calledWith("doclines:#{@doc_id}", @lines)
|
|
|
|
.should.equal true
|
|
|
|
|
|
|
|
it "should set the DocVersion again", ->
|
|
|
|
@rclient.set
|
|
|
|
.calledWith("DocVersion:#{@doc_id}", @version)
|
|
|
|
.should.equal true
|
|
|
|
|
|
|
|
it "should set the project id again", ->
|
|
|
|
@rclient.set
|
|
|
|
.calledWith("ProjectId:#{@doc_id}", @project_id)
|
|
|
|
.should.equal true
|
|
|
|
|
|
|
|
it "should set the doc ops again", ->
|
|
|
|
@rclient.del
|
|
|
|
.calledWith("DocOps:#{@doc_id}")
|
|
|
|
.should.equal true
|
|
|
|
@rclient.rpush
|
|
|
|
.calledWith("DocOps:#{@doc_id}", @doc_ops...)
|
|
|
|
.should.equal true
|
|
|
|
|
|
|
|
describe "getAndSetProject", ->
|
|
|
|
beforeEach ->
|
|
|
|
@rclient.smembers = sinon.stub()
|
|
|
|
@rclient.sadd = sinon.stub()
|
|
|
|
@rclient.smembers.withArgs("DocsIn:#{@project_id}").yields(null, @doc_ids = ["mock-doc-1", "mock-doc-2"])
|
|
|
|
@RedisManager.getAndSetProject @project_id, @callback
|
|
|
|
|
|
|
|
it "should set the doc ids again", ->
|
|
|
|
@rclient.sadd
|
|
|
|
.calledWith("DocsIn:#{@project_id}", @doc_ids...)
|
|
|
|
.should.equal true
|