Add support for Ceedling 1.0+ report_tests_log_factory plugin #146
Conversation
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
Add compatibility for Ceedling 1.0+ which uses the new report_tests_log_factory plugin with CppUnit format instead of the legacy xml_tests_report plugin. Changes: - Detect report_tests_log_factory plugin for Ceedling 1.0+ compatibility - Add isCeedling1Plus() helper for reliable version detection via plugin config - Support both legacy xml_tests_report and new report_tests_log_factory plugins - Add debugTestExecutablePath command for full executable path resolution - Update XML report path detection to handle both plugin configurations - Update sanity check to validate either plugin configuration - Add documentation for both Ceedling 0.31.x and 1.0+ configurations - Add unit tests for plugin detection and report path logic Addresses issues numaru#137 and numaru#145. Contributed by Generac
Filter out files from the build output directory when discovering tests. Ceedling's files:test command returns preprocessed files and test runners from the build directory, which caused duplicate entries in the test explorer tree. Uses the project's configured :build_root from project.yml to dynamically determine which paths to exclude.
NJGenerac
force-pushed
the
feature/ceedling-1.0-support
branch
from
a64affa to
7c4fc34
Compare
Owner
|
As you can see, I struggle to maintain this extension up to date. Thank you very much for this high quality contribution. I'm sure some people will be happy to see it work again. |
numaru
added a commit
that referenced
this pull request
Dec 17, 2025
Add support for Ceedling 1.0+ report_tests_log_factory plugin
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
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.
Summary
This PR adds compatibility for Ceedling 1.0+, which uses the new
report_tests_log_factoryplugin with CppUnit format instead of the legacyxml_tests_reportplugin.report_tests_log_factoryplugin for Ceedling 1.0+ compatibilityisCeedling1Plus()helper for reliable version detection via plugin configurationxml_tests_report(0.31.x) and newreport_tests_log_factory(1.0+) pluginsdebugTestExecutablePathcommand for full executable path resolutionAddresses issues #137 and #145.
Test Plan
report_tests_log_factoryplugin with CppUnit formatxml_tests_reportpluginContributed by Generac
I am going to admit that almost ALL of this was written by AI and Claude. HOWEVER - I have been using it for 3 weeks now. And I felt that if I didn't at least try to give it back considering the project has helped me so much., it wouldn't be right. I'm not trying to "pretend" to be anything I'm not. I am an embedded C programmer who knows very little JavaScript. But I updated this out of need. Not recognition. Thx!!! -- Nathan Jobe