Skip to content

Conversation

qz267
Copy link
Contributor

@qz267 qz267 commented Oct 24, 2024

This is a follow up of #2660.

To add coverage for the function implementations in cmd.go file.

Initialized two test cases:

  • TestCmdExecution(): using cobra.ExecuteC() to simulate command execution without actually running, enabled adding more cases to cover the subcommands in the future.
  • TestCreateBlueprintDisplayMetadata(): cover the scenarios @tjy9206 found in the previous PR.

@qz267 qz267 requested review from a team and q2w as code owners October 24, 2024 18:40
Copy link
Member

@bharathkkb bharathkkb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the followup!

@bharathkkb bharathkkb enabled auto-merge (squash) October 29, 2024 04:54
@bharathkkb bharathkkb merged commit 00febbb into GoogleCloudPlatform:master Oct 29, 2024
13 checks passed
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.

3 participants