V5.0.1

Author: ddc

.github/workflows/workflow.yml

@@ -37,13 +37,20 @@ jobs:
         run: poetry run poe tests
         shell: bash
 
-      - name: Upload coverage reports to Codecov
+      - name: Upload coverage to Codecov
         if: matrix.python-version == '3.13' && matrix.os == 'ubuntu-latest'
         uses: codecov/codecov-action@v5
         with:
             token: ${{ secrets.CODECOV_TOKEN }}
             slug: ddc/pythonLogs
 
+      - name: Upload test results to Codecov
+        if: matrix.python-version == '3.13' && matrix.os == 'ubuntu-latest'
+        uses: codecov/test-results-action@v1
+        with:
+          token: ${{ secrets.CODECOV_TOKEN }}
+          slug: ddc/pythonLogs
+
   build:
     name: Build for Python ${{ matrix.python-version }} on ${{ matrix.os }}
     runs-on: ${{ matrix.os }}

.gitignore

@@ -162,3 +162,4 @@ cython_debug/
 /profile_fixed.prof
 /profile_output.prof
 /profile_pytest.prof
+/junit.xml

poetry.lock

@@ -43,16 +43,14 @@ pydantic-settings = "^2.10.1"
 python-dotenv = "^1.1.1"
 
 [tool.poetry.group.test.dependencies]
-coverage = "^7.10.0"
 poethepoet = "^0.36.0"
 psutil = "^7.0.0"
 pytest = "^8.4.1"
+pytest-cov = "^6.2.1"
 
 [tool.poe.tasks]
-_test = "coverage run -m pytest -v"
-_coverage_report = "coverage report"
-_coverage_xml = "coverage xml"
-tests = ["_test", "_coverage_report", "_coverage_xml"]
+_test = "python -m pytest -v --cov=pythonLogs --cov-report=term --cov-report=xml --junitxml=junit.xml -o junit_family=legacy"
+tests = ["_test"]
 test = ["tests"]
 
 [tool.black]
@@ -66,6 +64,7 @@ markers = [
 
 [tool.coverage.run]
 omit = [
+    "build.py",
     "tests/*",
 ]
 

pyproject.toml

@@ -153,7 +153,7 @@ def is_older_than_x_days(path: str, days: int) -> bool:
 
 # Cache stderr timezone for better performance
 @lru_cache(maxsize=1)
-def _get_stderr_timezone():
+def get_stderr_timezone():
     timezone_name = os.getenv("LOG_TIMEZONE", "UTC")
     if timezone_name.lower() == "localtime":
         return None  # Use system local timezone
@@ -167,7 +167,7 @@ def _get_stderr_timezone():
 def write_stderr(msg: str) -> None:
     """Write msg to stderr with optimized timezone handling"""
     try:
-        tz = _get_stderr_timezone()
+        tz = get_stderr_timezone()
         if tz is None:
             # Use local timezone
             dt = datetime.now()
@@ -206,7 +206,7 @@ def get_log_path(directory: str, filename: str) -> str:
 
 
 @lru_cache(maxsize=32)
-def _get_timezone_offset(timezone_: str) -> str:
+def get_timezone_offset(timezone_: str) -> str:
     """Cache timezone offset calculation with fallback for missing timezone data"""
     if timezone_.lower() == "localtime":
         return time.strftime("%z")
@@ -230,7 +230,7 @@ def get_format(show_location: bool, name: str, timezone_: str) -> str:
     if show_location:
         _debug_fmt = "[%(filename)s:%(funcName)s:%(lineno)d]:"
 
-    utc_offset = _get_timezone_offset(timezone_)
+    utc_offset = get_timezone_offset(timezone_)
     return f"[%(asctime)s.%(msecs)03d{utc_offset}]:[%(levelname)s]:{_logger_name}{_debug_fmt}%(message)s"
 
 

pythonLogs/log_utils.py

@@ -175,12 +175,12 @@ def optimize_lru_cache_sizes() -> None:
     log_utils.get_timezone_function = lru_cache(maxsize=8)(log_utils.get_timezone_function.__wrapped__)
 
     # Clear and recreate timezone offset cache with smaller size  
-    log_utils._get_timezone_offset.cache_clear()
-    log_utils._get_timezone_offset = lru_cache(maxsize=8)(log_utils._get_timezone_offset.__wrapped__)
+    log_utils.get_timezone_offset.cache_clear()
+    log_utils.get_timezone_offset = lru_cache(maxsize=8)(log_utils.get_timezone_offset.__wrapped__)
 
     # Clear and recreate stderr timezone cache with smaller size
-    log_utils._get_stderr_timezone.cache_clear()
-    log_utils._get_stderr_timezone = lru_cache(maxsize=4)(log_utils._get_stderr_timezone.__wrapped__)
+    log_utils.get_stderr_timezone.cache_clear()
+    log_utils.get_stderr_timezone = lru_cache(maxsize=4)(log_utils.get_stderr_timezone.__wrapped__)
 
 
 def force_garbage_collection() -> Dict[str, int]:

pythonLogs/memory_utils.py

@@ -62,6 +62,7 @@ def test_factory_registry_cleanup(self):
 
         # Verify handlers were closed and removed
         assert len(logger.handlers) == 0
+        assert initial_handler_count > 0  # Ensure we actually had handlers to clean up
         assert len(LoggerFactory._logger_registry) == 0
 
     def test_shutdown_specific_logger(self):
@@ -236,6 +237,8 @@ def test_memory_usage_after_cleanup(self):
 
         # Logger should still exist due to registry
         assert logger_weakref() is not None
+        # Handlers should also still exist
+        assert all(ref() is not None for ref in handler_weakrefs)
 
         # Clear registry
         clear_logger_registry()
@@ -244,7 +247,7 @@ def test_memory_usage_after_cleanup(self):
         gc.collect()
 
         # Logger should be garbage collected
-        # Note: This test might be flaky depending on Python's garbage collector
+        # Note: This test might be flaky depending on Python's garbage collector,
         # but it helps verify we're not holding unnecessary references
         print(f"Logger weakref after cleanup: {logger_weakref()}")