Try using fireEvent for comboboxes. On linux machines, user event tri…

…ggers some errors.
adobe · Feb 10, 2023 · f38d865 · f38d865
1 parent 46f00de
commit f38d865
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 7 deletions.
diff --git a/src/view/__tests_helpers__/jsDomHelpers.js b/src/view/__tests_helpers__/jsDomHelpers.js
@@ -38,6 +38,10 @@ export const changeInputValue = async (input, value) => {
   }
 };
 
+export const changeComboboxValue = async (input, value) => {
+  fireEvent.change(input, { target: { value } });
+};
+
 export const click = async (newTab) => {
   await userEvent.click(newTab);
 };

diff --git a/src/view/actions/__tests__/currencyField.test.jsx b/src/view/actions/__tests__/currencyField.test.jsx
@@ -15,7 +15,7 @@ import renderView from '../../__tests_helpers__/renderView';
 import Event from '../sendAddToCartEvent';
 import createExtensionBridge from '../../__tests_helpers__/createExtensionBridge';
 
-import { changeInputValue } from '../../__tests_helpers__/jsDomHelpers';
+import { changeComboboxValue } from '../../__tests_helpers__/jsDomHelpers';
 
 let extensionBridge;
 
@@ -71,7 +71,7 @@ describe('Currency field', () => {
 
     const input = getField();
 
-    await changeInputValue(input, '%ab%');
+    await changeComboboxValue(input, '%ab%');
 
     expect(extensionBridge.getSettings()).toEqual({
       currency: '%ab%'
@@ -89,7 +89,7 @@ describe('Currency field', () => {
 
     const input = getField();
 
-    await changeInputValue(input, 'EUR');
+    await changeComboboxValue(input, 'EUR');
 
     expect(extensionBridge.getSettings()).toEqual({
       currency: 'EUR'
@@ -107,7 +107,7 @@ describe('Currency field', () => {
 
     const input = getField();
 
-    await changeInputValue(input, '');
+    await changeComboboxValue(input, '');
 
     expect(extensionBridge.getSettings()).toEqual({});
   });
@@ -120,7 +120,7 @@ describe('Currency field', () => {
     const input = getField();
     expect(input).not.toHaveAttribute('aria-invalid');
 
-    await changeInputValue(input, 'USD');
+    await changeComboboxValue(input, 'USD');
     await extensionBridge.validate();
 
     expect(input).not.toHaveAttribute('aria-invalid', 'true');
@@ -134,7 +134,7 @@ describe('Currency field', () => {
     const input = getField();
     expect(input).not.toHaveAttribute('aria-invalid');
 
-    await changeInputValue(input, '%a%');
+    await changeComboboxValue(input, '%a%');
     await extensionBridge.validate();
 
     expect(input).not.toHaveAttribute('aria-invalid', 'true');
@@ -148,7 +148,7 @@ describe('Currency field', () => {
     const input = getField();
     expect(input).not.toHaveAttribute('aria-invalid');
 
-    await changeInputValue(input, 'a');
+    await changeComboboxValue(input, 'a');
     await extensionBridge.validate();
 
     expect(input).toHaveAttribute('aria-invalid', 'true');