Fix unit tests

Signed-off-by: Craig Perkins <[email protected]>
opensearch-project · Nov 1, 2024 · db0d336 · db0d336
1 parent 244ea43
commit db0d336
Show file tree

Hide file tree

Showing 5 changed files with 54 additions and 102 deletions.
diff --git a/public/JobHandler/callbacks/force_merge.test.tsx b/public/JobHandler/callbacks/force_merge.test.tsx
@@ -41,10 +41,7 @@ describe("callbackForForceMerge spec", () => {
 
   it("callback when not complete", async () => {
     httpClientMock.fetch = getMockFn({
-      found: true,
-      _source: {
-        completed: false,
-      },
+      completed: false,
     });
     const result = await callbackForForceMerge(forceMergeMetaData, {
       core,
@@ -54,10 +51,7 @@ describe("callbackForForceMerge spec", () => {
 
   it("callback when complete", async () => {
     httpClientMock.fetch = getMockFn({
-      found: true,
-      _source: {
-        completed: true,
-      },
+      completed: true,
     });
     const result = await callbackForForceMerge(forceMergeMetaData, {
       core,
@@ -69,14 +63,11 @@ describe("callbackForForceMerge spec", () => {
 
   it("callback when some complete", async () => {
     httpClientMock.fetch = getMockFn({
-      found: true,
-      _source: {
-        completed: true,
-        response: {
-          _shards: {
-            successful: 9,
-            total: 10,
-          },
+      completed: true,
+      response: {
+        _shards: {
+          successful: 9,
+          total: 10,
         },
       },
     });
@@ -90,12 +81,9 @@ describe("callbackForForceMerge spec", () => {
 
   it("callback when failed", async () => {
     httpClientMock.fetch = getMockFn({
-      found: true,
-      _source: {
-        completed: true,
-        error: {
-          reason: "reason",
-        },
+      completed: true,
+      error: {
+        reason: "reason",
       },
     });
     const result = await callbackForForceMerge(forceMergeMetaData, {

diff --git a/public/JobHandler/callbacks/open.test.tsx b/public/JobHandler/callbacks/open.test.tsx
@@ -40,10 +40,7 @@ describe("callbackForOpen spec", () => {
 
   it("callback when not complete", async () => {
     httpClientMock.fetch = getMockFn({
-      found: true,
-      _source: {
-        completed: false,
-      },
+      completed: false,
     });
     const result = await callbackForOpen(openMetaData, {
       core,
@@ -53,13 +50,10 @@ describe("callbackForOpen spec", () => {
 
   it("callback when successfully complete", async () => {
     httpClientMock.fetch = getMockFn({
-      found: true,
-      _source: {
-        completed: true,
-        response: {
-          acknowledged: true,
-          shards_acknowledged: true,
-        },
+      completed: true,
+      response: {
+        acknowledged: true,
+        shards_acknowledged: true,
       },
     });
     const result = await callbackForOpen(openMetaData, {
@@ -72,12 +66,9 @@ describe("callbackForOpen spec", () => {
 
   it("callback when failed", async () => {
     httpClientMock.fetch = getMockFn({
-      found: true,
-      _source: {
-        completed: true,
-        error: {
-          reason: "reason",
-        },
+      completed: true,
+      error: {
+        reason: "reason",
       },
     });
     const result = await callbackForOpen(openMetaData, {

diff --git a/public/JobHandler/callbacks/reindex.test.tsx b/public/JobHandler/callbacks/reindex.test.tsx
@@ -43,10 +43,7 @@ describe("callbackForOpen spec", () => {
 
   it("callback when not complete", async () => {
     httpClientMock.fetch = getMockFn({
-      found: true,
-      _source: {
-        completed: false,
-      },
+      completed: false,
     });
     const result = await callbackForReindex(reindexMetaData, {
       core,
@@ -56,10 +53,7 @@ describe("callbackForOpen spec", () => {
 
   it("callback when successfully complete", async () => {
     httpClientMock.fetch = getMockFn({
-      found: true,
-      _source: {
-        completed: true,
-      },
+      completed: true,
     });
     const result = await callbackForReindex(reindexMetaData, {
       core,
@@ -71,20 +65,17 @@ describe("callbackForOpen spec", () => {
 
   it("callback when failed", async () => {
     httpClientMock.fetch = getMockFn({
-      found: true,
-      _source: {
-        completed: true,
-        error: {
-          reason: "reason",
-        },
-        failures: [
-          {
-            cause: {
-              reason: "cause",
-            },
-          },
-        ],
+      completed: true,
+      error: {
+        reason: "reason",
       },
+      failures: [
+        {
+          cause: {
+            reason: "cause",
+          },
+        },
+      ],
     });
     const result = await callbackForReindex(reindexMetaData, {
       core,

diff --git a/public/JobHandler/callbacks/shrink.test.tsx b/public/JobHandler/callbacks/shrink.test.tsx
@@ -41,10 +41,7 @@ describe("callbackForOpen spec", () => {
 
   it("callback when not complete", async () => {
     httpClientMock.fetch = getMockFn({
-      found: true,
-      _source: {
-        completed: false,
-      },
+      completed: false,
     });
     const result = await callbackForShrink(shrinkMetaData, {
       core,
@@ -54,10 +51,7 @@ describe("callbackForOpen spec", () => {
 
   it("callback when successfully complete", async () => {
     httpClientMock.fetch = getMockFn({
-      found: true,
-      _source: {
-        completed: true,
-      },
+      completed: true,
     });
     const result = await callbackForShrink(shrinkMetaData, {
       core,
@@ -69,20 +63,17 @@ describe("callbackForOpen spec", () => {
 
   it("callback when failed", async () => {
     httpClientMock.fetch = getMockFn({
-      found: true,
-      _source: {
-        completed: true,
-        error: {
-          reason: "reason",
-        },
-        failures: [
-          {
-            cause: {
-              reason: "cause",
-            },
-          },
-        ],
+      completed: true,
+      error: {
+        reason: "reason",
       },
+      failures: [
+        {
+          cause: {
+            reason: "cause",
+          },
+        },
+      ],
     });
     const result = await callbackForShrink(shrinkMetaData, {
       core,

diff --git a/public/JobHandler/callbacks/split.test.tsx b/public/JobHandler/callbacks/split.test.tsx
@@ -41,10 +41,7 @@ describe("callbackForOpen spec", () => {
 
   it("callback when not complete", async () => {
     httpClientMock.fetch = getMockFn({
-      found: true,
-      _source: {
-        completed: false,
-      },
+      completed: false,
     });
     const result = await callbackForSplit(splitMetaData, {
       core,
@@ -54,10 +51,7 @@ describe("callbackForOpen spec", () => {
 
   it("callback when successfully complete", async () => {
     httpClientMock.fetch = getMockFn({
-      found: true,
-      _source: {
-        completed: true,
-      },
+      completed: true,
     });
     const result = await callbackForSplit(splitMetaData, {
       core,
@@ -69,20 +63,17 @@ describe("callbackForOpen spec", () => {
 
   it("callback when failed", async () => {
     httpClientMock.fetch = getMockFn({
-      found: true,
-      _source: {
-        completed: true,
-        error: {
-          reason: "reason",
-        },
-        failures: [
-          {
-            cause: {
-              reason: "cause",
-            },
-          },
-        ],
+      completed: true,
+      error: {
+        reason: "reason",
       },
+      failures: [
+        {
+          cause: {
+            reason: "cause",
+          },
+        },
+      ],
     });
     const result = await callbackForSplit(splitMetaData, {
       core,