test: reset keyboardActive between tests (#8504)

* test: reset keyboardActive between tests

* reset state in date and time picker tests

packages/combo-box/test/visual/lumo/combo-box.test.js

@@ -1,4 +1,4 @@
-import { fixtureSync, nextFrame } from '@vaadin/testing-helpers';
+import { fixtureSync, mousedown, nextFrame } from '@vaadin/testing-helpers';
 import { sendKeys } from '@web/test-runner-commands';
 import { visualDiff } from '@web/test-runner-visual-regression';
 import '../common.js';
@@ -15,6 +15,13 @@ describe('combo-box', () => {
     element = fixtureSync('<vaadin-combo-box></vaadin-combo-box>', div);
   });
 
+  afterEach(() => {
+    // After tests which use sendKeys() the focus-utils.js -> isKeyboardActive is set to true.
+    // Click once here on body to reset it so other tests are not affected by it.
+    // An unwanted focus-ring would be shown in other tests otherwise.
+    mousedown(document.body);
+  });
+
   it('basic', async () => {
     await visualDiff(div, 'basic');
   });

packages/date-picker/test/visual/lumo/date-picker.test.js

@@ -1,4 +1,4 @@
-import { fixtureSync } from '@vaadin/testing-helpers/dist/fixture.js';
+import { fixtureSync, mousedown } from '@vaadin/testing-helpers';
 import { sendKeys } from '@web/test-runner-commands';
 import { visualDiff } from '@web/test-runner-visual-regression';
 import '../../../theme/lumo/vaadin-date-picker.js';
@@ -15,6 +15,13 @@ describe('date-picker', () => {
     element = fixtureSync('<vaadin-date-picker></vaadin-date-picker>', div);
   });
 
+  afterEach(() => {
+    // After tests which use sendKeys() the focus-utils.js -> isKeyboardActive is set to true.
+    // Click once here on body to reset it so other tests are not affected by it.
+    // An unwanted focus-ring would be shown in other tests otherwise.
+    mousedown(document.body);
+  });
+
   it('basic', async () => {
     await visualDiff(div, 'basic');
   });

packages/time-picker/test/visual/lumo/time-picker.test.js

@@ -15,6 +15,13 @@ describe('time-picker', () => {
     element.style.setProperty('--vaadin-time-picker-overlay-max-height', '300px');
   });
 
+  afterEach(() => {
+    // After tests which use sendKeys() the focus-utils.js -> isKeyboardActive is set to true.
+    // Click once here on body to reset it so other tests are not affected by it.
+    // An unwanted focus-ring would be shown in other tests otherwise.
+    mousedown(document.body);
+  });
+
   it('basic', async () => {
     await visualDiff(div, 'basic');
   });
@@ -23,11 +30,6 @@ describe('time-picker', () => {
     await sendKeys({ press: 'Tab' });
 
     await visualDiff(div, 'focus-ring');
-
-    // at this moment focus-utils.js -> isKeyboardActive is true.
-    // Click once here to reset it so other tests are not affected by it.
-    // A focus-ring would be shown in other tests otherwise.
-    mousedown(document.body);
   });
 
   it('disabled', async () => {