mirror/kumomta
1
0
Fork 0
mirror of https://github.com/mailscope/kumomta.git synced 2026-09-08 11:38:56 +00:00
Code Issues Packages Projects Releases Wiki Activity

set_scheduling: raise error if start > end

Browse Source
This commit is contained in:
Wez Furlong
2025-03-15 07:18:12 -07:00
parent e655236fe0
commit b138235aa1
1 changed files with 14 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files
crates/message/src/scheduling.rs
+14
View File
@@ -173,6 +173,12 @@ impl<'de> Deserialize<'de> for Scheduling {
// and we must report it.
let restriction = match (self.days_of_week, self.timezone, self.start, self.end) {
(Some(days_of_week), Some(timezone), Some(start), Some(end)) => {
if start > end {
return Err(M::Error::custom(format!(
"'start' must be before 'end' and define a time window \
within a given day. start={start:?}, end={end:?}"
)));
}
Some(ScheduleRestriction {
days_of_week,
timezone,
@@ -546,4 +552,12 @@ mod test {
// Expected to round into Friday, later that week
k9::assert_equal!(adjusted.to_string(), "2023-03-31 09:00:00 MST");
}
#[test]
fn start_after_end() {
k9::snapshot!(serde_json::from_str::<Scheduling>(
r#"{"dow":"Mon,Tue,Wed,Thu,Fri,Sat,Sun","tz":"Etc/UTC","end":"20:57:49","start":"21:10:49", "first_attempt":"2025-03-14T21:10:49Z"}"#,
)
.unwrap_err(), r#"Error("'start' must be before 'end' and define a time window within a given day. start=21:10:49, end=20:57:49", line: 1, column: 128)"#);
}
}