diff --git a/src/notification.rs b/src/notification.rs index 264c93e..ee4a709 100644 --- a/src/notification.rs +++ b/src/notification.rs @@ -86,19 +86,6 @@ pub async fn notify_build( } } -fn should_notify_since_last_notification( - last_notification: jiff::Timestamp, - now: jiff::Timestamp, -) -> bool { - now.since(last_notification).is_ok_and(|diff| { - diff.total(( - jiff::Unit::Month, - &last_notification.to_zoned(jiff::tz::TimeZone::UTC), - )) - .is_ok_and(|diff_months| diff_months >= 1.0) - }) -} - pub async fn notify_build_failure( github_client: &GitHubClient, db: &Db, @@ -126,7 +113,9 @@ pub async fn notify_build_failure( if issue.last_update_date.is_none_or(|last_update_date| { jiff::Timestamp::from_millisecond(last_update_date).is_ok_and(|last_update_date| { - should_notify_since_last_notification(last_update_date, jiff::Timestamp::now()) + jiff::Timestamp::now() + .since(last_update_date) + .is_ok_and(|diff| diff.get_months() > 0) }) }) { info!( @@ -285,22 +274,3 @@ pub async fn notify_build_pass( Ok(()) } - -#[cfg(test)] -mod tests { - #[test] - fn should_notify_since_last_notification() { - assert!(super::should_notify_since_last_notification( - "2025-11-01T00:00:00Z".parse().unwrap(), - "2025-12-02T00:00:00Z".parse().unwrap() - )); - assert!(super::should_notify_since_last_notification( - "2025-09-01T00:00:00Z".parse().unwrap(), - "2025-12-02T00:00:00Z".parse().unwrap() - )); - assert!(!super::should_notify_since_last_notification( - "2025-11-02T00:00:00Z".parse().unwrap(), - "2025-12-01T00:00:00Z".parse().unwrap() - )); - } -}