cli/command/registry: cleanup search tests

- TestSearchContext: don't use un-keyed structs
- TestSearchContext: don't use CompareMultipleValues as it was not needed
- TestSearchContextDescription: don't use un-keyed structs
- TestSearchContextDescription: don't use CompareMultipleValues as it was not needed
- TestSearchContextWrite: don't use un-keyed structs, and include the
  code-comments into the test-table as names for the tests to give them
  some context.
- TestSearchContextWriteJSON and TestSearchContextWriteJSONField were not
  validating the output format, but validating if the JSON output could
  be marshalled back to a struct. Let's just role them into TestSearchContextWrite.

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>

cli/command/registry/formatter_search_test.go

@@ -2,12 +2,9 @@ package registry
 
 import (
 	"bytes"
-	"encoding/json"
-	"strings"
 	"testing"
 
 	"github.com/docker/cli/cli/command/formatter"
-	"github.com/docker/cli/internal/test"
 	registrytypes "github.com/docker/docker/api/types/registry"
 	"gotest.tools/v3/assert"
 	is "gotest.tools/v3/assert/cmp"
@@ -15,50 +12,67 @@ import (
 )
 
 func TestSearchContext(t *testing.T) {
-	name := "nginx"
-	starCount := 5000
-
 	var ctx searchContext
 	cases := []struct {
 		searchCtx searchContext
 		expValue  string
 		call      func() string
 	}{
-		{searchContext{
+		{
-			s: registrytypes.SearchResult{Name: name},
+			searchCtx: searchContext{
-		}, name, ctx.Name},
+				s: registrytypes.SearchResult{Name: "nginx"},
-		{searchContext{
+			},
-			s: registrytypes.SearchResult{StarCount: starCount},
+			expValue: "nginx",
-		}, "5000", ctx.StarCount},
+			call:     ctx.Name,
-		{searchContext{
+		},
-			s: registrytypes.SearchResult{IsOfficial: true},
+		{
-		}, "[OK]", ctx.IsOfficial},
+			searchCtx: searchContext{
-		{searchContext{
+				s: registrytypes.SearchResult{StarCount: 5000},
-			s: registrytypes.SearchResult{IsOfficial: false},
+			},
-		}, "", ctx.IsOfficial},
+			expValue: "5000",
-		{searchContext{
+			call:     ctx.StarCount,
-			s: registrytypes.SearchResult{IsAutomated: true}, //nolint:staticcheck // ignore SA1019 (IsAutomated is deprecated).
+		},
-		}, "[OK]", ctx.IsAutomated},
+		{
-		{searchContext{
+			searchCtx: searchContext{
-			s: registrytypes.SearchResult{},
+				s: registrytypes.SearchResult{IsOfficial: true},
-		}, "", ctx.IsAutomated},
+			},
+			expValue: "[OK]",
+			call:     ctx.IsOfficial,
+		},
+		{
+			searchCtx: searchContext{
+				s: registrytypes.SearchResult{IsOfficial: false},
+			},
+			call: ctx.IsOfficial,
+		},
+		{
+			searchCtx: searchContext{
+				s: registrytypes.SearchResult{IsAutomated: true}, //nolint:staticcheck // ignore SA1019 (IsAutomated is deprecated).
+			},
+			expValue: "[OK]",
+			call:     ctx.IsAutomated,
+		},
+		{
+			searchCtx: searchContext{
+				s: registrytypes.SearchResult{},
+			},
+			call: ctx.IsAutomated,
+		},
 	}
 
 	for _, c := range cases {
 		ctx = c.searchCtx
 		v := c.call()
-		if strings.Contains(v, ",") {
+		assert.Check(t, is.Equal(v, c.expValue))
-			test.CompareMultipleValues(t, v, c.expValue)
-		} else if v != c.expValue {
-			t.Fatalf("Expected %s, was %s\n", c.expValue, v)
-		}
 	}
 }
 
 func TestSearchContextDescription(t *testing.T) {
-	shortDescription := "Official build of Nginx."
+	const (
-	longDescription := "Automated Nginx reverse proxy for docker containers"
+		shortDescription    = "Official build of Nginx."
-	descriptionWReturns := "Automated\nNginx reverse\rproxy\rfor docker\ncontainers"
+		longDescription     = "Automated Nginx reverse proxy for docker containers"
+		descriptionWReturns = "Automated\nNginx reverse\rproxy\rfor docker\ncontainers"
+	)
 
 	var ctx searchContext
 	cases := []struct {
@@ -66,80 +80,118 @@ func TestSearchContextDescription(t *testing.T) {
 		expValue  string
 		call      func() string
 	}{
-		{searchContext{
+		{
-			s:     registrytypes.SearchResult{Description: shortDescription},
+			searchCtx: searchContext{
-			trunc: true,
+				s:     registrytypes.SearchResult{Description: shortDescription},
-		}, shortDescription, ctx.Description},
+				trunc: true,
-		{searchContext{
+			},
-			s:     registrytypes.SearchResult{Description: shortDescription},
+			expValue: shortDescription,
-			trunc: false,
+			call:     ctx.Description,
-		}, shortDescription, ctx.Description},
+		},
-		{searchContext{
+		{
-			s:     registrytypes.SearchResult{Description: longDescription},
+			searchCtx: searchContext{
-			trunc: false,
+				s:     registrytypes.SearchResult{Description: shortDescription},
-		}, longDescription, ctx.Description},
+				trunc: false,
-		{searchContext{
+			},
-			s:     registrytypes.SearchResult{Description: longDescription},
+			expValue: shortDescription,
-			trunc: true,
+			call:     ctx.Description,
-		}, formatter.Ellipsis(longDescription, 45), ctx.Description},
+		},
-		{searchContext{
+		{
-			s:     registrytypes.SearchResult{Description: descriptionWReturns},
+			searchCtx: searchContext{
-			trunc: false,
+				s:     registrytypes.SearchResult{Description: longDescription},
-		}, longDescription, ctx.Description},
+				trunc: false,
-		{searchContext{
+			},
-			s:     registrytypes.SearchResult{Description: descriptionWReturns},
+			expValue: longDescription,
-			trunc: true,
+			call:     ctx.Description,
-		}, formatter.Ellipsis(longDescription, 45), ctx.Description},
+		},
+		{
+			searchCtx: searchContext{
+				s:     registrytypes.SearchResult{Description: longDescription},
+				trunc: true,
+			},
+			expValue: formatter.Ellipsis(longDescription, 45),
+			call:     ctx.Description,
+		},
+		{
+			searchCtx: searchContext{
+				s:     registrytypes.SearchResult{Description: descriptionWReturns},
+				trunc: false,
+			},
+			expValue: longDescription,
+			call:     ctx.Description,
+		},
+		{
+			searchCtx: searchContext{
+				s:     registrytypes.SearchResult{Description: descriptionWReturns},
+				trunc: true,
+			},
+			expValue: formatter.Ellipsis(longDescription, 45),
+			call:     ctx.Description,
+		},
 	}
 
 	for _, c := range cases {
 		ctx = c.searchCtx
 		v := c.call()
-		if strings.Contains(v, ",") {
+		assert.Check(t, is.Equal(v, c.expValue))
-			test.CompareMultipleValues(t, v, c.expValue)
-		} else if v != c.expValue {
-			t.Fatalf("Expected %s, was %s\n", c.expValue, v)
-		}
 	}
 }
 
 func TestSearchContextWrite(t *testing.T) {
 	cases := []struct {
-		context  formatter.Context
+		doc         string
-		expected string
+		format      formatter.Format
+		expected    string
+		expectedErr string
 	}{
-		// Errors
 		{
-			formatter.Context{Format: "{{InvalidFunction}}"},
+			doc:         "Errors",
-			`template parsing error: template: :1: function "InvalidFunction" not defined`,
+			format:      "{{InvalidFunction}}",
+			expectedErr: `template parsing error: template: :1: function "InvalidFunction" not defined`,
 		},
 		{
-			formatter.Context{Format: "{{nil}}"},
+			doc:         "Nil format",
-			`template parsing error: template: :1:2: executing "" at <nil>: nil is not a command`,
+			format:      "{{nil}}",
-		},
+			expectedErr: `template parsing error: template: :1:2: executing "" at <nil>: nil is not a command`,
-		// Table format
-		{
-			formatter.Context{Format: NewSearchFormat("table")},
-			string(golden.Get(t, "search-context-write-table.golden")),
 		},
 		{
-			formatter.Context{Format: NewSearchFormat("table {{.Name}}")},
+			doc:    "JSON format",
-			`NAME
+			format: "{{json .}}",
+			expected: `{"Description":"Official build","IsAutomated":"false","IsOfficial":"true","Name":"result1","StarCount":"5000"}
+{"Description":"Not official","IsAutomated":"true","IsOfficial":"false","Name":"result2","StarCount":"5"}
+`,
+		},
+		{
+			doc:    "JSON format, single field",
+			format: "{{json .Name}}",
+			expected: `"result1"
+"result2"
+`,
+		},
+		{
+			doc:      "Table format",
+			format:   NewSearchFormat("table"),
+			expected: string(golden.Get(t, "search-context-write-table.golden")),
+		},
+		{
+			doc:    "Table format, single column",
+			format: NewSearchFormat("table {{.Name}}"),
+			expected: `NAME
 result1
 result2
 `,
 		},
-		// Custom Format
 		{
-			formatter.Context{Format: NewSearchFormat("{{.Name}}")},
+			doc:    "Custom format, single field",
-			`result1
+			format: NewSearchFormat("{{.Name}}"),
+			expected: `result1
 result2
 `,
 		},
-		// Custom Format with CreatedAt
 		{
-			formatter.Context{Format: NewSearchFormat("{{.Name}} {{.StarCount}}")},
+			doc:    "Custom Format, two columns",
-			`result1 5000
+			format: NewSearchFormat("{{.Name}} {{.StarCount}}"),
+			expected: `result1 5000
 result2 5
 `,
 		},
@@ -152,61 +204,15 @@ result2 5
 
 	for _, tc := range cases {
 		tc := tc
-		t.Run(string(tc.context.Format), func(t *testing.T) {
+		t.Run(tc.doc, func(t *testing.T) {
 			var out bytes.Buffer
-			tc.context.Output = &out
+			err := SearchWrite(formatter.Context{Format: tc.format, Output: &out}, results)
-
+			if tc.expectedErr != "" {
-			err := SearchWrite(tc.context, results)
+				assert.Check(t, is.Error(err, tc.expectedErr))
-			if err != nil {
-				assert.Error(t, err, tc.expected)
 			} else {
-				assert.Equal(t, out.String(), tc.expected)
+				assert.Check(t, err)
 			}
+			assert.Check(t, is.Equal(out.String(), tc.expected))
 		})
 	}
 }
-
-func TestSearchContextWriteJSON(t *testing.T) {
-	results := []registrytypes.SearchResult{
-		{Name: "result1", Description: "Official build", StarCount: 5000, IsOfficial: true},
-		{Name: "result2", Description: "Not official", StarCount: 5, IsOfficial: false, IsAutomated: true}, //nolint:staticcheck // ignore SA1019 (IsAutomated is deprecated).
-	}
-	expectedJSONs := []map[string]interface{}{
-		{"Name": "result1", "Description": "Official build", "StarCount": "5000", "IsOfficial": "true", "IsAutomated": "false"},
-		{"Name": "result2", "Description": "Not official", "StarCount": "5", "IsOfficial": "false", "IsAutomated": "true"},
-	}
-
-	out := bytes.NewBufferString("")
-	err := SearchWrite(formatter.Context{Format: "{{json .}}", Output: out}, results)
-	if err != nil {
-		t.Fatal(err)
-	}
-	for i, line := range strings.Split(strings.TrimSpace(out.String()), "\n") {
-		t.Logf("Output: line %d: %s", i, line)
-		var m map[string]interface{}
-		if err := json.Unmarshal([]byte(line), &m); err != nil {
-			t.Fatal(err)
-		}
-		assert.Check(t, is.DeepEqual(m, expectedJSONs[i]))
-	}
-}
-
-func TestSearchContextWriteJSONField(t *testing.T) {
-	results := []registrytypes.SearchResult{
-		{Name: "result1", Description: "Official build", StarCount: 5000, IsOfficial: true},
-		{Name: "result2", Description: "Not official", StarCount: 5, IsOfficial: false, IsAutomated: true}, //nolint:staticcheck // ignore SA1019 (IsAutomated is deprecated).
-	}
-	out := bytes.NewBufferString("")
-	err := SearchWrite(formatter.Context{Format: "{{json .Name}}", Output: out}, results)
-	if err != nil {
-		t.Fatal(err)
-	}
-	for i, line := range strings.Split(strings.TrimSpace(out.String()), "\n") {
-		t.Logf("Output: line %d: %s", i, line)
-		var s string
-		if err := json.Unmarshal([]byte(line), &s); err != nil {
-			t.Fatal(err)
-		}
-		assert.Check(t, is.Equal(s, results[i].Name))
-	}
-}