refactor: remove now redundant mark as read on click functionality (#713)

* refactor: remove now redundant `mark as read on click` functionality

* refactor: remove now redundant `mark as read on click` functionality

* refactor: remove now redundant `mark as read on click` functionality

Author: setchy

Diff for: src/__mocks__/mock-state.ts

@@ -16,7 +16,6 @@ export const mockSettings: SettingsState = {
   participating: false,
   playSound: true,
   showNotifications: true,
-  markOnClick: false,
   openAtStartup: false,
   appearance: Appearance.SYSTEM,
   colors: false,

Diff for: src/components/NotificationRow.test.tsx

@@ -41,7 +41,7 @@ describe('components/Notification.js', () => {
     const { getByRole } = render(
       <AppContext.Provider
         value={{
-          settings: { ...mockSettings, markOnClick: true },
+          settings: mockSettings,
           markNotification,
           accounts: mockAccounts,
         }}
@@ -54,31 +54,6 @@ describe('components/Notification.js', () => {
     expect(shell.openExternal).toHaveBeenCalledTimes(1);
   });
 
-  it('should open a notification in browser & mark it as read', () => {
-    const markNotification = jest.fn();
-
-    const props = {
-      notification: mockedSingleNotification,
-      hostname: 'github.com',
-    };
-
-    const { getByRole } = render(
-      <AppContext.Provider
-        value={{
-          settings: { ...mockSettings, markOnClick: true },
-          markNotification,
-          accounts: mockAccounts,
-        }}
-      >
-        <NotificationRow {...props} />
-      </AppContext.Provider>,
-    );
-
-    fireEvent.click(getByRole('main'));
-    expect(shell.openExternal).toHaveBeenCalledTimes(1);
-    expect(markNotification).toHaveBeenCalledTimes(1);
-  });
-
   it('should mark a notification as read', () => {
     const markNotification = jest.fn();
 
@@ -90,7 +65,7 @@ describe('components/Notification.js', () => {
     const { getByTitle } = render(
       <AppContext.Provider
         value={{
-          settings: { ...mockSettings, markOnClick: false },
+          settings: mockSettings,
           accounts: mockAccounts,
         }}
       >

Diff for: src/components/NotificationRow.tsx

@@ -25,10 +25,6 @@ export const NotificationRow: React.FC<IProps> = ({
 
   const pressTitle = useCallback(() => {
     openBrowser();
-
-    if (settings.markOnClick) {
-      markNotification(notification.id, hostname);
-    }
   }, [settings]);
 
   const openBrowser = useCallback(

Diff for: src/components/Sidebar.test.tsx

@@ -35,7 +35,7 @@ describe('components/Sidebar.tsx', () => {
     const tree = TestRenderer.create(
       <AppContext.Provider
         value={{
-          settings: { ...mockSettings },
+          settings: mockSettings,
           notifications: mockedAccountNotifications,
         }}
       >

Diff for: src/context/App.test.tsx

@@ -259,7 +259,6 @@ describe('context/App.tsx', () => {
       { enterpriseAccounts: [], token: null, user: null },
       {
         appearance: 'SYSTEM',
-        markOnClick: false,
         openAtStartup: false,
         participating: true,
         playSound: true,
@@ -297,7 +296,6 @@ describe('context/App.tsx', () => {
       { enterpriseAccounts: [], token: null, user: null },
       {
         appearance: 'SYSTEM',
-        markOnClick: false,
         openAtStartup: true,
         participating: false,
         playSound: true,

Diff for: src/context/App.tsx

@@ -34,7 +34,6 @@ export const defaultSettings: SettingsState = {
   participating: false,
   playSound: true,
   showNotifications: true,
-  markOnClick: false,
   openAtStartup: false,
   appearance: Appearance.SYSTEM,
   colors: false,

Diff for: src/routes/Settings.test.tsx

@@ -182,34 +182,6 @@ describe('routes/Settings.tsx', () => {
     expect(updateSetting).toHaveBeenCalledWith('showNotifications', false);
   });
 
-  it('should toggle the onClickMarkAsRead checkbox', async () => {
-    let getByLabelText;
-
-    await act(async () => {
-      const { getByLabelText: getByLabelTextLocal } = render(
-        <AppContext.Provider
-          value={{
-            settings: mockSettings,
-            accounts: mockAccounts,
-            updateSetting,
-          }}
-        >
-          <MemoryRouter>
-            <SettingsRoute />
-          </MemoryRouter>
-        </AppContext.Provider>,
-      );
-      getByLabelText = getByLabelTextLocal;
-    });
-
-    fireEvent.click(getByLabelText('Mark as read on click'), {
-      target: { checked: true },
-    });
-
-    expect(updateSetting).toHaveBeenCalledTimes(1);
-    expect(updateSetting).toHaveBeenCalledWith('markOnClick', false);
-  });
-
   it('should toggle the openAtStartup checkbox', async () => {
     let getByLabelText;
 

Diff for: src/routes/Settings.tsx

@@ -139,12 +139,6 @@ export const SettingsRoute: React.FC = () => {
             updateSetting('showNotifications', evt.target.checked)
           }
         />
-        <FieldCheckbox
-          name="onClickMarkAsRead"
-          label="Mark as read on click"
-          checked={settings.markOnClick}
-          onChange={(evt) => updateSetting('markOnClick', evt.target.checked)}
-        />
         {!isLinux && (
           <FieldCheckbox
             name="openAtStartUp"

Diff for: src/routes/__snapshots__/Settings.test.tsx.snap

@@ -230,29 +230,6 @@ exports[`routes/Settings.tsx should render itself & its children 1`] = `
         </label>
       </div>
     </div>
-    <div
-      class="flex items-start mt-1 mb-3"
-    >
-      <div
-        class="flex items-center h-5"
-      >
-        <input
-          class="focus:ring-indigo-500 h-4 w-4 text-indigo-600 border-gray-300 rounded"
-          id="onClickMarkAsRead"
-          type="checkbox"
-        />
-      </div>
-      <div
-        class="ml-3 text-sm"
-      >
-        <label
-          class="font-medium text-gray-700 dark:text-gray-200"
-          for="onClickMarkAsRead"
-        >
-          Mark as read on click
-        </label>
-      </div>
-    </div>
     <div
       class="flex items-start mt-1 mb-3"
     >

Diff for: src/types.ts

@@ -10,7 +10,6 @@ export interface SettingsState {
   participating: boolean;
   playSound: boolean;
   showNotifications: boolean;
-  markOnClick: boolean;
   openAtStartup: boolean;
   appearance: Appearance;
   colors: boolean;