Commit 2ad0706
committed
[calendar] test: remove "Recurring event per timezone" test
Remove the "Recurring event per timezone" test that manipulated
Date.prototype.getTimezoneOffset to simulate 24 different timezones
for testing all-day recurring events.
Reasons for removal:
1. The test approach is incompatible with node-ical 0.22.0's
Intl-based timezone handling (which replaced moment-timezone).
Manipulating Date.prototype.getTimezoneOffset no longer affects
Intl.DateTimeFormat, which reads the system timezone directly.
2. node-ical 0.22.0 handles all-day events (VALUE=DATE) correctly
by preserving the calendar date without timezone conversions,
making cross-timezone testing unnecessary. The library includes
comprehensive tests for this behavior, particularly "keeps
whole-day recurrence across DST" in test/advanced.test.js.
3. The existing "Recurring event" test already verifies that
recurring events from the same ICS file are displayed correctly,
so a simplified version of this test is not necessary.
The old test attempted to work around timezone conversion issues
in node-ical 0.21.0 that are now properly resolved upstream.
Closes #39281 parent bc0d365 commit 2ad0706
2 files changed
+1
-16
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
18 | 18 | | |
19 | 19 | | |
20 | 20 | | |
| 21 | + | |
21 | 22 | | |
22 | 23 | | |
23 | 24 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
119 | 119 | | |
120 | 120 | | |
121 | 121 | | |
122 | | - | |
123 | | - | |
124 | | - | |
125 | | - | |
126 | | - | |
127 | | - | |
128 | | - | |
129 | | - | |
130 | | - | |
131 | | - | |
132 | | - | |
133 | | - | |
134 | | - | |
135 | | - | |
136 | | - | |
137 | | - | |
138 | 122 | | |
139 | 123 | | |
140 | 124 | | |
| |||
0 commit comments