test: fix flaky SpringAnnotationCompatibilityTest

The test asserted that the first ApolloConfigChangeEvent received by the
ApplicationListener probe is for namespace 'application'. However, config
change listeners are notified asynchronously on a shared thread pool
(AbstractConfig#notifyAsync), so events from different namespaces may
arrive in any order, occasionally failing CI with:

  expected:<application[]> but was:<application[.yaml]>

Replace the order-sensitive FIFO assertion with an order-independent
check that all expected namespaces are eventually observed, matching the
approach already used in ApolloSpringBootCompatibilityTest.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>

Author: Shawyeok

apollo-compat-tests/apollo-spring-compat-it/src/test/java/com/ctrip/framework/apollo/compat/spring/SpringAnnotationCompatibilityTest.java

@@ -122,8 +122,13 @@ public void shouldSupportAnnotationAndMultipleConfig() throws Exception {
     assertNotNull(anotherAppChange);
     assertNotNull(anotherAppChange.getChange("compat.origin"));
 
-    String namespace = apolloEventListenerProbe.pollNamespace(10, TimeUnit.SECONDS);
-    assertEquals("application", namespace);
+    // config change listeners are notified asynchronously, so events from different
+    // namespaces may arrive in any order
+    SpringCompatibilityTestSupport.waitForCondition(
+        "ApplicationListener should receive namespace updates",
+        () -> apolloEventListenerProbe.hasNamespace("application")
+            && apolloEventListenerProbe.hasNamespace("TEST1.apollo")
+            && apolloEventListenerProbe.hasNamespace("application.yaml"));
 
     SpringCompatibilityTestSupport.waitForCondition("public value should be updated",
         () -> "from-public-updated".equals(

apollo-compat-tests/apollo-spring-compat-it/src/test/java/com/ctrip/framework/apollo/compat/spring/SpringApolloEventListenerProbe.java

@@ -17,24 +17,24 @@
 package com.ctrip.framework.apollo.compat.spring;
 
 import com.ctrip.framework.apollo.spring.events.ApolloConfigChangeEvent;
-import java.util.concurrent.BlockingQueue;
-import java.util.concurrent.LinkedBlockingQueue;
-import java.util.concurrent.TimeUnit;
+import java.util.Collections;
+import java.util.HashSet;
+import java.util.Set;
 import org.springframework.context.ApplicationEvent;
 import org.springframework.context.ApplicationListener;
 
 public class SpringApolloEventListenerProbe implements ApplicationListener {
 
-  private final BlockingQueue<String> namespaces = new LinkedBlockingQueue<String>();
+  private final Set<String> namespaces = Collections.synchronizedSet(new HashSet<String>());
 
   @Override
   public void onApplicationEvent(ApplicationEvent event) {
     if (event instanceof ApolloConfigChangeEvent) {
-      namespaces.offer(((ApolloConfigChangeEvent) event).getConfigChangeEvent().getNamespace());
+      namespaces.add(((ApolloConfigChangeEvent) event).getConfigChangeEvent().getNamespace());
     }
   }
 
-  public String pollNamespace(long timeout, TimeUnit unit) throws InterruptedException {
-    return namespaces.poll(timeout, unit);
+  public boolean hasNamespace(String namespace) {
+    return namespaces.contains(namespace);
   }
 }