feat: Text encoding for BYHOUR, BYMINUTE, BYSECOND

JonasWanke · Apr 28, 2023 · 534cba0 · 534cba0
1 parent 8653e38
commit 534cba0
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 4 deletions.
diff --git a/lib/src/recurrence_rule.dart b/lib/src/recurrence_rule.dart
@@ -363,13 +363,10 @@ class RecurrenceRule {
 
   /// Whether this rule can be converted to a human-readable string.
   ///
-  /// - Unsupported attributes: [bySeconds], [byMinutes], [byHours]
   /// - Unsupported frequencies (if any by-parts are specified):
   ///   [Frequency.secondly], [Frequency.hourly], [Frequency.daily]
   bool get canFullyConvertToText {
-    if (hasBySeconds || hasByMinutes || hasByHours) {
-      return false;
-    } else if (frequency <= Frequency.daily) {
+    if (frequency <= Frequency.daily) {
       return true;
     } else if (hasBySetPositions ||
         hasBySeconds ||

diff --git a/test/codecs/text/daily_test.dart b/test/codecs/text/daily_test.dart
@@ -109,8 +109,18 @@ void main() {
   );
 
   // Time of day.
+
   testText(
     'Daily at 8:30 & 12:30',
     string: 'RRULE:FREQ=DAILY;BYHOUR=8,12;BYMINUTE=30',
   );
+
+  test('canConvertToText is true when byHours is specified', () async {
+    final rrule = RecurrenceRule(frequency: Frequency.daily, byHours: {8});
+    expect(rrule.canFullyConvertToText, isTrue);
+    expect(
+      rrule.toText(l10n: await RruleL10nEn.create()),
+      equals('Daily at 8'),
+    );
+  });
 }