Update specs_test.go

When checking storage options output printed, ignore the order
because Golang map iteration is randomized

Author: davidheryanto

cli/feast/pkg/printer/specs_test.go

@@ -2,6 +2,7 @@ package printer
 
 import (
 	"fmt"
+	"strings"
 	"testing"
 
 	"github.com/gojek/feast/cli/feast/pkg/timeutil"
@@ -132,22 +133,51 @@ func TestPrintStorage(t *testing.T) {
 		Spec: &specs.StorageSpec{
 			Id:   "REDIS1",
 			Type: "redis",
-			Options: map[string]string{
-				"option1": "value1",
-				"option2": "value2",
-			},
 		},
 		LastUpdated: &timestamp.Timestamp{Seconds: 1},
 	}
 	out := PrintStorageDetail(storageDetail)
 	expected := fmt.Sprintf(`Id:	REDIS1
 Type:	redis
 Options:
-  option1: value1
-  option2: value2
 LastUpdated:	%s`,
 		timeutil.FormatToRFC3339(timestamp.Timestamp{Seconds: 1}))
 	if out != expected {
 		t.Errorf("Expected output:\n%s \nActual:\n%s \n", expected, out)
 	}
 }
+
+func TestPrintStorageWithOptions(t *testing.T) {
+	storageDetail := &core.UIServiceTypes_StorageDetail{
+		Spec: &specs.StorageSpec{
+			Id:   "REDIS1",
+			Type: "redis",
+			Options: map[string]string{
+				"option1": "value1",
+				"option2": "value2",
+			},
+		},
+		LastUpdated: &timestamp.Timestamp{Seconds: 1},
+	}
+	out := PrintStorageDetail(storageDetail)
+
+	// Since map iteration order in Golang is radomized and "Options" is stored in a map,
+	// this test just check that the output "contains" the key and value
+	// rather than checking the exact match
+
+	expected := []string{
+		"Id:	REDIS1",
+		"Type:	redis",
+		"Options:",
+		"  option1: value1",
+		"  option2: value2",
+		fmt.Sprintf("LastUpdated:	%s", timeutil.FormatToRFC3339(timestamp.Timestamp{Seconds: 1})),
+	}
+
+	for _, want := range expected {
+		if !strings.Contains(out, want) {
+			t.Errorf("Missing \"%s\" from actual output\nActual output:\n%s", want, out)
+		}
+	}
+
+}