Skip to content

Commit 59af460

Browse files
EmilienMEmilienM
authored
Merge pull request #3005 from stephenfin/more-skips
tests: Add skips for feature- or configuration-dependent tests
2 parents ef98774 + 79b1324 commit 59af460

File tree

20 files changed

+105
-97
lines changed

20 files changed

+105
-97
lines changed

‎internal/acceptance/clients/clients.go‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -218,7 +218,7 @@ func NewBlockStorageV2NoAuthClient() (*gophercloud.ServiceClient, error) {
218218
}
219219

220220
// NewBlockStorageV3NoAuthClient returns a noauth *ServiceClient for
221-
// making calls to the OpenStack Block Storage v2 API. An error will be
221+
// making calls to the OpenStack Block Storage v3 API. An error will be
222222
// returned if client creation was not possible.
223223
func NewBlockStorageV3NoAuthClient() (*gophercloud.ServiceClient, error) {
224224
client, err := blockstorageNoAuth.NewClient(gophercloud.AuthOptions{

‎internal/acceptance/clients/conditions.go‎

Lines changed: 18 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -83,8 +83,24 @@ func RequireNovaNetwork(t *testing.T) {
8383
}
8484
}
8585

86-
// RequireIronicHTTPBasic will restric a test to be only run in
87-
// environments that have Ironic using http_basic.
86+
// RequireCinderNoAuth will restrict a test to be only run in environments that
87+
// have Cinder using noauth.
88+
func RequireCinderNoAuth(t *testing.T) {
89+
if os.Getenv("CINDER_ENDPOINT") == "" || os.Getenv("OS_USERNAME") == "" {
90+
t.Skip("this test requires Cinder using noauth, set OS_USERNAME and CINDER_ENDPOINT")
91+
}
92+
}
93+
94+
// RequireIronicNoAuth will restrict a test to be only run in environments that
95+
// have Ironic using noauth.
96+
func RequireIronicNoAuth(t *testing.T) {
97+
if os.Getenv("IRONIC_ENDPOINT") == "" || os.Getenv("OS_USERNAME") == "" {
98+
t.Skip("this test requires IRONIC using noauth, set OS_USERNAME and IRONIC_ENDPOINT")
99+
}
100+
}
101+
102+
// RequireIronicHTTPBasic will restrict a test to be only run in environments
103+
// that have Ironic using http_basic.
88104
func RequireIronicHTTPBasic(t *testing.T) {
89105
if os.Getenv("IRONIC_ENDPOINT") == "" || os.Getenv("OS_USERNAME") == "" || os.Getenv("OS_PASSWORD") == "" {
90106
t.Skip("this test requires Ironic using http_basic, set OS_USERNAME, OS_PASSWORD and IRONIC_ENDPOINT")

‎internal/acceptance/openstack/baremetal/noauth/allocations_test.go‎

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -15,6 +15,7 @@ import (
1515

1616
func TestAllocationsCreateDestroy(t *testing.T) {
1717
clients.RequireLong(t)
18+
clients.RequireIronicNoAuth(t)
1819

1920
client, err := clients.NewBareMetalV1NoAuthClient()
2021
th.AssertNoErr(t, err)

‎internal/acceptance/openstack/baremetal/noauth/nodes_test.go‎

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -16,6 +16,7 @@ import (
1616

1717
func TestNodesCreateDestroy(t *testing.T) {
1818
clients.RequireLong(t)
19+
clients.RequireIronicNoAuth(t)
1920

2021
client, err := clients.NewBareMetalV1NoAuthClient()
2122
th.AssertNoErr(t, err)
@@ -48,6 +49,7 @@ func TestNodesCreateDestroy(t *testing.T) {
4849

4950
func TestNodesUpdate(t *testing.T) {
5051
clients.RequireLong(t)
52+
clients.RequireIronicNoAuth(t)
5153

5254
client, err := clients.NewBareMetalV1NoAuthClient()
5355
th.AssertNoErr(t, err)
@@ -72,6 +74,7 @@ func TestNodesUpdate(t *testing.T) {
7274
func TestNodesRAIDConfig(t *testing.T) {
7375
clients.SkipReleasesBelow(t, "stable/ussuri")
7476
clients.RequireLong(t)
77+
clients.RequireIronicNoAuth(t)
7578

7679
client, err := clients.NewBareMetalV1NoAuthClient()
7780
th.AssertNoErr(t, err)

‎internal/acceptance/openstack/baremetal/noauth/ports_test.go‎

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -16,6 +16,7 @@ import (
1616

1717
func TestPortsCreateDestroy(t *testing.T) {
1818
clients.RequireLong(t)
19+
clients.RequireIronicNoAuth(t)
1920

2021
client, err := clients.NewBareMetalV1NoAuthClient()
2122
th.AssertNoErr(t, err)
@@ -50,6 +51,7 @@ func TestPortsCreateDestroy(t *testing.T) {
5051

5152
func TestPortsUpdate(t *testing.T) {
5253
clients.RequireLong(t)
54+
clients.RequireIronicNoAuth(t)
5355

5456
client, err := clients.NewBareMetalV1NoAuthClient()
5557
th.AssertNoErr(t, err)

‎internal/acceptance/openstack/blockstorage/noauth/snapshots_test.go‎

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -12,6 +12,8 @@ import (
1212
)
1313

1414
func TestSnapshotsList(t *testing.T) {
15+
clients.RequireCinderNoAuth(t)
16+
1517
client, err := clients.NewBlockStorageV3NoAuthClient()
1618
if err != nil {
1719
t.Fatalf("Unable to create a blockstorage client: %v", err)
@@ -33,6 +35,8 @@ func TestSnapshotsList(t *testing.T) {
3335
}
3436

3537
func TestSnapshotsCreateDelete(t *testing.T) {
38+
clients.RequireCinderNoAuth(t)
39+
3640
client, err := clients.NewBlockStorageV3NoAuthClient()
3741
if err != nil {
3842
t.Fatalf("Unable to create a blockstorage client: %v", err)

‎internal/acceptance/openstack/blockstorage/noauth/volumes_test.go‎

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -12,6 +12,8 @@ import (
1212
)
1313

1414
func TestVolumesList(t *testing.T) {
15+
clients.RequireCinderNoAuth(t)
16+
1517
client, err := clients.NewBlockStorageV3NoAuthClient()
1618
if err != nil {
1719
t.Fatalf("Unable to create a blockstorage client: %v", err)
@@ -33,6 +35,8 @@ func TestVolumesList(t *testing.T) {
3335
}
3436

3537
func TestVolumesCreateDestroy(t *testing.T) {
38+
clients.RequireCinderNoAuth(t)
39+
3640
client, err := clients.NewBlockStorageV3NoAuthClient()
3741
if err != nil {
3842
t.Fatalf("Unable to create blockstorage client: %v", err)

‎internal/acceptance/openstack/networking/v2/conditions.go‎

Lines changed: 18 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,18 @@
1+
package v2
2+
3+
import (
4+
"context"
5+
"testing"
6+
7+
"github.com/gophercloud/gophercloud/v2"
8+
"github.com/gophercloud/gophercloud/v2/openstack/common/extensions"
9+
)
10+
11+
// RequireNeutronExtension will restrict a test to be only run in environments
12+
// with the requested Neutron extension present.
13+
func RequireNeutronExtension(t *testing.T, client *gophercloud.ServiceClient, extension string) {
14+
_, err := extensions.Get(context.TODO(), client, extension).Extract()
15+
if err != nil {
16+
t.Skipf("this test requires %s Neutron extension", extension)
17+
}
18+
}

‎internal/acceptance/openstack/networking/v2/extensions/dns/dns_test.go‎

Lines changed: 6 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,6 @@ import (
1111
networking "github.com/gophercloud/gophercloud/v2/internal/acceptance/openstack/networking/v2"
1212
"github.com/gophercloud/gophercloud/v2/internal/acceptance/openstack/networking/v2/extensions/layer3"
1313
"github.com/gophercloud/gophercloud/v2/internal/acceptance/tools"
14-
"github.com/gophercloud/gophercloud/v2/openstack/common/extensions"
1514
"github.com/gophercloud/gophercloud/v2/openstack/networking/v2/extensions/dns"
1615
"github.com/gophercloud/gophercloud/v2/openstack/networking/v2/extensions/layer3/floatingips"
1716
"github.com/gophercloud/gophercloud/v2/openstack/networking/v2/networks"
@@ -26,11 +25,8 @@ func TestDNSPortCRUDL(t *testing.T) {
2625
client, err := clients.NewNetworkV2Client()
2726
th.AssertNoErr(t, err)
2827

29-
extension, err := extensions.Get(context.TODO(), client, "dns-integration").Extract()
30-
if err != nil {
31-
t.Skip("This test requires dns-integration Neutron extension")
32-
}
33-
tools.PrintResource(t, extension)
28+
// Skip these tests if we don't have the required extension
29+
networking.RequireNeutronExtension(t, client, "dns-integration")
3430

3531
// Create Network
3632
networkDNSDomain := "local."
@@ -153,11 +149,8 @@ func TestDNSFloatingIPCRDL(t *testing.T) {
153149
client, err := clients.NewNetworkV2Client()
154150
th.AssertNoErr(t, err)
155151

156-
extension, err := extensions.Get(context.TODO(), client, "dns-integration").Extract()
157-
if err != nil {
158-
t.Skip("This test requires dns-integration Neutron extension")
159-
}
160-
tools.PrintResource(t, extension)
152+
// Skip these tests if we don't have the required extension
153+
networking.RequireNeutronExtension(t, client, "dns-integration")
161154

162155
choices, err := clients.AcceptanceTestChoicesFromEnv()
163156
th.AssertNoErr(t, err)
@@ -215,11 +208,8 @@ func TestDNSNetwork(t *testing.T) {
215208
client, err := clients.NewNetworkV2Client()
216209
th.AssertNoErr(t, err)
217210

218-
extension, err := extensions.Get(context.TODO(), client, "dns-integration").Extract()
219-
if err != nil {
220-
t.Skip("This test requires dns-integration Neutron extension")
221-
}
222-
tools.PrintResource(t, extension)
211+
// Skip these tests if we don't have the required extension
212+
networking.RequireNeutronExtension(t, client, "dns-integration")
223213

224214
// Create Network
225215
networkDNSDomain := "local."

‎internal/acceptance/openstack/networking/v2/extensions/fwaas_v2/groups_test.go‎

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,7 @@ import (
77
"testing"
88

99
"github.com/gophercloud/gophercloud/v2/internal/acceptance/clients"
10+
networking "github.com/gophercloud/gophercloud/v2/internal/acceptance/openstack/networking/v2"
1011
"github.com/gophercloud/gophercloud/v2/internal/acceptance/tools"
1112
"github.com/gophercloud/gophercloud/v2/openstack/networking/v2/extensions/fwaas_v2/groups"
1213
th "github.com/gophercloud/gophercloud/v2/testhelper"
@@ -20,6 +21,9 @@ func TestGroupCRUD(t *testing.T) {
2021
client, err := clients.NewNetworkV2Client()
2122
th.AssertNoErr(t, err)
2223

24+
// Skip these tests if we don't have the required extension
25+
networking.RequireNeutronExtension(t, client, "fwaas_v2")
26+
2327
createdGroup, err := CreateGroup(t, client)
2428
th.AssertNoErr(t, err)
2529
defer DeleteGroup(t, client, createdGroup.ID)

0 commit comments

Comments
 (0)