Commands generating warnings about non existing table fields (#106)

privateip · web-flow · commit bf401b19220b · 2025-04-07T09:47:03.000-04:00
There are three commands that are generating warnings about table fields
that do not exist.  The output table configuration for would reference
fields not returned from the API call.  This change addresses those
issues and properly returns a table output
diff --git a/internal/runners/methods.go b/internal/runners/methods.go
@@ -5,8 +5,6 @@
 package runners
 
 import (
-	"fmt"
-
 	"github.com/itential/ipctl/pkg/client"
 	"github.com/itential/ipctl/pkg/config"
 	"github.com/itential/ipctl/pkg/logger"
@@ -33,14 +31,8 @@ func (r *MethodRunner) Get(in Request) (*Response, error) {
 		return nil, err
 	}
 
-	display := []string{"NAME\tTYPE"}
-	for _, ele := range methods {
-		line := fmt.Sprintf("%s\t%s", ele.Name, ele.Provenance)
-		display = append(display, line)
-	}
-
 	return &Response{
-		Keys:   []string{"name", "type"},
+		Keys:   []string{"name", "provenance"},
 		Object: methods,
 	}, nil
 }
diff --git a/internal/runners/roles.go b/internal/runners/roles.go
@@ -54,7 +54,7 @@ func (r *RoleRunner) Get(in Request) (*Response, error) {
 	}
 
 	return &Response{
-		Keys:   []string{"name", "type"},
+		Keys:   []string{"name", "provenance"},
 		Object: filtered,
 	}, nil
 
diff --git a/internal/runners/workflows.go b/internal/runners/workflows.go
@@ -57,7 +57,7 @@ func (r *WorkflowRunner) Get(in Request) (*Response, error) {
 	}
 
 	return &Response{
-		Keys:   []string{"name", "description"},
+		Keys:   []string{"name"},
 		Object: workflows,
 	}, nil
 

Original file line number	Diff line number	Diff line change
`@@ -54,7 +54,7 @@ func (r RoleRunner) Get(in Request) (Response, error) {`
`54`	`54`	`}`
`55`	`55`
`56`	`56`	`return &Response{`
`57`		`- Keys: []string{"name", "type"},`
	`57`	`+ Keys: []string{"name", "provenance"},`
`58`	`58`	`Object: filtered,`
`59`	`59`	`}, nil`
`60`	`60`
Original file line number	Diff line number	Diff line change
`@@ -57,7 +57,7 @@ func (r WorkflowRunner) Get(in Request) (Response, error) {`
`57`	`57`	`}`
`58`	`58`
`59`	`59`	`return &Response{`
`60`		`- Keys: []string{"name", "description"},`
	`60`	`+ Keys: []string{"name"},`
`61`	`61`	`Object: workflows,`
`62`	`62`	`}, nil`
`63`	`63`