fixed unit test

Author: rotem-codefresh

lib/interface/cli/commands/offline-logs/ensure-index.cmd.js

@@ -1,7 +1,7 @@
 const { MongoClient } = require("mongodb");
 const Command = require("../../Command");
 const cmd = require("./base.cmd");
-const { ensureIndex, defaultCollections } = require('./utils')
+const utils = require('./utils')
 
 const command = new Command({
   command: "ensure-index",
@@ -22,9 +22,9 @@ const command = new Command({
     await client.connect();
     const database = client.db(db);
     const failedCollections = [];
-    for (const collection of defaultCollections) {
+    for (const collection of utils.defaultCollections) {
       try {
-        await ensureIndex(database, collection);
+        await utils.ensureIndex(database, collection);
       } catch (error) {
         console.error(`failed to ensure index of collection '${collection}', error: ${error.message}`);
         failedCollections.push(collection);

lib/interface/cli/commands/offline-logs/ensure-index.spec.js

@@ -28,14 +28,14 @@ describe("ensure-index", () => {
       mockCollection.indexExists.mockReturnValue(false);
       utils.getUserInput.mockReturnValue(true);
       const collection = 'whatever';
-      const expectedIndexObj = { jobId: 1, accountId: 1, time: 1 }
+      const expectedIndexObj = { accountId: 1, jobId: 1, time: 1 }
 
       //execute
       await utils.ensureIndex(mockDatabase, collection);
 
       //check
       expect(mockCollection.indexExists).toBeCalledTimes(1);
-      expect(mockCollection.indexExists).toBeCalledWith("jobId_1_accountId_1_time_1");
+      expect(mockCollection.indexExists).toBeCalledWith("accountId_1_jobId_1_time_1");
       expect(mockCollection.estimatedDocumentCount).toBeCalledTimes(1);
       expect(utils.getUserInput).toBeCalledTimes(1);
       expect(mockCollection.createIndex).toBeCalledTimes(1);
@@ -52,7 +52,7 @@ describe("ensure-index", () => {
 
       //check
       expect(mockCollection.indexExists).toBeCalledTimes(1);
-      expect(mockCollection.indexExists).toBeCalledWith("jobId_1_accountId_1_time_1");
+      expect(mockCollection.indexExists).toBeCalledWith("accountId_1_jobId_1_time_1");
       expect(mockCollection.estimatedDocumentCount).toBeCalledTimes(1);
       expect(utils.getUserInput).toBeCalledTimes(1);
       expect(mockCollection.createIndex).toBeCalledTimes(0);
@@ -67,7 +67,7 @@ describe("ensure-index", () => {
 
       //check
       expect(mockCollection.indexExists).toBeCalledTimes(1);
-      expect(mockCollection.indexExists).toBeCalledWith("jobId_1_accountId_1_time_1");
+      expect(mockCollection.indexExists).toBeCalledWith("accountId_1_jobId_1_time_1");
       expect(mockCollection.estimatedDocumentCount).toBeCalledTimes(0);
       expect(utils.getUserInput).toBeCalledTimes(0);
       expect(mockCollection.createIndex).toBeCalledTimes(0);