fix: do not throw on closing popover when target is not set (#9942) (#9947)

Co-authored-by: Serhii Kulykov <iamkulykov@gmail.com>

Author: vaadin-bot

packages/popover/src/vaadin-popover.js

@@ -950,7 +950,7 @@ class Popover extends PopoverPositionMixin(
     }
 
     // Restore pointer-events set when opening on hover.
-    if (this.modal && this.target.style.pointerEvents) {
+    if (this.modal && this.target && this.target.style.pointerEvents) {
       this.target.style.pointerEvents = '';
     }
 

packages/popover/test/basic.test.js

@@ -149,6 +149,21 @@ describe('popover', () => {
 
       expect(document.activeElement).to.not.equal(target);
     });
+
+    it('should not throw when target is removed', async () => {
+      popover.modal = true;
+
+      // Clear target
+      popover.target = null;
+      await nextUpdate(popover);
+
+      popover.opened = true;
+      await oneEvent(overlay, 'vaadin-overlay-open');
+
+      // No error should be thrown
+      popover.opened = false;
+      await oneEvent(popover, 'closed');
+    });
   });
 
   describe('for', () => {