Skip to content

Commit 90e56b9

Browse files
committed
e2e: rename testdata package to testscenario
Signed-off-by: Jaime Caamaño Ruiz <[email protected]>
1 parent dc437b6 commit 90e56b9

9 files changed

+27
-27
lines changed

test/e2e/network_segmentation_api_validations.go

Lines changed: 12 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -6,13 +6,13 @@ import (
66

77
e2ekubectl "k8s.io/kubernetes/test/e2e/framework/kubectl"
88

9-
"github.com/ovn-org/ovn-kubernetes/test/e2e/testdata"
10-
testdatacudn "github.com/ovn-org/ovn-kubernetes/test/e2e/testdata/cudn"
9+
"github.com/ovn-org/ovn-kubernetes/test/e2e/testscenario"
10+
testscenariocudn "github.com/ovn-org/ovn-kubernetes/test/e2e/testscenario/cudn"
1111
)
1212

1313
var _ = Describe("Network Segmentation: API validations", func() {
1414
DescribeTable("api-server should reject invalid CRs",
15-
func(scenarios []testdata.ValidateCRScenario) {
15+
func(scenarios []testscenario.ValidateCRScenario) {
1616
DeferCleanup(func() {
1717
cleanupValidateCRsTest(scenarios)
1818
})
@@ -23,16 +23,16 @@ var _ = Describe("Network Segmentation: API validations", func() {
2323
Expect(stderr).To(ContainSubstring(s.ExpectedErr))
2424
}
2525
},
26-
Entry("ClusterUserDefinedNetwork, mismatch topology and config", testdatacudn.MismatchTopologyConfig),
27-
Entry("ClusterUserDefinedNetwork, localnet, invalid role", testdatacudn.LocalnetInvalidRole),
28-
Entry("ClusterUserDefinedNetwork, localnet, invalid physicalNetworkName", testdatacudn.LocalnetInvalidPhyNetName),
29-
Entry("ClusterUserDefinedNetwork, localnet, invalid subnets", testdatacudn.LocalnetInvalidSubnets),
30-
Entry("ClusterUserDefinedNetwork, localnet, invalid mtu", testdatacudn.LocalnetInvalidMTU),
31-
Entry("ClusterUserDefinedNetwork, localnet, invalid vlan", testdatacudn.LocalnetInvalidVLAN),
26+
Entry("ClusterUserDefinedNetwork, mismatch topology and config", testscenariocudn.MismatchTopologyConfig),
27+
Entry("ClusterUserDefinedNetwork, localnet, invalid role", testscenariocudn.LocalnetInvalidRole),
28+
Entry("ClusterUserDefinedNetwork, localnet, invalid physicalNetworkName", testscenariocudn.LocalnetInvalidPhyNetName),
29+
Entry("ClusterUserDefinedNetwork, localnet, invalid subnets", testscenariocudn.LocalnetInvalidSubnets),
30+
Entry("ClusterUserDefinedNetwork, localnet, invalid mtu", testscenariocudn.LocalnetInvalidMTU),
31+
Entry("ClusterUserDefinedNetwork, localnet, invalid vlan", testscenariocudn.LocalnetInvalidVLAN),
3232
)
3333

3434
DescribeTable("api-server should accept valid CRs",
35-
func(scenarios []testdata.ValidateCRScenario) {
35+
func(scenarios []testscenario.ValidateCRScenario) {
3636
DeferCleanup(func() {
3737
cleanupValidateCRsTest(scenarios)
3838
})
@@ -42,7 +42,7 @@ var _ = Describe("Network Segmentation: API validations", func() {
4242
Expect(err).NotTo(HaveOccurred(), "should create valid CR successfully")
4343
}
4444
},
45-
Entry("ClusterUserDefinedNetwork, localnet", testdatacudn.LocalnetValid),
45+
Entry("ClusterUserDefinedNetwork, localnet", testscenariocudn.LocalnetValid),
4646
)
4747
})
4848

@@ -52,7 +52,7 @@ func runKubectlInputWithFullOutput(namespace string, data string, args ...string
5252
return e2ekubectl.NewKubectlCommand(namespace, args...).WithStdinData(data).ExecWithFullOutput()
5353
}
5454

55-
func cleanupValidateCRsTest(scenarios []testdata.ValidateCRScenario) {
55+
func cleanupValidateCRsTest(scenarios []testscenario.ValidateCRScenario) {
5656
for _, s := range scenarios {
5757
e2ekubectl.RunKubectlInput("", s.Manifest, "delete", "-f", "-")
5858
}

test/e2e/testdata/cudn/invalid-scenarios-localnet-mtu.go renamed to test/e2e/testscenario/cudn/invalid-scenarios-localnet-mtu.go

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,8 @@
11
package cudn
22

3-
import "github.com/ovn-org/ovn-kubernetes/test/e2e/testdata"
3+
import "github.com/ovn-org/ovn-kubernetes/test/e2e/testscenario"
44

5-
var LocalnetInvalidMTU = []testdata.ValidateCRScenario{
5+
var LocalnetInvalidMTU = []testscenario.ValidateCRScenario{
66
{
77
Description: "invalid MTU - higher than 65536",
88
ExpectedErr: `spec.network.localnet.mtu in body should be less than or equal to 65536`,

test/e2e/testdata/cudn/invalid-scenarios-localnet-phynetname.go renamed to test/e2e/testscenario/cudn/invalid-scenarios-localnet-phynetname.go

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,8 @@
11
package cudn
22

3-
import "github.com/ovn-org/ovn-kubernetes/test/e2e/testdata"
3+
import "github.com/ovn-org/ovn-kubernetes/test/e2e/testscenario"
44

5-
var LocalnetInvalidPhyNetName = []testdata.ValidateCRScenario{
5+
var LocalnetInvalidPhyNetName = []testscenario.ValidateCRScenario{
66
{
77
Description: "unset PhysicalNetworkName",
88
ExpectedErr: `spec.network.localnet.physicalNetworkName: Required value`,

test/e2e/testdata/cudn/invalid-scenarios-localnet-role.go renamed to test/e2e/testscenario/cudn/invalid-scenarios-localnet-role.go

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,8 @@
11
package cudn
22

3-
import "github.com/ovn-org/ovn-kubernetes/test/e2e/testdata"
3+
import "github.com/ovn-org/ovn-kubernetes/test/e2e/testscenario"
44

5-
var LocalnetInvalidRole = []testdata.ValidateCRScenario{
5+
var LocalnetInvalidRole = []testscenario.ValidateCRScenario{
66
{
77
Description: "role unset",
88
ExpectedErr: `spec.network.localnet.role: Required value`,

test/e2e/testdata/cudn/invalid-scenarios-localnet-subnets.go renamed to test/e2e/testscenario/cudn/invalid-scenarios-localnet-subnets.go

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,8 @@
11
package cudn
22

3-
import "github.com/ovn-org/ovn-kubernetes/test/e2e/testdata"
3+
import "github.com/ovn-org/ovn-kubernetes/test/e2e/testscenario"
44

5-
var LocalnetInvalidSubnets = []testdata.ValidateCRScenario{
5+
var LocalnetInvalidSubnets = []testscenario.ValidateCRScenario{
66
{
77
Description: "unset subnets, and ipam.mode is unset",
88
ExpectedErr: `Subnets is required with ipam.mode is Enabled or unset, and forbidden otherwise`,

test/e2e/testdata/cudn/invalid-scenarios-localnet-vlan.go renamed to test/e2e/testscenario/cudn/invalid-scenarios-localnet-vlan.go

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,8 @@
11
package cudn
22

3-
import "github.com/ovn-org/ovn-kubernetes/test/e2e/testdata"
3+
import "github.com/ovn-org/ovn-kubernetes/test/e2e/testscenario"
44

5-
var LocalnetInvalidVLAN = []testdata.ValidateCRScenario{
5+
var LocalnetInvalidVLAN = []testscenario.ValidateCRScenario{
66
{
77
Description: "invalid VLAN - invalid mode",
88
ExpectedErr: `spec.network.localnet.vlan.mode: Unsupported value: "Disabled": supported values: "Access`,

test/e2e/testdata/cudn/invalid-scenarios-mismatch-topo-conf.go renamed to test/e2e/testscenario/cudn/invalid-scenarios-mismatch-topo-conf.go

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,8 @@
11
package cudn
22

3-
import "github.com/ovn-org/ovn-kubernetes/test/e2e/testdata"
3+
import "github.com/ovn-org/ovn-kubernetes/test/e2e/testscenario"
44

5-
var MismatchTopologyConfig = []testdata.ValidateCRScenario{
5+
var MismatchTopologyConfig = []testscenario.ValidateCRScenario{
66
{
77
Description: "topology is localnet but topology config is layer2",
88
ExpectedErr: `spec.localnet is required when topology is Localnet and forbidden otherwise`,

test/e2e/testdata/cudn/valid-scenarios-localnet.go renamed to test/e2e/testscenario/cudn/valid-scenarios-localnet.go

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,8 @@
11
package cudn
22

3-
import "github.com/ovn-org/ovn-kubernetes/test/e2e/testdata"
3+
import "github.com/ovn-org/ovn-kubernetes/test/e2e/testscenario"
44

5-
var LocalnetValid = []testdata.ValidateCRScenario{
5+
var LocalnetValid = []testscenario.ValidateCRScenario{
66
{
77
Description: "should create localnet topology successfully - minimal",
88
Manifest: `

test/e2e/testdata/scenario.go renamed to test/e2e/testscenario/scenario.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
package testdata
1+
package testscenario
22

33
// ValidateCRScenario represent test scenario where a manifest is applied and failed with the expected error
44
type ValidateCRScenario struct {

0 commit comments

Comments
 (0)