From 00fa8dee6678c7d43dc530b5b7c82310a69b07a1 Mon Sep 17 00:00:00 2001
From: coltMcKissick <colt.mckissick@gmail.com>
Date: Fri, 14 Nov 2025 15:01:34 -0500
Subject: [PATCH] fix: set lowercase when passing to handle fetch, update unit
 test

Signed-off-by: coltMcKissick <colt.mckissick@gmail.com>
---
 .../EntityOwnerPicker.test.tsx                | 42 ++++++++++++++++++-
 .../EntityOwnerPicker/EntityOwnerPicker.tsx   |  8 +++-
 2 files changed, 46 insertions(+), 4 deletions(-)

diff --git a/plugins/catalog-react/src/components/EntityOwnerPicker/EntityOwnerPicker.test.tsx b/plugins/catalog-react/src/components/EntityOwnerPicker/EntityOwnerPicker.test.tsx
index e89a3bcadd..e49d54b485 100644
--- a/plugins/catalog-react/src/components/EntityOwnerPicker/EntityOwnerPicker.test.tsx
+++ b/plugins/catalog-react/src/components/EntityOwnerPicker/EntityOwnerPicker.test.tsx
@@ -365,6 +365,37 @@ describe('<EntityOwnerPicker mode="all" />', () => {
 
   it('calls fetch with lowercased input and displays results', async () => {
     const updateFilters = jest.fn();
+    const someOwnerEntities: Entity[] = [
+      {
+        apiVersion: '1',
+        kind: 'Group',
+        metadata: {
+          name: 'some-owner',
+        },
+      },
+      {
+        apiVersion: '1',
+        kind: 'Group',
+        metadata: {
+          name: 'some-owner-2',
+        },
+        spec: {
+          profile: {
+            displayName: 'Some Owner 2',
+          },
+        },
+      },
+    ];
+    mockCatalogApi.queryEntities.mockImplementation(async _request => {
+      const totalItems = 2;
+      return {
+        items: someOwnerEntities,
+        pageInfo: {
+          nextCursor: '',
+        },
+        totalItems,
+      };
+    });
     await renderInTestApp(
       <ApiProvider apis={mockApis}>
         <MockEntityListContextProvider
@@ -382,13 +413,20 @@ describe('<EntityOwnerPicker mode="all" />', () => {
       owners: undefined,
     });
 
-    fireEvent.click(screen.getByTestId('owner-picker-expand'));
-    const input = screen.getByRole('textbox', { name: 'Owner' });
+    // fireEvent.click(screen.getByTestId('owner-picker-expand'));
+    const input = screen.getByRole('textbox');
     fireEvent.change(input, { target: { value: 'Some-Owner' } });
 
     await waitFor(() =>
       expect(screen.getByText('some-owner')).toBeInTheDocument(),
     );
+    expect(mockCatalogApi.queryEntities).toHaveBeenLastCalledWith(
+      expect.objectContaining({
+        fullTextFilter: expect.objectContaining({
+          term: 'some-owner',
+        }),
+      }),
+    );
 
     fireEvent.click(screen.getByText('some-owner'));
     expect(updateFilters).toHaveBeenLastCalledWith({
diff --git a/plugins/catalog-react/src/components/EntityOwnerPicker/EntityOwnerPicker.tsx b/plugins/catalog-react/src/components/EntityOwnerPicker/EntityOwnerPicker.tsx
index cbfd20ad3e..ed200c77c4 100644
--- a/plugins/catalog-react/src/components/EntityOwnerPicker/EntityOwnerPicker.tsx
+++ b/plugins/catalog-react/src/components/EntityOwnerPicker/EntityOwnerPicker.tsx
@@ -146,7 +146,11 @@ export const EntityOwnerPicker = (props?: EntityOwnerPickerProps) => {
     mode,
     initialSelectedOwnersRefs: selectedOwners,
   });
-  useDebouncedEffect(() => handleFetch({ text }), [text, handleFetch], 250);
+  useDebouncedEffect(
+    () => handleFetch({ text: text.toLocaleLowerCase('en-US') }),
+    [text, handleFetch],
+    250,
+  );
 
   const availableOwners = value?.items || [];
 
@@ -221,7 +225,7 @@ export const EntityOwnerPicker = (props?: EntityOwnerPickerProps) => {
         }}
         name="owner-picker"
         onInputChange={(_e, inputValue) => {
-          setText(inputValue.toLocaleLowerCase('en-US'));
+          setText(inputValue);
         }}
         ListboxProps={{
           onScroll: (e: MouseEvent) => {