Return only found PackageManifests (#78)

cdjohnson · web-flow · commit 975751e323bb · 2022-11-22T16:19:40.000-05:00
Signed-off-by: cdjohnson &lt;cdjohnson@us.ibm.com&gt;

Signed-off-by: cdjohnson &lt;cdjohnson@us.ibm.com&gt;
diff --git a/internal/pkg/action/operator_list_available.go b/internal/pkg/action/operator_list_available.go
@@ -39,10 +39,10 @@ func (l *OperatorListAvailable) Run(ctx context.Context) ([]operator.PackageMani
 	if err := l.config.Client.List(ctx, &pms, labelSelector); err != nil {
 		return nil, err
 	}
-	pkgs := make([]operator.PackageManifest, len(pms.Items))
+	pkgs := make([]operator.PackageManifest, 0, len(pms.Items))
 	for i := range pms.Items {
 		if l.Package == "" || l.Package == pms.Items[i].GetName() {
-			pkgs[i] = operator.PackageManifest{PackageManifest: pms.Items[i]}
+			pkgs = append(pkgs, operator.PackageManifest{PackageManifest: pms.Items[i]})
 		}
 	}
 	return pkgs, nil

Original file line number	Diff line number	Diff line change
`@@ -39,10 +39,10 @@ func (l *OperatorListAvailable) Run(ctx context.Context) ([]operator.PackageMani`
`39`	`39`	`if err := l.config.Client.List(ctx, &pms, labelSelector); err != nil {`
`40`	`40`	`return nil, err`
`41`	`41`	`}`
`42`		`- pkgs := make([]operator.PackageManifest, len(pms.Items))`
	`42`	`+ pkgs := make([]operator.PackageManifest, 0, len(pms.Items))`
`43`	`43`	`for i := range pms.Items {`
`44`	`44`	`if l.Package == "" \|\| l.Package == pms.Items[i].GetName() {`
`45`		`- pkgs[i] = operator.PackageManifest{PackageManifest: pms.Items[i]}`
	`45`	`+ pkgs = append(pkgs, operator.PackageManifest{PackageManifest: pms.Items[i]})`
`46`	`46`	`}`
`47`	`47`	`}`
`48`	`48`	`return pkgs, nil`