fix: fire opened-change only when opened state changes (#8230) (#8234)

Co-authored-by: Sascha Ißbrücker <sissbruecker@vaadin.com>

Author: vaadin-bot

vaadin-popover-flow-parent/vaadin-popover-flow/src/main/java/com/vaadin/flow/component/popover/Popover.java

@@ -28,7 +28,6 @@
 import com.vaadin.flow.component.Component;
 import com.vaadin.flow.component.ComponentEvent;
 import com.vaadin.flow.component.ComponentEventListener;
-import com.vaadin.flow.component.DomEvent;
 import com.vaadin.flow.component.HasAriaLabel;
 import com.vaadin.flow.component.HasComponents;
 import com.vaadin.flow.component.Synchronize;
@@ -93,6 +92,9 @@ public Popover() {
 
         updateTrigger();
         setOverlayRole("dialog");
+
+        getElement().addPropertyChangeListener("opened", event -> fireEvent(
+                new OpenedChangeEvent(this, event.isUserOriginated())));
     }
 
     /**
@@ -192,7 +194,6 @@ private static void applyConfigurationForUI(UI ui) {
      * {@code opened-changed} event is sent when the overlay opened state
      * changes.
      */
-    @DomEvent("opened-changed")
     public static class OpenedChangeEvent extends ComponentEvent<Popover> {
         private final boolean opened;
 
@@ -215,7 +216,6 @@ public boolean isOpened() {
     public void setOpened(boolean opened) {
         if (opened != isOpened()) {
             getElement().setProperty("opened", opened);
-            fireEvent(new OpenedChangeEvent(this, false));
         }
     }
 

vaadin-popover-flow-parent/vaadin-popover-flow/src/test/java/com/vaadin/flow/component/popover/PopoverOpenedChangeListenerTest.java

@@ -25,8 +25,13 @@
 
 import com.vaadin.flow.component.ComponentEventListener;
 import com.vaadin.flow.component.UI;
+import com.vaadin.flow.dom.DomEvent;
+import com.vaadin.flow.dom.Element;
+import com.vaadin.flow.internal.nodefeature.ElementListenerMap;
 import com.vaadin.flow.server.VaadinSession;
 
+import elemental.json.Json;
+
 public class PopoverOpenedChangeListenerTest {
     private final UI ui = new UI();
     private Popover popover;
@@ -87,6 +92,18 @@ public void close() {
         assertListenerCalls(0);
     }
 
+    @Test
+    public void openedChangeFromClient_noChangeEvent() {
+        Element element = popover.getElement();
+        element.getNode().getFeature(ElementListenerMap.class).fireEvent(
+                new DomEvent(element, "opened-changed", Json.createObject()));
+
+        // The event should only be fired when the opened state changes, not for
+        // any opened-changed event from the client
+        Assert.assertNull(event.get());
+        assertListenerCalls(0);
+    }
+
     private void assertListenerCalls(int expectedCount) {
         Mockito.verify(mockListener, Mockito.times(expectedCount))
                 .onComponentEvent(Mockito.any());