From 2a73abf8ae5dd54a01d9b2a42a3411ff2e5effa9 Mon Sep 17 00:00:00 2001 From: ramaniprateek Date: Fri, 8 Dec 2023 13:44:41 +0530 Subject: [PATCH] issue 902 fixed --- bigip/resource_bigip_ltm_profile_ssl_client.go | 3 ++- bigip/resource_bigip_ltm_profile_ssl_client_test.go | 13 +++++++++++-- 2 files changed, 13 insertions(+), 3 deletions(-) diff --git a/bigip/resource_bigip_ltm_profile_ssl_client.go b/bigip/resource_bigip_ltm_profile_ssl_client.go index ca2ed5279..0e26042dd 100644 --- 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 index f27c5e4f5..ae94041ba 100644 --- 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"), ), }, {