chore: remove temp comment - getting these WARN logs out of the unit tests would require properly mocking out the firebase, which is messy and likely not worth investing effort into until we're on a more recent firebase version.

Mephistic · Mephistic · commit 69fb65dcbc97 · 2025-03-08T12:38:29.000-05:00
diff --git a/functions/src/notifications/helpers.ts b/functions/src/notifications/helpers.ts
@@ -32,7 +32,6 @@ export const getNextDigestAt = (notificationFrequency: Frequency) => {
   return nextDigestAt
 }
 
-// TODO: Get the stupid Firebase WARN logs out of the unit tests
 export const getNotificationStartDate = (
   notificationFrequency: Frequency,
   now: Timestamp

Original file line number	Diff line number	Diff line change
`@@ -32,7 +32,6 @@ export const getNextDigestAt = (notificationFrequency: Frequency) => {`
`32`	`32`	`return nextDigestAt`
`33`	`33`	`}`
`34`	`34`
`35`		`-// TODO: Get the stupid Firebase WARN logs out of the unit tests`
`36`	`35`	`export const getNotificationStartDate = (`
`37`	`36`	`notificationFrequency: Frequency,`
`38`	`37`	`now: Timestamp`