Skip to content

Report template processing errors#110

Merged
glopesdev merged 4 commits intoharp-tech:mainfrom
glopesdev:report-generation-errors
Mar 13, 2026
Merged

Report template processing errors#110
glopesdev merged 4 commits intoharp-tech:mainfrom
glopesdev:report-generation-errors

Conversation

@glopesdev
Copy link
Contributor

@glopesdev glopesdev commented Mar 13, 2026

Surface any errors raised during the code generation process through a single Errors property in each generator API, and include the target file name of the template raising the code generation error for improved debugging experience.

Added assertions to ensure no errors are raised during code generation tests. In case the error collection is not empty, an exception is thrown concatenating all reported errors.

Finally, regression tests were added for expected generator errors. Infrastructure is provided for matching expected error messages against template processing errors and added tests for invalid IO pin configuration.

Any errors raised during the code generation process are surfaced
through a single collection.

Each code generation target is marked with its default target file name.
In case the error collection is not empty, an assertion error is raised
concatenating all reported errors.
Added infrastructure for matching expected error messages against
template processing errors.

Added tests for invalid IO pin configuration.
@glopesdev glopesdev requested a review from a team March 13, 2026 17:11
@glopesdev glopesdev added the feature New planned feature label Mar 13, 2026
@glopesdev glopesdev merged commit e9c24dc into harp-tech:main Mar 13, 2026
9 checks passed
@glopesdev glopesdev deleted the report-generation-errors branch March 13, 2026 17:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feature New planned feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant