renamed functions to NewCmd

Author: Devansh Thakur

internal/cmd/beta/intake/runner/create/create.go

@@ -39,7 +39,7 @@ type inputModel struct {
 	Labels             *map[string]string
 }
 
-func NewCreateCmd(p *params.CmdParams) *cobra.Command {
+func NewCmd(p *params.CmdParams) *cobra.Command {
 	cmd := &cobra.Command{
 		Use:   "create",
 		Short: "Creates a new Intake Runner",

internal/cmd/beta/intake/runner/create/create_test.go

@@ -176,7 +176,7 @@ func TestParseInput(t *testing.T) {
 			parseInputWrapper := func(p *print.Printer, cmd *cobra.Command, _ []string) (*inputModel, error) {
 				return parseInput(p, cmd)
 			}
-			testutils.TestParseInput(t, NewCreateCmd, parseInputWrapper, tt.expectedModel, tt.argValues, tt.flagValues, tt.isValid)
+			testutils.TestParseInput(t, NewCmd, parseInputWrapper, tt.expectedModel, tt.argValues, tt.flagValues, tt.isValid)
 		})
 	}
 }
@@ -282,7 +282,7 @@ func TestOutputResult(t *testing.T) {
 		},
 	}
 	p := print.NewPrinter()
-	p.Cmd = NewCreateCmd(&params.CmdParams{Printer: p})
+	p.Cmd = NewCmd(&params.CmdParams{Printer: p})
 	for _, tt := range tests {
 		t.Run(tt.name, func(t *testing.T) {
 			if err := outputResult(p, tt.args.model, tt.args.projectLabel, tt.args.resp); (err != nil) != tt.wantErr {

internal/cmd/beta/intake/runner/delete/delete.go

@@ -28,8 +28,8 @@ type inputModel struct {
 	RunnerId string
 }
 
-// NewDeleteCmd creates a new cobra command for deleting an Intake Runner
-func NewDeleteCmd(p *params.CmdParams) *cobra.Command {
+// NewCmd creates a new cobra command for deleting an Intake Runner
+func NewCmd(p *params.CmdParams) *cobra.Command {
 	cmd := &cobra.Command{
 		Use:   fmt.Sprintf("delete %s", runnerIdArg),
 		Short: "Deletes an Intake Runner",

internal/cmd/beta/intake/runner/delete/delete_test.go

@@ -121,7 +121,7 @@ func TestParseInput(t *testing.T) {
 
 	for _, tt := range tests {
 		t.Run(tt.description, func(t *testing.T) {
-			testutils.TestParseInput(t, NewDeleteCmd, parseInput, tt.expectedModel, tt.argValues, tt.flagValues, tt.isValid)
+			testutils.TestParseInput(t, NewCmd, parseInput, tt.expectedModel, tt.argValues, tt.flagValues, tt.isValid)
 		})
 	}
 }

internal/cmd/beta/intake/runner/describe/describe.go

@@ -27,7 +27,7 @@ type inputModel struct {
 	RunnerId string
 }
 
-func NewDescribeCmd(p *params.CmdParams) *cobra.Command {
+func NewCmd(p *params.CmdParams) *cobra.Command {
 	cmd := &cobra.Command{
 		Use:   fmt.Sprintf("describe %s", runnerIdArg),
 		Short: "Shows details of an Intake Runner",

internal/cmd/beta/intake/runner/describe/describe_test.go

@@ -116,7 +116,7 @@ func TestParseInput(t *testing.T) {
 
 	for _, tt := range tests {
 		t.Run(tt.description, func(t *testing.T) {
-			testutils.TestParseInput(t, NewDescribeCmd, parseInput, tt.expectedModel, tt.argValues, tt.flagValues, tt.isValid)
+			testutils.TestParseInput(t, NewCmd, parseInput, tt.expectedModel, tt.argValues, tt.flagValues, tt.isValid)
 		})
 	}
 }
@@ -181,7 +181,7 @@ func TestOutputResult(t *testing.T) {
 		},
 	}
 	p := print.NewPrinter()
-	p.Cmd = NewDescribeCmd(&params.CmdParams{Printer: p})
+	p.Cmd = NewCmd(&params.CmdParams{Printer: p})
 	for _, tt := range tests {
 		t.Run(tt.name, func(t *testing.T) {
 			if err := outputResult(p, tt.args.outputFormat, tt.args.runner); (err != nil) != tt.wantErr {

internal/cmd/beta/intake/runner/list/list.go

@@ -28,8 +28,8 @@ type inputModel struct {
 	Limit *int64
 }
 
-// NewListCmd creates a new cobra command for listing Intake Runners
-func NewListCmd(p *params.CmdParams) *cobra.Command {
+// NewCmd creates a new cobra command for listing Intake Runners
+func NewCmd(p *params.CmdParams) *cobra.Command {
 	cmd := &cobra.Command{
 		Use:   "list",
 		Short: "Lists all Intake Runners",

internal/cmd/beta/intake/runner/list/list_test.go

@@ -112,7 +112,7 @@ func TestParseInput(t *testing.T) {
 
 	for _, tt := range tests {
 		t.Run(tt.description, func(t *testing.T) {
-			testutils.TestParseInput(t, NewListCmd, func(p *print.Printer, cmd *cobra.Command, _ []string) (*inputModel, error) {
+			testutils.TestParseInput(t, NewCmd, func(p *print.Printer, cmd *cobra.Command, _ []string) (*inputModel, error) {
 				return parseInput(p, cmd)
 			}, tt.expectedModel, tt.argValues, tt.flagValues, tt.isValid)
 		})
@@ -186,7 +186,7 @@ func TestOutputResult(t *testing.T) {
 		},
 	}
 	p := print.NewPrinter()
-	p.Cmd = NewListCmd(&params.CmdParams{Printer: p})
+	p.Cmd = NewCmd(&params.CmdParams{Printer: p})
 	for _, tt := range tests {
 		t.Run(tt.name, func(t *testing.T) {
 			if err := outputResult(p, tt.args.outputFormat, "dummy-projectlabel", tt.args.runners); (err != nil) != tt.wantErr {

internal/cmd/beta/intake/runner/runner.go

@@ -21,11 +21,11 @@ func NewCmd(params *params.CmdParams) *cobra.Command {
 		Run:   utils.CmdHelp,
 	}
 	// Pass the params down to each action command
-	cmd.AddCommand(create.NewCreateCmd(params))
-	cmd.AddCommand(delete.NewDeleteCmd(params))
-	cmd.AddCommand(describe.NewDescribeCmd(params))
-	cmd.AddCommand(list.NewListCmd(params))
-	cmd.AddCommand(update.NewUpdateCmd(params))
+	cmd.AddCommand(create.NewCmd(params))
+	cmd.AddCommand(delete.NewCmd(params))
+	cmd.AddCommand(describe.NewCmd(params))
+	cmd.AddCommand(list.NewCmd(params))
+	cmd.AddCommand(update.NewCmd(params))
 
 	return cmd
 }

internal/cmd/beta/intake/runner/update/update.go

@@ -43,7 +43,7 @@ type inputModel struct {
 	Labels             *map[string]string
 }
 
-func NewUpdateCmd(p *params.CmdParams) *cobra.Command {
+func NewCmd(p *params.CmdParams) *cobra.Command {
 	cmd := &cobra.Command{
 		Use:   fmt.Sprintf("update %s", runnerIdArg),
 		Short: "Updates an Intake Runner",