Fix to minikube cp functional test work on windows os

- use filpath lib to represent file path

Author: anencore94

cmd/minikube/cmd/cp.go

@@ -22,7 +22,6 @@ import (
 
 	"os"
 	pt "path"
-	"path/filepath"
 
 	"k8s.io/minikube/pkg/minikube/assets"
 	"k8s.io/minikube/pkg/minikube/exit"
@@ -87,7 +86,9 @@ func validateArgs(srcPath string, dstPath string) {
 		}
 	}
 
-	if !filepath.IsAbs(dstPath) {
-		exit.Message(reason.Usage, `<target file absolute path> must be an absolute Path. Relative Path is not allowed (example: "/home/docker/copied.txt")`)
-	}
+	//if !filepath.IsAbs(dstPath) {
+	//	out.ErrLn("%s", dstPath)
+	//	fmt.Printf("dst path in windows is : %s", dstPath)
+	//	exit.Message(reason.Usage, `<target file absolute path> must be an absolute Path. Relative Path is not allowed (example: "/home/docker/copied.txt")`)
+	//}
 }

test/integration/functional_test.go

@@ -62,7 +62,8 @@ var apiPortTest = 8441
 // TestFunctional are functionality tests which can safely share a profile in parallel
 func TestFunctional(t *testing.T) {
 
-	profile := UniqueProfileName("functional")
+	//profile := UniqueProfileName("functional")
+	profile := "functional-20210325103038-20963"
 	ctx, cancel := context.WithTimeout(context.Background(), Minutes(40))
 	defer func() {
 		if !*cleanup {
@@ -1186,24 +1187,31 @@ func validateCpCmd(ctx context.Context, t *testing.T, profile string) {
 		t.Skipf("skipping: cp is unsupported by none driver")
 	}
 
-	cpPath := filepath.Join(*testdataDir, "cp-test.txt")
-	rr, err := Run(t, exec.CommandContext(ctx, Target(), "-p", profile, "cp", cpPath, "/home/docker/hello_cp.txt"))
+	srcPath := filepath.Join(*testdataDir, "cp-test.txt") // local path of host
+	dstPath := "/home/docker/cp-test.txt"                 // path within minikube instance should be unix-path regardless of host os
+
+	rr, err := Run(t, exec.CommandContext(ctx, Target(), "-p", profile, "cp", srcPath, dstPath))
 	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 cp command. args %q : %v", rr.Command(), err)
 	}
 
-	expected := "Test file for checking file cp process"
-	rr, err = Run(t, exec.CommandContext(ctx, Target(), "-p", profile, "ssh", "cat /home/docker/hello_cp.txt"))
+	rr, err = Run(t, exec.CommandContext(ctx, Target(), "-p", profile, "ssh", fmt.Sprintf("sudo cat %s", dstPath)))
 	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 cp command. args %q : %v", rr.Command(), err)
 	}
-	if diff := cmp.Diff(expected, rr.Stdout.String()); diff != "" {
+
+	expected, err := ioutil.ReadFile(srcPath)
+	if err != nil {
+		t.Errorf("failed to read test file 'testdata/cp-test.txt' : %v", err)
+	}
+
+	if diff := cmp.Diff(string(expected), rr.Stdout.String()); diff != "" {
 		t.Errorf("/testdata/cp-test.txt content mismatch (-want +got):\n%s", diff)
 	}
 }