admin/n8n-enterprise-unlocked
1
0
Fork 0
mirror of https://github.com/Abdulazizzn/n8n-enterprise-unlocked.git synced 2025-12-18 02:21:13 +00:00
Code Issues Packages Projects Releases Wiki Activity

fix(core): Handle infinite max history for insights date range (#14794)

Browse Source
This commit is contained in:
Guillaume Jacquart
2025-04-22 17:44:28 +02:00
committed by GitHub
parent f835c66d98
commit e83a64b84a
2 changed files with 20 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

16
packages/cli/src/modules/insights/__tests__/insights.service.test.ts
View File

@@ -505,6 +505,22 @@ describe('getAvailableDateRanges', () => {
);
});
test('returns correct ranges when hourly data is enabled and max history is unlimited', () => {
licenseMock.getInsightsMaxHistory.mockReturnValue(-1);
licenseMock.isInsightsHourlyDataEnabled.mockReturnValue(true);
const result = insightsService.getAvailableDateRanges();
expect(result).toEqual([
{ key: 'day', licensed: true, granularity: 'hour' },
{ key: 'week', licensed: true, granularity: 'day' },
{ key: '2weeks', licensed: true, granularity: 'day' },
{ key: 'month', licensed: true, granularity: 'day' },
{ key: 'quarter', licensed: true, granularity: 'week' },
{ key: 'year', licensed: true, granularity: 'week' },
]);
});
test('returns correct ranges when hourly data is enabled and max history is 365 days', () => {
licenseMock.getInsightsMaxHistory.mockReturnValue(365);
licenseMock.isInsightsHourlyDataEnabled.mockReturnValue(true);

5
packages/cli/src/modules/insights/insights.service.ts
View File

@@ -182,7 +182,10 @@ export class InsightsService {
}
getAvailableDateRanges(): InsightsDateRange[] {
const maxHistoryInDays = this.license.getInsightsMaxHistory();
const maxHistoryInDays =
this.license.getInsightsMaxHistory() === -1
? Number.MAX_SAFE_INTEGER
: this.license.getInsightsMaxHistory();
const isHourlyDateEnabled = this.license.isInsightsHourlyDataEnabled();
return [