fix: Handle undefined histories in test details

dashboard/src/api/testDetails.ts

@@ -1,5 +1,5 @@
 import type { UseQueryResult } from '@tanstack/react-query';
-import { useQuery } from '@tanstack/react-query';
+import { useQuery, useQueryClient } from '@tanstack/react-query';
 
 import type {
   TTestDetails,
@@ -9,6 +9,7 @@ import type {
 
 import type { TIssue } from '@/types/issues';
 import type { ApiUseQueryOptions } from '@/types/api';
+import { isBadRequestError, DEFAULT_QUERY_RETRY_COUNT } from '@/utils/query';
 
 import { RequestData } from './commonRequest';
 
@@ -64,9 +65,30 @@ const fetchTestStatusHistory = async (
 export const useTestStatusHistory = (
   params?: TestStatusHistoryParams,
 ): UseQueryResult<TestStatusHistory> => {
+  const queryClient = useQueryClient();
+
   return useQuery({
     queryKey: ['testStatusHistoryData', params],
     queryFn: () => fetchTestStatusHistory(params),
+    retry: (failureCount, error) => {
+      if (isBadRequestError(error)) {
+        return false;
+      }
+
+      const globalRetry =
+        queryClient.getDefaultOptions().queries?.retry ??
+        DEFAULT_QUERY_RETRY_COUNT;
+
+      if (typeof globalRetry === 'function') {
+        return globalRetry(failureCount, error);
+      }
+
+      if (typeof globalRetry === 'number') {
+        return failureCount < globalRetry;
+      }
+
+      return globalRetry;
+    },
     enabled: params !== undefined && Object.keys(params).length > 0,
   });
 };

dashboard/src/components/TestDetails/TestDetails.tsx

@@ -67,6 +67,7 @@ import { processLogData } from '@/hooks/useLogData';
 
 import { dateObjectToTimestampInSeconds, daysToSeconds } from '@/utils/date';
 import { REDUCED_TIME_SEARCH } from '@/utils/constants/general';
+import { isBadRequestError } from '@/utils/query';
 
 import { MemoizedKcidevFooter } from '@/components/Footer/KcidevFooter';
 
@@ -223,9 +224,8 @@ const TestDetailsSections = ({
   }, [statusHistory?.status_history, test.id]);
 
   const regressionSection: ISection | undefined = useMemo(() => {
-    if (statusHistoryStatus === 'error') {
-      return;
-    }
+    const cannotFetchHistory =
+      statusHistoryStatus === 'error' && isBadRequestError(statusHistoryError);
 
     const regressionTypeTooltip: string | null = ((): string | null => {
       switch (statusHistory?.regression_type) {
@@ -294,12 +294,18 @@ const TestDetailsSections = ({
                   status={statusHistoryStatus}
                   data={statusHistory}
                   customError={
-                    <MemoizedSectionError
-                      isLoading={statusHistoryStatus === 'pending'}
-                      errorMessage={statusHistoryError?.message}
-                      emptyLabel="global.error"
-                      variant="warning"
-                    />
+                    cannotFetchHistory ? (
+                      <div className="text-weak-gray flex flex-col items-center py-6 text-2xl font-semibold">
+                        <FormattedMessage id="testDetails.cannotFetchHistory" />
+                      </div>
+                    ) : (
+                      <MemoizedSectionError
+                        isLoading={statusHistoryStatus === 'pending'}
+                        errorMessage={statusHistoryError?.message}
+                        emptyLabel="global.error"
+                        variant="warning"
+                      />
+                    )
                   }
                 >
                   <div className="flex items-center">{regressionData}</div>
@@ -314,7 +320,7 @@ const TestDetailsSections = ({
     formatMessage,
     regressionData,
     statusHistory,
-    statusHistoryError?.message,
+    statusHistoryError,
     statusHistoryStatus,
   ]);
 

dashboard/src/locales/messages/index.ts

@@ -336,6 +336,7 @@ export const messages = {
       'Inconclusive - test concluded with inconclusive results such as infrastructure errors.{br}{br}' +
       'Inconclusive groups tests with ERROR, MISS, SKIP, DONE, and unknown statuses defined by KCIDB.',
     'testDetails.buildInfo': 'Build Info',
+    'testDetails.cannotFetchHistory': 'No tracking information available',
     'testDetails.notFound': 'Test not found',
     'testDetails.regressionTooltip.fixed':
       'Test was failing but passed in the last iterations',

dashboard/src/utils/query.ts

@@ -1,3 +1,7 @@
+import { AxiosError, HttpStatusCode } from 'axios';
+
+export const DEFAULT_QUERY_RETRY_COUNT = 3;
+
 export const retryHandler = (retryCount: number | boolean = 3) => {
   return (failureCount: number, error: Error): boolean => {
     const splittedError = error.message.split(':');
@@ -19,3 +23,16 @@ export const retryHandler = (retryCount: number | boolean = 3) => {
     return true;
   };
 };
+
+export const isBadRequestError = (error: unknown): boolean => {
+  if (error instanceof AxiosError) {
+    return error.response?.status === HttpStatusCode.BadRequest;
+  }
+
+  if (error instanceof Error) {
+    const statusCode = Number(error.message.split(':')[0]);
+    return statusCode === HttpStatusCode.BadRequest;
+  }
+
+  return false;
+};