[aws-codebuild] ReportGroup does not use provided name property

[BrianFarnhill]

When using the ReportGroup construct, if you specify the reportGroupName property the resulting CloudFormation does not output the Name property at all.

Reproduction Steps

Code to reproduce this:

import * as cdk from '@aws-cdk/core';
import * as codebuild from '@aws-cdk/aws-codebuild';

export class ReportGroupBugStack extends cdk.Stack {
  constructor(scope: cdk.Construct, id: string, props?: cdk.StackProps) {
    super(scope, id, props);

    new codebuild.ReportGroup(this, "DemoGroupName", {
      reportGroupName: "Example Group Name"
    })
  }
}

What did you expect to happen?

The resource I expect would look like this:

    "DemoGroupName5FAD2957": {
      "Type": "AWS::CodeBuild::ReportGroup",
      "Properties": {
        "Name": "Example Group Name",
        "ExportConfig": {
          "ExportConfigType": "NO_EXPORT"
        },
        "Type": "TEST"
      },
      "UpdateReplacePolicy": "Retain",
      "DeletionPolicy": "Retain",
      "Metadata": {
        "aws:cdk:path": "ReportGroupBugStack/DemoGroupName/Resource"
      }
    },

What actually happened?

This is the output I get (note the missing name property)

    "DemoGroupName5FAD2957": {
      "Type": "AWS::CodeBuild::ReportGroup",
      "Properties": {
        "ExportConfig": {
          "ExportConfigType": "NO_EXPORT"
        },
        "Type": "TEST"
      },
      "UpdateReplacePolicy": "Retain",
      "DeletionPolicy": "Retain",
      "Metadata": {
        "aws:cdk:path": "ReportGroupBugStack/DemoGroupName/Resource"
      }
    },

Environment

• CLI Version : 1.69.0 (build 2b474b9)
• Framework Version: 1.69.0
• Node.js Version: v12.18.4
• OS : Ubuntu 18.04 (Windows Subsystem For Linux)
• Language (Version): TypeScript (3.9.7)

---

This is 🐛 Bug Report

[skinny85]

Hey @BrianFarnhill ,

thanks for opening the issue. You are absolutely right that this is a bug.

Any chance you can open us a PR fixing it? It should be as simple as adding the line name: props.reportGroupName here (plus a unit test, of course 😉). Our contributing docs are here.

Thanks,
Adam

[github-actions]

⚠️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.