Skip to content

Commit 0981352

Browse files
committed
Fix test: #7 broke test by re-adding DTEND to recurring events
- After #7, the length of recurring events is not made null - This means that all-day recurring events now have a DTEND property which is one day after the start day. This appears to be intentional: an all-day recurring event on day N has DTSTART=(N) and DTEND=(N+1). Furthermore, tests with Thunderbird and Google Calendar report that the new (post-#7) behavior is imported correctly. - However, #7 has broken the tests, undetected to me. This fixes the test by removing the check which is no longer needed.
1 parent 68ec495 commit 0981352

1 file changed

Lines changed: 2 additions & 1 deletion

File tree

tests/test_events.py

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -60,7 +60,8 @@ def test_rrule():
6060
)
6161
)
6262
event_str = event.serialize()
63-
assert 'DTEND' not in event_str
63+
# DTEND exists and is the next day, calendar tools import this
64+
# correctly as being a one-day event
6465
assert 'RRULE:FREQ=YEARLY;UNTIL=20300422T000000' in event_str
6566

6667

0 commit comments

Comments
 (0)