Merge pull request #137 from adhocteam/274-allow-case-insensitive-search

Allow search to be case-insensitive

frontend/src/pages/Admin/__tests__/index.js

@@ -29,15 +29,15 @@ describe('Admin Page', () => {
     const users = [
       {
         id: 2,
-        email: 'email',
+        email: '[email protected]',
         name: undefined,
         homeRegionId: 1,
         role: 'Grantee Specialist',
         permissions: [],
       },
       {
         id: 3,
-        email: 'email',
+        email: '[email protected]',
         name: 'Harry Potter',
         homeRegionId: 1,
         role: 'Grantee Specialist',
@@ -63,6 +63,23 @@ describe('Admin Page', () => {
         expect(links[0]).toHaveTextContent('Harry Potter');
       });
 
+      it('User list is filterable by email', async () => {
+        const filter = await screen.findByLabelText('Filter Users');
+        userEvent.type(filter, '@hogwarts.com');
+        const sideNav = screen.getByTestId('sidenav');
+        const links = within(sideNav).getAllByRole('link');
+        expect(links.length).toBe(2);
+      });
+
+      it('user filtering is case-insentive', async () => {
+        const filter = await screen.findByLabelText('Filter Users');
+        userEvent.type(filter, 'harry');
+        const sideNav = screen.getByTestId('sidenav');
+        const links = within(sideNav).getAllByRole('link');
+        expect(links.length).toBe(1);
+        expect(links[0]).toHaveTextContent('Harry Potter');
+      });
+
       it('allows a user to be selected', async () => {
         const button = await screen.findByText('Harry Potter');
         userEvent.click(button);

frontend/src/pages/Admin/index.js

@@ -81,7 +81,7 @@ function Admin(props) {
 
   const filteredUsers = _.filter(users, (u) => {
     const { email, name } = u;
-    return `${email}${name}`.includes(userSearch);
+    return `${email}${name}`.toLowerCase().includes(userSearch.toLowerCase());
   });
 
   const onSave = async (newUser) => {