Fix linter issues

vivgoyal · vivgoyal · commit 1f051c02afbc · 2024-12-17T12:08:11.000-08:00
Signed-off-by: Vivek Goyal &lt;vivek.goyal@broadcom.com&gt;
diff --git a/.golangci.yml b/.golangci.yml
@@ -26,7 +26,6 @@ linters:
     - asciicheck
     - bodyclose
     - copyloopvar
-    - depguard
     - dogsled
     - dupl
     - errcheck
diff --git a/internal/client/proxy/tls_test.go b/internal/client/proxy/tls_test.go
@@ -47,6 +47,7 @@ J+guVb41E7ckHez4ENTEj2hrqrYifViGvOaSAG3w1d8PW+wIj3jI6vQk7vTO6mbt
 ws3Up/6R4aI0ohB4wjOfUCL6x9L3pyQcDoumDuy8ToyL5Pk=
 -----END CERTIFICATE-----
 `
+	// nolint: gosec
 	clientKey = `
 -----BEGIN RSA PRIVATE KEY-----
 MIIEpQIBAAKCAQEA51mGnwWS2HJTzbfGgALTgBz69OMEgPGljrfv58lOU50V+bU2
@@ -281,15 +282,19 @@ func TestGetConnectorTLSConfig(t *testing.T) {
 				require.EqualError(t, actualErr, tc.expectedErr.Error())
 				return
 			}
+
 			require.NoError(t, actualErr)
+
 			if tc.expectedTLSConfig.GetClientCertificate != nil {
 				require.NotNil(t, actualTLSConfig.GetClientCertificate)
 				return
 			}
+
 			if tc.expectedTLSConfig.RootCAs != nil {
 				require.NotNil(t, actualTLSConfig.RootCAs)
 				return
 			}
+
 			require.Equal(t, actualTLSConfig, tc.expectedTLSConfig)
 		})
 	}
diff --git a/internal/client/transport/methods_test.go b/internal/client/transport/methods_test.go
@@ -61,5 +61,6 @@ func TestConcurrentAccessOfInvokeAction(t *testing.T) {
 			require.Error(t, actual)
 		}()
 	}
+
 	waitGroup.Wait()
 }
diff --git a/internal/helper/helper.go b/internal/helper/helper.go
@@ -81,6 +81,7 @@ func readString(input interface{}, key string) string {
 	return data
 }
 
+// nolint:gosec
 func SetPrimitiveValue(input, model interface{}, key string) {
 	switch m := model.(type) {
 	case *float64:
diff --git a/internal/helper/retry.go b/internal/helper/retry.go
@@ -28,6 +28,7 @@ func Retry(f Retryable, interval time.Duration, attempts int) (int, error) {
 		}
 
 		time.Sleep(interval)
+
 		retries++
 	}
 
diff --git a/internal/resources/akscluster/resource_akscluster_test.go b/internal/resources/akscluster/resource_akscluster_test.go
@@ -841,6 +841,7 @@ func Test_pollUntilReady(t *testing.T) {
 						tt.args.interval, tt.wantError, gotErr)
 				}
 			}
+
 			if tt.validation != nil {
 				tt.validation(t, tt.args)
 			}
diff --git a/internal/resources/cluster/tkgaws/resource_tkg_aws.go b/internal/resources/cluster/tkgaws/resource_tkg_aws.go
@@ -275,6 +275,7 @@ func expandTKGAWSClusterNetwork(data []interface{}) (cluster *tkgawsmodel.Vmware
 
 	if v, ok := lookUpCluster[apiServerPortKey]; ok {
 		apiServerPort := v.(int)
+		// nolint:gosec
 		cluster.APIServerPort = int32(apiServerPort)
 	}
 
diff --git a/internal/resources/clusterclass/datasource_cluster_class.go b/internal/resources/clusterclass/datasource_cluster_class.go
@@ -41,7 +41,7 @@ func dataSourceClusterClassRead(ctx context.Context, data *schema.ResourceData,
 	switch {
 	case err != nil:
 		return diag.FromErr(errors.Wrap(err, "Couldn't read cluster class"))
-	case resp.ClusterClasses == nil || len(resp.ClusterClasses) == 0:
+	case len(resp.ClusterClasses) == 0:
 		data.SetId("NO_DATA")
 	default:
 		clusterClass := resp.ClusterClasses[0]
diff --git a/internal/resources/ekscluster/ekscluster_flatten_test.go b/internal/resources/ekscluster/ekscluster_flatten_test.go
@@ -444,6 +444,7 @@ func TestFlattenClusterSpec(t *testing.T) {
 		t.Run(test.description, func(t *testing.T) {
 			spec, nps := test.getInput()
 			output := flattenClusterSpec(spec, nps)
+
 			print(test.expected)
 			print(output)
 			require.Equal(t, test.expected, output)
diff --git a/internal/resources/ekscluster/helpers_test.go b/internal/resources/ekscluster/helpers_test.go
@@ -321,8 +321,9 @@ func TestNodepoolSpecEqual(t *testing.T) {
 	for _, test := range tests {
 		t.Run(test.name, func(t *testing.T) {
 			spec1 := getNodepoolSpec()
-			spec2 := getNodepoolSpec()
 			test.modifySpec1(spec1)
+
+			spec2 := getNodepoolSpec()
 			test.modifySpec2(spec2)
 
 			require.Equal(t, test.result, nodepoolSpecEqual(spec1, spec2), "return didn't match the expected output")
@@ -693,8 +694,9 @@ func TestClusterSpecEqual(t *testing.T) {
 	for _, test := range tests {
 		t.Run(test.name, func(t *testing.T) {
 			spec1, _ := getClusterSpec()
-			spec2, _ := getClusterSpec()
 			test.modifySpec1(spec1)
+
+			spec2, _ := getClusterSpec()
 			test.modifySpec2(spec2)
 
 			require.Equal(t, test.result, clusterSpecEqual(spec1, spec2), "return didn't match the expected output")
diff --git a/internal/resources/gitrepository/data_source_git_repository.go b/internal/resources/gitrepository/data_source_git_repository.go
@@ -65,7 +65,7 @@ func dataSourceGitRepositoryRead(ctx context.Context, d *schema.ResourceData, m
 	if err != nil {
 		if clienterrors.IsNotFoundError(err) && !helper.IsDataRead(ctx) {
 			_ = schema.RemoveFromState(d, m)
-			return
+			return diags
 		}
 
 		return diag.FromErr(err)
diff --git a/internal/resources/gitrepository/resource_git_repository.go b/internal/resources/gitrepository/resource_git_repository.go
@@ -226,7 +226,7 @@ func resourceGitRepositoryInPlaceUpdate(ctx context.Context, d *schema.ResourceD
 
 	if !updateAvailable {
 		log.Printf("[INFO] git repository update is not required")
-		return
+		return diags
 	}
 
 	switch scopedFullnameData.Scope {
diff --git a/internal/resources/helmcharts/helm_charts_mock_test.go b/internal/resources/helmcharts/helm_charts_mock_test.go
@@ -20,13 +20,6 @@ import (
 	objectmetamodel "github.com/vmware/terraform-provider-tanzu-mission-control/internal/models/objectmeta"
 )
 
-const (
-	https                = "https:/"
-	clAPIVersionAndGroup = "v1alpha1/organization/fluxcd/helm/repositories"
-	apiSubGroup          = "chartmetadatas"
-	apiKind              = "charts"
-)
-
 // nolint: unused
 func bodyInspectingResponder(t *testing.T, expectedContent interface{}, successResponse int, successResponseBody interface{}) httpmock.Responder {
 	return func(r *http.Request) (*http.Response, error) {
@@ -120,6 +113,13 @@ func (testConfig *testAcceptanceConfig) setupHTTPMocks(t *testing.T) {
 		},
 	}
 
+	const (
+		https                = "https:/"
+		clAPIVersionAndGroup = "v1alpha1/organization/fluxcd/helm/repositories"
+		apiSubGroup          = "chartmetadatas"
+		apiKind              = "charts"
+	)
+
 	getPkgEndpoint := (helper.ConstructRequestURL(https, endpoint, clAPIVersionAndGroup, "*", apiSubGroup, testConfig.ChartMetadataName, apiKind)).String()
 
 	httpmock.RegisterResponder("GET", getPkgEndpoint,
diff --git a/internal/resources/helmfeature/data_source_helm_feature.go b/internal/resources/helmfeature/data_source_helm_feature.go
@@ -53,7 +53,7 @@ func dataSourceHelmRead(ctx context.Context, d *schema.ResourceData, m interface
 	if err != nil {
 		if clienterrors.IsNotFoundError(err) && !helper.IsDataRead(ctx) {
 			_ = schema.RemoveFromState(d, m)
-			return
+			return diags
 		}
 
 		return diag.FromErr(err)
diff --git a/internal/resources/helmrelease/data_source_helm_release.go b/internal/resources/helmrelease/data_source_helm_release.go
@@ -65,7 +65,7 @@ func dataSourceHelmReleaseRead(ctx context.Context, d *schema.ResourceData, m in
 	if err != nil {
 		if clienterrors.IsNotFoundError(err) && !helper.IsDataRead(ctx) {
 			_ = schema.RemoveFromState(d, m)
-			return
+			return diags
 		}
 
 		return diag.FromErr(err)
diff --git a/internal/resources/helmrelease/resource_helm_release.go b/internal/resources/helmrelease/resource_helm_release.go
@@ -267,7 +267,7 @@ func resourceHelmReleaseInPlaceUpdate(ctx context.Context, d *schema.ResourceDat
 
 	if !updateAvailable {
 		log.Printf("[INFO] helm release update is not required")
-		return
+		return diags
 	}
 
 	switch scopedFullnameData.Scope {
diff --git a/internal/resources/helmrepository/helm_repository_data_source.go b/internal/resources/helmrepository/helm_repository_data_source.go
@@ -84,7 +84,7 @@ func dataSourceHelmRepositoryRead(ctx context.Context, d *schema.ResourceData, m
 	if err != nil {
 		if clienterrors.IsNotFoundError(err) && !helper.IsDataRead(ctx) {
 			_ = schema.RemoveFromState(d, m)
-			return
+			return diags
 		}
 
 		return diag.FromErr(err)
diff --git a/internal/resources/helmrepository/helm_repository_mock_test.go b/internal/resources/helmrepository/helm_repository_mock_test.go
@@ -20,13 +20,6 @@ import (
 	objectmetamodel "github.com/vmware/terraform-provider-tanzu-mission-control/internal/models/objectmeta"
 )
 
-const (
-	https                = "https:/"
-	clAPIVersionAndGroup = "v1alpha1/clusters"
-	apiSubGroup          = "namespaces"
-	apiKind              = "fluxcd/helm/repositories"
-)
-
 // nolint: unused
 func bodyInspectingResponder(t *testing.T, expectedContent interface{}, successResponse int, successResponseBody interface{}) httpmock.Responder {
 	return func(r *http.Request) (*http.Response, error) {
@@ -124,6 +117,13 @@ func (testConfig *testAcceptanceConfig) setupHTTPMocks(t *testing.T) {
 		},
 	}
 
+	const (
+		https                = "https:/"
+		clAPIVersionAndGroup = "v1alpha1/clusters"
+		apiSubGroup          = "namespaces"
+		apiKind              = "fluxcd/helm/repositories"
+	)
+
 	getPkgEndpoint := (helper.ConstructRequestURL(https, endpoint, clAPIVersionAndGroup, testConfig.ScopeHelperResources.Cluster.Name, apiSubGroup, testConfig.Namespace, apiKind)).String()
 
 	httpmock.RegisterResponder("GET", getPkgEndpoint,
diff --git a/internal/resources/iampolicy/resource_iam_policy.go b/internal/resources/iampolicy/resource_iam_policy.go
@@ -223,7 +223,7 @@ func resourceIAMPolicyRead(_ context.Context, d *schema.ResourceData, m interfac
 	// when iam policy resource is empty: no role bindings are existing, this is equivalent to not found condition.
 	if len(policyList) == 0 {
 		_ = schema.RemoveFromState(d, m)
-		return
+		return diags
 	}
 
 	d.SetId(d.State().ID)
diff --git a/internal/resources/kustomization/resource_kustomization.go b/internal/resources/kustomization/resource_kustomization.go
@@ -86,7 +86,7 @@ func resourceKustomizationRead(ctx context.Context, d *schema.ResourceData, m in
 	if err != nil {
 		if clienterrors.IsNotFoundError(err) {
 			_ = schema.RemoveFromState(d, m)
-			return
+			return diags
 		}
 
 		return diag.FromErr(err)
@@ -346,7 +346,7 @@ func resourceKustomizationInPlaceUpdate(ctx context.Context, d *schema.ResourceD
 
 	if !updateAvailable {
 		log.Printf("[INFO] kustomization update is not required")
-		return
+		return diags
 	}
 
 	switch scopedFullnameData.Scope {
diff --git a/internal/resources/permissiontemplate/datasource_permission_template.go b/internal/resources/permissiontemplate/datasource_permission_template.go
@@ -69,7 +69,7 @@ func dataSourcePermissionTemplateRead(ctx context.Context, data *schema.Resource
 		}
 	}
 
-	if response.TemplateValues != nil && len(response.TemplateValues) > 0 {
+	if len(response.TemplateValues) > 0 {
 		// This is necessary because sometimes the template parameters definition and the template values returned from the API do not match.
 		err = removeUndefinedTemplateValues(response)
 
diff --git a/internal/resources/permissiontemplate/schema.go b/internal/resources/permissiontemplate/schema.go
@@ -113,6 +113,7 @@ func buildCapabilityProviderDescription(schemaKey string) (description string) {
 		for k, v := range capabilityProviderMap {
 			valueDescription := fmt.Sprintf("When %s is set to '%s' %s must be set to '%s'.", ProviderKey, v, CapabilityKey, k)
 			description = fmt.Sprintf("%s\n%s", description, valueDescription)
+
 			validValues = append(validValues, v)
 		}
 
diff --git a/internal/resources/policy/kind/network/recipe/custom_common_schema.go b/internal/resources/policy/kind/network/recipe/custom_common_schema.go
@@ -295,7 +295,6 @@ func flattenCustomRule(rule *policyrecipenetworkcommonmodel.V1alpha1CommonPolicy
 		var ports []interface{}
 
 		for _, port := range *rule.Ports {
-			port := port
 			ports = append(ports, flattenPort(&port))
 		}
 
diff --git a/internal/resources/policy/kind/network/recipe/custom_egress_schema.go b/internal/resources/policy/kind/network/recipe/custom_egress_schema.go
@@ -129,7 +129,6 @@ func FlattenCustomEgress(customEgress *policyrecipenetworkmodel.V1alpha1CommonPo
 		var rules []interface{}
 
 		for _, rule := range customEgress.Rules {
-			rule := rule
 			rules = append(rules, flattenCustomRule(&rule))
 		}
 
diff --git a/internal/resources/policy/kind/network/recipe/custom_ingress_schema.go b/internal/resources/policy/kind/network/recipe/custom_ingress_schema.go
@@ -129,7 +129,6 @@ func FlattenCustomIngress(customIngress *policyrecipenetworkmodel.V1alpha1Common
 		var rules []interface{}
 
 		for _, rule := range customIngress.Rules {
-			rule := rule
 			rules = append(rules, flattenCustomRule(&rule))
 		}
 
diff --git a/internal/resources/policy/operations/update.go b/internal/resources/policy/operations/update.go
@@ -64,7 +64,7 @@ func ResourcePolicyInPlaceUpdate(ctx context.Context, d *schema.ResourceData, m
 	if !updateAvailable {
 		log.Printf("[INFO] %s policy update is not required", rn)
 
-		return
+		return diags
 	}
 
 	switch scopedFullnameData.Scope {
diff --git a/internal/resources/provisioner/data_source_provisioner.go b/internal/resources/provisioner/data_source_provisioner.go
@@ -75,19 +75,20 @@ func dataSourceProvisionerRead(ctx context.Context, d *schema.ResourceData, m in
 		if err != nil {
 			if clienterrors.IsNotFoundError(err) && !helper.IsDataRead(ctx) {
 				_ = schema.RemoveFromState(d, m)
-				return
+				return diags
 			}
 
-			return
+			return diags
 		}
 	} else {
 		getResp, err := config.TMCConnection.ProvisionerResourceService.ProvisionerResourceServiceGet(model.Provisioners[0].FullName)
 		if err != nil {
 			if clienterrors.IsNotFoundError(err) && !helper.IsDataRead(ctx) {
 				_ = schema.RemoveFromState(d, m)
-				return
+				return diags
 			}
-			return
+
+			return diags
 		}
 
 		p := &provisioner.VmwareTanzuManageV1alpha1ManagementclusterProvisionerListprovisionersResponse{
diff --git a/internal/resources/sourcesecret/resource_sourcesecret.go b/internal/resources/sourcesecret/resource_sourcesecret.go
@@ -262,7 +262,7 @@ func resourceSourcesecretInPlaceUpdate(ctx context.Context, d *schema.ResourceDa
 	}
 
 	if !updateAvailable {
-		return
+		return diags
 	}
 
 	switch scopedFullnameData.Scope {
diff --git a/internal/resources/tanzukubernetescluster/helper.go b/internal/resources/tanzukubernetescluster/helper.go
@@ -205,7 +205,7 @@ func waitNodePoolsReady(ctx context.Context, config *authctx.TanzuContext, clust
 						errMsg = fmt.Sprintf("%s\n%s", errMsg, npStatusMsg)
 					}
 
-					err = errors.Wrapf(err, errMsg)
+					err = errors.Wrapf(err, "%s", errMsg)
 				}
 
 				return err
@@ -461,6 +461,7 @@ func removeModelVariable(clusterClassSchema interface{}, modelVariable interface
 		} else {
 			modelVarAdditionalProperties, additionalPropertiesExist := clusterClassSchema.(map[string]interface{})[string(openapiv3.AdditionalPropertiesKey)]
 			preserve, preserveExists := clusterClassSchema.(map[string]interface{})[string(openapiv3.PreserveUnknownFieldKey)]
+
 			if additionalPropertiesExist {
 				_, propertiesExist := modelVarAdditionalProperties.(map[string]interface{})[string(openapiv3.PropertiesKey)]
 
diff --git a/internal/resources/tanzukubernetescluster/resource_tanzu_kuberenetes_cluster.go b/internal/resources/tanzukubernetescluster/resource_tanzu_kuberenetes_cluster.go
@@ -392,7 +392,7 @@ func validateSchema(_ context.Context, data *schema.ResourceDiff, value interfac
 	if err != nil {
 		return errors.Wrapf(err, "Couldn't find cluster class.\nManagement Cluster Name: %s, Provisioner: %s, Cluster Class Name: %s.",
 			clusterClassFn.ManagementClusterName, clusterClassFn.ProvisionerName, clusterClassFn.Name)
-	} else if resp.ClusterClasses == nil || len(resp.ClusterClasses) == 0 {
+	} else if len(resp.ClusterClasses) == 0 {
 		return errors.Errorf("Couldn't find cluster class.\nManagement Cluster Name: %s, Provisioner: %s, Cluster Class Name: %s.",
 			clusterClassFn.ManagementClusterName, clusterClassFn.ProvisionerName, clusterClassFn.Name)
 	}
diff --git a/internal/resources/tanzupackageinstall/spec/cluster_scope.go b/internal/resources/tanzupackageinstall/spec/cluster_scope.go
@@ -18,7 +18,10 @@ import (
 	"github.com/vmware/terraform-provider-tanzu-mission-control/internal/resources/common"
 )
 
-func ConstructSpecForClusterScope(d *schema.ResourceData) (spec *packageinstallmodel.VmwareTanzuManageV1alpha1ClusterNamespaceTanzupackageInstallSpec, err error) {
+// nolint: gocognit
+func ConstructSpecForClusterScope(d *schema.ResourceData) (
+	spec *packageinstallmodel.VmwareTanzuManageV1alpha1ClusterNamespaceTanzupackageInstallSpec,
+	err error) {
 	value, ok := d.GetOk(SpecKey)
 	if !ok {
 		return spec, nil
diff --git a/internal/resources/tanzupackagerepository/resource_package_repository.go b/internal/resources/tanzupackagerepository/resource_package_repository.go
@@ -239,7 +239,7 @@ func resourcePackageRepositoryInPlaceUpdate(ctx context.Context, d *schema.Resou
 	}
 
 	if !updateCheckForMeta(d, pkgRepoDataFromServer.meta) && !updateCheckForSpec(d, pkgRepoDataFromServer.spec) && !d.HasChange(disabledKey) {
-		return
+		return diags
 	}
 
 	return dataPackageRepositoryRead(ctx, d, m)

Original file line number	Diff line number	Diff line change
`@@ -47,6 +47,7 @@ J+guVb41E7ckHez4ENTEj2hrqrYifViGvOaSAG3w1d8PW+wIj3jI6vQk7vTO6mbt`
`47`	`47`	`ws3Up/6R4aI0ohB4wjOfUCL6x9L3pyQcDoumDuy8ToyL5Pk=`
`48`	`48`	`-----END CERTIFICATE-----`
`49`	`49`	`
	`50`	`+ // nolint: gosec`
`50`	`51`	clientKey = `
`51`	`52`	`-----BEGIN RSA PRIVATE KEY-----`
`52`	`53`	`MIIEpQIBAAKCAQEA51mGnwWS2HJTzbfGgALTgBz69OMEgPGljrfv58lOU50V+bU2`
`@@ -281,15 +282,19 @@ func TestGetConnectorTLSConfig(t *testing.T) {`
`281`	`282`	`require.EqualError(t, actualErr, tc.expectedErr.Error())`
`282`	`283`	`return`
`283`	`284`	`}`
	`285`	`+`
`284`	`286`	`require.NoError(t, actualErr)`
	`287`	`+`
`285`	`288`	`if tc.expectedTLSConfig.GetClientCertificate != nil {`
`286`	`289`	`require.NotNil(t, actualTLSConfig.GetClientCertificate)`
`287`	`290`	`return`
`288`	`291`	`}`
	`292`	`+`
`289`	`293`	`if tc.expectedTLSConfig.RootCAs != nil {`
`290`	`294`	`require.NotNil(t, actualTLSConfig.RootCAs)`
`291`	`295`	`return`
`292`	`296`	`}`
	`297`	`+`
`293`	`298`	`require.Equal(t, actualTLSConfig, tc.expectedTLSConfig)`
`294`	`299`	`})`
`295`	`300`	`}`
Original file line number	Diff line number	Diff line change
`@@ -61,5 +61,6 @@ func TestConcurrentAccessOfInvokeAction(t *testing.T) {`
`61`	`61`	`require.Error(t, actual)`
`62`	`62`	`}()`
`63`	`63`	`}`
	`64`	`+`
`64`	`65`	`waitGroup.Wait()`
`65`	`66`	`}`
Original file line number	Diff line number	Diff line change
`@@ -81,6 +81,7 @@ func readString(input interface{}, key string) string {`
`81`	`81`	`return data`
`82`	`82`	`}`
`83`	`83`
	`84`	`+// nolint:gosec`
`84`	`85`	`func SetPrimitiveValue(input, model interface{}, key string) {`
`85`	`86`	`switch m := model.(type) {`
`86`	`87`	`case *float64:`
Original file line number	Diff line number	Diff line change
`@@ -28,6 +28,7 @@ func Retry(f Retryable, interval time.Duration, attempts int) (int, error) {`
`28`	`28`	`}`
`29`	`29`
`30`	`30`	`time.Sleep(interval)`
	`31`	`+`
`31`	`32`	`retries++`
`32`	`33`	`}`
`33`	`34`
Original file line number	Diff line number	Diff line change
`@@ -841,6 +841,7 @@ func Test_pollUntilReady(t *testing.T) {`
`841`	`841`	`tt.args.interval, tt.wantError, gotErr)`
`842`	`842`	`}`
`843`	`843`	`}`
	`844`	`+`
`844`	`845`	`if tt.validation != nil {`
`845`	`846`	`tt.validation(t, tt.args)`
`846`	`847`	`}`
Original file line number	Diff line number	Diff line change
`@@ -275,6 +275,7 @@ func expandTKGAWSClusterNetwork(data []interface{}) (cluster *tkgawsmodel.Vmware`
`275`	`275`
`276`	`276`	`if v, ok := lookUpCluster[apiServerPortKey]; ok {`
`277`	`277`	`apiServerPort := v.(int)`
	`278`	`+ // nolint:gosec`
`278`	`279`	`cluster.APIServerPort = int32(apiServerPort)`
`279`	`280`	`}`
`280`	`281`
Original file line number	Diff line number	Diff line change
`@@ -65,7 +65,7 @@ func dataSourceGitRepositoryRead(ctx context.Context, d *schema.ResourceData, m`
`65`	`65`	`if err != nil {`
`66`	`66`	`if clienterrors.IsNotFoundError(err) && !helper.IsDataRead(ctx) {`
`67`	`67`	`_ = schema.RemoveFromState(d, m)`
`68`		`- return`
	`68`	`+ return diags`
`69`	`69`	`}`
`70`	`70`
`71`	`71`	`return diag.FromErr(err)`
Original file line number	Diff line number	Diff line change
`@@ -226,7 +226,7 @@ func resourceGitRepositoryInPlaceUpdate(ctx context.Context, d *schema.ResourceD`
`226`	`226`
`227`	`227`	`if !updateAvailable {`
`228`	`228`	`log.Printf("[INFO] git repository update is not required")`
`229`		`- return`
	`229`	`+ return diags`
`230`	`230`	`}`
`231`	`231`
`232`	`232`	`switch scopedFullnameData.Scope {`
Original file line number	Diff line number	Diff line change
`@@ -53,7 +53,7 @@ func dataSourceHelmRead(ctx context.Context, d *schema.ResourceData, m interface`
`53`	`53`	`if err != nil {`
`54`	`54`	`if clienterrors.IsNotFoundError(err) && !helper.IsDataRead(ctx) {`
`55`	`55`	`_ = schema.RemoveFromState(d, m)`
`56`		`- return`
	`56`	`+ return diags`
`57`	`57`	`}`
`58`	`58`
`59`	`59`	`return diag.FromErr(err)`
Original file line number	Diff line number	Diff line change
`@@ -65,7 +65,7 @@ func dataSourceHelmReleaseRead(ctx context.Context, d *schema.ResourceData, m in`
`65`	`65`	`if err != nil {`
`66`	`66`	`if clienterrors.IsNotFoundError(err) && !helper.IsDataRead(ctx) {`
`67`	`67`	`_ = schema.RemoveFromState(d, m)`
`68`		`- return`
	`68`	`+ return diags`
`69`	`69`	`}`
`70`	`70`
`71`	`71`	`return diag.FromErr(err)`