9350893073

Committed 03 Jun 2024 01:16PM UTC coverage: 95.761% (-1.1%) from 96.898%

Build # 9350893073

Build Type

Pull #1172

github

Committed by

web-flow

Commit Message

Merge e0169f5a7 into 1ca2243f5

Pull Request Pull Request #1172: feat: new account management view

Run Details

410 of 424 branches covered (96.7%)

Branch coverage included in aggregate %.

24 of 40 new or added lines in 5 files covered. (60.0%)

5 existing lines in 2 files now uncovered.

1081 of 1133 relevant lines covered (95.41%)

19.38 hits per line

Source File
Press 'n' to go to next uncovered line, 'b' for previous

87.5

/src/components/Sidebar.tsx

import {
  BellIcon,
  GearIcon,
  PersonIcon,
  SyncIcon,
  XCircleIcon,
} from '@primer/octicons-react';
import { ipcRenderer } from 'electron';
import { type FC, useCallback, useContext, useMemo } from 'react';
import { useLocation, useNavigate } from 'react-router-dom';
import { Logo } from '../components/Logo';
import { AppContext } from '../context/App';
import { openExternalLink } from '../utils/comms';
import { Constants } from '../utils/constants';
import { getNotificationCount } from '../utils/notifications';

export const Sidebar: FC = () => {
  const navigate = useNavigate();
  const location = useLocation();

  const { notifications, fetchNotifications, isLoggedIn, status } =
    useContext(AppContext);

  const onOpenBrowser = useCallback(() => {
    openExternalLink(`https://github.com/${Constants.REPO_SLUG}`);
  }, []);

  const onOpenGitHubNotifications = useCallback(() => {
    openExternalLink('https://github.com/notifications');
  }, []);

  const quitApp = useCallback(() => {
    ipcRenderer.send('app-quit');
  }, []);

  const toggleSettings = () => {
    if (location.pathname.startsWith('/settings')) {
      navigate('/', { replace: true });
    } else {
      navigate('/settings');
    }
  };

  const notificationsCount = useMemo(() => {
    return getNotificationCount(notifications);
  }, [notifications]);

  const sidebarButtonClasses =
    'flex justify-evenly items-center bg-transparent border-0 w-full text-sm text-white my-1 py-2 cursor-pointer hover:text-gray-500 focus:outline-none disabled:text-gray-500 disabled:cursor-default';

  return (
    <div className="flex flex-col fixed left-14 w-14 -ml-14 h-full bg-gray-sidebar overflow-y-auto">
      <div className="flex flex-col flex-1 items-center py-4">
        <button
          type="button"
          className="w-5 my-3 mx-auto cursor-pointer outline-none"
          title="Open Gitify on GitHub"
          onClick={onOpenBrowser}
          data-testid="gitify-logo"
        >
          <Logo aria-label="Open Gitify" />
        </button>

        <button
          type="button"
          className={`flex justify-around self-stretch items-center my-1 py-1 px-2 text-xs font-extrabold cursor-pointer ${
            notificationsCount > 0 ? 'text-green-500' : 'text-white'
          }`}
          onClick={onOpenGitHubNotifications}
          title={`${notificationsCount} Unread Notifications`}
        >
          <BellIcon
            size={12}
            aria-label={`${notificationsCount} Unread Notifications`}
          />
          {notificationsCount > 0 && notificationsCount}
        </button>
      </div>

      <div className="py-4 px-3">
        {isLoggedIn && (
          <>
            <button
              type="button"
              className={sidebarButtonClasses}
              title="Refresh Notifications"
              onClick={() => {
                navigate('/', { replace: true });
                fetchNotifications();
              }}
              disabled={status === 'loading'}
            >
              <SyncIcon
                size={16}
                aria-label="Refresh Notifications"
                className={status === 'loading' ? 'animate-spin' : undefined}
              />
            </button>
            <button
              type="button"
              className={sidebarButtonClasses}
              title="Accounts"
              onClick={() => {
                if (location.pathname.startsWith('/accounts')) {
                  navigate('/', { replace: true });
                } else {
                  navigate('/accounts');
                }
              }}
            >
              <PersonIcon size={16} aria-label="Accounts" />
            </button>
            <button
              type="button"
              className={sidebarButtonClasses}
              title="Settings"
              onClick={toggleSettings}
            >
              <GearIcon size={16} aria-label="Settings" />
            </button>
          </>
        )}

        {!isLoggedIn && (
          <button
            type="button"
            className={sidebarButtonClasses}
            title="Quit Gitify"
            aria-label="Quit Gitify"
            onClick={quitApp}
          >
            <XCircleIcon size={16} aria-label="Quit Gitify" />
          </button>
        )}
      </div>
    </div>
  );
};

1	import {	2✔
2	BellIcon,
3	GearIcon,
4	PersonIcon,
5	SyncIcon,
6	XCircleIcon,
7	} from '@primer/octicons-react';
8	import { ipcRenderer } from 'electron';	2✔
9	import { type FC, useCallback, useContext, useMemo } from 'react';	2✔
10	import { useLocation, useNavigate } from 'react-router-dom';	2✔
11	import { Logo } from '../components/Logo';	2✔
12	import { AppContext } from '../context/App';	2✔
13	import { openExternalLink } from '../utils/comms';	2✔
14	import { Constants } from '../utils/constants';	2✔
15	import { getNotificationCount } from '../utils/notifications';	2✔
16
17	export const Sidebar: FC = () => {	2✔
18	const navigate = useNavigate();	22✔
19	const location = useLocation();	22✔
20
21	const { notifications, fetchNotifications, isLoggedIn, status } =
22	useContext(AppContext);	22✔
23
24	const onOpenBrowser = useCallback(() => {	22✔
25	openExternalLink(`https://github.com/${Constants.REPO_SLUG}`);	2✔
26	}, []);
27
28	const onOpenGitHubNotifications = useCallback(() => {	22✔
29	openExternalLink('https://github.com/notifications');	2✔
30	}, []);
31
32	const quitApp = useCallback(() => {	22✔
33	ipcRenderer.send('app-quit');	2✔
34	}, []);
35
36	const toggleSettings = () => {	22✔
37	if (location.pathname.startsWith('/settings')) {	4✔
38	navigate('/', { replace: true });	2✔
39	} else {
40	navigate('/settings');	2✔
41	}
42	};
43
44	const notificationsCount = useMemo(() => {	22✔
45	return getNotificationCount(notifications);	22✔
46	}, [notifications]);
47
48	const sidebarButtonClasses =
49	'flex justify-evenly items-center bg-transparent border-0 w-full text-sm text-white my-1 py-2 cursor-pointer hover:text-gray-500 focus:outline-none disabled:text-gray-500 disabled:cursor-default';	22✔
50
51	return (	22✔
52	<div className="flex flex-col fixed left-14 w-14 -ml-14 h-full bg-gray-sidebar overflow-y-auto">
53	<div className="flex flex-col flex-1 items-center py-4">
54	<button
55	type="button"
56	className="w-5 my-3 mx-auto cursor-pointer outline-none"
57	title="Open Gitify on GitHub"
58	onClick={onOpenBrowser}
59	data-testid="gitify-logo"
60	>
61	<Logo aria-label="Open Gitify" />
62	</button>
63
64	<button
65	type="button"
66	className={`flex justify-around self-stretch items-center my-1 py-1 px-2 text-xs font-extrabold cursor-pointer ${
67	notificationsCount > 0 ? 'text-green-500' : 'text-white'	11✔
68	}`}
69	onClick={onOpenGitHubNotifications}
70	title={`${notificationsCount} Unread Notifications`}
71	>
72	<BellIcon
73	size={12}
74	aria-label={`${notificationsCount} Unread Notifications`}
75	/>
76	{notificationsCount > 0 && notificationsCount}	15✔
77	</button>
78	</div>
79
80	<div className="py-4 px-3">
81	{isLoggedIn && (	18✔
82	<>
83	<button
84	type="button"
85	className={sidebarButtonClasses}
86	title="Refresh Notifications"
87	onClick={() => {
88	navigate('/', { replace: true });	2✔
89	fetchNotifications();	2✔
90	}}
91	disabled={status === 'loading'}
92	>
93	<SyncIcon
94	size={16}
95	aria-label="Refresh Notifications"
96	className={status === 'loading' ? 'animate-spin' : undefined}	7✔
97	/>
98	</button>
99	<button
100	type="button"
101	className={sidebarButtonClasses}
102	title="Accounts"
103	onClick={() => {
NEW UNCOV 104	if (location.pathname.startsWith('/accounts')) {	×
NEW UNCOV 105	navigate('/', { replace: true });	×
106	} else {
NEW UNCOV 107	navigate('/accounts');	×
108	}
109	}}
110	>
111	<PersonIcon size={16} aria-label="Accounts" />
112	</button>
113	<button
114	type="button"
115	className={sidebarButtonClasses}
116	title="Settings"
117	onClick={toggleSettings}
118	>
119	<GearIcon size={16} aria-label="Settings" />
120	</button>
121	</>
122	)}
123
124	{!isLoggedIn && (	15✔
125	<button
126	type="button"
127	className={sidebarButtonClasses}
128	title="Quit Gitify"
129	aria-label="Quit Gitify"
130	onClick={quitApp}
131	>
132	<XCircleIcon size={16} aria-label="Quit Gitify" />
133	</button>
134	)}
135	</div>
136	</div>
137	);
138	};

gitify-app / gitify / 9350893073

Source File Press 'n' to go to next uncovered line, 'b' for previous

Source File
Press 'n' to go to next uncovered line, 'b' for previous