Feature/fix docs (#2199)

* fix docs

* fix go fmt

Author: HemanthDogiparthi12

docs/resources/outbound_digitalruleset.md

@@ -63,14 +63,11 @@ resource "genesyscloud_outbound_digitalruleset" "test_ruleset_sample" {
 <!-- schema generated by tfplugindocs -->
 ## Schema
 
-### Required
-
-- `rules` (Block List, Min: 1) The list of rules. (see [below for nested schema](#nestedblock--rules))
-
 ### Optional
 
 - `contact_list_id` (String) A ContactList to provide suggestions for contact columns on relevant conditions and actions.
 - `name` (String) The name of the digital rule set
+- `rules` (Block List) The list of rules. (see [below for nested schema](#nestedblock--rules))
 
 ### Read-Only
 

genesyscloud/bcp_tf_exporter/genesyscloud_bcp_tf_exporter_proxy.go

@@ -65,7 +65,7 @@ func getFlowDependenciesFn(ctx context.Context, p *BcpExporterProxy, resourceInf
 	freshConfig.BasePath = p.ClientConfig.BasePath
 	freshConfig.AccessToken = p.ClientConfig.AccessToken
 	freshConfig.DefaultHeader = p.ClientConfig.DefaultHeader // Direct assignment is safe for read-only use
-	
+
 	depConsumerProxy := dependentconsumers.GetDependentConsumerProxy(freshConfig)
 	resources, dependsMap, _, err := depConsumerProxy.GetDependentConsumers(ctx, resourceInfo, []string{})
 	if err != nil {

genesyscloud/dependent_consumers/genesyscloud_dependent_consumer_proxy_test.go

@@ -3,8 +3,8 @@ package dependent_consumers
 import (
 	"testing"
 
-	resourceExporter "github.com/mypurecloud/terraform-provider-genesyscloud/genesyscloud/resource_exporter"
 	"github.com/mypurecloud/platform-client-sdk-go/v179/platformclientv2"
+	resourceExporter "github.com/mypurecloud/terraform-provider-genesyscloud/genesyscloud/resource_exporter"
 	"github.com/stretchr/testify/assert"
 )
 
@@ -15,13 +15,13 @@ func TestBuildDependsMap(t *testing.T) {
 		"id3": {BlockLabel: "genesyscloud_flow::::id3"},
 	}
 	dependsMap := make(map[string][]string)
-	
+
 	result := buildDependsMap(resources, dependsMap, "main-id")
-	
+
 	assert.NotNil(t, result)
 	assert.Contains(t, result, "main-id")
 	assert.Len(t, result["main-id"], 3)
-	
+
 	// Verify all expected dependencies are present
 	deps := result["main-id"]
 	assert.Contains(t, deps, "genesyscloud_user.id1")
@@ -35,19 +35,19 @@ func TestBuildDependsMap_ExcludesSelf(t *testing.T) {
 		"id2": {BlockLabel: "genesyscloud_queue::::id2"},
 	}
 	dependsMap := make(map[string][]string)
-	
+
 	result := buildDependsMap(resources, dependsMap, "id1")
-	
+
 	assert.Len(t, result["id1"], 1)
 	assert.Equal(t, "genesyscloud_queue.id2", result["id1"][0])
 }
 
 func TestBuildDependsMap_EmptyResources(t *testing.T) {
 	resources := resourceExporter.ResourceIDMetaMap{}
 	dependsMap := make(map[string][]string)
-	
+
 	result := buildDependsMap(resources, dependsMap, "main-id")
-	
+
 	assert.NotNil(t, result)
 	assert.Contains(t, result, "main-id")
 	assert.Empty(t, result["main-id"])
@@ -59,15 +59,15 @@ func TestBuildDependsMap_Deterministic(t *testing.T) {
 		"a-id": {BlockLabel: "genesyscloud_queue::::a-id"},
 		"m-id": {BlockLabel: "genesyscloud_flow::::m-id"},
 	}
-	
+
 	// Run multiple times to verify consistent ordering
 	results := make([][]string, 5)
 	for i := 0; i < 5; i++ {
 		dependsMap := make(map[string][]string)
 		result := buildDependsMap(resources, dependsMap, "main-id")
 		results[i] = result["main-id"]
 	}
-	
+
 	// All results should be identical
 	for i := 1; i < 5; i++ {
 		assert.Equal(t, results[0], results[i])
@@ -76,7 +76,7 @@ func TestBuildDependsMap_Deterministic(t *testing.T) {
 
 func TestGetResourceType(t *testing.T) {
 	dependentConsumerMap := SetDependentObjectMaps()
-	
+
 	tests := []struct {
 		name         string
 		varType      string
@@ -88,15 +88,15 @@ func TestGetResourceType(t *testing.T) {
 		{"valid flow", "BOTFLOW", "genesyscloud_flow", true},
 		{"invalid type", "INVALID_TYPE", "", false},
 	}
-	
+
 	for _, tt := range tests {
 		t.Run(tt.name, func(t *testing.T) {
 			consumer := platformclientv2.Dependency{
 				VarType: &tt.varType,
 			}
-			
+
 			resourceType, ok := getResourceType(consumer, dependentConsumerMap)
-			
+
 			assert.Equal(t, tt.expectedOk, ok)
 			assert.Equal(t, tt.expectedType, resourceType)
 		})
@@ -107,35 +107,35 @@ func TestGetResourceFilter(t *testing.T) {
 	varType := "QUEUE"
 	id := "test-id-123"
 	name := "Test Queue"
-	
+
 	consumer := platformclientv2.Dependency{
 		VarType: &varType,
 		Id:      &id,
 		Name:    &name,
 	}
-	
+
 	result := getResourceFilter(consumer, "genesyscloud_routing_queue")
-	
+
 	assert.Equal(t, "genesyscloud_routing_queue::::test-id-123", result)
 }
 
 func TestProcessResource(t *testing.T) {
 	varType := "QUEUE"
 	id := "queue-123"
 	name := "Test Queue"
-	
+
 	consumer := platformclientv2.Dependency{
 		VarType: &varType,
 		Id:      &id,
 		Name:    &name,
 	}
-	
+
 	resources := make(resourceExporter.ResourceIDMetaMap)
 	architectDependencies := make(map[string][]string)
 	key := "flow-123"
-	
+
 	resources, architectDependencies = processResource(consumer, "genesyscloud_routing_queue", resources, architectDependencies, key)
-	
+
 	assert.Contains(t, resources, "queue-123")
 	assert.Equal(t, "genesyscloud_routing_queue::::queue-123", resources["queue-123"].BlockLabel)
 	assert.Contains(t, architectDependencies, key)
@@ -146,20 +146,20 @@ func TestProcessResource_DoesNotDuplicate(t *testing.T) {
 	varType := "QUEUE"
 	id := "queue-123"
 	name := "Test Queue"
-	
+
 	consumer := platformclientv2.Dependency{
 		VarType: &varType,
 		Id:      &id,
 		Name:    &name,
 	}
-	
+
 	resources := make(resourceExporter.ResourceIDMetaMap)
 	resources["queue-123"] = &resourceExporter.ResourceMeta{BlockLabel: "existing"}
 	architectDependencies := make(map[string][]string)
 	key := "flow-123"
-	
+
 	resources, architectDependencies = processResource(consumer, "genesyscloud_routing_queue", resources, architectDependencies, key)
-	
+
 	// Should not overwrite existing resource
 	assert.Equal(t, "existing", resources["queue-123"].BlockLabel)
 }
@@ -169,7 +169,7 @@ func TestIsDependencyPresent(t *testing.T) {
 		"flow1": {"dep1", "dep2", "dep3"},
 		"flow2": {"dep4", "dep5"},
 	}
-	
+
 	assert.True(t, isDependencyPresent(architectDependencies, "flow1", "dep2"))
 	assert.False(t, isDependencyPresent(architectDependencies, "flow2", "dep2"))
 	assert.False(t, isDependencyPresent(architectDependencies, "flow1", "dep6"))
@@ -181,51 +181,51 @@ func TestSearchForKeyValue(t *testing.T) {
 		"key1": {"value1", "value2", "value3"},
 		"key2": {"value4", "value5"},
 	}
-	
+
 	assert.True(t, searchForKeyValue(m, "key1", "value2"))
 	assert.False(t, searchForKeyValue(m, "key1", "value4"))
 	assert.False(t, searchForKeyValue(m, "key3", "value1"))
 }
 
 func TestStringInSlice(t *testing.T) {
 	slice := []string{"apple", "banana", "cherry"}
-	
+
 	assert.True(t, stringInSlice("banana", slice))
 	assert.False(t, stringInSlice("grape", slice))
 	assert.False(t, stringInSlice("", slice))
 }
 
 func TestGetDependentConsumerProxy_Singleton(t *testing.T) {
 	config := &platformclientv2.Configuration{}
-	
+
 	proxy1 := GetDependentConsumerProxy(config)
 	proxy2 := GetDependentConsumerProxy(config)
-	
+
 	// Should return same instance
 	assert.Equal(t, proxy1, proxy2)
 }
 
 func TestNewDependentConsumerProxy_ThreadSafe(t *testing.T) {
 	// Reset singleton for test
 	InternalProxy = nil
-	
+
 	config := &platformclientv2.Configuration{}
 	done := make(chan *DependentConsumerProxy, 10)
-	
+
 	// Run concurrent initializations
 	for i := 0; i < 10; i++ {
 		go func() {
 			proxy := newDependentConsumerProxy(config)
 			done <- proxy
 		}()
 	}
-	
+
 	// Collect all proxies
 	proxies := make([]*DependentConsumerProxy, 10)
 	for i := 0; i < 10; i++ {
 		proxies[i] = <-done
 	}
-	
+
 	// All should be the same instance
 	for i := 1; i < 10; i++ {
 		assert.Equal(t, proxies[0], proxies[i])
@@ -235,9 +235,9 @@ func TestNewDependentConsumerProxy_ThreadSafe(t *testing.T) {
 func TestGetDependentConsumerProxy_NilConfig(t *testing.T) {
 	// Reset singleton
 	InternalProxy = nil
-	
+
 	proxy := GetDependentConsumerProxy(nil)
-	
+
 	assert.NotNil(t, proxy)
 	assert.NotNil(t, proxy.GetPooledClientAttr)
 }

genesyscloud/tfexporter/genesyscloud_resource_exporter_test.go

@@ -376,12 +376,12 @@ func TestUnitTfExportBuildDependsOnResources(t *testing.T) {
 
 	dependentconsumers.InternalProxy = dependencyProxy
 	defer func() { dependentconsumers.InternalProxy = nil }()
-	
+
 	ctx := context.Background()
 
 	gre := &GenesysCloudResourceExporter{
-		ctx:          ctx,
-		dependsList:  make(map[string][]string),
+		ctx:               ctx,
+		dependsList:       make(map[string][]string),
 		flowResourcesList: []string{},
 	}