fix: further sonar issues (#1819)

* fix: issues identified by sonar

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

* fix: issues identified by sonar

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

---------

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

Author: setchy

src/renderer/components/avatars/AvatarWithFallback.tsx

@@ -26,6 +26,7 @@ export const AvatarWithFallback: React.FC<IAvatarWithFallback> = ({
   const [isBroken, setIsBroken] = useState(false);
 
   const isNonHuman = isNonHumanUser(userType);
+  const DefaultUserIcon = isNonHuman ? MarkGithubIcon : FeedPersonIcon;
 
   // TODO explore using AnchoredOverlay component (https://primer.style/components/anchored-overlay/react/alpha) to render Avatar Card on hover
   return (
@@ -36,11 +37,7 @@ export const AvatarWithFallback: React.FC<IAvatarWithFallback> = ({
       data-testid="avatar"
     >
       {!src || isBroken ? (
-        isNonHuman ? (
-          <MarkGithubIcon size={size} />
-        ) : (
-          <FeedPersonIcon size={size} />
-        )
+        <DefaultUserIcon size={size} />
       ) : (
         <Avatar
           src={src}

src/renderer/components/fields/Checkbox.tsx

@@ -54,11 +54,11 @@ export const Checkbox: FC<ICheckbox> = ({
           <Tooltip name={`tooltip-${props.name}`} tooltip={props.tooltip} />
         )}
 
-        {counter && (
+        {counter ? (
           <CounterLabel scheme={props.checked ? 'primary' : 'secondary'}>
             {counter}
           </CounterLabel>
-        )}
+        ) : null}
       </Stack>
     )
   );

src/renderer/components/metrics/MetricPill.tsx

@@ -23,7 +23,9 @@ export const MetricPill: FC<IMetricPill> = (props: IMetricPill) => {
     >
       <Stack direction="horizontal" align="center" gap="none">
         <props.icon size={Size.XSMALL} className={props.color} />
-        {props.metric && <Text className="text-xxs px-1">{props.metric}</Text>}
+        {props.metric ? (
+          <Text className="text-xxs px-1">{props.metric}</Text>
+        ) : null}
       </Stack>
     </Label>
   );

src/renderer/components/notifications/NotificationRow.tsx

@@ -85,8 +85,9 @@ export const NotificationRow: FC<INotificationRow> = ({
     ? `#${notification.subject.number}`
     : '';
 
-  const notificationTitle =
-    `${notification.subject.title} ${notificationNumber && `[${notificationNumber}]`}`.trim();
+  const notificationTitle = notificationNumber
+    ? `${notification.subject.title} [${notificationNumber}]`
+    : notification.subject.title;
 
   const groupByDate = settings.groupBy === GroupBy.DATE;
 

src/renderer/typesGitHub.ts

@@ -94,7 +94,7 @@ export type PullRequestReviewAuthorAssociation =
   | 'NONE'
   | 'OWNER';
 
-// TODO: Add explicit types for GitHub API response vs Gitify Notifications object
+// TODO: #828 Add explicit types for GitHub API response vs Gitify Notifications object
 export type Notification = GitHubNotification & GitifyNotification;
 
 export interface GitHubNotification {

src/renderer/utils/auth/utils.test.ts

@@ -101,7 +101,9 @@ describe('renderer/utils/auth/utils.ts', () => {
       });
 
       await expect(async () => await auth.authGitHub()).rejects.toEqual(
-        "Oops! Something went wrong and we couldn't log you in using GitHub. Please try again. Reason: The redirect_uri is missing or invalid. Docs: https://docs.github.com/en/developers/apps/troubleshooting-oauth-errors",
+        new Error(
+          "Oops! Something went wrong and we couldn't log you in using GitHub. Please try again. Reason: The redirect_uri is missing or invalid. Docs: https://docs.github.com/en/developers/apps/troubleshooting-oauth-errors",
+        ),
       );
 
       expect(openExternalLinkMock).toHaveBeenCalledTimes(1);

src/renderer/utils/auth/utils.ts

@@ -54,7 +54,9 @@ export function authGitHub(
         });
       } else if (error) {
         reject(
-          `Oops! Something went wrong and we couldn't log you in using GitHub. Please try again. Reason: ${errorDescription} Docs: ${errorUri}`,
+          new Error(
+            `Oops! Something went wrong and we couldn't log you in using GitHub. Please try again. Reason: ${errorDescription} Docs: ${errorUri}`,
+          ),
         );
       }
     };
@@ -257,9 +259,7 @@ export function getNewOAuthAppURL(hostname: Hostname): Link {
 }
 
 export function isValidHostname(hostname: Hostname) {
-  return /^([a-zA-Z0-9]([a-zA-Z0-9\-]{0,61}[a-zA-Z0-9])?\.)+[a-zA-Z]{2,6}$/i.test(
-    hostname,
-  );
+  return /^([A-Z0-9]([A-Z0-9-]{0,61}[A-Z0-9])?\.)+[A-Z]{2,6}$/i.test(hostname);
 }
 
 export function isValidClientId(clientId: ClientID) {

src/renderer/utils/constants.ts

@@ -1,5 +1,4 @@
-import type { Link } from '../types';
-import type { ClientID, ClientSecret, Hostname } from '../types';
+import type { ClientID, ClientSecret, Hostname, Link } from '../types';
 
 export const Constants = {
   REPO_SLUG: 'gitify-app/gitify',