fix: render weekdays as valid rrule strings

Previously, weekdays would've been rendered by their built-in name.
fmeringdal · May 3, 2024 · 20b06ed · 20b06ed
1 parent ec5a9b9
commit 20b06ed
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 2 deletions.
diff --git a/rrule/src/core/rrule.rs b/rrule/src/core/rrule.rs
@@ -656,7 +656,7 @@ impl<S> Display for RRule<S> {
 
         // Monday is the default, no need to expose it.
         if self.week_start != Weekday::Mon {
-            res.push(format!("WKST={}", &self.week_start));
+            res.push(format!("WKST={}", weekday_to_str(self.week_start)));
         }
 
         if !self.by_set_pos.is_empty() {

diff --git a/rrule/src/tests/regression.rs b/rrule/src/tests/regression.rs
@@ -1,7 +1,7 @@
 use chrono::Month;
 
 use crate::tests::common;
-use crate::{Frequency, RRule, RRuleSet};
+use crate::{Frequency, RRule, RRuleSet, Unvalidated};
 
 #[test]
 fn issue_34() {
@@ -60,3 +60,13 @@ fn issue_97() {
         "FREQ=YEARLY;BYMONTH=12;BYMONTHDAY=24,25,26"
     );
 }
+
+#[test]
+fn issue_111() {
+    let rrule = "RRULE:FREQ=WEEKLY;INTERVAL=1;BYDAY=TU;WKST=SU"
+        .parse::<RRule<Unvalidated>>();
+
+        // Convert to string...
+    let rrule_str = format!("{}", rrule.unwrap());
+    assert!(rrule_str.contains("WKST=SU"));
+}