Return Promise in workflow history test (#808)

Return Promise in workflow history test to fix flakiness with rendering
Use waitFor instead of waitForElementToBeRemoved

Author: adhityamamallan

src/views/workflow-history/__tests__/workflow-history.test.tsx

@@ -8,6 +8,7 @@ import {
   render,
   screen,
   userEvent,
+  waitFor,
   waitForElementToBeRemoved,
 } from '@/test-utils/rtl';
 
@@ -168,7 +169,9 @@ describe('WorkflowHistory', () => {
       });
     });
 
-    await waitForElementToBeRemoved(loadingIndicator);
+    await waitFor(() => {
+      expect(loadingIndicator).not.toBeInTheDocument();
+    });
   });
 });
 
@@ -223,7 +226,7 @@ async function setup({
           httpResolver: async () => {
             requestIndex = requestIndex + 1;
             if (requestIndex > 0 && resolveLoadMoreManually) {
-              await new Promise((resolve, reject) => {
+              return await new Promise((resolve, reject) => {
                 requestResolver = (result: GetWorkflowHistoryResponse) =>
                   resolve(HttpResponse.json(result, { status: 200 }));
                 requestRejector = () =>