Skip to content

Conversation

eyck
Copy link
Contributor

@eyck eyck commented Jul 13, 2022

This PR adds 2 extensions thus addressing #16 :

  • import an UCIS XML including the per_instance and merge_instances property
  • adds update_coverage() method to the CoverageReportBuilder and modif report accordingly.
    This method creates the union-merged coverage for instances having the merge_instances property set to true. The reporting uses the merge_instances and per_instance properties to calculate the coverage for reporting.

The implementation shall be seen as proof-of-concept. Naming can be improved and maybe it is not the right location. Maybe it is more a database functtionality rather than a report builder function.

eyck and others added 11 commits July 7, 2022 18:49
Signed-off-by: Matthew Ballance <matt.ballance@gmail.com>
Signed-off-by: Matthew Ballance <matt.ballance@gmail.com>
Signed-off-by: Matthew Ballance <matt.ballance@gmail.com>
Signed-off-by: Matthew Ballance <matt.ballance@gmail.com>
Signed-off-by: Matthew Ballance <matt.ballance@gmail.com>
Signed-off-by: Matthew Ballance <matt.ballance@gmail.com>
@eyck
Copy link
Contributor Author

eyck commented Jul 13, 2022

If this is acceptable I can work on fixing and extending the CI tests

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants