-
Notifications
You must be signed in to change notification settings - Fork 288
Description
It would be nice to have access to the TestReport object within the post_run callback of class vunit.ui.VUnit, as the current Results object only allows for merging coverage reports. This would be useful for programmatically constructing different reports, rather than re-parsing the XML which may or may not have been included in the CLI arguments.
Currently, both the TestReport object for printing to the screen and the Results object for the callback are both constructed in the _main_run function of ui.py, before post_run is invoked with just the Results object.
As Results seems to only be a wrapper to pass to the callback, I propose extending this object to take a second constructor argument of the existing TestReport object within _main_run.
This should not break any existing interfaces, and provided this is an acceptable approach to the project maintainers I am happy to submit a pull request.