Skip to content

Commit 5f159ec

Browse files
committed
fixes bug in TestAcc for iploadbalancing http/tcp server
Removing the introduced test in #428, it turns out the whole API response is written inside the resource. Including the null values. Which means the TestCheckNoResourceAttr was not fitting. Not replacing it as there's no built-in func that allows for checking that a resource is set while having a nil value. Signed-off-by: Arnaud SINAYS <[email protected]>
1 parent dd6b330 commit 5f159ec

File tree

2 files changed

+0
-2
lines changed

2 files changed

+0
-2
lines changed

ovh/resource_iploadbalancing_http_farm_server_test.go

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -318,7 +318,6 @@ func TestAccIpLoadbalancingHttpFarmServerBasic(t *testing.T) {
318318
resource.TestCheckResourceAttr(TEST_ACC_IPLOADBALANCING_HTTP_FARM_SRV_RES_NAME, "ssl", "true"),
319319
resource.TestCheckResourceAttr(TEST_ACC_IPLOADBALANCING_HTTP_FARM_SRV_RES_NAME, "backup", "true"),
320320
resource.TestCheckResourceAttr(TEST_ACC_IPLOADBALANCING_HTTP_FARM_SRV_RES_NAME, "on_marked_down", "shutdown-sessions"),
321-
resource.TestCheckNoResourceAttr(TEST_ACC_IPLOADBALANCING_HTTP_FARM_SRV_RES_NAME, "proxy_protocol_version"),
322321
),
323322
},
324323
{

ovh/resource_iploadbalancing_tcp_farm_server_test.go

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -303,7 +303,6 @@ func TestAccIpLoadbalancingTcpFarmServerBasic(t *testing.T) {
303303
resource.TestCheckResourceAttr("ovh_iploadbalancing_tcp_farm_server.testacc", "weight", "1"),
304304
resource.TestCheckResourceAttr("ovh_iploadbalancing_tcp_farm_server.testacc", "ssl", "true"),
305305
resource.TestCheckResourceAttr("ovh_iploadbalancing_tcp_farm_server.testacc", "backup", "true"),
306-
resource.TestCheckNoResourceAttr("ovh_iploadbalancing_tcp_farm_server.testacc", "proxy_protocol_version"),
307306
),
308307
},
309308
{

0 commit comments

Comments
 (0)