perf(notifier): project only notifications field when loading rules (#551)

* perf(notifier): project only notifications field when loading rules

* test(notifier): update findOne assertion for projection arg

Author: Kuchizu

workers/notifier/src/index.ts

@@ -193,9 +193,12 @@ export default class NotifierWorker extends Worker {
    */
   private async getProjectNotificationRules(projectId: string): Promise<Rule[]> {
     const connection = this.accountsDb.getConnection();
-    const projects = connection.collection('projects');
+    const projects = connection.collection<{ notifications?: Rule[] }>('projects');
 
-    const project = await projects.findOne({ _id: new ObjectID(projectId) });
+    const project = await projects.findOne(
+      { _id: new ObjectID(projectId) },
+      { projection: { notifications: 1 } }
+    );
 
     if (!project) {
       throw new Error('There is no project with given id');

workers/notifier/tests/worker.test.ts

@@ -242,7 +242,10 @@ describe('NotifierWorker', () => {
 
       await worker.handle(message);
 
-      expect(dbQueryMock).toBeCalledWith({ _id: new ObjectID(message.projectId) });
+      expect(dbQueryMock).toBeCalledWith(
+        { _id: new ObjectID(message.projectId) },
+        { projection: { notifications: 1 } }
+      );
     });
 
     it('should close db connection on finish', async () => {