feat(search-input): add onFocus

[Valentin-Lachere]

PR Description

...

GitHub issues resolved by PR

...

Bug fix checklist

• Units tests have been adjusted to account for bug.
• The fix has been tested in multiple Storybook stories.
• All GitHub checks are successful.

New component checklist

• The new component and its tests are in the same component folder.
• The component is unit tested and/or snapshot tested.
• All of the relevant Storybook stories have been added to the storybook package.
• There are no linting errors or warnings in the modified/new code.
• All GitHub checks are successful.

[meriouma]

C'est quoi le use case pour avoir besoin du click sur le input? C'est rarement un event qu'on met sur un input 🤔

[Valentin-Lachere]

C'est pour ouvrir un dropdown quand la personne clique le input avant de saisir du texte car le dropdown contient un historique des recherches précédentes.

[meriouma]

Dans ce cas c'est plus onFocus que tu as besoin probablement? Aussi, ce serait bien d'être plus spécifique. Quelque chose comme onInputFocus?

[maxime-gendron]

Je suis d'accord avec @meriouma , le onClick peut porter à confusion avec le search-button dans SearchContextual.

[maxime-gendron]

J'ajouterais ce test dans search-input.test.tsx et ce serait all good.

        it('should call onClick when input is clicked', () => {
            const onClick = jest.fn();
            const wrapper = shallow(
                <SearchInput onClick={onClick} />,
            );

            getByTestId(wrapper, 'search-input').simulate('click');

            expect(onClick).toHaveBeenCalledTimes(1);
        });

[maxime-gendron]

Ça devrait être onInputFocus car ça peut porter à confusion avec le focus du bouton.

[maxime-gendron]

Avant de merger il faudrait cleaner les commits et changer le titre de la PR. Pour sauver des étapes tu peux seulement changer le titre de la PR et faire un squash and merge. Ça va faire un merge commit avec le titre de la PR comme commit message.