mirror of
https://github.com/overleaf/overleaf.git
synced 2025-03-06 07:43:24 +00:00
unit tests for locking timeouts
This commit is contained in:
parent
bb0dc4b4d5
commit
d2730c8d6b
1 changed files with 37 additions and 1 deletions
|
@ -19,7 +19,7 @@ describe 'LockManager - trying the lock', ->
|
|||
key_schema:
|
||||
blockingKey: ({doc_id}) -> "Blocking:#{doc_id}"
|
||||
}
|
||||
"./Profiler": class Profiler
|
||||
"./Profiler": @Profiler = class Profiler
|
||||
log: sinon.stub().returns { end: sinon.stub() }
|
||||
end: sinon.stub()
|
||||
|
||||
|
@ -48,3 +48,39 @@ describe 'LockManager - trying the lock', ->
|
|||
it "should return the callback with false", ->
|
||||
@callback.calledWith(null, false).should.equal true
|
||||
|
||||
describe "when it takes a long time for redis to set the lock", ->
|
||||
beforeEach ->
|
||||
@Profiler.prototype.end = () -> 7000 # take a long time
|
||||
@Profiler.prototype.log = sinon.stub().returns { end: @Profiler.prototype.end }
|
||||
@lockValue = "mock-lock-value"
|
||||
@LockManager.randomLock = sinon.stub().returns @lockValue
|
||||
@LockManager.releaseLock = sinon.stub().callsArgWith(2,null)
|
||||
@set.callsArgWith(5, null, "OK")
|
||||
|
||||
describe "in all cases", ->
|
||||
beforeEach ->
|
||||
@LockManager.tryLock @doc_id, @callback
|
||||
|
||||
it "should set the lock key with an expiry if it is not set", ->
|
||||
@set.calledWith("Blocking:#{@doc_id}", @lockValue, "EX", 30, "NX")
|
||||
.should.equal true
|
||||
|
||||
it "should try to release the lock", ->
|
||||
@LockManager.releaseLock.calledWith(@doc_id, @lockValue).should.equal true
|
||||
|
||||
describe "if the lock is released successfully", ->
|
||||
beforeEach ->
|
||||
@LockManager.releaseLock = sinon.stub().callsArgWith(2,null)
|
||||
@LockManager.tryLock @doc_id, @callback
|
||||
|
||||
it "should return the callback with false", ->
|
||||
@callback.calledWith(null, false).should.equal true
|
||||
|
||||
describe "if the lock has already timed out", ->
|
||||
beforeEach ->
|
||||
@LockManager.releaseLock = sinon.stub().callsArgWith(2, new Error("tried to release timed out lock"))
|
||||
@LockManager.tryLock @doc_id, @callback
|
||||
|
||||
it "should return the callback with an error", ->
|
||||
e = new Error("tried to release timed out lock")
|
||||
@callback.calledWith(e).should.equal true
|
||||
|
|
Loading…
Reference in a new issue