Skip to content

Commit

Permalink
fix typo in timeout interval
Browse files Browse the repository at this point in the history
  • Loading branch information
Maksim Fedotov committed Aug 31, 2020
1 parent f6aace2 commit d7dd352
Show file tree
Hide file tree
Showing 11 changed files with 41 additions and 41 deletions.
12 changes: 6 additions & 6 deletions e2e/ingress_class_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -68,8 +68,8 @@ var _ = Describe("when Tenant handles Ingress classes", func() {
ns := NewNamespace("ingress-class-disallowed")
cs := ownerClient(tnt)

NamespaceCreationShouldSucceed(ns, tnt, defaultPollInterval)
NamespaceShouldBeManagedByTenant(ns, tnt, defaultPollInterval)
NamespaceCreationShouldSucceed(ns, tnt, defaultTimeoutInterval)
NamespaceShouldBeManagedByTenant(ns, tnt, defaultTimeoutInterval)

By("non-specifying the class", func() {
Eventually(func() (err error) {
Expand Down Expand Up @@ -131,8 +131,8 @@ var _ = Describe("when Tenant handles Ingress classes", func() {
ns := NewNamespace("ingress-class-allowed-annotation")
cs := ownerClient(tnt)

NamespaceCreationShouldSucceed(ns, tnt, defaultPollInterval)
NamespaceShouldBeManagedByTenant(ns, tnt, defaultPollInterval)
NamespaceCreationShouldSucceed(ns, tnt, defaultTimeoutInterval)
NamespaceShouldBeManagedByTenant(ns, tnt, defaultTimeoutInterval)

for _, c := range tnt.Spec.IngressClasses {
Eventually(func() (err error) {
Expand Down Expand Up @@ -171,8 +171,8 @@ var _ = Describe("when Tenant handles Ingress classes", func() {
Skip("Running test ont Kubernetes " + v.String() + ", doesn't provide .spec.ingressClassName")
}

NamespaceCreationShouldSucceed(ns, tnt, defaultPollInterval)
NamespaceShouldBeManagedByTenant(ns, tnt, defaultPollInterval)
NamespaceCreationShouldSucceed(ns, tnt, defaultTimeoutInterval)
NamespaceShouldBeManagedByTenant(ns, tnt, defaultTimeoutInterval)

for _, c := range tnt.Spec.IngressClasses {
Eventually(func() (err error) {
Expand Down
4 changes: 2 additions & 2 deletions e2e/new_namespace_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -55,7 +55,7 @@ var _ = Describe("creating a Namespace as Tenant owner", func() {
})
It("should be available in Tenant namespaces list", func() {
ns := NewNamespace("new-namespace")
NamespaceCreationShouldSucceed(ns, tnt, defaultPollInterval)
NamespaceShouldBeManagedByTenant(ns, tnt, defaultPollInterval)
NamespaceCreationShouldSucceed(ns, tnt, defaultTimeoutInterval)
NamespaceShouldBeManagedByTenant(ns, tnt, defaultTimeoutInterval)
})
})
4 changes: 2 additions & 2 deletions e2e/overquota_namespace_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -57,8 +57,8 @@ var _ = Describe("creating a Namespace over-quota", func() {
By("creating three Namespaces", func() {
for _, name := range []string{"bob-dev", "bob-staging", "bob-production"} {
ns := NewNamespace(name)
NamespaceCreationShouldSucceed(ns, tnt, defaultPollInterval)
NamespaceShouldBeManagedByTenant(ns, tnt, defaultPollInterval)
NamespaceCreationShouldSucceed(ns, tnt, defaultTimeoutInterval)
NamespaceShouldBeManagedByTenant(ns, tnt, defaultTimeoutInterval)
}
})

Expand Down
28 changes: 14 additions & 14 deletions e2e/owner_webhooks_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -106,8 +106,8 @@ var _ = Describe("when Tenant owner interacts with the webhooks", func() {
It("should disallow deletions", func() {
By("blocking Capsule Limit ranges", func() {
ns := NewNamespace("limit-range-disallow")
NamespaceCreationShouldSucceed(ns, tnt, defaultPollInterval)
NamespaceShouldBeManagedByTenant(ns, tnt, defaultPollInterval)
NamespaceCreationShouldSucceed(ns, tnt, defaultTimeoutInterval)
NamespaceShouldBeManagedByTenant(ns, tnt, defaultTimeoutInterval)

lr := &corev1.LimitRange{}
Eventually(func() error {
Expand All @@ -120,8 +120,8 @@ var _ = Describe("when Tenant owner interacts with the webhooks", func() {
})
By("blocking Capsule Network Policy", func() {
ns := NewNamespace("network-policy-disallow")
NamespaceCreationShouldSucceed(ns, tnt, defaultPollInterval)
NamespaceShouldBeManagedByTenant(ns, tnt, defaultPollInterval)
NamespaceCreationShouldSucceed(ns, tnt, defaultTimeoutInterval)
NamespaceShouldBeManagedByTenant(ns, tnt, defaultTimeoutInterval)

np := &networkingv1.NetworkPolicy{}
Eventually(func() error {
Expand All @@ -134,8 +134,8 @@ var _ = Describe("when Tenant owner interacts with the webhooks", func() {
})
By("blocking blocking Capsule Resource Quota", func() {
ns := NewNamespace("resource-quota-disallow")
NamespaceCreationShouldSucceed(ns, tnt, defaultPollInterval)
NamespaceShouldBeManagedByTenant(ns, tnt, defaultPollInterval)
NamespaceCreationShouldSucceed(ns, tnt, defaultTimeoutInterval)
NamespaceShouldBeManagedByTenant(ns, tnt, defaultTimeoutInterval)

rq := &corev1.ResourceQuota{}
Eventually(func() error {
Expand All @@ -150,8 +150,8 @@ var _ = Describe("when Tenant owner interacts with the webhooks", func() {
It("should allow listing", func() {
By("Limit Range resources", func() {
ns := NewNamespace("limit-range-list")
NamespaceCreationShouldSucceed(ns, tnt, defaultPollInterval)
NamespaceShouldBeManagedByTenant(ns, tnt, defaultPollInterval)
NamespaceCreationShouldSucceed(ns, tnt, defaultTimeoutInterval)
NamespaceShouldBeManagedByTenant(ns, tnt, defaultTimeoutInterval)

Eventually(func() (err error) {
cs := ownerClient(tnt)
Expand All @@ -161,8 +161,8 @@ var _ = Describe("when Tenant owner interacts with the webhooks", func() {
})
By("Network Policy resources", func() {
ns := NewNamespace("network-policy-list")
NamespaceCreationShouldSucceed(ns, tnt, defaultPollInterval)
NamespaceShouldBeManagedByTenant(ns, tnt, defaultPollInterval)
NamespaceCreationShouldSucceed(ns, tnt, defaultTimeoutInterval)
NamespaceShouldBeManagedByTenant(ns, tnt, defaultTimeoutInterval)

Eventually(func() (err error) {
cs := ownerClient(tnt)
Expand All @@ -172,8 +172,8 @@ var _ = Describe("when Tenant owner interacts with the webhooks", func() {
})
By("Resource Quota resources", func() {
ns := NewNamespace("resource-quota-list")
NamespaceCreationShouldSucceed(ns, tnt, defaultPollInterval)
NamespaceShouldBeManagedByTenant(ns, tnt, defaultPollInterval)
NamespaceCreationShouldSucceed(ns, tnt, defaultTimeoutInterval)
NamespaceShouldBeManagedByTenant(ns, tnt, defaultTimeoutInterval)

Eventually(func() (err error) {
cs := ownerClient(tnt)
Expand All @@ -184,8 +184,8 @@ var _ = Describe("when Tenant owner interacts with the webhooks", func() {
})
It("should allow all actions to Tenant owner Network Policy resources", func() {
ns := NewNamespace("network-policy-allow")
NamespaceCreationShouldSucceed(ns, tnt, defaultPollInterval)
NamespaceShouldBeManagedByTenant(ns, tnt, defaultPollInterval)
NamespaceCreationShouldSucceed(ns, tnt, defaultTimeoutInterval)
NamespaceShouldBeManagedByTenant(ns, tnt, defaultTimeoutInterval)

cs := ownerClient(tnt)
np := &networkingv1.NetworkPolicy{
Expand Down
4 changes: 2 additions & 2 deletions e2e/resource_quota_exceeded_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -128,8 +128,8 @@ var _ = Describe("exceeding Tenant resource quota", func() {
By("creating the Namespaces", func() {
for _, i := range nsl {
ns := NewNamespace(i)
NamespaceCreationShouldSucceed(ns, tnt, defaultPollInterval)
NamespaceShouldBeManagedByTenant(ns, tnt, defaultPollInterval)
NamespaceCreationShouldSucceed(ns, tnt, defaultTimeoutInterval)
NamespaceShouldBeManagedByTenant(ns, tnt, defaultTimeoutInterval)
}
})
})
Expand Down
4 changes: 2 additions & 2 deletions e2e/selecting_tenant_no_label_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -93,7 +93,7 @@ var _ = Describe("creating a Namespace without a Tenant selector", func() {
})
It("should be assigned to the first Tenant", func() {
ns := NewNamespace("tenant-1-ns")
NamespaceCreationShouldSucceed(ns, t3, defaultPollInterval)
NamespaceShouldBeManagedByTenant(ns, t1, defaultPollInterval)
NamespaceCreationShouldSucceed(ns, t3, defaultTimeoutInterval)
NamespaceShouldBeManagedByTenant(ns, t1, defaultTimeoutInterval)
})
})
4 changes: 2 additions & 2 deletions e2e/selecting_tenant_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -81,7 +81,7 @@ var _ = Describe("creating a Namespace with Tenant selector", func() {
l: t2.Name,
}
})
NamespaceCreationShouldSucceed(ns, t2, defaultPollInterval)
NamespaceShouldBeManagedByTenant(ns, t2, defaultPollInterval)
NamespaceCreationShouldSucceed(ns, t2, defaultTimeoutInterval)
NamespaceShouldBeManagedByTenant(ns, t2, defaultTimeoutInterval)
})
})
8 changes: 4 additions & 4 deletions e2e/storage_class_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -63,8 +63,8 @@ var _ = Describe("when Tenant handles Storage classes", func() {
})
It("should block non allowed Storage Class", func() {
ns := NewNamespace("storage-class-disallowed")
NamespaceCreationShouldSucceed(ns, tnt, defaultPollInterval)
NamespaceShouldBeManagedByTenant(ns, tnt, defaultPollInterval)
NamespaceCreationShouldSucceed(ns, tnt, defaultTimeoutInterval)
NamespaceShouldBeManagedByTenant(ns, tnt, defaultTimeoutInterval)

By("non-specifying the class", func() {
Eventually(func() (err error) {
Expand Down Expand Up @@ -111,8 +111,8 @@ var _ = Describe("when Tenant handles Storage classes", func() {
ns := NewNamespace("storage-class-allowed")
cs := ownerClient(tnt)

NamespaceCreationShouldSucceed(ns, tnt, defaultPollInterval)
NamespaceShouldBeManagedByTenant(ns, tnt, defaultPollInterval)
NamespaceCreationShouldSucceed(ns, tnt, defaultTimeoutInterval)
NamespaceShouldBeManagedByTenant(ns, tnt, defaultTimeoutInterval)

for _, c := range tnt.Spec.StorageClasses {
Eventually(func() (err error) {
Expand Down
6 changes: 3 additions & 3 deletions e2e/tenant_owner_group_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -56,8 +56,8 @@ var _ = Describe("creating a Namespace with group Tenant owner", func() {
})
It("should succeed and be available in Tenant namespaces list", func() {
ns := NewNamespace("gto-namespace")
NamespaceCreationShouldSucceed(ns, tnt, defaultPollInterval)
NamespaceShouldBeManagedByTenant(ns, tnt, defaultPollInterval)
GroupShouldBeUsedInTenantRoleBinding(ns, tnt, defaultPollInterval)
NamespaceCreationShouldSucceed(ns, tnt, defaultTimeoutInterval)
NamespaceShouldBeManagedByTenant(ns, tnt, defaultTimeoutInterval)
GroupShouldBeUsedInTenantRoleBinding(ns, tnt, defaultTimeoutInterval)
})
})
4 changes: 2 additions & 2 deletions e2e/tenant_resources_changes_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -171,8 +171,8 @@ var _ = Describe("changing Tenant managed Kubernetes resources", func() {
By("creating the Namespaces", func() {
for _, i := range nsl {
ns := NewNamespace(i)
NamespaceCreationShouldSucceed(ns, tnt, defaultPollInterval)
NamespaceShouldBeManagedByTenant(ns, tnt, defaultPollInterval)
NamespaceCreationShouldSucceed(ns, tnt, defaultTimeoutInterval)
NamespaceShouldBeManagedByTenant(ns, tnt, defaultTimeoutInterval)
}
})
})
Expand Down
4 changes: 2 additions & 2 deletions e2e/tenant_resources_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -174,8 +174,8 @@ var _ = Describe("creating namespaces within a Tenant with resources", func() {
By("creating the Namespaces", func() {
for _, i := range nsl {
ns := NewNamespace(i)
NamespaceCreationShouldSucceed(ns, tnt, defaultPollInterval)
NamespaceShouldBeManagedByTenant(ns, tnt, defaultPollInterval)
NamespaceCreationShouldSucceed(ns, tnt, defaultTimeoutInterval)
NamespaceShouldBeManagedByTenant(ns, tnt, defaultTimeoutInterval)
}
})
})
Expand Down

0 comments on commit d7dd352

Please sign in to comment.