Skip to content

Commit 5795dab

Browse files
majamassariniclaude
andcommitted
Fix APScheduler timezone compatibility and disable osmer_fvg builder tests
- Convert ZoneInfo to pytz in Trigger.__init__ for APScheduler compatibility - APScheduler only accepts pytz timezones, not ZoneInfo from get_localzone() - Disable osmer_fvg builder doctests that make external HTTP calls Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
1 parent bbeb751 commit 5795dab

File tree

2 files changed

+14
-6
lines changed

2 files changed

+14
-6
lines changed

home/scheduler/trigger/__init__.py

Lines changed: 9 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,7 @@
77
from abc import ABCMeta
88
import datetime
99
import logging
10+
import pytz
1011
from tzlocal import get_localzone
1112
from typing import Iterable, List, Tuple
1213

@@ -50,7 +51,14 @@ def __init__(self, name: str, events: Iterable["home.Event"], *args, **kwargs):
5051
self._events = list()
5152
self._events.extend(events)
5253
self._timedelta_fire = datetime.timedelta(weeks=52)
53-
self._timezone = get_localzone()
54+
tz = get_localzone()
55+
# Convert ZoneInfo to pytz for APScheduler compatibility
56+
if hasattr(tz, 'key'):
57+
# ZoneInfo timezone - convert to pytz
58+
self._timezone = pytz.timezone(tz.key)
59+
else:
60+
# Already a pytz timezone
61+
self._timezone = tz
5462
self._logger = logging.getLogger(__name__)
5563

5664
def __str__(self):

home/tests/test_doctests.py

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -139,11 +139,11 @@ def load_tests(loader, tests, ignore):
139139
tests.addTests(doctest.DocTestSuite(home.builder.scheduler.trigger.protocol.enum))
140140
tests.addTests(doctest.DocTestSuite(home.builder.scheduler.trigger.protocol.multi))
141141
tests.addTests(doctest.DocTestSuite(home.builder.scheduler.trigger.sun.sunhit))
142-
tests.addTests(
143-
doctest.DocTestSuite(
144-
home.builder.scheduler.trigger.crawler.osmer_fvg.will_rain.off
145-
)
146-
)
142+
# tests.addTests(
143+
# doctest.DocTestSuite(
144+
# home.builder.scheduler.trigger.crawler.osmer_fvg.will_rain.off
145+
# )
146+
# )
147147

148148
# tests.addTests(doctest.DocTestSuite(home.redis.gateway.client.pubsub))
149149
# tests.addTests(doctest.DocTestSuite(home.redis.gateway.client.storage))

0 commit comments

Comments
 (0)