feat: lowercase search input value when looking for redirects

DAU-249
sajari · Feb 7, 2023 · 9966eb0 · 9966eb0
1 parent 6797d7e
commit 9966eb0
Show file tree

Hide file tree

Showing 3 changed files with 23 additions and 2 deletions.
diff --git a/.changeset/plenty-goats-tan.md b/.changeset/plenty-goats-tan.md
@@ -0,0 +1,5 @@
+---
+'@sajari/react-search-ui': minor
+---
+
+feat: lowercase search input value when looking for redirects
diff --git a/packages/search-ui/src/Input/index.test.tsx b/packages/search-ui/src/Input/index.test.tsx
@@ -101,6 +101,22 @@ describe('Input', () => {
     });
   });
 
+  it('ignores letter casing of input query on redirect lookup', async () => {
+    const onRedirectSpy = jest.spyOn(eventTrackingPipeline.getTracking(), 'onRedirect');
+    customRender(<Input data-testid="mysearch" mode="suggestions" />, {
+      search: { pipeline: eventTrackingPipeline },
+    });
+    const input = screen.getByTestId('mysearch');
+    input.focus(); // need this else we get TypeError: element.ownerDocument.getSelection is not a function
+    await user.keyboard('ShEeTs');
+    await waitFor(() => expect(input).toHaveTextContent('ShEeTs'));
+    await user.keyboard('{enter}');
+    expect(onRedirectSpy).toHaveBeenCalledWith({
+      ...redirectTarget,
+      token: `https://re.sajari.com/token/${redirectTarget.token}`, // sdk-js prepends the clickTokenURL
+    });
+  });
+
   it('calls onResultClick on results click', async () => {
     const onResultClickSpy = jest.spyOn(eventTrackingPipeline.getTracking(), 'onResultClick');
     customRender(<Input data-testid="mysearch" mode="results" />, {

diff --git a/packages/search-ui/src/Input/index.tsx b/packages/search-ui/src/Input/index.tsx
@@ -174,7 +174,7 @@ const Input = React.forwardRef((props: InputProps<any>, ref: React.ForwardedRef<
           if (!retainFilters) {
             resetFilters();
           }
-          const redirectValue = redirectsRef.current[value];
+          const redirectValue = redirectsRef.current[value.toLowerCase()];
           if (!disableRedirects && redirectValue) {
             tracking.onRedirect(redirectValue);
             window.location.assign(redirectValue.token || redirectValue.target);
@@ -183,7 +183,7 @@ const Input = React.forwardRef((props: InputProps<any>, ref: React.ForwardedRef<
             // If we're performing an autocomplete search, wait a tick to recheck redirects before unloading
             e.preventDefault();
             setTimeout(() => {
-              const redirectTarget = redirectsRef.current[value];
+              const redirectTarget = redirectsRef.current[value.toLowerCase()];
               if (redirectTarget) {
                 tracking.onRedirect(redirectTarget);
                 window.location.assign(redirectTarget.token || redirectTarget.target);