From 1ae4e751967850e8146f8f3fa04c0dd302ef15bf Mon Sep 17 00:00:00 2001
From: izturn <44051386+izturn@users.noreply.github.com>
Date: Fri, 19 Apr 2024 20:15:50 +0800
Subject: [PATCH] refactor: replace util.SliceContainsString with
 slices.Contains & make fmt (#1041)

* use std package's func instead

Signed-off-by: gang.liu <gang.liu@daocloud.io>

* refactor: replace util.SliceContainsString with slices.Contains  & make fmt

Signed-off-by: gang.liu <gang.liu@daocloud.io>

---------

Signed-off-by: gang.liu <gang.liu@daocloud.io>
Co-authored-by: Alex Jones <alexsimonjones@gmail.com>
---
 cmd/filters/list.go   |  7 +++----
 pkg/util/util.go      | 14 +++-----------
 pkg/util/util_test.go | 24 +-----------------------
 3 files changed, 7 insertions(+), 38 deletions(-)

diff --git a/cmd/filters/list.go b/cmd/filters/list.go
index 1332773254..51e6fe119d 100644
--- a/cmd/filters/list.go
+++ b/cmd/filters/list.go
@@ -15,6 +15,7 @@ package filters
 
 import (
 	"fmt"
+	"slices"
 
 	"github.com/fatih/color"
 	"github.com/k8sgpt-ai/k8sgpt/pkg/analyzer"
@@ -40,10 +41,9 @@ var listCmd = &cobra.Command{
 		inactiveFilters := util.SliceDiff(availableFilters, activeFilters)
 		fmt.Print(color.YellowString("Active: \n"))
 		for _, filter := range activeFilters {
-
 			// if the filter is an integration, mark this differently
 			// but if the integration is inactive, remove
-			if util.SliceContainsString(integrationFilters, filter) {
+			if slices.Contains(integrationFilters, filter) {
 				fmt.Printf("> %s\n", color.BlueString("%s (integration)", filter))
 			} else {
 				// This strange bit of logic will loop through every integration via
@@ -60,13 +60,12 @@ var listCmd = &cobra.Command{
 			fmt.Print(color.YellowString("Unused: \n"))
 			for _, filter := range inactiveFilters {
 				// if the filter is an integration, mark this differently
-				if util.SliceContainsString(integrationFilters, filter) {
+				if slices.Contains(integrationFilters, filter) {
 					fmt.Printf("> %s\n", color.BlueString("%s (integration)", filter))
 				} else {
 					fmt.Printf("> %s\n", color.RedString(filter))
 				}
 			}
 		}
-
 	},
 }
diff --git a/pkg/util/util.go b/pkg/util/util.go
index 4527c58d8a..0083a00447 100644
--- a/pkg/util/util.go
+++ b/pkg/util/util.go
@@ -33,15 +33,6 @@ import (
 
 var anonymizePattern = []rune("abcdefghijklmnopqrstuvwxyzABCDEFGHIJKLMNOPQRSTUVWXYZ0123456789!@#$%^&*()-_=+[]{}|;':\",./<>?")
 
-func SliceContainsString(slice []string, s string) bool {
-	for _, item := range slice {
-		if item == s {
-			return true
-		}
-	}
-	return false
-}
-
 func GetParent(client *kubernetes.Client, meta metav1.ObjectMeta) (string, bool) {
 	if meta.OwnerReferences != nil {
 		for _, owner := range meta.OwnerReferences {
@@ -183,7 +174,8 @@ func GetCacheKey(provider string, language string, sEnc string) string {
 
 func GetPodListByLabels(client k.Interface,
 	namespace string,
-	labels map[string]string) (*v1.PodList, error) {
+	labels map[string]string,
+) (*v1.PodList, error) {
 	pods, err := client.CoreV1().Pods(namespace).List(context.Background(), metav1.ListOptions{
 		LabelSelector: metav1.FormatLabelSelector(&metav1.LabelSelector{
 			MatchLabels: labels,
@@ -207,7 +199,7 @@ func FileExists(path string) (bool, error) {
 }
 
 func EnsureDirExists(dir string) error {
-	err := os.MkdirAll(dir, 0755)
+	err := os.MkdirAll(dir, 0o755)
 
 	if errors.Is(err, os.ErrExist) {
 		return nil
diff --git a/pkg/util/util_test.go b/pkg/util/util_test.go
index 018cc885e4..609b2d7995 100644
--- a/pkg/util/util_test.go
+++ b/pkg/util/util_test.go
@@ -26,29 +26,6 @@ import (
 	"k8s.io/client-go/kubernetes/fake"
 )
 
-func TestSliceContainsString(t *testing.T) {
-	tests := []struct {
-		slice    []string
-		s        string
-		expected bool
-	}{
-		{
-			expected: false,
-		},
-		{
-			slice:    []string{"temp", "value"},
-			s:        "value",
-			expected: true,
-		},
-	}
-	for _, tt := range tests {
-		tt := tt
-		t.Run(tt.s, func(t *testing.T) {
-			require.Equal(t, tt.expected, SliceContainsString(tt.slice, tt.s))
-		})
-	}
-}
-
 func TestGetParent(t *testing.T) {
 	ownerName := "test-name"
 	namespace := "test"
@@ -409,6 +386,7 @@ func TestGetPodListByLabels(t *testing.T) {
 		})
 	}
 }
+
 func TestFileExists(t *testing.T) {
 	tests := []struct {
 		filePath  string