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"),
),
},
{