Multiple preparations for extended testing#285
Merged
Conversation
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Contributor
|
Looks like your PR has code that needs to be changed in order to meet our coding standards!
|
Member
Author
|
This PR has to be merged AFTER #284 |
There was a problem hiding this comment.
Pull request overview
This PR prepares the repository for more extensive host-based testing by tightening the native PlatformIO test environment, adding coverage tooling/config, and documenting both the workflow and a longer-term refactor/testing plan.
Changes:
- Hardened the PlatformIO
nativeunit-test environment with stricter warnings and coverage instrumentation. - Added gcovr configuration and helper scripts to generate text and HTML coverage reports, plus README instructions.
- Added an architecture/refactor plan document to guide future work, and small housekeeping updates (.gitignore, unit test main signature).
Reviewed changes
Copilot reviewed 7 out of 9 changed files in this pull request and generated 3 comments.
Show a summary per file
| File | Description |
|---|---|
| unit_tests/test_common/test_MappedDict.cpp | Removes unused argc/argv from the host main() entrypoint for stricter warning builds. |
| specs/plan.md | Adds a detailed clean-architecture refactor & testing roadmap document. |
| scripts/native_coverage.py | Ensures --coverage is applied at link time and disables SCons cache to keep gcov data valid. |
| scripts/coverage_html.sh | Adds a helper script to generate an HTML coverage report via gcovr. |
| README.md | Documents how to run native unit tests and generate coverage reports. |
| platformio.ini | Adds stricter warnings and coverage flags to [env:native] and wires in the coverage extra-script. |
| gcovr.cfg | Adds a gcovr config for consistent coverage output/filtering. |
| .gitignore | Ignores macOS .DS_Store files. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
ClutchplateDude
requested changes
May 12, 2026
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
…ement - Introduced .clang-format-ignore files to exclude specific directories and files from formatting. - Added a new script (format.sh) to automate code formatting using clang-format. - Updated platformio.ini to include environment-specific build flags. - Cleaned up code formatting in Mount.cpp, Mount.hpp, b_setup.hpp, fonts128x64.h, and other files for consistency.
…orkflow (honoring .clang-format-ignore)
…ge.py for improved coverage reporting
…p-python, add coverage summary publishing step
…ndency updates, and unit testing improvements
ClutchplateDude
requested changes
May 14, 2026
ClutchplateDude
approved these changes
May 14, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.