Skip to content

Commit d5bdbfa

Browse files
move datadog to before callback, maybe it will fix test
1 parent 8c5edd1 commit d5bdbfa

2 files changed

Lines changed: 4 additions & 4 deletions

File tree

lib/models/mongo/context-version.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -781,11 +781,11 @@ ContextVersionSchema.methods.dedupeBuild = function (callback) {
781781

782782
function replaceIfDupe(dupe, cb) {
783783
if (dupe) {
784-
self.copyBuildFromContextVersion(dupe, cb);
785784
dogstatsd.increment('api.contextVersion.build.deduped');
785+
self.copyBuildFromContextVersion(dupe, cb);
786786
} else {
787-
cb(null, self);
788787
dogstatsd.increment('api.contextVersion.build.noDupe');
788+
cb(null, self);
789789
}
790790
}
791791
};

lib/models/mongo/infra-code-version.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -607,9 +607,9 @@ function hashString(data, cb) {
607607
.replace(/^[\s\uFEFF\xA0]*\n/g, '') // remove start of file blank lines
608608
.replace(/[\s\uFEFF\xA0]+$/g, '\n'), salt, function(err, hash) {
609609
if (err) { return cb(err); }
610+
dogstatsd.timing('api.infraCodeVersion.hashTime', new Date()-start, 1,
611+
['length:'+data.length]);
610612
cb(null, hash);
611-
dogstatsd.timing('api.infraCodeVersion.hashTime',
612-
new Date()-start, 1, ['length:'+data.length]);
613613
});
614614
}
615615

0 commit comments

Comments
 (0)