This is an automated email from the ASF dual-hosted git repository.

ronething pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/apisix-ingress-controller.git


The following commit(s) were added to refs/heads/master by this push:
     new d1abfd59 test: fix unstable websocket annotation test (#2637)
d1abfd59 is described below

commit d1abfd59e51d4c442daa47b6c6a27441c5312acc
Author: Ashing Zheng <[email protected]>
AuthorDate: Thu Oct 30 15:09:59 2025 +0800

    test: fix unstable websocket annotation test (#2637)
---
 test/e2e/ingress/annotations.go | 17 +++++++++++++----
 1 file changed, 13 insertions(+), 4 deletions(-)

diff --git a/test/e2e/ingress/annotations.go b/test/e2e/ingress/annotations.go
index a9c86e08..30b8ee75 100644
--- a/test/e2e/ingress/annotations.go
+++ b/test/e2e/ingress/annotations.go
@@ -266,10 +266,19 @@ spec:
                It("websocket", func() {
                        
Expect(s.CreateResourceFromString(fmt.Sprintf(ingressWebSocket, 
s.Namespace()))).ShouldNot(HaveOccurred(), "creating Ingress")
 
-                       routes, err := 
s.DefaultDataplaneResource().Route().List(context.Background())
-                       Expect(err).NotTo(HaveOccurred(), "listing Route")
-                       Expect(routes).To(HaveLen(1), "checking Route length")
-                       
Expect(routes[0].EnableWebsocket).To(Equal(ptr.To(true)), "checking Route 
EnableWebsocket")
+                       Eventually(func() bool {
+                               routes, err := 
s.DefaultDataplaneResource().Route().List(context.Background())
+                               if err != nil {
+                                       return false
+                               }
+                               if len(routes) != 1 {
+                                       return false
+                               }
+                               if routes[0].EnableWebsocket == nil || 
!*routes[0].EnableWebsocket {
+                                       return false
+                               }
+                               return true
+                       }).WithTimeout(30 * time.Second).ProbeEvery(2 * 
time.Second).Should(BeTrue())
                })
        })
 

Reply via email to