Parcourir la source

Merge pull request #31 from overleaf/jpa-fix-unit-tests

[misc] fix unit tests
Jakob Ackermann il y a 6 ans
Parent
commit
ea435c88ea

+ 2 - 2
libraries/metrics/test/unit/coffee/timeAsyncMethodTests.coffee

@@ -53,7 +53,7 @@ describe 'timeAsyncMethod', ->
 		@timeAsyncMethod @testObject, 'nextNumber', 'someContext.TestObject'
 		@testObject.nextNumber 2, (err, result) =>
 			expect(@metrics.inc.callCount).to.equal 1
-			expect(@metrics.inc.calledWith('someContext.TestObject.nextNumber.success')).to.equal true
+			expect(@metrics.inc.calledWith('someContext_result', 1, { method: 'TestObject_nextNumber', status: 'success'})).to.equal true
 			done()
 
 	describe 'when base method produces an error', ->
@@ -78,7 +78,7 @@ describe 'timeAsyncMethod', ->
 			@timeAsyncMethod @testObject, 'nextNumber', 'someContext.TestObject'
 			@testObject.nextNumber 2, (err, result) =>
 				expect(@metrics.inc.callCount).to.equal 1
-				expect(@metrics.inc.calledWith('someContext.TestObject.nextNumber.failure')).to.equal true
+				expect(@metrics.inc.calledWith('someContext_result', 1, { method: 'TestObject_nextNumber', status: 'failed'})).to.equal true
 				done()
 
 	describe 'when a logger is supplied', ->