Skip to content

Commit 861d983

Browse files
committed
add more logs
1 parent 8c7b9e5 commit 861d983

File tree

2 files changed

+14
-1
lines changed

2 files changed

+14
-1
lines changed

test/pytest.ini

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -12,3 +12,7 @@ timeout = 900
1212

1313
# Disable legacy paths.
1414
addopts = -p no:legacypath
15+
16+
log_level = DEBUG
17+
log_format = %(asctime)s [%(levelname)s] %(name)s/%(funcName)s: %(message)s
18+
log_date_format = %Y-%m-%d %H:%M:%S.%f

test/utils.py

Lines changed: 10 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -648,29 +648,38 @@ def __del__(self):
648648

649649
class Timeout(Timer):
650650
__timeout: float
651+
__is_running: bool
651652

652653
def __init__(self, name, timeout, callback, *args) -> None:
653654
self.__timeout = timeout
655+
self.__is_running = False
654656
if self.is_timeout:
655657
super().__init__(timeout, callback, *args)
656658
self.setName(name)
657659

658660
@property
659661
def is_timeout(self) -> bool:
660-
return self.__timeout > 0
662+
return self.__timeout is not None and self.__timeout > 0
661663

662664
def __enter__(self):
665+
assert not self.__is_running, "This timer already in use"
663666
if self.is_timeout:
667+
self.__is_running = True
668+
logging.getLogger(__name__).debug(
669+
f"Start timer='{self.name}' for {self.__timeout} seconds {self.args}"
670+
)
664671
self.start()
665672
return self
666673

667674
def __exit__(self, unused_exc_type, unused_value, unused_traceback) -> None:
668675
_ = (unused_exc_type, unused_value, unused_traceback)
669676
if self.is_timeout:
677+
logging.getLogger(__name__).debug(f"Stop(1) timer='{self.name}' {self.args}")
670678
self.cancel()
671679

672680
def __del__(self) -> None:
673681
if self.is_timeout:
682+
logging.getLogger(__name__).debug(f"Stop(2) timer='{self.name}' {self.args}")
674683
self.cancel()
675684

676685

0 commit comments

Comments
 (0)