[Fix] Upload content databricks_workspace_file using raw format (#4200

)

## Changes
<!-- Summary of your changes that are easy to understand -->

This fixes a problem with uploading zip-based files with the
`databricks_workspace_file` resource.

## Tests
<!-- 
How is this tested? Please see the checklist below and also describe any
other relevant tests
-->

- [x] `make test` run locally
- [ ] ~relevant change in `docs/` folder~
- [x] covered with integration tests in `internal/acceptance`
- [ ] relevant acceptance tests are passing
- [x] using Go SDK

internal/acceptance/permissions_test.go

@@ -466,17 +466,22 @@ func TestAccPermissions_WorkspaceFile_Path(t *testing.T) {
 		}
 		resource "databricks_workspace_file" "this" {
 			source = "{var.CWD}/../../storage/testdata/tf-test-python.py"
-			path = "${databricks_directory.this.path}/test_notebook"
+			path = "${databricks_directory.this.path}/test_ws_file"
 		}`
 	WorkspaceLevel(t, Step{
-		Template: workspaceFile + makePermissionsTestStage("workspace_file_path", "databricks_workspace_file.this.id", groupPermissions("CAN_RUN")),
+		Template: workspaceFile + makePermissionsTestStage("workspace_file_path", "databricks_workspace_file.this.id",
+			groupPermissions("CAN_RUN")),
 	}, Step{
-		Template: workspaceFile + makePermissionsTestStage("workspace_file_path", "databricks_workspace_file.this.id", currentPrincipalPermission(t, "CAN_MANAGE"), allPrincipalPermissions("CAN_RUN", "CAN_READ", "CAN_EDIT", "CAN_MANAGE")),
+		Template: workspaceFile + makePermissionsTestStage("workspace_file_path", "databricks_workspace_file.this.id",
+			currentPrincipalPermission(t, "CAN_MANAGE"),
+			allPrincipalPermissions("CAN_RUN", "CAN_READ", "CAN_EDIT", "CAN_MANAGE")),
 	}, Step{
 		// The current user can be removed from permissions since they inherit permissions from the directory they created.
-		Template: workspaceFile + makePermissionsTestStage("workspace_file_path", "databricks_workspace_file.this.id", allPrincipalPermissions("CAN_RUN", "CAN_READ", "CAN_EDIT", "CAN_MANAGE")),
+		Template: workspaceFile + makePermissionsTestStage("workspace_file_path", "databricks_workspace_file.this.id",
+			allPrincipalPermissions("CAN_RUN", "CAN_READ", "CAN_EDIT", "CAN_MANAGE")),
 	}, Step{
-		Template:    workspaceFile + makePermissionsTestStage("workspace_file_path", "databricks_workspace_file.this.id", currentPrincipalPermission(t, "CAN_READ")),
+		Template: workspaceFile + makePermissionsTestStage("workspace_file_path", "databricks_workspace_file.this.id",
+			currentPrincipalPermission(t, "CAN_READ")),
 		ExpectError: regexp.MustCompile("cannot remove management permissions for the current user for file, allowed levels: CAN_MANAGE"),
 	})
 }
@@ -489,17 +494,22 @@ func TestAccPermissions_WorkspaceFile_Id(t *testing.T) {
 		}
 		resource "databricks_workspace_file" "this" {
 			source = "{var.CWD}/../../storage/testdata/tf-test-python.py"
-			path = "${databricks_directory.this.path}/test_notebook"
+			path = "${databricks_directory.this.path}/test_ws_file"
 		}`
 	WorkspaceLevel(t, Step{
-		Template: workspaceFile + makePermissionsTestStage("workspace_file_id", "databricks_workspace_file.this.object_id", groupPermissions("CAN_RUN")),
+		Template: workspaceFile + makePermissionsTestStage("workspace_file_id", "databricks_workspace_file.this.object_id",
+			groupPermissions("CAN_RUN")),
 	}, Step{
-		Template: workspaceFile + makePermissionsTestStage("workspace_file_id", "databricks_workspace_file.this.object_id", currentPrincipalPermission(t, "CAN_MANAGE"), allPrincipalPermissions("CAN_RUN", "CAN_READ", "CAN_EDIT", "CAN_MANAGE")),
+		Template: workspaceFile + makePermissionsTestStage("workspace_file_id", "databricks_workspace_file.this.object_id",
+			currentPrincipalPermission(t, "CAN_MANAGE"),
+			allPrincipalPermissions("CAN_RUN", "CAN_READ", "CAN_EDIT", "CAN_MANAGE")),
 	}, Step{
 		// The current user can be removed from permissions since they inherit permissions from the directory they created.
-		Template: workspaceFile + makePermissionsTestStage("workspace_file_id", "databricks_workspace_file.this.object_id", allPrincipalPermissions("CAN_RUN", "CAN_READ", "CAN_EDIT", "CAN_MANAGE")),
+		Template: workspaceFile + makePermissionsTestStage("workspace_file_id", "databricks_workspace_file.this.object_id",
+			allPrincipalPermissions("CAN_RUN", "CAN_READ", "CAN_EDIT", "CAN_MANAGE")),
 	}, Step{
-		Template:    workspaceFile + makePermissionsTestStage("workspace_file_id", "databricks_workspace_file.this.object_id", currentPrincipalPermission(t, "CAN_READ")),
+		Template: workspaceFile + makePermissionsTestStage("workspace_file_id", "databricks_workspace_file.this.object_id",
+			currentPrincipalPermission(t, "CAN_READ")),
 		ExpectError: regexp.MustCompile("cannot remove management permissions for the current user for file, allowed levels: CAN_MANAGE"),
 	})
 }

internal/acceptance/workspace_file_test.go

@@ -27,6 +27,15 @@ func TestAccWorkspaceFileEmptyFile(t *testing.T) {
 	})
 }
 
+func TestAccWorkspaceFileZipFile(t *testing.T) {
+	WorkspaceLevel(t, Step{
+		Template: `resource "databricks_workspace_file" "zipfile" {
+			source = "{var.CWD}/../../workspace/acceptance/testdata/zipfile.zip"
+			path = "/Shared/provider-test/zipfile_{var.RANDOM}.zip"
+		}`,
+	})
+}
+
 func TestAccWorkspaceFileBase64(t *testing.T) {
 	WorkspaceLevel(t, Step{
 		Template: `resource "databricks_workspace_file" "this2" {

workspace/resource_workspace_file.go

@@ -44,7 +44,7 @@ func ResourceWorkspaceFile() common.Resource {
 			path := d.Get("path").(string)
 			importReq := workspace.Import{
 				Content:         base64.StdEncoding.EncodeToString(content),
-				Format:          workspace.ImportFormatAuto,
+				Format:          workspace.ImportFormatRaw,
 				Path:            path,
 				Overwrite:       true,
 				ForceSendFields: []string{"Content"},
@@ -93,7 +93,7 @@ func ResourceWorkspaceFile() common.Resource {
 			}
 			return client.Workspace.Import(ctx, workspace.Import{
 				Content:         base64.StdEncoding.EncodeToString(content),
-				Format:          workspace.ImportFormatAuto,
+				Format:          workspace.ImportFormatRaw,
 				Overwrite:       true,
 				Path:            d.Id(),
 				ForceSendFields: []string{"Content"},

workspace/resource_workspace_file_test.go

@@ -120,7 +120,7 @@ func TestResourceWorkspaceFileCreate_DirectoryExist(t *testing.T) {
 					Content:   dummyWorkspaceFilePayload,
 					Path:      dummyWorkspaceFilePath,
 					Overwrite: true,
-					Format:    "AUTO",
+					Format:    "RAW",
 				},
 			},
 			{
@@ -168,7 +168,7 @@ func TestResourceWorkspaceFileCreate_DirectoryDoesntExist(t *testing.T) {
 					Content:   dummyWorkspaceFilePayload,
 					Path:      dummyWorkspaceFilePath,
 					Overwrite: true,
-					Format:    "AUTO",
+					Format:    "RAW",
 				},
 				Response: map[string]string{
 					"error_code": "RESOURCE_DOES_NOT_EXIST",
@@ -183,7 +183,7 @@ func TestResourceWorkspaceFileCreate_DirectoryDoesntExist(t *testing.T) {
 					Content:   dummyWorkspaceFilePayload,
 					Path:      dummyWorkspaceFilePath,
 					Overwrite: true,
-					Format:    "AUTO",
+					Format:    "RAW",
 				},
 			},
 			{
@@ -236,7 +236,7 @@ func TestResourceWorkspaceFileCreate_DirectoryCreateError(t *testing.T) {
 					Content:   dummyWorkspaceFilePayload,
 					Path:      dummyWorkspaceFilePath,
 					Overwrite: true,
-					Format:    "AUTO",
+					Format:    "RAW",
 				},
 				Response: map[string]string{
 					"error_code": "RESOURCE_DOES_NOT_EXIST",
@@ -267,7 +267,7 @@ func TestResourceWorkspaceFileCreateSource(t *testing.T) {
 						"NPTU1BTkQgLS0tLS0tLS0tLQoKCg==",
 					Path:      "/Dashboard",
 					Overwrite: true,
-					Format:    "AUTO",
+					Format:    "RAW",
 				},
 			},
 			{
@@ -301,7 +301,7 @@ func TestResourceWorkspaceFileCreateEmptyFileSource(t *testing.T) {
 					Content:         "",
 					Path:            "/__init__.py",
 					Overwrite:       true,
-					Format:          "AUTO",
+					Format:          "RAW",
 					ForceSendFields: []string{"Content"},
 				},
 			},
@@ -334,7 +334,7 @@ func TestResourceWorkspaceFileCreate_Error(t *testing.T) {
 				Resource: "/api/2.0/workspace/import",
 				ExpectedRequest: map[string]interface{}{
 					"content":   dummyWorkspaceFilePayload,
-					"format":    "AUTO",
+					"format":    "RAW",
 					"overwrite": true,
 					"path":      "/path.py",
 				},
@@ -384,7 +384,7 @@ func TestResourceWorkspaceFileUpdate(t *testing.T) {
 				Method:   "POST",
 				Resource: "/api/2.0/workspace/import",
 				ExpectedRequest: ws_api.Import{
-					Format:    "AUTO",
+					Format:    "RAW",
 					Overwrite: true,
 					Content:   dummyWorkspaceFilePayload,
 					Path:      "abc",