Chore: change helper texts and add debugger

Author: rkpattnaik780

docs/commands/rhoas_kafka_list.adoc

@@ -25,7 +25,7 @@ rhoas kafka list [flags]
       --limit int       The maximum number of Kafka instances to be returned (default 100)
   -o, --output string   Format in which to display the Kafka instances. Choose from: "json", "yml", "yaml"
       --page int        Display the Kafka instances from the specified page number.
-      --search string   Text search to filter the Kafka instances
+      --search string   Text search to filter the Kafka instances by name, owner, cloud_provider, region and status
 ....
 
 === Options inherited from parent commands

locales/cmd/kafka/list/active.en.toml

@@ -42,7 +42,11 @@ one = 'The maximum number of Kafka instances to be returned'
 
 [kafka.list.flag.search]
 description = 'Description for the --search flag'
-one = 'Text search to filter the Kafka instances'
+one = 'Text search to filter the Kafka instances by name, owner, cloud_provider, region and status'
+
+[kafka.list.log.debug.filteringKafkaList]
+description = 'Debugging message when filtering the list of Kafka instances'
+one = 'Filtering Kafka instances with the query "{{.Search}}"'
 
 [kafka.list.log.info.noKafkaInstances]
 description = 'Info message when no Kafkas were found'

pkg/cmd/kafka/list/list.go

@@ -99,7 +99,17 @@ func runList(opts *options) error {
 	a := api.Kafka().ListKafkas(context.Background())
 	a = a.Page(strconv.Itoa(opts.page))
 	a = a.Size(strconv.Itoa(opts.limit))
-	a = a.Search(buildQuery(opts.search))
+
+	if opts.search != "" {
+		logger.Debug(localizer.MustLocalize(&localizer.Config{
+			MessageID: "kafka.list.log.debug.filteringKafkaList",
+			TemplateData: map[string]interface{}{
+				"Search": buildQuery(opts.search),
+			},
+		}))
+		a = a.Search(buildQuery(opts.search))
+	}
+
 	response, _, apiErr := a.Execute()
 
 	if apiErr.Error() != "" {
@@ -148,14 +158,10 @@ func mapResponseItemsToRows(kafkas []kasclient.KafkaRequest) []kafkaRow {
 
 func buildQuery(search string) string {
 
-	var queryString string
-
-	if search != "" {
-		queryString = fmt.Sprintf(
-			"name like %%%[1]v%% or owner like %%%[1]v%% or cloud_provider like %%%[1]v%% or region like %%%[1]v%% or status like %%%[1]v%%",
-			search,
-		)
-	}
+	queryString := fmt.Sprintf(
+		"name like %%%[1]v%% or owner like %%%[1]v%% or cloud_provider like %%%[1]v%% or region like %%%[1]v%% or status like %%%[1]v%%",
+		search,
+	)
 
 	return queryString