chore(deps): Remove history dependency (#648)

* fix(deps): update dependency axios to v1

* Use new adapter setting

See axios/axios#5277

* Fix tests

* Remove unnecessary transform

* chore(deps): Remove history dependency

* chore: update tests to remove `history`

---------

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: Afonso Jorge Ramos <[email protected]>

Author: 3 people

package.json

@@ -100,7 +100,6 @@
     "date-fns": "2.30.0",
     "electron-updater": "6.1.4",
     "final-form": "4.20.10",
-    "history": "4.10.1",
     "menubar": "9.3.0",
     "nprogress": "0.2.0",
     "react": "18.2.0",

pnpm-lock.yaml

@@ -1,7 +1,5 @@
 import { fireEvent, render, screen } from '@testing-library/react';
-import { createMemoryHistory } from 'history';
 import * as React from 'react';
-import { Router } from 'react-router';
 import { MemoryRouter } from 'react-router-dom';
 import * as TestRenderer from 'react-test-renderer';
 
@@ -20,7 +18,6 @@ jest.mock('react-router-dom', () => ({
 
 describe('components/Sidebar.tsx', () => {
   const fetchNotifications = jest.fn();
-  const history = createMemoryHistory();
 
   beforeEach(() => {
     fetchNotifications.mockReset();
@@ -81,9 +78,9 @@ describe('components/Sidebar.tsx', () => {
   it('go to the settings route', () => {
     const { getByLabelText } = render(
       <AppContext.Provider value={{ isLoggedIn: true, notifications: [] }}>
-        <Router location={history.location} navigator={history}>
+        <MemoryRouter>
           <Sidebar />
-        </Router>
+        </MemoryRouter>
       </AppContext.Provider>,
     );
     fireEvent.click(getByLabelText('Settings'));

src/components/Sidebar.test.tsx

@@ -1,8 +1,6 @@
 import React from 'react';
 import TestRenderer from 'react-test-renderer';
-import { Router } from 'react-router';
 import { MemoryRouter } from 'react-router-dom';
-import { createMemoryHistory } from 'history';
 import { render, fireEvent } from '@testing-library/react';
 
 const { ipcRenderer } = require('electron');
@@ -17,8 +15,6 @@ jest.mock('react-router-dom', () => ({
 }));
 
 describe('routes/Login.tsx', () => {
-  const history = createMemoryHistory();
-
   beforeEach(() => {
     mockNavigate.mockReset();
     jest.spyOn(ipcRenderer, 'send');
@@ -37,17 +33,17 @@ describe('routes/Login.tsx', () => {
   it('should redirect to notifications once logged in', () => {
     const { rerender } = render(
       <AppContext.Provider value={{ isLoggedIn: false }}>
-        <Router location={history.location} navigator={history}>
+        <MemoryRouter>
           <LoginRoute />
-        </Router>
+        </MemoryRouter>
       </AppContext.Provider>,
     );
 
     rerender(
       <AppContext.Provider value={{ isLoggedIn: true }}>
-        <Router location={history.location} navigator={history}>
+        <MemoryRouter>
           <LoginRoute />
-        </Router>
+        </MemoryRouter>
       </AppContext.Provider>,
     );
 
@@ -58,9 +54,9 @@ describe('routes/Login.tsx', () => {
 
   it('should navigate to login with github enterprise', () => {
     const { getByLabelText } = render(
-      <Router location={history.location} navigator={history}>
+      <MemoryRouter>
         <LoginRoute />
-      </Router>,
+      </MemoryRouter>,
     );
 
     fireEvent.click(getByLabelText('Login with GitHub Enterprise'));

src/routes/Login.test.tsx

@@ -1,9 +1,7 @@
 import * as React from 'react';
 import * as TestRenderer from 'react-test-renderer';
 import { fireEvent, render } from '@testing-library/react';
-import { Router } from 'react-router';
 import { MemoryRouter } from 'react-router-dom';
-import { createMemoryHistory } from 'history';
 
 const { ipcRenderer } = require('electron');
 
@@ -19,8 +17,6 @@ jest.mock('react-router-dom', () => ({
 }));
 
 describe('routes/LoginEnterprise.js', () => {
-  const history = createMemoryHistory();
-
   const mockAccounts: AuthState = {
     enterpriseAccounts: [],
     user: null,
@@ -47,9 +43,9 @@ describe('routes/LoginEnterprise.js', () => {
   it('let us go back', () => {
     const { getByLabelText } = render(
       <AppContext.Provider value={{ accounts: mockAccounts }}>
-        <Router location={history.location} navigator={history}>
+        <MemoryRouter>
           <LoginEnterpriseRoute />
-        </Router>
+        </MemoryRouter>
       </AppContext.Provider>,
     );
 
@@ -86,9 +82,9 @@ describe('routes/LoginEnterprise.js', () => {
   it('should receive a logged-in enterprise account', () => {
     const { rerender } = render(
       <AppContext.Provider value={{ accounts: mockAccounts }}>
-        <Router location={history.location} navigator={history}>
+        <MemoryRouter>
           <LoginEnterpriseRoute />
-        </Router>
+        </MemoryRouter>
       </AppContext.Provider>,
     );
 
@@ -101,9 +97,9 @@ describe('routes/LoginEnterprise.js', () => {
           },
         }}
       >
-        <Router location={history.location} navigator={history}>
+        <MemoryRouter>
           <LoginEnterpriseRoute />
-        </Router>
+        </MemoryRouter>
       </AppContext.Provider>,
     );
 

src/routes/LoginEnterprise.test.tsx

@@ -1,9 +1,7 @@
 import React from 'react';
 import TestRenderer from 'react-test-renderer';
 import { act, fireEvent, render, waitFor } from '@testing-library/react';
-import { Router } from 'react-router';
 import { MemoryRouter } from 'react-router-dom';
-import { createMemoryHistory } from 'history';
 import { shell } from 'electron';
 
 import { AppContext } from '../context/App';
@@ -16,7 +14,6 @@ jest.mock('react-router-dom', () => ({
 }));
 
 describe('routes/LoginWithToken.js', () => {
-  const history = createMemoryHistory();
   const openExternalMock = jest.spyOn(shell, 'openExternal');
 
   const mockValidateToken = jest.fn();
@@ -43,9 +40,9 @@ describe('routes/LoginWithToken.js', () => {
 
   it('let us go back', () => {
     const { getByLabelText } = render(
-      <Router location={history.location} navigator={history}>
+      <MemoryRouter>
         <LoginWithToken />
-      </Router>,
+      </MemoryRouter>,
     );
 
     fireEvent.click(getByLabelText('Go Back'));
@@ -77,9 +74,9 @@ describe('routes/LoginWithToken.js', () => {
   it("should click on the 'personal access tokens' link and open the browser", async () => {
     const { getByText } = render(
       <AppContext.Provider value={{ validateToken: mockValidateToken }}>
-        <Router location={history.location} navigator={history}>
+        <MemoryRouter>
           <LoginWithToken />
-        </Router>
+        </MemoryRouter>
       </AppContext.Provider>,
     );
 
@@ -93,9 +90,9 @@ describe('routes/LoginWithToken.js', () => {
 
     const { getByLabelText, getByTitle } = render(
       <AppContext.Provider value={{ validateToken: mockValidateToken }}>
-        <Router location={history.location} navigator={history}>
+        <MemoryRouter>
           <LoginWithToken />
-        </Router>
+        </MemoryRouter>
       </AppContext.Provider>,
     );
 
@@ -119,9 +116,9 @@ describe('routes/LoginWithToken.js', () => {
 
     const { getByLabelText, getByTitle } = render(
       <AppContext.Provider value={{ validateToken: mockValidateToken }}>
-        <Router location={history.location} navigator={history}>
+        <MemoryRouter>
           <LoginWithToken />
-        </Router>
+        </MemoryRouter>
       </AppContext.Provider>,
     );
 

src/routes/LoginWithToken.test.tsx

@@ -1,9 +1,7 @@
 import React from 'react';
 import TestRenderer, { act } from 'react-test-renderer';
 import { render, fireEvent } from '@testing-library/react';
-import { Router } from 'react-router';
 import { MemoryRouter } from 'react-router-dom';
-import { createMemoryHistory } from 'history';
 
 const { ipcRenderer } = require('electron');
 
@@ -18,7 +16,6 @@ jest.mock('react-router-dom', () => ({
 }));
 
 describe('routes/Settings.tsx', () => {
-  const history = createMemoryHistory();
   const updateSetting = jest.fn();
 
   beforeEach(() => {
@@ -50,9 +47,9 @@ describe('routes/Settings.tsx', () => {
         <AppContext.Provider
           value={{ settings: mockSettings, logout: logoutMock }}
         >
-          <Router location={history.location} navigator={history}>
+          <MemoryRouter>
             <SettingsRoute />
-          </Router>
+          </MemoryRouter>
         </AppContext.Provider>,
       );
 
@@ -74,9 +71,9 @@ describe('routes/Settings.tsx', () => {
     await act(async () => {
       const { getByLabelText: getByLabelTextLocal } = render(
         <AppContext.Provider value={{ settings: mockSettings }}>
-          <Router location={history.location} navigator={history}>
+          <MemoryRouter>
             <SettingsRoute />
-          </Router>
+          </MemoryRouter>
         </AppContext.Provider>,
       );
 
@@ -222,9 +219,9 @@ describe('routes/Settings.tsx', () => {
     await act(async () => {
       const { getByLabelText: getByLabelTextLocal } = render(
         <AppContext.Provider value={{ settings: mockSettings }}>
-          <Router location={history.location} navigator={history}>
+          <MemoryRouter>
             <SettingsRoute />
-          </Router>
+          </MemoryRouter>
         </AppContext.Provider>,
       );
       getByLabelText = getByLabelTextLocal;