Improve databricks_git_credential resource by correctly handle new fields (#4854)

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

SDK 0.75.0 brought two new optional fields, so documented them and
correctly handle zero-values.

Also, refactored unit tests to use Mock framework instead of HTTP
fixtures

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

- [x] `make test` run locally
- [x] relevant change in `docs/` folder
- [x] covered with integration tests in `internal/acceptance`
- [x] using Go SDK
- [ ] using TF Plugin Framework
- [x] has entry in `NEXT_CHANGELOG.md` file

Author: alexott

NEXT_CHANGELOG.md

@@ -13,9 +13,10 @@
 * Document new fields in `databricks_model_serving` and deprecate `invalid_keywords` and `valid_topics` in AI Gateway configuration ([#4851](https://github.com/databricks/terraform-provider-databricks/pull/4851)).
 * Added support for Alert V2 in `databricks_permissions` ([#4831](https://github.com/databricks/terraform-provider-databricks/pull/4831)).
 * Replace instead of dropping Delta `databricks_sql_table` ([#2424](https://github.com/databricks/terraform-provider-databricks/pull/2424)).
-* Added `clean_rooms_clean_room` resource and data sources ([#4844](https://github.com/databricks/terraform-provider-databricks/pull/4844)).
-* Added `external_metadata` resource and data sources ([#4844](https://github.com/databricks/terraform-provider-databricks/pull/4844)).
-* Added `materialized_features_feature_tag` resource and data sources ([#4844](https://github.com/databricks/terraform-provider-databricks/pull/4844)).
+* Added `databricks_clean_rooms_clean_room` resource and data sources ([#4844](https://github.com/databricks/terraform-provider-databricks/pull/4844)).
+* Added `databricks_external_metadata` resource and data sources ([#4844](https://github.com/databricks/terraform-provider-databricks/pull/4844)).
+* Added `databricks_materialized_features_feature_tag` resource and data sources ([#4844](https://github.com/databricks/terraform-provider-databricks/pull/4844)).
+* Improve `databricks_git_credential` resource by correctly handle new fields ([#4854](https://github.com/databricks/terraform-provider-databricks/pull/4854)).
 
 ### Bug Fixes
 

docs/resources/git_credential.md

@@ -38,6 +38,8 @@ The following arguments are supported:
 * `personal_access_token` - (Optional, required for some Git providers) The personal access token used to authenticate to the corresponding Git provider. If value is not provided, it's sourced from the first environment variable of [`GITHUB_TOKEN`](https://registry.terraform.io/providers/integrations/github/latest/docs#oauth--personal-access-token), [`GITLAB_TOKEN`](https://registry.terraform.io/providers/gitlabhq/gitlab/latest/docs#required), or [`AZDO_PERSONAL_ACCESS_TOKEN`](https://registry.terraform.io/providers/microsoft/azuredevops/latest/docs#argument-reference), that has a non-empty value.
 * `git_username` - (Optional, required for some Git providers) user name at Git provider.
 * `git_provider` -  (Required) case insensitive name of the Git provider.  Following values are supported right now (could be a subject for a change, consult [Git Credentials API documentation](https://docs.databricks.com/dev-tools/api/latest/gitcredentials.html)): `gitHub`, `gitHubEnterprise`, `bitbucketCloud`, `bitbucketServer`, `azureDevOpsServices`, `gitLab`, `gitLabEnterpriseEdition`, `awsCodeCommit`, `azureDevOpsServicesAad`.
+* `is_default_for_provider` - (Optional) boolean flag specifying if the credential is the default for the given provider type.
+* `name` - (Optional) the name of the git credential, used for identification and ease of lookup.
 * `force` - (Optional) specify if settings need to be enforced.
 
 ## Attribute Reference

repos/resource_git_credential.go

@@ -92,6 +92,12 @@ func ResourceGitCredential() common.Resource {
 			if err != nil {
 				return err
 			}
+			if d.HasChange("is_default_for_provider") {
+				req.ForceSendFields = append(req.ForceSendFields, "IsDefaultForProvider")
+			}
+			if d.HasChange("name") {
+				req.ForceSendFields = append(req.ForceSendFields, "Name")
+			}
 			return w.GitCredentials.Update(ctx, req)
 		},
 		Delete: func(ctx context.Context, d *schema.ResourceData, c *common.DatabricksClient) error {