chore: Remove the debug flag (#358)

aws · Sep 16, 2021 · 801d4d8 · 801d4d8
1 parent 34661af
commit 801d4d8
Show file tree

Hide file tree

Showing 4 changed files with 12 additions and 13 deletions.
diff --git a/src/components/Autosuggest/Autosuggest.md b/src/components/Autosuggest/Autosuggest.md
@@ -66,10 +66,9 @@ React.useEffect(() => {
         <Autosuggest
             controlId="formFieldId1"
             loadingText="Loading services"
-            onFocus={() => {
+            onFocus={(e) => {
                 setOptions([]);
                 setLoadingStatus(true);
-                action('onFocus');
             }}
             statusType={status}
             options={options}
@@ -166,7 +165,6 @@ import { awsServices } from './data/data';
             icon={Computer}
             filteringType="manual"
             freeSolo={true}
-            disableClearable={true}
             options={awsServices}
             controlId="formFieldId1"
             ariaDescribedby="This is a description"

diff --git a/src/components/Autosuggest/index.stories.tsx b/src/components/Autosuggest/index.stories.tsx
@@ -40,6 +40,8 @@ export const Default = () => (
             controlId="formFieldId1"
             ariaDescribedby="This is a description"
             onChange={action('onChange')}
+            onFocus={action('onFocus')}
+            onBlur={action('onBlur')}
         />
     </FormField>
 );
@@ -105,11 +107,12 @@ export const AsyncLoading = () => {
             <Autosuggest
                 controlId="formFieldId1"
                 loadingText="Loading services"
-                onFocus={() => {
+                onFocus={(e) => {
                     setOptions([]);
                     setLoadingStatus(true);
-                    action('onFocus');
+                    action('onFocus')(e);
                 }}
+                onBlur={action('onBlur')}
                 statusType={status}
                 options={options}
                 empty="No matching service found"
@@ -259,7 +262,6 @@ export const WithFreeSolo = () => (
             icon={Computer}
             freeSolo={true}
             filteringType="manual"
-            disableClearable={true}
             options={awsServices}
             controlId="formFieldId1"
             ariaDescribedby="This is a description"

diff --git a/src/components/Autosuggest/index.test.tsx b/src/components/Autosuggest/index.test.tsx
@@ -197,7 +197,7 @@ describe('Autosuggest', () => {
             const input = getByPlaceholderText('input-1');
 
             expect(mockEvent).toHaveBeenCalledTimes(0);
-            fireEvent.change(input, { target: { value: 'Amazon Pay' } });
+            fireEvent.click(input);
             expect(mockEvent).toHaveBeenCalledTimes(1);
         });
 
@@ -208,8 +208,9 @@ describe('Autosuggest', () => {
             const input = getByPlaceholderText('input-1');
 
             expect(mockEvent).toHaveBeenCalledTimes(0);
+            fireEvent.click(input);
             fireEvent.change(input, { target: { value: 'Amazon Pay' } });
-            fireEvent.keyDown(input, { key: 'Escape', code: '27' });
+            fireEvent.blur(input);
             expect(mockEvent).toHaveBeenCalledTimes(1);
         });
 

diff --git a/src/components/Autosuggest/index.tsx b/src/components/Autosuggest/index.tsx
@@ -370,7 +370,6 @@ const AutosuggestBase: FunctionComponent<AutosuggestBaseProps> = ({
     return (
         <Stack>
             <MaterialUIAutocomplete
-                debug
                 multiple={props.multiple}
                 data-testid={props.multiple ? 'multiselect' : 'autosuggest'}
                 disabled={disabled}
@@ -389,13 +388,12 @@ const AutosuggestBase: FunctionComponent<AutosuggestBaseProps> = ({
                 onChange={handleOnChange}
                 onInputChange={handleOnInput}
                 onFocus={onFocus}
-                onOpen={(e) => {
+                onBlur={onBlur}
+                onOpen={() => {
                     setOpen(true);
-                    onFocus?.(e as React.FocusEvent<HTMLElement>);
                 }}
-                onClose={(e) => {
+                onClose={() => {
                     setOpen(false);
-                    onBlur?.(e as React.FocusEvent<HTMLElement>);
                 }}
                 loading={statusType !== 'finished'}
                 renderOption={renderOption}