Skip to content

Commit b23897c

Browse files
authored
fix: remove duplicate addDays utility (#2059)
1 parent 8976969 commit b23897c

File tree

2 files changed

+0
-60
lines changed

2 files changed

+0
-60
lines changed

app/utils/charts.ts

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -32,12 +32,6 @@ export function buildWeeklyEvolutionFromDaily(
3232
})
3333
}
3434

35-
export function addDays(date: Date, days: number): Date {
36-
const d = new Date(date)
37-
d.setUTCDate(d.getUTCDate() + days)
38-
return d
39-
}
40-
4135
// Statistics & Interpretation utilities
4236

4337
export function clamp(value: number, minValue: number, maxValue: number): number {

test/unit/app/utils/charts.spec.ts

Lines changed: 0 additions & 54 deletions
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,6 @@ import {
33
sum,
44
chunkIntoWeeks,
55
buildWeeklyEvolutionFromDaily,
6-
addDays,
76
clamp,
87
quantile,
98
winsorize,
@@ -325,59 +324,6 @@ describe('buildWeeklyEvolutionFromDaily', () => {
325324
})
326325
})
327326

328-
describe('addDays', () => {
329-
it('returns a new Date instance (does not mutate original)', () => {
330-
const original = new Date('2028-01-01T00:00:00Z')
331-
const result = addDays(original, 5)
332-
333-
expect(result).not.toBe(original)
334-
expect(original.toISOString()).toBe('2028-01-01T00:00:00.000Z')
335-
})
336-
337-
it('adds positive days correctly', () => {
338-
const date = new Date('2028-01-01T00:00:00Z')
339-
const result = addDays(date, 10)
340-
341-
expect(result.toISOString()).toBe('2028-01-11T00:00:00.000Z')
342-
})
343-
344-
it('subtracts days when negative value is provided', () => {
345-
const date = new Date('2028-01-10T00:00:00Z')
346-
const result = addDays(date, -5)
347-
348-
expect(result.toISOString()).toBe('2028-01-05T00:00:00.000Z')
349-
})
350-
351-
it('handles month overflow correctly', () => {
352-
const date = new Date('2028-01-28T00:00:00Z')
353-
const result = addDays(date, 5)
354-
355-
expect(result.toISOString()).toBe('2028-02-02T00:00:00.000Z')
356-
})
357-
358-
it('handles year overflow correctly', () => {
359-
const date = new Date('2027-12-29T00:00:00Z')
360-
const result = addDays(date, 5)
361-
362-
expect(result.toISOString()).toBe('2028-01-03T00:00:00.000Z')
363-
})
364-
365-
it('handles leap year correctly', () => {
366-
const date = new Date('2028-02-27T00:00:00Z') // 2028 is leap year
367-
const result = addDays(date, 2)
368-
369-
expect(result.toISOString()).toBe('2028-02-29T00:00:00.000Z')
370-
})
371-
372-
it('keeps UTC behavior consistent (no timezone drift)', () => {
373-
const date = new Date('2028-03-01T00:00:00Z')
374-
const result = addDays(date, 1)
375-
376-
expect(result.getUTCHours()).toBe(0)
377-
expect(result.toISOString()).toBe('2028-03-02T00:00:00.000Z')
378-
})
379-
})
380-
381327
describe('clamp', () => {
382328
it('returns the value when it is within bounds', () => {
383329
expect(clamp(5, 0, 10)).toBe(5)

0 commit comments

Comments
 (0)