[Enhancement] Add functional test for scp command

Signed-off-by: anencore94 <anencore94@kaist.ac.kr>

cmd/minikube/cmd/scp.go

@@ -22,12 +22,14 @@ import (
 	"path/filepath"
 
 	"github.com/spf13/cobra"
+
 	"k8s.io/minikube/pkg/minikube/config"
 	"k8s.io/minikube/pkg/minikube/exit"
 	"k8s.io/minikube/pkg/minikube/localpath"
 	"k8s.io/minikube/pkg/minikube/mustload"
 	"k8s.io/minikube/pkg/minikube/node"
 	"k8s.io/minikube/pkg/minikube/out"
+
 	"k8s.io/minikube/pkg/minikube/reason"
 )
 
@@ -45,7 +47,7 @@ var (
 var scpCmd = &cobra.Command{
 	Use:   "scp [flags] <source file or directory path> <target file or directory path followed by '/home/docker/'>",
 	Short: "Copy the specified file or directory into minikube",
-	Long:  "Copy the specified file or directory into minikube, it will be saved at path \"/home/docker/<target file or directory path>\" in your minikube.\n" +
+	Long: "Copy the specified file or directory into minikube, it will be saved at path \"/home/docker/<target file or directory path>\" in your minikube.\n" +
 		"Example Command : \"minikube scp a.txt b.txt\"\n",
 	Run: func(cmd *cobra.Command, args []string) {
 		// validate args

test/integration/functional_test.go

@@ -121,6 +121,7 @@ func TestFunctional(t *testing.T) {
 			{"PersistentVolumeClaim", validatePersistentVolumeClaim},
 			{"TunnelCmd", validateTunnelCmd},
 			{"SSHCmd", validateSSHCmd},
+			{"ScpCmd", validateScpCmd},
 			{"MySQL", validateMySQL},
 			{"FileSync", validateFileSync},
 			{"CertSync", validateCertSync},
@@ -981,6 +982,34 @@ func validateSSHCmd(ctx context.Context, t *testing.T, profile string) {
 	}
 }
 
+// validateScpCmd asserts basic "scp" command functionality
+func validateScpCmd(ctx context.Context, t *testing.T, profile string) {
+	if NoneDriver() {
+		t.Skipf("skipping: scp unsupported by none")
+	}
+
+	scpPath := filepath.Join(*testdataDir, "scp.test")
+	rr, err := Run(t, exec.CommandContext(ctx, Target(), "-p", profile, "scp", scpPath, "hello_scp.txt"))
+	if ctx.Err() == context.DeadlineExceeded {
+		t.Errorf("failed to run command by deadline. exceeded timeout : %s", rr.Command())
+	}
+	if err != nil {
+		t.Errorf("failed to run an scp command. args %q : %v", rr.Command(), err)
+	}
+
+	expected := "Test file for checking file scp process"
+	rr, err = Run(t, exec.CommandContext(ctx, Target(), "-p", profile, "ssh", "cat hello_scp.txt"))
+	if ctx.Err() == context.DeadlineExceeded {
+		t.Errorf("failed to run command by deadline. exceeded timeout : %s", rr.Command())
+	}
+	if err != nil {
+		t.Errorf("failed to run an scp command. args %q : %v", rr.Command(), err)
+	}
+	if diff := cmp.Diff(expected, rr.Stdout.String()); diff != "" {
+		t.Errorf("/testdata/scp.test content mismatch (-want +got):\n%s", diff)
+	}
+}
+
 // validateMySQL validates a minimalist MySQL deployment
 func validateMySQL(ctx context.Context, t *testing.T, profile string) {
 	if arm64Platform() {