Uh oh!
There was an error while loading. Please reload this page.
Generate acceptance tests for help output for all CLI commands - #2260
Closed
andrewnester wants to merge 1 commit into
Closed
Generate acceptance tests for help output for all CLI commands#2260andrewnester wants to merge 1 commit into
andrewnester wants to merge 1 commit into
Conversation
andrewnester
requested review from
denik, pietern and shreyas-goenka
as code ownersJanuary 29, 2025 11:58
andrewnestertemporarily deployed
to
test-trigger-is
January 29, 2025 11:59 — with 
GitHub Actions
Inactive
pietern
reviewed
Jan 29, 2025
| "services": { | ||
| ".codegen/service.go.tmpl": "cmd/{{if .IsAccounts}}account{{else}}workspace{{end}}/{{(.TrimPrefix \"account\").KebabName}}/{{(.TrimPrefix \"account\").KebabName}}.go" | ||
| ".codegen/service.go.tmpl": "cmd/{{if .IsAccounts}}account{{else}}workspace{{end}}/{{(.TrimPrefix \"account\").KebabName}}/{{(.TrimPrefix \"account\").KebabName}}.go", | ||
| ".codegen/cmd_script.tmpl": "acceptance/help/cmd/{{if .IsAccounts}}account{{else}}workspace{{end}}/{{(.TrimPrefix \"account\").KebabName}}/{{(.TrimPrefix \"account\").KebabName}}/script" |
Contributor
There was a problem hiding this comment.
This repeats the KebabName twice. Intentional or can we use 1x?
| "go" | ||
| ], | ||
| "post_generate": [ | ||
| "go test ./acceptance -v -update -run 'TestAccept/help/cmd' ", |
Contributor
There was a problem hiding this comment.
Trailing whitespace + quotes are not needed.
andrewnester
marked this pull request as draft
February 25, 2025 19:58
Contributor
This PR has not received an update in a while. If you want to keep this PR open, please leave a comment below or push a new commit and auto-close will be canceled. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Changes
This PR helps to make sure that there is no unexpected issues in help output.
It automatically generates scripts used for acceptance test for all CLI commands with
--helpflag.If there's any changes to the output, the test output will be autogenerated and can be reviewed later as part of CLI regeneration PR