fix: Handle date ranges with same start and end date

Author: pulpdrew

packages/app/src/hooks/__tests__/useChartConfig.test.tsx

@@ -286,6 +286,32 @@ describe('useChartConfig', () => {
         },
       ]);
     });
+
+    it('returns a single window matching the input date range if the input date range is empty', () => {
+      expect(
+        getGranularityAlignedTimeWindows(
+          {
+            dateRange: [
+              new Date('2023-01-10 00:00:30'),
+              new Date('2023-01-10 00:00:30'),
+            ],
+            granularity: '1 minute',
+            timestampValueExpression: 'TimestampTime',
+          } as ChartConfigWithDateRange & { granularity: string },
+          [
+            60, // 1m
+            5 * 60, // 5m
+          ],
+        ),
+      ).toEqual([
+        {
+          dateRange: [
+            new Date('2023-01-10 00:00:30'),
+            new Date('2023-01-10 00:00:30'),
+          ],
+        },
+      ]);
+    });
   });
 
   describe('useQueriedChartConfig', () => {
@@ -936,5 +962,36 @@ describe('useChartConfig', () => {
       // The original query should still have its chunked data
       expect(result1.current.data?.rows).toBeGreaterThan(1);
     });
+
+    it('returns an empty result if the given date range is empty', async () => {
+      const config = createMockChartConfig({
+        dateRange: [
+          new Date('2025-10-01 00:00:00Z'),
+          new Date('2025-10-01 00:00:00Z'),
+        ],
+        granularity: '1 hour',
+      });
+
+      const mockResponse = createMockQueryResponse([]);
+      mockClickhouseClient.queryChartConfig.mockResolvedValue(mockResponse);
+
+      const { result } = renderHook(() => useQueriedChartConfig(config), {
+        wrapper,
+      });
+
+      await waitFor(() => expect(result.current.isSuccess).toBe(true));
+      await waitFor(() => expect(result.current.isFetching).toBe(false));
+
+      expect(mockClickhouseClient.queryChartConfig).toHaveBeenCalledTimes(1);
+
+      expect(result.current.data).toEqual({
+        data: [],
+        meta: mockResponse.meta,
+        rows: 0,
+        isComplete: true,
+      });
+      expect(result.current.isLoading).toBe(false);
+      expect(result.current.isPending).toBe(false);
+    });
   });
 });

packages/app/src/hooks/__tests__/useOffsetPaginatedQuery.test.tsx

@@ -309,6 +309,40 @@ describe('useOffsetPaginatedQuery', () => {
       // Should have more pages available due to large time range
       expect(result.current.hasNextPage).toBe(true);
     });
+
+    it('should handle a time range with the same start and end date by generating one window', async () => {
+      const config = createMockChartConfig({
+        dateRange: [
+          new Date('2024-01-01T00:00:00Z'),
+          new Date('2024-01-01T00:00:00Z'), // same start and end date
+        ] as [Date, Date],
+      });
+
+      // Mock the reader to return data for first window
+      mockReader.read
+        .mockResolvedValueOnce({
+          done: false,
+          value: [
+            { json: () => ['timestamp', 'message'] },
+            { json: () => ['DateTime', 'String'] },
+            { json: () => ['2024-01-01T01:00:00Z', 'test log 1'] },
+          ],
+        })
+        .mockResolvedValueOnce({ done: true });
+
+      const { result } = renderHook(() => useOffsetPaginatedQuery(config), {
+        wrapper,
+      });
+
+      await waitFor(() => expect(result.current.isLoading).toBe(false));
+
+      // Should have data from the first window
+      expect(result.current.data).toBeDefined();
+      expect(result.current.data?.window.windowIndex).toBe(0);
+
+      // Should have more pages available due to large time range
+      expect(result.current.hasNextPage).toBe(true);
+    });
   });
 
   describe('Pagination Within Time Windows', () => {

packages/app/src/utils/searchWindows.ts

@@ -18,6 +18,17 @@ export function generateTimeWindowsDescending(
   endDate: Date,
   windowDurationsSeconds: number[] = DEFAULT_TIME_WINDOWS_SECONDS,
 ): TimeWindow[] {
+  if (startDate.getTime() === endDate.getTime()) {
+    return [
+      {
+        startTime: startDate,
+        endTime: endDate,
+        windowIndex: 0,
+        direction: 'DESC',
+      },
+    ];
+  }
+
   const windows: TimeWindow[] = [];
   let currentEnd = new Date(endDate);
   let windowIndex = 0;
@@ -50,7 +61,18 @@ export function generateTimeWindowsAscending(
   startDate: Date,
   endDate: Date,
   windowDurationsSeconds: number[] = DEFAULT_TIME_WINDOWS_SECONDS,
-) {
+): TimeWindow[] {
+  if (startDate.getTime() === endDate.getTime()) {
+    return [
+      {
+        startTime: startDate,
+        endTime: endDate,
+        windowIndex: 0,
+        direction: 'ASC',
+      },
+    ];
+  }
+
   const windows: TimeWindow[] = [];
   let currentStart = new Date(startDate);
   let windowIndex = 0;