Merge pull request #909 from F5Networks/profile_ssl_client

issue #902 fixed
F5Networks · Dec 21, 2023 · 26edc10 · 26edc10
2 parents 05f985e + 2a73abf
commit 26edc10
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 3 deletions.
diff --git a/bigip/resource_bigip_ltm_profile_ssl_client.go b/bigip/resource_bigip_ltm_profile_ssl_client.go
@@ -267,7 +267,8 @@ func resourceBigipLtmProfileClientSsl() *schema.Resource {
 			"cipher_group": {
 				Type:          schema.TypeString,
 				Optional:      true,
-				Default:       "none",
+				Computed: true,	
+				// Default:       "none",
 				Description:   "Cipher group for the ssl client profile",
 				ConflictsWith: []string{"ciphers"},
 			},

diff --git a/bigip/resource_bigip_ltm_profile_ssl_client_test.go b/bigip/resource_bigip_ltm_profile_ssl_client_test.go
@@ -8,7 +8,6 @@ package bigip
 import (
 	"fmt"
 	"testing"
-
 	bigip "github.com/f5devcentral/go-bigip"
 	"github.com/hashicorp/terraform-plugin-sdk/v2/helper/resource"
 	"github.com/hashicorp/terraform-plugin-sdk/v2/terraform"
@@ -413,7 +412,17 @@ func TestAccBigipLtmProfileClientSsl_UpdateCipherGroup(t *testing.T) {
 					resource.TestCheckResourceAttr(resFullName, "name", instFullName),
 					resource.TestCheckResourceAttr(resFullName, "partition", "Common"),
 					resource.TestCheckResourceAttr(resFullName, "defaults_from", "/Common/clientssl"),
-					resource.TestCheckResourceAttr(resFullName, "cipher_group", "none"),
+					// resource.TestCheckResourceAttr(resFullName, "cipher_group", "none"),
+				),
+			},
+			{
+				Config: testaccbigipltmprofileclientsslUpdateparam(instName, "cipher_group"),
+				Check: resource.ComposeTestCheckFunc(
+					testCheckClientSslExists(instFullName),
+					resource.TestCheckResourceAttr(resFullName, "name", instFullName),
+					resource.TestCheckResourceAttr(resFullName, "partition", "Common"),
+					resource.TestCheckResourceAttr(resFullName, "defaults_from", "/Common/clientssl"),
+					resource.TestCheckResourceAttr(resFullName, "cipher_group", "/Common/f5-aes"),
 				),
 			},
 			{