Shane Kilkelly 9 лет назад
Родитель
Сommit
a64f890ed0

+ 7 - 1
services/web/app/coffee/Features/Project/ProjectCreationHandler.coffee

@@ -11,7 +11,8 @@ fs = require('fs')
 Path = require "path"
 _ = require "underscore"
 
-module.exports =
+module.exports = ProjectCreationHandler =
+
 	createBlankProject : (owner_id, projectName, callback = (error, project) ->)->
 		metrics.inc("project-creation")
 		logger.log owner_id:owner_id, projectName:projectName, "creating blank project"
@@ -79,5 +80,10 @@ module.exports =
 				output = _.template(template.toString(), data)
 				callback null, output.split("\n")
 
+metrics.timeAsyncMethod(
+	ProjectCreationHandler, 'createBlankProject',
+	'mongo.ProjectCreationHandler',
+	logger
+)
 
 

+ 5 - 1
services/web/app/coffee/Features/Project/ProjectGetter.coffee

@@ -60,4 +60,8 @@ module.exports = ProjectGetter =
 				callback null, projects, readAndWriteProjects, readOnlyProjects
 
 
-metrics.timeAsyncMethod ProjectGetter, 'getProject', 'ProjectGetter.getProject', logger
+[
+	'getProject',
+	'getProjectWithoutDocLines'
+].map (method) ->
+	metrics.timeAsyncMethod(ProjectGetter, method, 'mongo.ProjectGetter', logger)

+ 9 - 1
services/web/app/coffee/Features/User/UserCreator.coffee

@@ -1,8 +1,10 @@
 User = require("../../models/User").User
 UserLocator = require("./UserLocator")
 logger = require("logger-sharelatex")
+metrics = require('../../infrastructure/Metrics')
 
-module.exports =
+
+module.exports = UserCreator =
 
 	getUserOrCreateHoldingAccount: (email, callback = (err, user)->)->
 		self = @
@@ -36,3 +38,9 @@ module.exports =
 
 		user.save (err)->
 			callback(err, user)
+
+metrics.timeAsyncMethod(
+	UserCreator, 'createNewUser',
+	'mongo.UserCreator',
+	logger
+)

+ 6 - 2
services/web/app/coffee/Features/User/UserGetter.coffee

@@ -27,5 +27,9 @@ module.exports = UserGetter =
 		
 		db.users.find { _id: { $in: user_ids} }, projection, callback
 
-metrics.timeAsyncMethod UserGetter, 'getUser', 'UserGetter.getUser', logger
-metrics.timeAsyncMethod UserGetter, 'getUsers', 'UserGetter.getUsers', logger
+
+[
+	'getUser',
+	'getUsers'
+].map (method) ->
+	metrics.timeAsyncMethod UserGetter, method, 'mongo.UserGetter', logger

+ 5 - 2
services/web/app/coffee/Features/User/UserLocator.coffee

@@ -14,5 +14,8 @@ module.exports = UserLocator =
 	findById: (_id, callback)->
 		db.users.findOne _id:ObjectId(_id+""), callback
 
-metrics.timeAsyncMethod UserLocator, 'findById', 'UserLocator.findById', logger
-metrics.timeAsyncMethod UserLocator, 'findByEmail', 'UserLocator.findByEmail', logger
+[
+	'findById',
+	'findByEmail'
+].map (method) ->
+	metrics.timeAsyncMethod UserLocator, method, 'mongo.UserLocator', logger

+ 1 - 1
services/web/app/coffee/Features/User/UserUpdater.coffee

@@ -30,4 +30,4 @@ module.exports = UserUpdater =
 				callback()
 
 
-metrics.timeAsyncMethod UserUpdater, 'updateUser', 'UserUpdater.updateUser', logger
+metrics.timeAsyncMethod UserUpdater, 'updateUser', 'mongo.UserUpdater', logger

+ 4 - 3
services/web/test/UnitTests/coffee/Project/ProjectCreationHandlerTests.coffee

@@ -50,9 +50,10 @@ describe 'ProjectCreationHandler', ->
 			'./ProjectEntityHandler':@ProjectEntityHandler
 			"settings-sharelatex": @Settings = {}
 			'logger-sharelatex': {log:->}
-			"../../infrastructure/Metrics": inc:->
-				
-
+			"../../infrastructure/Metrics": {
+				inc: ()->,
+				timeAsyncMethod: ()->
+			}
 
 	describe 'Creating a Blank project', ->
 		beforeEach ->