v2.1.x Chore - Refactor colored logging implementation in tests module

[coderabbitai]

Description

This issue tracks the refactoring of the colored logging implementation in tests/__init__.py.

Current Implementation

Currently, the logging setup code is defined directly in tests/__init__.py to ensure it's loaded very early during test module initialization (before loading multicast).

This includes:

• ANSIColors class for ANSI color codes
• Mappings for log levels to colors and logging module levels
• ColoredStreamHandler class extending logging.StreamHandler
• Root logger configuration

Proposed Refactoring

Move the implementation to a separate module (e.g., tests/logging_setup.py or tests/utils/logging.py) and import it early in tests/__init__.py to ensure it still takes effect at the appropriate time.

This refactoring would follow SOLID principles:

• Single Responsibility: Separate logging concerns from the test module initialization
• Open/Closed: Allow for future extensions to logging without modifying tests/__init__.py
• Interface Segregation: Provide a clean interface for the logging functionality

Origin

This refactoring was identified in PR #354: #354 (comment)

Implementation Notes

• Ensure the new module is imported early enough to maintain the current behavior
• Maintain compatibility with existing logging calls
• Add appropriate tests for the new module

[coderabbitai]

Testing comment creation for issue #355. Will add detailed improvements in a follow-up comment.

[coderabbitai]

Suggested code improvements for the refactored logging module

As discussed in PR #354, here are some improvements to implement when moving the logging code to its own module:

1. Add type hints to the emit method:

   • Change from: def emit(self, record)
   • To: def emit(self, record: logging.LogRecord) -> None

2. Simplify validation logic:

   • Replace two separate condition checks with a single combined check
   • Add the actual invalid value to the error message for better debugging

3. Use string literals instead of str() for empty strings:

   • Change from: colorPrefix = str(), endColor = str()
   • To: colorPrefix = "", endColor = ""

These improvements follow Python best practices and align with the SOLID principles mentioned in the issue description while improving code readability without changing functionality.