fix

Author: ScottZhuMS

.github/workflows/windows.yml

@@ -4,7 +4,7 @@ jobs:
   integration_tests:
     strategy:
       matrix:
-        go: ['1.22']
+        go: ['1.21']
         platform: [windows-latest]
     runs-on: ${{ matrix.platform }}
     steps:
@@ -31,11 +31,11 @@ jobs:
           Write-Output "getting named pipes"
           [System.IO.Directory]::GetFiles("\\.\\pipe\\")
           $env:CSI_PROXY_GH_ACTIONS="TRUE"
-          go test -timeout 20m -v -race ./integrationtests/...
+          go test -timeout 5m -v -race ./integrationtests/...
   unit_tests:
     strategy:
       matrix:
-        go: ['1.22']
+        go: ['1.21']
         platform: [windows-latest]
     runs-on: ${{ matrix.platform }}
     steps:
@@ -53,7 +53,7 @@ jobs:
   bump_version_test:
     strategy:
       matrix:
-        go: ['1.22']
+        go: ['1.21']
         platform: [ubuntu-latest]
     runs-on: ${{ matrix.platform }}
     steps:

integrationtests/smb_test.go

@@ -35,8 +35,7 @@ func setupUser(username, password string) error {
 	cmd := exec.Command("powershell", "/c", cmdLine)
 	cmd.Env = append(os.Environ(),
 		fmt.Sprintf("username=%s", username),
-		fmt.Sprintf("password=%s", password),
-		"PSModulePath=") // Set the PSModulePath environment variable
+		fmt.Sprintf("password=%s", password))
 	if output, err := cmd.CombinedOutput(); err != nil {
 		return fmt.Errorf("setupUser failed: %v, output: %q", err, string(output))
 	}

integrationtests/smb_v1alpha1_test.go

@@ -13,11 +13,13 @@ import (
 )
 
 func v1alpha1SmbTests(t *testing.T) {
+	t.Logf("Before NewClient")
 	client, err := client.NewClient()
 	if err != nil {
 		t.Fatalf("Fail to get smb API group client %v", err)
 	}
 	defer client.Close()
+	t.Logf("After NewClient")
 
 	username := randomString(5)
 	password := randomString(10) + "!"
@@ -26,16 +28,21 @@ func v1alpha1SmbTests(t *testing.T) {
 
 	localPath := fmt.Sprintf("C:\\localpath%s", randomString(5))
 
+	t.Logf("Before setupUser")
 	if err = setupUser(username, password); err != nil {
 		t.Fatalf("TestSmbAPIGroup %v", err)
 	}
 	defer removeUser(t, username)
+	t.Logf("After setupUser")
 
+	t.Logf("Before setupSmbShare")
 	if err = setupSmbShare(smbShare, sharePath, username); err != nil {
 		t.Fatalf("TestSmbAPIGroup %v", err)
 	}
 	defer removeSmbShare(t, smbShare)
+	t.Logf("After setupSmbShare")
 
+	t.Logf("Before NewSmbGlobalMapping")
 	hostname, err := os.Hostname()
 	assert.Nil(t, err)
 
@@ -51,23 +58,34 @@ func v1alpha1SmbTests(t *testing.T) {
 	if err != nil {
 		t.Fatalf("TestSmbAPIGroup %v", err)
 	}
+	t.Logf("After NewSmbGlobalMapping")
 
+	t.Logf("Before getSmbGlobalMapping")
 	err = getSmbGlobalMapping(remotePath)
 	assert.Nil(t, err)
+	t.Logf("After getSmbGlobalMapping")
 
+	t.Logf("Before writeReadFile")
 	err = writeReadFile(remotePath)
 	assert.Nil(t, err)
+	t.Logf("After writeReadFile")
 
+	t.Logf("Before RemoveSmbGlobalMapping")
 	unmountSmbShareReq := &v1alpha1.RemoveSmbGlobalMappingRequest{
 		RemotePath: remotePath,
 	}
 	_, err = client.RemoveSmbGlobalMapping(context.Background(), unmountSmbShareReq)
 	if err != nil {
 		t.Fatalf("TestSmbAPIGroup %v", err)
 	}
+	t.Logf("After RemoveSmbGlobalMapping")
+	t.Logf("Before getSmbGlobalMapping")
 	err = getSmbGlobalMapping(remotePath)
 	assert.NotNil(t, err)
+	t.Logf("After getSmbGlobalMapping")
+	t.Logf("Before writeReadFile")
 	err = writeReadFile(localPath)
 	assert.NotNil(t, err)
+	t.Logf("After writeReadFile")
 
 }