(CodeBuild Report Group): Defining the report type in CDK

When you create a new Report Group you should be able to select the report type (e,g,. Coverage)

There is no way to change the report type from Test to Coverage in props new ReportGroup(this.scope, 'test-coverage', {reportGroupName: 'test-coverage'});

Use Case

Proposed Solution

Other

• 👋 I may be able to implement this feature request
• ⚠️ This feature might incur a breaking change

---

This is a 🚀 Feature Request

[bilalq]

I don’t really feel strongly either way. The only reason I suggested using an adjustment on the existing class was that it mirrors the underlying CFN resource more that way.

I suppose a better implementation here would be to have ReportGroup be an interface that TestReportGroup and CoverageReportGroup could both implement, but I guess it’s too late to do that without impacting backwards compatibility.

Another option would be to make ReportGroup take a property like I suggested earlier and have two subclasses while preserving the existing behavior as the default. Thoughts?

Today, they don’t have different properties, but they do have different valid combinations with the buildspec property on the related CodeBuild project object. The reports/<report-group>/file-format property of buildspecs can have test reports can be one of CUCUMBERJSON | JUNITXML | NUNITXML | NUNIT3XML | TESTNGXML | VISUALSTUDIOTRX, while coverage reports must have it be one of CLOVERXML | COBERTURAXML | JACOCOXML | SIMPLECOV.

It’s worth calling out that buildspecs can be imported from files, so compile time guarantees can’t be made without additional runtime checking.

[github-actions[bot]]

⚠️COMMENT VISIBILITY WARNING⚠️

Comments on closed issues are hard for our team to see. If you need more assistance, please either tag a team member or open a new issue that references this one. If you wish to keep having a conversation with other community members under this issue feel free to do so.