This is an automated email from the ASF dual-hosted git repository. lburgazzoli pushed a commit to branch main in repository https://gitbox.apache.org/repos/asf/camel-k.git
commit c9776e11e4371429e8163f52ecab10dc8bab633a Author: Luca Burgazzoli <lburgazz...@gmail.com> AuthorDate: Thu Jan 19 14:55:03 2023 +0100 chore(test): improve knative kamelet binding test I've removed some expectation as based on timing an likely to randomly fail. --- e2e/global/knative/kamelet_test.go | 22 ++++------------------ e2e/support/test_support.go | 7 +++++++ 2 files changed, 11 insertions(+), 18 deletions(-) diff --git a/e2e/global/knative/kamelet_test.go b/e2e/global/knative/kamelet_test.go index 420a791a4..811149ccb 100644 --- a/e2e/global/knative/kamelet_test.go +++ b/e2e/global/knative/kamelet_test.go @@ -42,6 +42,7 @@ func TestKameletChange(t *testing.T) { WithNewTestNamespace(t, func(ns string) { operatorID := "camel-k-kamelet-change" timerSource := "my-timer-source" + Expect(KamelInstallWithID(operatorID, ns).Execute()).To(Succeed()) Expect(CreateTimerKamelet(ns, timerSource)()).To(Succeed()) Expect(CreateKnativeChannel(ns, "messages")()).To(Succeed()) @@ -66,22 +67,15 @@ func TestKameletChange(t *testing.T) { "trait.camel.apache.org/health.readiness-initial-delay": "10", } + // Create the KameletBinding Expect(BindKameletTo(ns, timerBinding, annotations, from, to, map[string]string{"message": "message is Hello"}, map[string]string{})()).To(Succeed()) - Eventually(KameletBindingConditionStatus(ns, timerBinding, v1alpha1.KameletBindingConditionReady), TestTimeoutShort).Should(Equal(corev1.ConditionFalse)) - Eventually(KameletBindingCondition(ns, timerBinding, v1alpha1.KameletBindingConditionReady), TestTimeoutMedium).Should(And( - WithTransform(KameletBindingConditionReason, Equal(v1.IntegrationConditionDeploymentProgressingReason)), - WithTransform(KameletBindingConditionMessage, Or( - Equal("0/1 updated replicas"), - Equal("0/1 ready replicas"), - )))) - Eventually(IntegrationPodPhase(ns, timerBinding), TestTimeoutLong).Should(Equal(corev1.PodRunning)) Eventually(IntegrationConditionStatus(ns, "timer-binding", v1.IntegrationConditionReady), TestTimeoutShort).Should(Equal(corev1.ConditionTrue)) Eventually(IntegrationLogs(ns, "display"), TestTimeoutShort).Should(ContainSubstring("message is Hello")) - Eventually(KameletBindingConditionStatus(ns, timerBinding, v1alpha1.KameletBindingConditionReady), TestTimeoutShort).Should(Equal(corev1.ConditionTrue)) Eventually(KameletBindingCondition(ns, timerBinding, v1alpha1.KameletBindingConditionReady), TestTimeoutMedium).Should(And( + WithTransform(KameletBindingConditionStatusExtract, Equal(corev1.ConditionTrue)), WithTransform(KameletBindingConditionReason, Equal(v1.IntegrationConditionDeploymentReadyReason)), WithTransform(KameletBindingConditionMessage, Equal(fmt.Sprintf("1/1 ready replicas"))), )) @@ -89,21 +83,13 @@ func TestKameletChange(t *testing.T) { // Update the KameletBinding Expect(BindKameletTo(ns, "timer-binding", annotations, from, to, map[string]string{"message": "message is Hi"}, map[string]string{})()).To(Succeed()) - Eventually(KameletBindingConditionStatus(ns, timerBinding, v1alpha1.KameletBindingConditionReady), TestTimeoutShort).Should(Equal(corev1.ConditionFalse)) - Eventually(KameletBindingCondition(ns, timerBinding, v1alpha1.KameletBindingConditionReady), TestTimeoutMedium).Should(And( - WithTransform(KameletBindingConditionReason, Equal(v1.IntegrationConditionDeploymentProgressingReason)), - WithTransform(KameletBindingConditionMessage, Or( - Equal("0/1 updated replicas"), - Equal("0/1 ready replicas"), - )))) - Eventually(IntegrationPodPhase(ns, "timer-binding"), TestTimeoutLong).Should(Equal(corev1.PodRunning)) Eventually(IntegrationConditionStatus(ns, "timer-binding", v1.IntegrationConditionReady), TestTimeoutShort).Should(Equal(corev1.ConditionTrue)) Eventually(IntegrationLogs(ns, "display"), TestTimeoutShort).Should(ContainSubstring("message is Hi")) - Eventually(KameletBindingConditionStatus(ns, timerBinding, v1alpha1.KameletBindingConditionReady), TestTimeoutShort).Should(Equal(corev1.ConditionTrue)) Eventually(KameletBindingCondition(ns, timerBinding, v1alpha1.KameletBindingConditionReady), TestTimeoutMedium). Should(And( + WithTransform(KameletBindingConditionStatusExtract, Equal(corev1.ConditionTrue)), WithTransform(KameletBindingConditionReason, Equal(v1.IntegrationConditionDeploymentReadyReason)), WithTransform(KameletBindingConditionMessage, Equal("1/1 ready replicas")), )) diff --git a/e2e/support/test_support.go b/e2e/support/test_support.go index c5f041e7c..0030cacc3 100644 --- a/e2e/support/test_support.go +++ b/e2e/support/test_support.go @@ -1089,6 +1089,13 @@ func KameletBindingCondition(ns string, name string, conditionType v1alpha1.Kame } } +func KameletBindingConditionStatusExtract(c *v1alpha1.KameletBindingCondition) corev1.ConditionStatus { + if c == nil { + return "" + } + return c.Status +} + func KameletBindingConditionReason(c *v1alpha1.KameletBindingCondition) string { if c == nil { return ""