command: merge two container rename tests into one using table driven…

… tests

Co-authored-by: Laura Brehm <[email protected]>

cli/command/container/rename_test.go

@@ -28,34 +28,33 @@ func TestRunRename(t *testing.T) {
 	assert.Equal(t, cli.OutBuffer().String(), expected)
 }
 
-func TestRunRenameOldNameError(t *testing.T) {
-	cli := test.NewFakeCli(&fakeClient{})
-
-	cmd := NewRenameCommand(cli)
-	cmd.SetOut(io.Discard)
-	cmd.SetArgs([]string{"", "newName"})
-
-	err := cmd.Execute()
-
-	assert.Assert(t, err != nil)
-
-	expected := "Error: Neither old nor new names may be empty"
-	assert.ErrorContains(t, err, expected)
-}
-
-func TestRunRenameNewNameError(t *testing.T) {
-	cli := test.NewFakeCli(&fakeClient{})
-
-	cmd := NewRenameCommand(cli)
-	cmd.SetOut(io.Discard)
-	cmd.SetArgs([]string{"oldName", ""})
+func TestRunRenameInputs(t *testing.T) {
+	testCases := []struct {
+		args []string
+		expectedErr string
+	} {
+		{
+			args: []string{"", "newName"},
+			expectedErr: "Error: Neither old nor new names may be empty",
+		},
+		{
+			args: []string{"oldName", ""}
+			expectedErr: "Error: Neither old nor new names may be empty",
+		},
+	}
+
+	for _, tc := range testCases {
+		cli := test.NewFakeCli(&fakeClient{})
 
-	err := cmd.Execute()
+		cmd := NewRenameCommand(cli)
+		cmd.SetOut(io.Discard)
+		cmd.SetArgs(tc.args)
 
-	assert.Assert(t, err != nil)
+		err := cmd.Execute()
 
-	expected := "Error: Neither old nor new names may be empty"
-	assert.ErrorContains(t, err, expected)
+		assert.Assert(t, err != nil)
+		assert.ErrorContains(t, err, tc.expectedErr)
+	}
 }
 
 func TestRunRenameClientError(t *testing.T) {