fix: filtered dropdown items by display value instead of event target (…

…#2547)
openedx · Aug 18, 2023 · cac6984 · cac6984
1 parent c803a4d
commit cac6984
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/src/Form/FormAutosuggest.jsx b/src/Form/FormAutosuggest.jsx
@@ -79,15 +79,15 @@ function FormAutosuggest({
     return childrenOpt;
   }
 
-  const handleExpand = (e) => {
+  const handleExpand = () => {
     setIsMenuClosed(!isMenuClosed);
 
     const newState = {
       dropDownItems: [],
     };
 
     if (isMenuClosed) {
-      newState.dropDownItems = getItems(e.target.value);
+      newState.dropDownItems = getItems(state.displayValue);
       newState.errorMessage = '';
     }
 

diff --git a/src/Form/tests/FormAutosuggest.test.jsx b/src/Form/tests/FormAutosuggest.test.jsx
@@ -140,7 +140,7 @@ describe('FormAutosuggest', () => {
       expect(container.find(dropdownContainer).find('button').length).toEqual(0);
 
       container.find('button.pgn__form-autosuggest__icon-button').simulate('click');
-      expect(container.find(dropdownContainer).find('button').length).toEqual(3);
+      expect(container.find(dropdownContainer).find('button').length).toEqual(1);
     });
 
     it('shows options list depends on field value', () => {