From c5a284f7400724b7ede04c29334816a2f1149724 Mon Sep 17 00:00:00 2001
From: Stefan Prodan <stefan.prodan@gmail.com>
Date: Tue, 10 Oct 2023 18:01:18 +0300
Subject: [PATCH] Use tabular view for `timoni inspect resources` Print
 resources managed by an instance in a tabular view with name, namespace and
 API version columns.

Signed-off-by: Stefan Prodan <stefan.prodan@gmail.com>
---
 cmd/timoni/inspect_resources.go | 30 ++++++++++++++++++++++++++----
 cmd/timoni/inspect_test.go      |  4 ++--
 2 files changed, 28 insertions(+), 6 deletions(-)

diff --git a/cmd/timoni/inspect_resources.go b/cmd/timoni/inspect_resources.go
index 3154ee85..3e691f82 100644
--- a/cmd/timoni/inspect_resources.go
+++ b/cmd/timoni/inspect_resources.go
@@ -19,9 +19,10 @@ package main
 import (
 	"context"
 	"fmt"
+	"strings"
 
-	"github.com/fluxcd/pkg/ssa"
 	"github.com/spf13/cobra"
+
 	"github.com/stefanprodan/timoni/internal/runtime"
 )
 
@@ -74,14 +75,35 @@ func runInspectResourcesCmd(cmd *cobra.Command, args []string) error {
 
 	iManager := runtime.InstanceManager{Instance: *inst}
 
-	metas, err := iManager.ListMeta()
+	//metas, err := iManager.ListMeta()
+	//if err != nil {
+	//	return err
+	//}
+	//
+	//for _, meta := range metas {
+	//	fmt.Fprintln(cmd.OutOrStdout(), colorizeSubject(ssa.FmtObjMetadata(meta)))
+	//}
+
+	objects, err := iManager.ListObjects()
 	if err != nil {
 		return err
 	}
 
-	for _, meta := range metas {
-		fmt.Fprintln(cmd.OutOrStdout(), colorizeSubject(ssa.FmtObjMetadata(meta)))
+	var rows [][]string
+	for _, obj := range objects {
+		ns := "-"
+		if obj.GetNamespace() != "" {
+			ns = obj.GetNamespace()
+		}
+		row := []string{
+			strings.ToLower(obj.GetKind() + "/" + obj.GetName()),
+			ns,
+			obj.GetAPIVersion(),
+		}
+		rows = append(rows, row)
 	}
 
+	printTable(rootCmd.OutOrStdout(), []string{"name", "namespace", "API version"}, rows)
+
 	return nil
 }
diff --git a/cmd/timoni/inspect_test.go b/cmd/timoni/inspect_test.go
index 8c7f4cc0..4625a269 100644
--- a/cmd/timoni/inspect_test.go
+++ b/cmd/timoni/inspect_test.go
@@ -96,8 +96,8 @@ func TestInspect(t *testing.T) {
 		g.Expect(err).ToNot(HaveOccurred())
 
 		// Verify inspect output contains the expected resources
-		g.Expect(output).To(ContainSubstring(fmt.Sprintf("ConfigMap/%s/%s-client", namespace, name)))
-		g.Expect(output).To(ContainSubstring(fmt.Sprintf("ConfigMap/%s/%s-server", namespace, name)))
+		g.Expect(output).To(ContainSubstring(fmt.Sprintf("configmap/%s-client", name)))
+		g.Expect(output).To(ContainSubstring(fmt.Sprintf("configmap/%s-server", name)))
 	})
 }