Skip to content

Commit d4a64b4

Browse files
authored
fix: increased waiting timeout for integration tests (#221)
* fix: increased waiting timeout for authorization models On-behalf-of: SAP [email protected] * chore: increased timeout for every waiting operation On-behalf-of: SAP [email protected]
1 parent 3fca9ad commit d4a64b4

File tree

1 file changed

+6
-6
lines changed

1 file changed

+6
-6
lines changed

internal/test/integration/authorization_model_generation_test.go

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -57,7 +57,7 @@ func (suite *IntegrationSuite) TestAuthorizationModelGeneration_Process() {
5757
suite.Assert().Eventually(func() bool {
5858
err := suite.platformMeshSystemClient.Get(ctx, client.ObjectKey{Name: expectedModelName}, &model)
5959
return err == nil
60-
}, 5*time.Second, 200*time.Millisecond, "authorizationModel should be created by controller")
60+
}, 10*time.Second, 200*time.Millisecond, "authorizationModel should be created by controller")
6161

6262
suite.Assert().Equal(testOrgName, model.Spec.StoreRef.Name)
6363
suite.Assert().Equal(testOrgPath.String(), model.Spec.StoreRef.Path)
@@ -106,7 +106,7 @@ func (suite *IntegrationSuite) TestAuthorizationModelGeneration_Finalize() {
106106
suite.Assert().Eventually(func() bool {
107107
err := suite.platformMeshSystemClient.Get(ctx, client.ObjectKey{Name: expectedModelName}, &model)
108108
return err == nil
109-
}, 5*time.Second, 200*time.Millisecond, "authorizationModel should exist after reconciliations")
109+
}, 10*time.Second, 200*time.Millisecond, "authorizationModel should exist after reconciliations")
110110

111111
var testApiBinding1, testApiBinding2 kcpapiv1alpha1.APIBinding
112112
suite.Require().NoError(testAccount1Client.Get(ctx, client.ObjectKey{Name: apiBinding1.Name}, &testApiBinding1))
@@ -122,13 +122,13 @@ func (suite *IntegrationSuite) TestAuthorizationModelGeneration_Finalize() {
122122
var binding kcpapiv1alpha1.APIBinding
123123
err := testAccount1Client.Get(ctx, client.ObjectKey{Name: apiBinding1.Name}, &binding)
124124
return kerrors.IsNotFound(err)
125-
}, 5*time.Second, 200*time.Millisecond, "APIBinding1 should be deleted")
125+
}, 10*time.Second, 200*time.Millisecond, "APIBinding1 should be deleted")
126126

127127
suite.Assert().Eventually(func() bool {
128128
var authModel securityv1alpha1.AuthorizationModel
129129
err := suite.platformMeshSystemClient.Get(ctx, client.ObjectKey{Name: expectedModelName}, &authModel)
130130
return err == nil && authModel.DeletionTimestamp.IsZero()
131-
}, 5*time.Second, 200*time.Millisecond, "authorizationModel should still exist after deleting first binding")
131+
}, 10*time.Second, 200*time.Millisecond, "authorizationModel should still exist after deleting first binding")
132132

133133
err = testAccount2Client.Delete(ctx, apiBinding2)
134134
suite.Require().NoError(err)
@@ -137,13 +137,13 @@ func (suite *IntegrationSuite) TestAuthorizationModelGeneration_Finalize() {
137137
var binding kcpapiv1alpha1.APIBinding
138138
err := testAccount2Client.Get(ctx, client.ObjectKey{Name: apiBinding2.Name}, &binding)
139139
return kerrors.IsNotFound(err)
140-
}, 5*time.Second, 200*time.Millisecond, "APIBinding2 should be deleted")
140+
}, 10*time.Second, 200*time.Millisecond, "APIBinding2 should be deleted")
141141

142142
suite.Assert().Eventually(func() bool {
143143
var authModel securityv1alpha1.AuthorizationModel
144144
err := suite.platformMeshSystemClient.Get(ctx, client.ObjectKey{Name: expectedModelName}, &authModel)
145145
return kerrors.IsNotFound(err)
146-
}, 5*time.Second, 200*time.Millisecond, "authorizationModel should be deleted after deleting both bindings")
146+
}, 10*time.Second, 200*time.Millisecond, "authorizationModel should be deleted after deleting both bindings")
147147
}
148148

149149
func (suite *IntegrationSuite) createTestAPIResourceSchema(ctx context.Context, client client.Client, name, group, plural, singular string, scope apiextensionsv1.ResourceScope) {

0 commit comments

Comments
 (0)