Skip to content

Commit 71542b5

Browse files
committed
test: lower constistent timeouts to speed up pipelines
1 parent 84df511 commit 71542b5

5 files changed

+6
-8
lines changed

test/integration/admission/managementcontext_create_withBadCredentials_test.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -56,6 +56,6 @@ var _ = Describe("Validate create", labels.WithContext, func() {
5656
),
5757
err,
5858
)
59-
}, constants.EventualTimeout, interval).Should(Succeed())
59+
}, constants.ConsistentTimeout, interval).Should(Succeed())
6060
})
6161
})

test/integration/admission/managementcontext_create_withEnvironmentNotFound_test.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -49,6 +49,6 @@ var _ = Describe("Validate create", labels.WithContext, func() {
4949
Consistently(func() error {
5050
_, err := admissionCtrl.ValidateCreate(ctx, fixtures.Context)
5151
return assert.NotNil("error", err)
52-
}, constants.EventualTimeout, interval).Should(Succeed())
52+
}, constants.ConsistentTimeout, interval).Should(Succeed())
5353
})
5454
})

test/integration/admission/managementcontext_create_withMissingSecret_test.go

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -29,7 +29,6 @@ import (
2929
)
3030

3131
var _ = Describe("Validate create", labels.WithContext, func() {
32-
timeout := constants.EventualTimeout / 10
3332
interval := constants.Interval
3433
ctx := context.Background()
3534
admissionCtrl := mctx.AdmissionCtrl{}
@@ -42,6 +41,6 @@ var _ = Describe("Validate create", labels.WithContext, func() {
4241
Consistently(func() error {
4342
_, err := admissionCtrl.ValidateCreate(ctx, fixtures.Context)
4443
return err
45-
}, timeout, interval).Should(HaveOccurred())
44+
}, constants.ConsistentTimeout, interval).Should(HaveOccurred())
4645
})
4746
})

test/integration/admission/managementcontext_update_withNetworkError_test.go

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -34,7 +34,6 @@ import (
3434
)
3535

3636
var _ = Describe("Validate update", labels.WithContext, func() {
37-
timeout := constants.EventualTimeout
3837
interval := constants.Interval
3938
admissionCtrl := mctx.AdmissionCtrl{}
4039
ctx := context.Background()
@@ -66,11 +65,11 @@ var _ = Describe("Validate update", labels.WithContext, func() {
6665
Namespace: mCtx.Namespace,
6766
Name: mCtx.Name,
6867
}, mCtx)
69-
}, timeout, interval).Should(Succeed())
68+
}, constants.EventualTimeout, interval).Should(Succeed())
7069

7170
Consistently(func() error {
7271
warnings, _ := admissionCtrl.ValidateUpdate(ctx, mCtx, mCtx)
7372
return assert.SliceOfSize("warnings", warnings, 1)
74-
}, timeout, interval).ShouldNot(Succeed())
73+
}, constants.ConsistentTimeout, interval).ShouldNot(Succeed())
7574
})
7675
})

test/internal/integration/constants/constants.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,7 @@ import (
2525
const (
2626
Namespace = "default"
2727

28-
ConsistentTimeout = time.Second * 3
28+
ConsistentTimeout = time.Second * 2
2929
EventualTimeout = time.Second * 30
3030
Interval = time.Millisecond * 250
3131

0 commit comments

Comments
 (0)