feat: change icon navigation to go home (#1533)

* feat: change icon to navigate home

Signed-off-by: Adam Setch <[email protected]>

* feat: change icon to navigate home

Signed-off-by: Adam Setch <[email protected]>

---------

Signed-off-by: Adam Setch <[email protected]>

Author: setchy

src/components/Sidebar.test.tsx

@@ -60,7 +60,7 @@ describe('components/Sidebar.tsx', () => {
     expect(tree).toMatchSnapshot();
   });
 
-  it('should open the gitify repository', () => {
+  it('should navigate home when clicking the gitify logo', () => {
     render(
       <AppContext.Provider
         value={{
@@ -76,12 +76,8 @@ describe('components/Sidebar.tsx', () => {
       </AppContext.Provider>,
     );
 
-    fireEvent.click(screen.getByTestId('gitify-logo'));
-
-    expect(openExternalLinkMock).toHaveBeenCalledTimes(1);
-    expect(openExternalLinkMock).toHaveBeenCalledWith(
-      'https://github.com/gitify-app/gitify',
-    );
+    fireEvent.click(screen.getByTitle('Home'));
+    expect(mockNavigate).toHaveBeenNthCalledWith(1, '/', { replace: true });
   });
 
   describe('quick links', () => {

src/components/Sidebar.tsx

@@ -18,7 +18,6 @@ import {
   openGitHubIssues,
   openGitHubNotifications,
   openGitHubPulls,
-  openGitifyRepository,
 } from '../utils/links';
 import { getNotificationCount } from '../utils/notifications';
 import { SidebarButton } from './buttons/SidebarButton';
@@ -77,9 +76,8 @@ export const Sidebar: FC = () => {
         <button
           type="button"
           className="mx-auto my-3 cursor-pointer outline-none"
-          title="Open Gitify on GitHub"
-          onClick={() => openGitifyRepository()}
-          data-testid="gitify-logo"
+          title="Home"
+          onClick={() => navigate('/', { replace: true })}
         >
           <LogoIcon size={Size.SMALL} aria-label="Open Gitify" />
         </button>

src/components/__snapshots__/Sidebar.test.tsx.snap

@@ -15,7 +15,6 @@ import {
   openGitHubParticipatingDocs,
   openGitHubPulls,
   openGitifyReleaseNotes,
-  openGitifyRepository,
   openHost,
   openNotification,
   openRepository,
@@ -31,13 +30,6 @@ describe('utils/links.ts', () => {
     jest.clearAllMocks();
   });
 
-  it('openGitifyRepository', () => {
-    openGitifyRepository();
-    expect(openExternalLinkMock).toHaveBeenCalledWith(
-      'https://github.com/gitify-app/gitify',
-    );
-  });
-
   it('openGitifyReleaseNotes', () => {
     openGitifyReleaseNotes('v1.0.0');
     expect(openExternalLinkMock).toHaveBeenCalledWith(

src/utils/links.test.ts

@@ -5,10 +5,6 @@ import { openExternalLink } from './comms';
 import { Constants } from './constants';
 import { generateGitHubWebUrl } from './helpers';
 
-export function openGitifyRepository() {
-  openExternalLink(`https://github.com/${Constants.REPO_SLUG}` as Link);
-}
-
 export function openGitifyReleaseNotes(version: string) {
   openExternalLink(
     `https://github.com/${Constants.REPO_SLUG}/releases/tag/${version}` as Link,