refactor: update sidebar labels

Signed-off-by: Adam Setch <[email protected]>
gitify-app · Sep 17, 2024 · 2072b27 · 2072b27
1 parent 25faf92
commit 2072b27
Show file tree

Hide file tree

Showing 3 changed files with 41 additions and 41 deletions.
diff --git a/src/components/Sidebar.test.tsx b/src/components/Sidebar.test.tsx
@@ -102,13 +102,13 @@ describe('components/Sidebar.tsx', () => {
           </AppContext.Provider>,
         );
 
-        const notificationsIcon = screen.getByTitle('0 Unread Notifications');
+        const notificationsIcon = screen.getByTitle('0 unread notifications');
 
         expect(notificationsIcon.className).toContain('text-white');
         expect(notificationsIcon.childNodes.length).toBe(1);
         expect(notificationsIcon.childNodes[0].nodeName).toBe('svg');
 
-        fireEvent.click(screen.getByLabelText('0 Unread Notifications'));
+        fireEvent.click(screen.getByLabelText('0 unread notifications'));
 
         expect(openExternalLinkMock).toHaveBeenCalledTimes(1);
         expect(openExternalLinkMock).toHaveBeenCalledWith(
@@ -132,14 +132,14 @@ describe('components/Sidebar.tsx', () => {
           </AppContext.Provider>,
         );
 
-        const notificationsIcon = screen.getByTitle('4 Unread Notifications');
+        const notificationsIcon = screen.getByTitle('4 unread notifications');
 
         expect(notificationsIcon.className).toContain(IconColor.GREEN);
         expect(notificationsIcon.childNodes.length).toBe(2);
         expect(notificationsIcon.childNodes[0].nodeName).toBe('svg');
         expect(notificationsIcon.childNodes[1].nodeValue).toBe('4');
 
-        fireEvent.click(screen.getByLabelText('4 Unread Notifications'));
+        fireEvent.click(screen.getByLabelText('4 unread notifications'));
 
         expect(openExternalLinkMock).toHaveBeenCalledTimes(1);
         expect(openExternalLinkMock).toHaveBeenCalledWith(
@@ -165,7 +165,7 @@ describe('components/Sidebar.tsx', () => {
       </AppContext.Provider>,
     );
 
-    fireEvent.click(screen.getByLabelText('My Issues'));
+    fireEvent.click(screen.getByLabelText('My issues'));
 
     expect(openExternalLinkMock).toHaveBeenCalledTimes(1);
     expect(openExternalLinkMock).toHaveBeenCalledWith(
@@ -189,7 +189,7 @@ describe('components/Sidebar.tsx', () => {
       </AppContext.Provider>,
     );
 
-    fireEvent.click(screen.getByLabelText('My Pull Requests'));
+    fireEvent.click(screen.getByLabelText('My pull requests'));
 
     expect(openExternalLinkMock).toHaveBeenCalledTimes(1);
     expect(openExternalLinkMock).toHaveBeenCalledWith(
@@ -216,7 +216,7 @@ describe('components/Sidebar.tsx', () => {
         </AppContext.Provider>,
       );
 
-      fireEvent.click(screen.getByTitle('Refresh Notifications'));
+      fireEvent.click(screen.getByTitle('Refresh notifications'));
 
       expect(fetchNotifications).toHaveBeenCalledTimes(1);
     });
@@ -239,7 +239,7 @@ describe('components/Sidebar.tsx', () => {
         </AppContext.Provider>,
       );
 
-      fireEvent.click(screen.getByTitle('Refresh Notifications'));
+      fireEvent.click(screen.getByTitle('Refresh notifications'));
 
       expect(fetchNotifications).not.toHaveBeenCalled();
     });
@@ -348,7 +348,7 @@ describe('components/Sidebar.tsx', () => {
         </MemoryRouter>
       </AppContext.Provider>,
     );
-    fireEvent.click(screen.getByLabelText('4 Unread Notifications'));
+    fireEvent.click(screen.getByLabelText('4 unread notifications'));
     expect(openExternalLinkMock).toHaveBeenCalledTimes(1);
     expect(openExternalLinkMock).toHaveBeenCalledWith(
       'https://github.com/notifications',
@@ -372,7 +372,7 @@ describe('components/Sidebar.tsx', () => {
         </MemoryRouter>
       </AppContext.Provider>,
     );
-    fireEvent.click(screen.getByLabelText('My Issues'));
+    fireEvent.click(screen.getByLabelText('My issues'));
     expect(openExternalLinkMock).toHaveBeenCalledTimes(1);
     expect(openExternalLinkMock).toHaveBeenCalledWith(
       'https://github.com/issues',
@@ -396,7 +396,7 @@ describe('components/Sidebar.tsx', () => {
         </MemoryRouter>
       </AppContext.Provider>,
     );
-    fireEvent.click(screen.getByLabelText('My Pull Requests'));
+    fireEvent.click(screen.getByLabelText('My pull requests'));
     expect(openExternalLinkMock).toHaveBeenCalledTimes(1);
     expect(openExternalLinkMock).toHaveBeenCalledWith(
       'https://github.com/pulls',
@@ -443,7 +443,7 @@ describe('components/Sidebar.tsx', () => {
         </AppContext.Provider>,
       );
 
-      const notificationsIcon = screen.getByTitle('0 Unread Notifications');
+      const notificationsIcon = screen.getByTitle('0 unread notifications');
       expect(notificationsIcon.className).toContain('text-white');
       expect(notificationsIcon.childNodes.length).toBe(1);
       expect(notificationsIcon.childNodes[0].nodeName).toBe('svg');
@@ -465,7 +465,7 @@ describe('components/Sidebar.tsx', () => {
         </AppContext.Provider>,
       );
 
-      const notificationsIcon = screen.getByTitle('4 Unread Notifications');
+      const notificationsIcon = screen.getByTitle('4 unread notifications');
       expect(notificationsIcon.className).toContain(IconColor.GREEN);
       expect(notificationsIcon.childNodes.length).toBe(2);
       expect(notificationsIcon.childNodes[0].nodeName).toBe('svg');

diff --git a/src/components/Sidebar.tsx b/src/components/Sidebar.tsx
@@ -85,20 +85,20 @@ export const Sidebar: FC = () => {
         </button>
 
         <SidebarButton
-          title={`${notificationsCount} Unread Notifications`}
+          title={`${notificationsCount} unread notifications`}
           metric={isLoggedIn ? notificationsCount : null}
           icon={BellIcon}
           onClick={() => openGitHubNotifications(primaryAccountHostname)}
         />
 
         <SidebarButton
-          title="My Issues"
+          title="My issues"
           icon={IssueOpenedIcon}
           onClick={() => openGitHubIssues(primaryAccountHostname)}
         />
 
         <SidebarButton
-          title="My Pull Requests"
+          title="My pull requests"
           icon={GitPullRequestIcon}
           onClick={() => openGitHubPulls(primaryAccountHostname)}
         />
@@ -108,7 +108,7 @@ export const Sidebar: FC = () => {
         {isLoggedIn && (
           <>
             <SidebarButton
-              title="Refresh Notifications"
+              title="Refresh notifications"
               icon={SyncIcon}
               size={Size.MEDIUM}
               loading={status === 'loading'}

diff --git a/src/components/__snapshots__/Sidebar.test.tsx.snap b/src/components/__snapshots__/Sidebar.test.tsx.snap