From 770b53f6afc474c2299d20b2c5452dd109f58917 Mon Sep 17 00:00:00 2001 From: James Allen Date: Thu, 8 Jun 2017 16:43:27 +0100 Subject: [PATCH] Add missing mocks to speed up tests and clean up output --- .../unit/coffee/DispatchManager/DispatchManagerTests.coffee | 1 + .../test/unit/coffee/LockManager/CheckingTheLock.coffee | 3 +++ .../test/unit/coffee/LockManager/ReleasingTheLock.coffee | 3 +++ .../test/unit/coffee/LockManager/getLockTests.coffee | 3 +++ .../test/unit/coffee/LockManager/tryLockTests.coffee | 3 +++ .../test/unit/coffee/UpdateManager/UpdateManagerTests.coffee | 3 +++ 6 files changed, 16 insertions(+) diff --git a/services/document-updater/test/unit/coffee/DispatchManager/DispatchManagerTests.coffee b/services/document-updater/test/unit/coffee/DispatchManager/DispatchManagerTests.coffee index dcd643fcfe..b749e83a5d 100644 --- a/services/document-updater/test/unit/coffee/DispatchManager/DispatchManagerTests.coffee +++ b/services/document-updater/test/unit/coffee/DispatchManager/DispatchManagerTests.coffee @@ -13,6 +13,7 @@ describe "DispatchManager", -> redis: realtime: {} "redis-sharelatex": @redis = {} + "./RateLimitManager": {} @callback = sinon.stub() @RateLimiter = { run: (task,cb) -> task(cb) } # run task without rate limit diff --git a/services/document-updater/test/unit/coffee/LockManager/CheckingTheLock.coffee b/services/document-updater/test/unit/coffee/LockManager/CheckingTheLock.coffee index f6670c8b35..a080e563f1 100644 --- a/services/document-updater/test/unit/coffee/LockManager/CheckingTheLock.coffee +++ b/services/document-updater/test/unit/coffee/LockManager/CheckingTheLock.coffee @@ -19,6 +19,9 @@ describe 'LockManager - checking the lock', ()-> auth:-> exists: existsStub "./Metrics": {inc: () ->} + "./Profiler": class Profiler + log: sinon.stub().returns { end: sinon.stub() } + end: sinon.stub() LockManager = SandboxedModule.require(modulePath, requires: mocks) it 'should return true if the key does not exists', (done)-> diff --git a/services/document-updater/test/unit/coffee/LockManager/ReleasingTheLock.coffee b/services/document-updater/test/unit/coffee/LockManager/ReleasingTheLock.coffee index 36c458cb71..06dd2aa6bf 100644 --- a/services/document-updater/test/unit/coffee/LockManager/ReleasingTheLock.coffee +++ b/services/document-updater/test/unit/coffee/LockManager/ReleasingTheLock.coffee @@ -26,6 +26,9 @@ describe 'LockManager - releasing the lock', ()-> blockingKey: ({doc_id}) -> "Blocking:#{doc_id}" } "./Metrics": {inc: () ->} + "./Profiler": class Profiler + log: sinon.stub().returns { end: sinon.stub() } + end: sinon.stub() @LockManager = SandboxedModule.require(modulePath, requires: mocks) @lockValue = "lock-value-stub" diff --git a/services/document-updater/test/unit/coffee/LockManager/getLockTests.coffee b/services/document-updater/test/unit/coffee/LockManager/getLockTests.coffee index 84cc3208a3..7093ab223a 100644 --- a/services/document-updater/test/unit/coffee/LockManager/getLockTests.coffee +++ b/services/document-updater/test/unit/coffee/LockManager/getLockTests.coffee @@ -12,6 +12,9 @@ describe 'LockManager - getting the lock', -> createClient : () => auth:-> "./Metrics": {inc: () ->} + "./Profiler": class Profiler + log: sinon.stub().returns { end: sinon.stub() } + end: sinon.stub() @callback = sinon.stub() @doc_id = "doc-id-123" diff --git a/services/document-updater/test/unit/coffee/LockManager/tryLockTests.coffee b/services/document-updater/test/unit/coffee/LockManager/tryLockTests.coffee index b3ff7cdd7e..7b52f416ab 100644 --- a/services/document-updater/test/unit/coffee/LockManager/tryLockTests.coffee +++ b/services/document-updater/test/unit/coffee/LockManager/tryLockTests.coffee @@ -19,6 +19,9 @@ describe 'LockManager - trying the lock', -> key_schema: blockingKey: ({doc_id}) -> "Blocking:#{doc_id}" } + "./Profiler": class Profiler + log: sinon.stub().returns { end: sinon.stub() } + end: sinon.stub() @callback = sinon.stub() @doc_id = "doc-id-123" diff --git a/services/document-updater/test/unit/coffee/UpdateManager/UpdateManagerTests.coffee b/services/document-updater/test/unit/coffee/UpdateManager/UpdateManagerTests.coffee index 2de6e93e44..4f456992f0 100644 --- a/services/document-updater/test/unit/coffee/UpdateManager/UpdateManagerTests.coffee +++ b/services/document-updater/test/unit/coffee/UpdateManager/UpdateManagerTests.coffee @@ -22,6 +22,9 @@ describe "UpdateManager", -> "settings-sharelatex": Settings = {} "./DocumentManager": @DocumentManager = {} "./RangesManager": @RangesManager = {} + "./Profiler": class Profiler + log: sinon.stub().returns { end: sinon.stub() } + end: sinon.stub() describe "processOutstandingUpdates", -> beforeEach ->