Add EZLog utility tests #5

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Open

sensenmann wants to merge 1 commit into main from codex/add-tests-for-ezlog-functions

Owner

sensenmann commented Jan 12, 2026

Motivation

Prevent regressions in EZLog string utilities and filter logic by adding unit tests for core helper functions.
Verify EZLog::split, EZLog::trim, and EZLog::formatNumber handle whitespace, delimiters and thousand separators correctly.
Ensure _shouldLog filtering works with LoggingConfig::customLoggingElements and overrideLogAll behavior.

Description

Add test file test/test_log_utils/test_log_utils.cpp with Unity tests covering EZLog::split, EZLog::trim, EZLog::formatNumber and _shouldLog scenarios.
Expose internals for testing by including EZLog.h with #define private public in the test file.
Add a PlatformIO test environment [env:esp32_test] to platformio.ini configured with test_framework = unity, test_filter = test_log_utils and test_build_project_src = true.

Testing

Attempted to run the test suite with pio test -e esp32_test, but the command failed because the pio executable is not available in the current environment.


          Add EZLog utility and filter tests

0a4afac

sensenmann added the codex label

— with

ChatGPT Codex Connector

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels