Merge pull request #3195 from gophercloud/bp-v2-9c94b95

kayrus · web-flow · commit 75a970da6dfd · 2024-10-04T22:08:46.000+02:00
[v2] [core]: handle empty response body
diff --git a/openstack/networking/v2/extensions/bgp/speakers/requests.go b/openstack/networking/v2/extensions/bgp/speakers/requests.go
@@ -142,7 +142,7 @@ func RemoveBGPPeer(ctx context.Context, c *gophercloud.ServiceClient, bgpSpeaker
 		r.Err = err
 		return
 	}
-	resp, err := c.Put(ctx, removeBGPPeerURL(c, bgpSpeakerID), b, &r.Body, &gophercloud.RequestOpts{
+	resp, err := c.Put(ctx, removeBGPPeerURL(c, bgpSpeakerID), b, nil, &gophercloud.RequestOpts{
 		OkCodes: []int{200},
 	})
 	_, r.Header, r.Err = gophercloud.ParseResponse(resp, err)
@@ -207,7 +207,7 @@ func RemoveGatewayNetwork(ctx context.Context, c *gophercloud.ServiceClient, bgp
 		r.Err = err
 		return
 	}
-	resp, err := c.Put(ctx, removeGatewayNetworkURL(c, bgpSpeakerID), b, &r.Body, &gophercloud.RequestOpts{
+	resp, err := c.Put(ctx, removeGatewayNetworkURL(c, bgpSpeakerID), b, nil, &gophercloud.RequestOpts{
 		OkCodes: []int{200},
 	})
 	_, r.Header, r.Err = gophercloud.ParseResponse(resp, err)
diff --git a/openstack/networking/v2/extensions/bgp/speakers/testing/requests_test.go b/openstack/networking/v2/extensions/bgp/speakers/testing/requests_test.go
@@ -3,7 +3,6 @@ package testing
 import (
 	"context"
 	"fmt"
-	"io"
 	"net/http"
 	"testing"
 
@@ -195,7 +194,7 @@ func TestRemoveBGPPeer(t *testing.T) {
 
 	opts := speakers.RemoveBGPPeerOpts{BGPPeerID: bgpPeerID}
 	err := speakers.RemoveBGPPeer(context.TODO(), fake.ServiceClient(), bgpSpeakerID, opts).ExtractErr()
-	th.AssertEquals(t, err, io.EOF)
+	th.AssertNoErr(t, err)
 }
 
 func TestGetAdvertisedRoutes(t *testing.T) {
@@ -279,5 +278,5 @@ func TestRemoveGatewayNetwork(t *testing.T) {
 
 	opts := speakers.RemoveGatewayNetworkOpts{NetworkID: networkID}
 	err := speakers.RemoveGatewayNetwork(context.TODO(), fake.ServiceClient(), bgpSpeakerID, opts).ExtractErr()
-	th.AssertEquals(t, err, io.EOF)
+	th.AssertNoErr(t, err)
 }
