DOP-4490 fixing find options

mongodb · Mar 28, 2024 · 2c7b18e · 2c7b18e
1 parent 24c8bd2
commit 2c7b18e
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 10 deletions.
diff --git a/src/repositories/baseRepository.ts b/src/repositories/baseRepository.ts
@@ -84,16 +84,11 @@ export abstract class BaseRepository {
     }
   }
 
-  protected async find(
-    query: any,
-    errorMsg: string,
-    options?: mongodb.FindOptions,
-    projection?: any
-  ): Promise<mongodb.FindCursor> {
+  protected async find(query: any, errorMsg: string, options?: mongodb.FindOptions): Promise<mongodb.FindCursor> {
     try {
       return await this.promiseTimeoutS(
         this._config.get('MONGO_TIMEOUT_S'),
-        this._collection.find(query, projection, options),
+        this._collection.find(query, options),
         errorMsg
       );
     } catch (error) {

diff --git a/src/repositories/repoBranchesRepository.ts b/src/repositories/repoBranchesRepository.ts
@@ -34,14 +34,13 @@ export class RepoBranchesRepository extends BaseRepository {
 
   async getProdDeployableRepoBranches(directoryPath?: string): Promise<any> {
     const query = { prodDeployable: true, internalOnly: false };
-    const projection = { _id: 0, repoName: 1 };
+    const findOptions = { projection: { _id: 0, repoName: 1 } };
     const repos = await this.find(
       query,
       `Mongo Timeout Error: Timedout while retrieving repo branches entries ${
         directoryPath ? `/${directoryPath}` : ''
       }`,
-      undefined,
-      projection
+      findOptions
     );
 
     return repos ?? [];