[CORE-737] Do not refresh submissions history when viewing all submissions (#5444)

Author: samanehsan

src/pages/workspaces/workspace/SubmissionHistory.js

@@ -173,6 +173,9 @@ export const SubmissionHistory = _.flow(
 
   // Helpers
   const refresh = Utils.withBusyState(setLoading, async () => {
+    if (scheduledRefresh.current) {
+      clearTimeout(scheduledRefresh.current);
+    }
     try {
       let startDate;
       if (dateRange === '30') {
@@ -184,7 +187,6 @@ export const SubmissionHistory = _.flow(
 
       const params = {};
       if (startDate) params.startDate = startDate.toISOString().split('T')[0];
-
       const submissions = _.flow(
         _.orderBy('submissionDate', 'desc'),
         _.map((sub) => {
@@ -215,8 +217,7 @@ export const SubmissionHistory = _.flow(
         })
       )(await Workspaces(signal).workspace(namespace, name).listSubmissions(params));
       setSubmissions(submissions);
-
-      if (_.some(({ status }) => !isTerminal(status), submissions)) {
+      if (dateRange !== 'all' && _.some(({ status }) => !isTerminal(status), submissions)) {
         scheduledRefresh.current = setTimeout(refresh, 1000 * 60);
       }
     } catch (error) {
@@ -239,8 +240,6 @@ export const SubmissionHistory = _.flow(
 
   // Lifecycle
   useOnMount(() => {
-    refresh();
-
     return () => {
       if (scheduledRefresh.current) {
         clearTimeout(scheduledRefresh.current);

src/pages/workspaces/workspace/SubmissionHistory.test.tsx

@@ -274,4 +274,28 @@ describe('SubmissionHistory date range filter', () => {
     // Verify the URL parameter is set to default instead of invalid value
     expect(window.location.hash).toContain('?dateRange=30');
   });
+
+  test('does not schedule refresh when "All Submissions" is selected', async () => {
+    // Set the URL to "all" to ensure that the refresh logic is tested correctly
+    window.location.hash = '#workspaces/test-ns/test-ws/submission_history?dateRange=all';
+
+    jest.useFakeTimers();
+    const setTimeoutSpy = jest.spyOn(global, 'setTimeout');
+    const runningSubmission = { ...newerSubmission, status: 'Running' };
+    mockListSubmissions([runningSubmission]);
+    await act(async () => {
+      renderSubmissionHistory();
+    });
+
+    // Wait for UI update
+    await waitFor(() => {
+      expect(screen.getByText('Recent Submission')).toBeInTheDocument();
+    });
+
+    // Assert setTimeout was not called to schedule a refresh
+    expect(setTimeoutSpy).not.toHaveBeenCalledWith(expect.any(Function), 1000 * 60);
+
+    setTimeoutSpy.mockRestore();
+    jest.useRealTimers();
+  });
 });