Test both imageOptions and imageDestOptions in TestTLSVerifyFlags

Signed-off-by: Miloslav Trmač <[email protected]>
lukewarmtemp · Jul 23, 2021 · bb447f2 · bb447f2
1 parent 2a98df6
commit bb447f2
Showing 1 changed file with 52 additions and 26 deletions.
diff --git a/cmd/skopeo/utils_test.go b/cmd/skopeo/utils_test.go
@@ -170,41 +170,67 @@ func TestImageDestOptionsNewSystemContext(t *testing.T) {
 		BigFilesTemporaryDir:              "/srv",
 	}, res)
 
+	// Global/per-command tlsVerify behavior is tested in TestTLSVerifyFlags.
+
 	// Invalid option values in imageOptions
 	opts = fakeImageDestOptions(t, "dest-", []string{}, []string{"--dest-creds", ""})
 	_, err = opts.newSystemContext()
 	assert.Error(t, err)
 }
 
 func TestTLSVerifyFlags(t *testing.T) {
-	for _, c := range []struct {
-		global, cmd          string
-		expectedDocker       types.OptionalBool
-		expectedDockerDaemon bool
+	type systemContextOpts interface { // Either *imageOptions or *imageDestOptions
+		newSystemContext() (*types.SystemContext, error)
+	}
+
+	for _, creator := range []struct {
+		name    string
+		newOpts func(globalFlags, cmdFlags []string) systemContextOpts
 	}{
-		{"", "", types.OptionalBoolUndefined, false},
-		{"", "false", types.OptionalBoolTrue, true},
-		{"", "true", types.OptionalBoolFalse, false},
-		{"false", "", types.OptionalBoolTrue, false},
-		{"false", "false", types.OptionalBoolTrue, true},
-		{"false", "true", types.OptionalBoolFalse, false},
-		{"true", "", types.OptionalBoolFalse, false},
-		{"true", "false", types.OptionalBoolTrue, true},
-		{"true", "true", types.OptionalBoolFalse, false},
+		{
+			"imageFlags",
+			func(globalFlags, cmdFlags []string) systemContextOpts {
+				return fakeImageOptions(t, "dest-", globalFlags, cmdFlags)
+			},
+		},
+		{
+			"imageDestFlags",
+			func(globalFlags, cmdFlags []string) systemContextOpts {
+				return fakeImageDestOptions(t, "dest-", globalFlags, cmdFlags)
+			},
+		},
 	} {
-		globalFlags := []string{}
-		if c.global != "" {
-			globalFlags = append(globalFlags, "--tls-verify="+c.global)
-		}
-		cmdFlags := []string{}
-		if c.cmd != "" {
-			cmdFlags = append(cmdFlags, "--dest-tls-verify="+c.cmd)
-		}
-		opts := fakeImageOptions(t, "dest-", globalFlags, cmdFlags)
-		res, err := opts.newSystemContext()
-		require.NoError(t, err)
-		assert.Equal(t, c.expectedDocker, res.DockerInsecureSkipTLSVerify, "%#v", c)
-		assert.Equal(t, c.expectedDockerDaemon, res.DockerDaemonInsecureSkipTLSVerify, "%#v", c)
+		t.Run(creator.name, func(t *testing.T) {
+			for _, c := range []struct {
+				global, cmd          string
+				expectedDocker       types.OptionalBool
+				expectedDockerDaemon bool
+			}{
+				{"", "", types.OptionalBoolUndefined, false},
+				{"", "false", types.OptionalBoolTrue, true},
+				{"", "true", types.OptionalBoolFalse, false},
+				{"false", "", types.OptionalBoolTrue, false},
+				{"false", "false", types.OptionalBoolTrue, true},
+				{"false", "true", types.OptionalBoolFalse, false},
+				{"true", "", types.OptionalBoolFalse, false},
+				{"true", "false", types.OptionalBoolTrue, true},
+				{"true", "true", types.OptionalBoolFalse, false},
+			} {
+				globalFlags := []string{}
+				if c.global != "" {
+					globalFlags = append(globalFlags, "--tls-verify="+c.global)
+				}
+				cmdFlags := []string{}
+				if c.cmd != "" {
+					cmdFlags = append(cmdFlags, "--dest-tls-verify="+c.cmd)
+				}
+				opts := creator.newOpts(globalFlags, cmdFlags)
+				res, err := opts.newSystemContext()
+				require.NoError(t, err)
+				assert.Equal(t, c.expectedDocker, res.DockerInsecureSkipTLSVerify, "%#v", c)
+				assert.Equal(t, c.expectedDockerDaemon, res.DockerDaemonInsecureSkipTLSVerify, "%#v", c)
+			}
+		})
 	}
 }