Explorar o código

Merge pull request #22646 from overleaf/ar-address-mongo-count-deprecation-warning

[web] Switch from find.count to countDocuments

GitOrigin-RevId: cc607868334d54b3d5c375c06fec97a482a16cc5
Andrew Rumble hai 1 ano
pai
achega
ccb2791a0d

+ 4 - 6
services/web/app/src/Features/User/UserHandler.js

@@ -15,12 +15,10 @@ function populateTeamInvites(user, callback) {
 async function countActiveUsers() {
   const oneYearAgo = new Date()
   oneYearAgo.setFullYear(oneYearAgo.getFullYear() - 1)
-  return await db.users
-    .find(
-      { lastActive: { $gte: oneYearAgo } },
-      { readPreference: READ_PREFERENCE_SECONDARY }
-    )
-    .count()
+  return await db.users.countDocuments(
+    { lastActive: { $gte: oneYearAgo } },
+    { readPreference: READ_PREFERENCE_SECONDARY }
+  )
 }
 
 module.exports = {

+ 1 - 3
services/web/test/unit/src/User/UserHandlerTests.js

@@ -17,9 +17,7 @@ describe('UserHandler', function () {
 
     this.db = {
       users: {
-        find: sinon.stub().returns({
-          count: sinon.stub().resolves(2),
-        }),
+        countDocuments: sinon.stub().resolves(2),
       },
     }